<div dir="ltr">Matt,<div><br></div><div>My understanding is that there will be a nova.virt.baremetal.ironic driver in Nova which will talk to Ironic API to manage bare-metal instances. So, Ironic will be actually providing the diagnostics data about bm instance via its API eventually.</div>

<div><br></div><div>Hope someone will correct me if I'm wrong.</div><div><br></div><div>--</div><div>Best regards,</div><div>Oleg Gelbukh</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Dec 20, 2013 at 7:12 PM, Matt Riedemann <span dir="ltr"><<a href="mailto:mriedem@linux.vnet.ibm.com" target="_blank">mriedem@linux.vnet.ibm.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
<br>
On Friday, December 20, 2013 3:57:15 AM, Daniel P. Berrange wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Fri, Dec 20, 2013 at 12:56:47PM +0400, Oleg Gelbukh wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi everyone,<br>
<br>
I'm sorry for being late to the thread, but what about baremetal driver?<br>
Should it support the get_diagnostics() as well?<br>
</blockquote>
<br>
Of course, where practical, every driver should aim to support every<br>
method in the virt driver class API.<br>
<br>
Regards,<br>
Daniel<br>
</blockquote>
<br></div>
Although isn't the baremetal driver moving to ironic, or there is an ironic driver moving into nova?  I'm a bit fuzzy on what's going on there.  Point is, if we're essentially halting feature development on the nova baremetal driver I'd hold off on implementing get_diagnostics there for now.<br>


<br>
--<br>
<br>
Thanks,<br>
<br>
Matt Riedemann<br>
<br>
</blockquote></div><br></div>