<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jul 1, 2015 at 5:49 AM, Kevin Benton <span dir="ltr"><<a href="mailto:blak111@gmail.com" target="_blank">blak111@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"><div dir="ltr"><span class="">><span style="font-size:12.8px">For the second one, I think we made it clear before that no plugins </span><span style="font-size:12.8px">must rely on neutron.openstack.* contents.</span><div><span style="font-size:12.8px"><br></span></div></span><div><span style="font-size:12.8px">Where was this made clear? I didn't know about this until it broke and I'm a very frequent contributor and reviewer, which is the core of my complaint. </span><span style="font-size:12.8px">The updated wiki is helpful, but doing it after-the-fact only helps when people try to figure out why they can't make changes to their repos.</span></div><div><br></div></div></blockquote><div><br></div><div>I agree, and I'd also like to mention that it would be good to move this type of documentation in-tree so it can be autogenerated rather than using the wiki.<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></div><div><span style="font-size:12.8px">In that patch there were 3 in-tree third party plugins that had to be updated. That should have given a hint that it was likely to break other plugins as well. The other annoying lesson from that is that plugins/drivers still in tree are getting better treatment than the ones that went through the split process.</span></div><div><span style="font-size:12.8px"><br></span></div></div></blockquote><div><br></div><div>Also agree. The current plan of record is that plugins/drivers which have not started to decompose will be marked as deprecated in Liberty, and we'll remove them in Mxx. I hope Henry can update his "Phase 2 Decomp" devref [1] to indicate this.<br><br>[1] <a href="https://review.openstack.org/#/c/187267/">https://review.openstack.org/#/c/187267/</a><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><span style="font-size:12.8px"></span></div><div><span style="font-size:12.8px">We need to be much more diligent about announcing these things ahead of time or we need to be clear that Neutron is completely unstable as a framework and third party drivers should basically never import anything from the neutron namespace.</span></div><div><span style="font-size:12.8px"><br></span></div></div></blockquote><div><br></div><div>We're working on that. The decomp of the reference plugin will help this situation because you won't be able to pass the gate if you break these types of things, and that will give patch authors time to notify everyone else of breaking changes, or even just submit patches to the other repositories.<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><span style="font-size:12.8px"></span></div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Tue, Jun 30, 2015 at 11:58 PM, Ihar Hrachyshka <span dir="ltr"><<a href="mailto:ihrachys@redhat.com" target="_blank">ihrachys@redhat.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">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA256<br>
<div><div><br>
On 07/01/2015 08:22 AM, Kevin Benton wrote:<br>
> Hi,<br>
><br>
> We have had at least two breaking changes merge this week for<br>
> out-of-tree drivers/plugins. These are just the two I noticed that<br>
> broke the Big Switch CI (the one I keep an eye on since I had set<br>
> it up):<br>
><br>
> 1. Removed test_lib that changes config files.<br>
> <a href="https://review.openstack.org/#/c/196583/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/196583/</a> 2. Removed the loopingcall<br>
> common util with no deprecation cycle or announcement.<br>
> <a href="https://review.openstack.org/#/c/192999/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/192999/</a><br>
><br>
> I proposed a revert for 1 that merged, but I don't particularly<br>
> want to keep fighting this. What is our current policy on this?<br>
> Just change whatever we want and tell plugin maintainers this is<br>
> just the way things work?<br>
><br>
<br>
</div></div>For the first one, the revert is justified.<br>
<br>
For the second one, I think we made it clear before that no plugins<br>
must rely on neutron.openstack.* contents.<br>
<br>
Now I've made it mentioned at:<br>
<a href="https://wiki.openstack.org/wiki/Neutron/LibraryAPIBreakage" rel="noreferrer" target="_blank">https://wiki.openstack.org/wiki/Neutron/LibraryAPIBreakage</a><br>
<br>
If your plugin is using any of neutron.openstack.* contents, just stop<br>
doing it. It's going to break.<br>
<br>
Ihar<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v2<br>
<br>
iQEcBAEBCAAGBQJVk4+AAAoJEC5aWaUY1u57OE8IAKDqKudi5zOUxRPc4Elzsdw4<br>
mASF5Mtguj5q9OUpYIyeOkbsIKmOwop4tbGjz52L+OZ8aLq1XptpKLUuX6mBL3lS<br>
m0/DtD2RlBRTmiO/kyBxeqJbsj7nZU9eoiDJ88gN51IetN1kIR09rAbmdkoduhK2<br>
FIrFLJhhuVqmb8S377cbSgJ46kC1DeDa2xhtFWB39iIKO3ZTwO7ia5KRipTSTyNV<br>
4ngB0+d8EgMfmsKj4Bd7/btkg5rI+o4qNgd4L1Ncd1BQzPiOzzeeGo0lAe86Kf7z<br>
KUH2Sw6n6mIUZJSGzDP4cigGqSsilBaurryK90G/Go1q7BEMRFVyrGHObnMBZTw=<br>
=qSnW<br>
-----END PGP SIGNATURE-----<br>
<br>
__________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class=""><font color="#888888">-- <br><div><div>Kevin Benton</div></div>
</font></span></div></div>
<br>__________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div></div>