<div dir="ltr"><div><div><div>Can the *ED states be optional?  If a state would be a NO-OP, then don't use it.  But if it is useful then use it.<br><br></div>It seems ugly to me to just have two 'change state' calls to move it through the *ED state.<br><br></div>If an *ED state needs to be added later I think it could be added without too much difficulty and the code will be cleaner without having 'no-op' states.<br><br></div>WDYT?<br><br>John<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 19, 2015 at 7:25 AM, 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 everyone!<br>
<br>
On one of our meetings [1] we agreed on keeping *ED states (DELETED, INSPECTED, CLEANED, etc) as no-ops for now. Since then, however, the inspection patch [2] got several comments from cores requesting removal of INSPECTED state. That was done by Nisha.<br>
<br>
Today we decided to approve [2] and bring the discussion here. We can always add the state later, and blocking this patch again would be a bit unfair IMO. We'll create a follow-up to [2] if we decide that we need INSPECTED state.<br>
<br>
So now, are we keeping/adding these *ED states to our state machine? I personally agree with what was discussed on the meeting, namely:<br>
1. Keep *ED states<br>
2. Make them no-ops, so that code that does INSPECTING -> MANAGEABLE right now will do INSPECTING -> INSPECTED -> MANAGEABLE instead.<br>
<br>
Having INSPECTED is also useful for distinguish between OOB case (when inspect_hardware returns after having everything done) and in-band case (when inspect_hardware returns after initializing inspection). We could use return value of inspect_hardware being INSPECTING or INSPECTED.<br>
<br>
WDYT?<br>
<br>
Dmitry.<br>
<br>
[1] <a href="http://eavesdrop.openstack.org/meetings/ironic/2015/ironic.2015-02-09-17.00.log.html" target="_blank">http://eavesdrop.openstack.<u></u>org/meetings/ironic/2015/<u></u>ironic.2015-02-09-17.00.log.<u></u>html</a> starting 17:47<br>
[2] <a href="https://review.openstack.org/#/c/147857/" target="_blank">https://review.openstack.org/#<u></u>/c/147857/</a><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>