<div dir="ltr"><br><div>+1 from me.  Since we don't have ENROLL state as per the state machine, I think it should be MANAGEABLE when we enroll a node.  At least, it can also prevent nodes getting into a ready state even before an operator getting hands on it.</div><div><br></div><div>One comment on #2.  Before we make a new client release with v1.6, shouldn't the behaviour of the 0.x.x python-ironicclient be that, newly enroll nodes have provision_state as NOSTATE again instead of AVAILABLE ?</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 3, 2015 at 1:59 PM, Dmitry Tantsur <span dir="ltr"><<a href="mailto:dtantsur@redhat.com" target="_blank">dtantsur@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all!<br>
<br>
Today I got an internal email, stating that new ironicclient brakes ironic-discoverd. Indeed, after rebase to the latest ironicclient git master, discoverd started receiving "AVAILABLE" state instead of "None" for newly enrolled nodes. It's not a valid state for introspection, valid are "MANAGEABLE" (discoverd stand-alone usage), "INSPECTING" (discoverd via Ironic driver) and None (Juno + discoverd stand-alone). Looks like despite introducing microversions we did manage to break 3rdparty apps relying on states... Also we're in a bit weird situation where nodes appear ready for scheduling, despite us having a special state for managing nodes _before_ being ready for scheduling.<br>
<br>
I find the situation pretty confusing, and I'd like your comments on the following proposal to be implemented before RC1:<br>
<br>
1. add new micro-version 1.7. nodes created by API with this version will appear in state MANAGEABLE;<br>
2. make a client release with current API version set to 1.6 (thus excluding change #1 from users of this version);<br>
3. set current API version for ironicclient to 1.7 and release ironicclient version 2.0.0 to designate behavior changes;<br>
4. document the whole thingy properly<br>
<br>
#1 should be a small change, but it definitely requires FFE.<br>
Thoughts?<br>
<br>
Dmitry<br>
<br>
______________________________<u></u>______________________________<u></u>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.<u></u>openstack.org?subject:<u></u>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</blockquote></div><br></div>