<div dir="ltr">On Wed, Dec 7, 2016 at 1:52 PM, Pavlo Shchelokovskyy <span dir="ltr"><<a href="mailto:pshchelokovskyy@mirantis.com" target="_blank">pshchelokovskyy@mirantis.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">HI all,<div><br></div><div>we (ironic community) some time ago decided [0] to require third-party CI for any driver that is present in the main ironic code tree. I'd like to discuss the state of currently unsupported drivers and how to proceed with them.</div><div><br></div><div>Here is the current rundown, please correct me if I've got something wrong:</div><div><br></div><div>* AMT - already in ironic-staging-drivers repo, patch removing those from ironic is on review [1]</div></div></blockquote><div><br>Agree - as a note, those were deprecated September 16, so we need to wait for December 16 to land that.<br><br></div><div>I've -2'd the patch, but reviews welcome so we can land that on the 16th.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>* iBoot - already in ironic-staging-drivers repo, patch removing those from ironic is on review [1]</div></div></blockquote><div><br></div><div>Ditto.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>* WakeOnLan - already in ironic-staging-drivers repo, patch removing those from ironic is on review [1]</div></div></blockquote><div><br></div><div>Ditto.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>* IPMINative/Pyghmi - community driver, AFAIU community still considers those as a viable alternative for the future and is constantly re-evaluating maturity of pyghmi IPMI implementation, so these are to stay for now</div></div></blockquote><div><br></div><div>Well, we deprecated these, unless someone gets CI running for it, I plan to drop them.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>* SSH - community driver, still used on several ironic gate jobs and in jobs of other projects under Baremetal program (like bifrost). Besides AFAIK quite a number of people use it for development. So it is to stay in the tree for some more time too, at least until all upstream gate jobs are moved to ipmitool-based drivers.</div></div></blockquote><div><br></div><div>Like Dmitry said, we should move these jobs over ASAP and then drop this driver.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>* SNMP - people are working to enable testing it in CI, patches are landing, stays in tree</div></div></blockquote><div><br></div><div>Agree.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>* VirtualBox - community driver, for testing only, VirtualBox can be used via SSH driver and I am not aware of any plans for (third-party) CI for it (although it would in principle be possible even in upstream). Is anyone actually using this driver?</div></div></blockquote><div><br></div><div>Someone that no longer works on Ironic submitted this. It's for using VirtualBox on a Windows host where we don't have SSH (it uses VBox's web API). This is also deprecated and I plan to remove it.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>* MSFTOCS - vendor driver, I am not aware of any plans for third-party CI</div></div></blockquote><div><br></div><div>Agree, I plan to drop this.<br> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>* SeaMicro - vendor driver, I am not aware of any plans for third-party CI</div></div></blockquote><div><br>Agree, I plan to drop this.<br><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>Based on that I propose to remove VirtualBox, MSFTOCS and SeaMicro drivers from ironic right away. If anybody is interested in supporting them they would have to extract those drivers (together with unit tests and docs) to separate repos or propose them to ironic-staging-drivers minding the warning [2]. <br></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>[0] <a href="https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/third-party-ci.html" target="_blank">https://specs.openstack.<wbr>org/openstack/ironic-specs/<wbr>specs/not-implemented/third-<wbr>party-ci.html</a></div><div>[1] <a href="https://review.openstack.org/#/c/397847" target="_blank">https://review.openstack.<wbr>org/#/c/397847</a></div><div>[2] <a href="http://ironic-staging-drivers.readthedocs.io/en/latest/README.html#what-the-ironic-staging-drivers-is-not" target="_blank">http://ironic-staging-<wbr>drivers.readthedocs.io/en/<wbr>latest/README.html#what-the-<wbr>ironic-staging-drivers-is-not</a></div></div></blockquote><div><br></div><div>Thanks for sending this, Pavlo. :)<br><br></div><div>// jim</div><br></div><br></div></div>