<div dir="ltr">Hi all,<div><br></div><div>Looking at the state machine <a href="https://specs.openstack.org/openstack/ironic-specs/specs/kilo-implemented/new-ironic-state-machine.html">spec</a>, this might be a leftover from the older state machine.<br>When the node is in deleting state, we only clean up the kernel/ramdisk and clear DHCP</div><div>options, so rebuild may be completed successfully afterwards. I agree that this transition</div><div>does not make much sense, and removing it will help fixing the bug you've referenced,</div><div>but I'm not sure how to handle backwards compatibility in this case (and whether we need</div><div>to do this).</div><div><br></div><div>Vlad</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 5, 2016 at 4:00 PM, Yuriy Zveryanskyy <span dir="ltr"><<a href="mailto:yzveryanskyy@mirantis.com" target="_blank">yzveryanskyy@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi.<br>
<br>
We have a followed transition in common/states.py:<br>
<br>
# An errored instance can be rebuilt<br>
# ironic/conductor/manager.py:do_node_deploy()<br>
machine.add_transition(ERROR, DEPLOYING, 'rebuild')<br>
<br>
At first glance it looks correct. But ERROR state is<br>
used only for error after deleting, see<br>
<a href="http://docs.openstack.org/developer/ironic/_images/states.svg" rel="noreferrer" target="_blank">http://docs.openstack.org/developer/ironic/_images/states.svg</a><br>
So ERROR is delete error, at least now, and transition<br>
error (delete error) -> deploying (on_rebuild)<br>
is possible.<br>
Looks strange if operator wants to remove an instance completely and then<br>
does rebuild after error (non-error targets for deleting is cleaning -> available).<br>
I think this transition should be removed. Without this strange transition<br>
bug <a href="https://bugs.launchpad.net/ironic/+bug/1522008" rel="noreferrer" target="_blank">https://bugs.launchpad.net/ironic/+bug/1522008</a> can be fixed by simple way,<br>
port's vif id can be removed via Ironic virt driver request before waiting of CLEANING<br>
(it's no more needed).<br>
<br>
Yuriy Zveryanskyy<br>
<br>
<br>
<br>
<br>
<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br></div>