<div dir="ltr">Thanks for the writeup, Melanie :)<div class="gmail_extra">
<br><div class="gmail_quote">On Mon, Mar 19, 2018 at 8:31 PM, melanie witt <span dir="ltr"><<a href="mailto:melwittt@gmail.com" target="_blank">melwittt@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
  * For the issue of nova-compute crashing on startup, we could add a try-except around the call site at startup and ignore a "NotReadyYet" or similar exception from the Ironic driver<br></blockquote><div><br></div><div>This is here: <a href="https://review.openstack.org/#/c/545479/">https://review.openstack.org/#/c/545479/</a></div><div><br></div><div>Just doing a bit more testing and should have a new version up shortly.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
  * On Ironic API version negotiation, the ironicclient already has some version negotiation built-in, so there are some options. 1) update Ironic driver to handle return/error codes from ironicclient version-negotiated calls, 2) add per-call microversion support to ironicclient and use it in the Ironic driver, 3) convert all Ironic driver calls to use raw REST<br>
    * Option 1) would be the most expedient, but it's up to the Ironic team how they will want to proceed. Option 3 is the desired ideal solution but will take a rewrite of the related Ironic driver unit tests as they currently all mock ironicclient<br></blockquote><div><br></div><div>We discussed this further in IRC yesterday, and Julia is going to explore option 2 for now.</div><div><br></div><div>// jim</div></div></div></div>