<div dir="ltr">Hi all,<div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div dir="ltr">On Thu, Dec 8, 2016 at 6:58 PM, Jim Rollenhagen <span dir="ltr"><<a href="mailto:jim@jimrollenhagen.com" target="_blank">jim@jimrollenhagen.com</a>></span> wrote:<br></div></div></div></div><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"><span class="gmail-">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></span><div class="gmail_extra"><div class="gmail_quote"><span class="gmail-"><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></span><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></div></div></blockquote><div><br></div><div>Thanks for reminding about deprecation period. This also means that the last patch for removal of deprecated agent vendor passthru is blocked until Dec 16 too. I will rebase that chain again and change commit order so we could in principle land the updates to iLO and OneView drivers without being blocked for a week.</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 class="gmail_extra"><div class="gmail_quote"><div></div><span class="gmail-"><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></span><div>Ditto.<br> <br></div><span class="gmail-"><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></span><div>Ditto.<br> <br></div><span class="gmail-"><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></span><div>Well, we deprecated these, unless someone gets CI running for it, I plan to drop them.<br></div></div></div></div></blockquote><div><br></div><div>Thanks for clarification. </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 class="gmail_extra"><div class="gmail_quote"><div> <br></div><span class="gmail-"><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></span><div>Like Dmitry said, we should move these jobs over ASAP and then drop this driver.<br></div></div></div></div></blockquote><div><br></div><div><div>Basically those are grenade job, couple of jobs in ironicclient and IPA, bifrost itself and many (all?) inspector-related ones.</div><div>We also have our devstack plugin defaulting to pxe_ssh for IRONIC_DEPLOY_DRIVER, and have it too in the default for IRONIC_ENABLED_DRIVERS. Only one job seems to use these defaults implicitly though.</div><div><br></div><div>Complete list of jobs still using *_ssh drivers is below.</div></div><div><br></div><div>Ironic<br></div><div><div>- gate-grenade-dsvm-ironic (pxe_ssh)</div><div>- gate-tempest-dsvm-ironic-<wbr>multitenant-network-ubuntu-<wbr>xenial (agent_ssh)</div><div>- gate-tempest-dsvm-ironic-<wbr>inspector-nv (agent_ssh)</div></div><div><br></div><div>The inspector one seems easy.<br></div><div>The multitenant one - the multinode job is also testing Ironic with network isolation on and is using ipmitool driver, so we might consider just dropping the single node multitenant job and make the multinode job voting. If not, single node multitenant could also be easily fixed.</div><div>Most problems I would expect with grenade...</div><div><br></div><div>Other projects under ironic governance:<br></div><div><br></div><div>ironic-ui, molteniron, ironic-tempest-plugin, ironic-inspector-tempest-plugin do not have any functional/integration jobs (last two are empty actually :) )</div><div><br></div><div>ironic-lib, virtualbmc jobs are using only ipmitool drivers</div><div><br></div><div>python-ironicclient</div><div>- gate-tempest-dsvm-python-ironicclient-src (pxe_ssh)</div><div>- gate-ironicclient-dsvm-functional (pxe_ssh as implicit default of ironic devstack plugin, also implicitly uses default enabled drivers)<br></div><div><div><br></div><div>ironic-python-agent</div><div>- gate-tempest-dsvm-ironic-inspector-src-ubuntu-xenial-nv (agent_ssh)</div><div><br></div></div><div>ironic-inspector<br></div><div>- gate-tempest-dsvm-ironic-inspector (agent_ssh)</div><div>- gate-grenade-dsvm-ironic-inspector (pxe_ssh)</div><div>- gate-tempest-dsvm-ironic-inspector-discovery (agent_ssh)</div><div><div><br class="gmail-Apple-interchange-newline">python-ironic-inspector-client<br></div><div>- gate-tempest-dsvm-python-ironic-inspector-client-ubuntu-xenial (agent_ssh)</div></div><div><br></div><div>bifrost<br></div><div>uses *_ssh drivers when in testing mode. Switching to virtualbmc would require some effort.</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 class="gmail_extra"><div class="gmail_quote"><div> <br></div><span class="gmail-"><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></span><div>Agree.<br> <br></div><span class="gmail-"><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></span><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><span class="gmail-"><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></span><div>Agree, I plan to drop this.<br> <br></div><span class="gmail-"><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></span><div><br>Agree, I plan to drop this.<br><br> </div><span class="gmail-"><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.or<wbr>g/openstack/ironic-specs/specs<wbr>/not-implemented/third-party-<wbr>ci.html</a></div><div>[1] <a href="https://review.openstack.org/#/c/397847" target="_blank">https://review.openstack.o<wbr>rg/#/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-driv<wbr>ers.readthedocs.io/en/latest/<wbr>README.html#what-the-ironic-<wbr>staging-drivers-is-not</a></div></div></blockquote><div><br></div></span><div>Thanks for sending this, Pavlo. :)<span class="gmail-HOEnZb"><font color="#888888"><br><br></font></span></div><span class="gmail-HOEnZb"><font color="#888888"><div>// jim</div><br></font></span></div><br></div></div>
<br>______________________________<wbr>______________________________<wbr>______________<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.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
<br></blockquote><div><br></div><div>Cheers,</div><div> </div></div>Dr. Pavlo Shchelokovskyy<div>Senior Software Engineer</div><div>Mirantis Inc</div><div><a href="http://www.mirantis.com/" target="_blank">www.mirantis.com</a></div></div></div>