<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 18, 2017 at 11:29 PM,  <span dir="ltr"><<a href="mailto:Arkady.Kanevsky@dell.com" target="_blank">Arkady.Kanevsky@dell.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">Thanks for response.<br>
My recommendation is:<br>
1. only allow patches into openstack/ironic-tempest-<wbr>plugin<br>
2. Give Ironic CI owners time period (3 weeks?) to switch their setup to only use openstack/ironic-tempest-<wbr>plugin and not master and report back to Ironic CI team if it works for them. If yes, go ahead and switch if not, report back.<br>
3. At the end of that time, if majority of Ironic CI site complete their  transition to ironic-tempest-plugin we switch.<br>
<br>
Thanks,<br>
Arkady<br>
<span class="gmail-im gmail-HOEnZb"><br></span></blockquote><div><br></div><div>I think this is reasonable. I cringe at knowingly doing something that will most likely break the third party CI when people are away and cannot address it. Also, apart from the danger that the third party CIs are running old versions of the tempest plugins (I doubt that we'll be doing major overhaul of that in the next 2+ weeks), the reason we have 3rd party CI is to make sure patches work against those HW. Having most or all of them being unavailable due to this breakage means that there won't be any feedback that someone's code would have broken 3rd party CI outside of the tempest changes.</div><div><br></div><div>John, don't you have a patch that removes the tempest plugin from master? Do most/all of the 3rd party CIs fail due to that? (Or is it not as simple as that?) Ah, yes, <a href="https://review.openstack.org/#/c/527733/">https://review.openstack.org/#/c/527733/</a> shows that all the 3rd party CI fail (although I don't know if it is due to that patch or if they were already failing prior to that).</div><div><br></div><div>--ruby </div></div></div></div>