<div dir="ltr"><div>Hi, </div><div><br></div><div>Thank you for putting it up Dmitry.</div><div><br></div><div>As I wrote into the blueprint,</div><div>if there are requests to implement API aborting introspection from operators,</div><div>we should introduce this feature as API, I think.</div><div><br></div><div>But if we just want to use this feature as debug,</div><div>we had better not to introduce it as API.</div><div>And, instead of introducing as API,</div><div>I suppose implement only client library and call it from shell script or something like "tool".</div><div><br></div><div><br></div><div>Best Regards,</div><div>Yuiko Takada</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-04-21 20:42 GMT+09:00 Dmitry Tantsur <span dir="ltr"><<a href="mailto:dtantsur@redhat.com" target="_blank">dtantsur@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi folks.<br>
<br>
Recently I got several requests to implement API aborting introspection in discoverd. This is useful mostly when debugging, when you know that introspection has failed, and you want to stop it right now. I've put a blueprint <a href="https://blueprints.launchpad.net/ironic-discoverd/+spec/abort-introspection" target="_blank">https://blueprints.launchpad.net/ironic-discoverd/+spec/abort-introspection</a> to track it.<br>
<br>
Such API would cause discoverd to set error state for the node immediately and issue a power off request for it. The technical side is not a big deal.<br>
<br>
What I'm worried about is whether we want to introduce such a feature at all. Some Ironic processes (deploy, in-band cleaning) are async as well, they also may hang, and IIUC we don't have means of aborting them. Does debugging justify introducing new API?<br>
<br>
This looks somewhat similar to our discussion about breaking locks in Ironic: it might be useful, but it's an API which we'll support and which can be easily misused.<br>
<br>
But with introspection the only case where lack of this feature can't be easily worked around is when people want to recreate a node. I've been arguing for some time that recreating a node is not a good way to solve problems. In other cases one can just power off the node via Ironic API and restart the introspection again afterwards.<br>
<br>
What do you think?<br>
<br>
Cheers,<br>
Dmitry<br>
<br>
__________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br></div>