<div dir="ltr">Matt,<div><br></div><div>Thank you for bring this topic to the ML.</div><div><br></div><div>In cinder, we've merged [1] patch to unblock gates. I've proposed other patch [2] to fix global-requirements for the stable/liberty branch.</div><div><br></div><div><br></div><div>[1] <a href="https://review.openstack.org/#/c/239837/">https://review.openstack.org/#/c/239837/</a></div><div>[2] <a href="https://review.openstack.org/#/c/239799/">https://review.openstack.org/#/c/239799/</a></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div>Regards,<br>Ivan Kolodyazhny,<br><a href="http://blog.e0ne.info/" target="_blank">http://blog.e0ne.info/</a></div></div></div></div>
<br><div class="gmail_quote">On Thu, Oct 29, 2015 at 12:13 AM, 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="HOEnZb"><div class="h5"><br>
<br>
On 10/28/2015 9:22 AM, Matt Riedemann wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
On 10/28/2015 9:06 AM, Yuriy Nesenenko wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi. Look at <a href="https://review.openstack.org/#/c/239837/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/239837/</a><br>
<br>
On Wed, Oct 28, 2015 at 3:52 PM, Matt Riedemann<br>
<<a href="mailto:mriedem@linux.vnet.ibm.com" target="_blank">mriedem@linux.vnet.ibm.com</a> <mailto:<a href="mailto:mriedem@linux.vnet.ibm.com" target="_blank">mriedem@linux.vnet.ibm.com</a>>> wrote:<br>
<br>
    That job is failing at a decent rate, tracking with bug:<br>
<br>
    <a href="https://bugs.launchpad.net/cinder/+bug/1510656" rel="noreferrer" target="_blank">https://bugs.launchpad.net/cinder/+bug/1510656</a><br>
<br>
    It lines up with the novaclient 2.33 release on 10/27, I'm checking<br>
    out what the change was that caused the regression.<br>
<br>
    This is a heads up that rechecks on this failure probably won't help.<br>
<br>
    So far I haven't seen any related patches up to fix it although<br>
    there were already 2 bugs reported when I got in this morning.<br>
<br>
    --<br>
<br>
    Thanks,<br>
<br>
    Matt Riedemann<br>
<br>
<br>
<br>
__________________________________________________________________________<br>
<br>
    OpenStack Development Mailing List (not for usage questions)<br>
    Unsubscribe:<br>
    <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>
<br>
<<a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">http://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>
<br>
<br>
<br>
__________________________________________________________________________<br>
<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe:<br>
<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>
<br>
Heh, well that's 3 bugs then, I didn't see that one. jgriffith and I<br>
were talking in IRC about just handling both exceptions in cinder to fix<br>
this but we also agreed that this is a backward incompatible change on<br>
the novaclient side, which was also discussed in the original novaclient<br>
wishlist bug that prompted the breaking change.<br>
<br>
Given the backward compat issues, we might not just be breaking cinder<br>
here, so I've proposed a revert of the novaclient change with<br>
justification in the commit message:<br>
<br>
<a href="https://review.openstack.org/#/c/239941/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/239941/</a><br>
<br>
At least with the cinder change above we're OK for mitaka, and logstash<br>
isn't yet showing failures for cinder in stable/liberty, but given the<br>
requirements there it will be a failure in cinder python34 tests in<br>
stalbe/liberty also - so we can backport the cinder fix or block the<br>
2.33 novaclient version on stable/liberty global-requirements depending<br>
on what we do with the proposed novaclient revert.<br>
<br>
</blockquote>
<br></div></div>
I have an alternative to the revert here:<br>
<br>
<a href="https://review.openstack.org/#/c/239963/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/239963/</a><br>
<br>
That makes novaclient.exceptions.RequestTimeout extend requests.Timeout so that older cinder continues to work.<br>
<br>
I also have changes to block novaclient 2.33.0 in g-r on master and stable/liberty:<br>
<br>
<a href="https://review.openstack.org/#/q/I6e7657b60308b30eed89b269810c1f37cce43063,n,z" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/I6e7657b60308b30eed89b269810c1f37cce43063,n,z</a><br>
<br>
I personally think we need to block 2.33.0 since it breaks cinder, then release a new version of novaclient with either the revert or the alternative change to extend requests.Timeout.<br>
<br>
If we block novaclient 2.33.0 then we can also revert the workaround in cinder (which would start breaking if we reverted the new exception type out of novaclient w/o blacklisting 2.33 first).<div class="HOEnZb"><div class="h5"><br>
<br>
-- <br>
<br>
Thanks,<br>
<br>
Matt Riedemann<br>
<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>
</div></div></blockquote></div><br></div>