<div dir="ltr">Yeah. The twisted take-away from this thread is to not test your changes against openstack so you have plausible deniability. :)</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 16, 2015 at 9:04 AM, Mike Bayer <span dir="ltr"><<a href="mailto:mbayer@redhat.com" target="_blank">mbayer@redhat.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 12/16/2015 11:53 AM, Sean Dague wrote:<br>
> On 12/16/2015 11:37 AM, Sean Dague wrote:<br>
>> On 12/16/2015 11:22 AM, Mike Bayer wrote:<br>
>>><br>
>>><br>
>>> On 12/16/2015 09:10 AM, Sylvain Bauza wrote:<br>
>>>><br>
>>>><br>
>>>> Le 16/12/2015 14:59, Sean Dague a écrit :<br>
>>>>> oslo.db test_migrations is using methods for alembic, which changed in<br>
>>>>> the 0.8.4 release. This ends up causing a unit test failure (at least in<br>
>>>>> the Nova case) that looks like this -<br>
>>>>> <a href="http://logs.openstack.org/44/258444/1/check/gate-nova-python27/2ed0401/console.html#_2015-12-16_12_20_17_404" rel="noreferrer" target="_blank">http://logs.openstack.org/44/258444/1/check/gate-nova-python27/2ed0401/console.html#_2015-12-16_12_20_17_404</a><br>
>>>>><br>
>>>>><br>
>>>>> There is an oslo.db patch out there<br>
>>>>> <a href="https://review.openstack.org/#/c/258478" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/258478</a> to fix it, but #openstack-oslo<br>
>>>>> has been pretty quiet this morning, so no idea how fast this can get out<br>
>>>>> into a release.<br>
>>>>><br>
>>>>>     -Sean<br>
>>>>><br>
>>>><br>
>>>> So, it seems that the issue came when<br>
>>>> <a href="https://bitbucket.org/zzzeek/alembic/issues/341" rel="noreferrer" target="_blank">https://bitbucket.org/zzzeek/alembic/issues/341</a> was merged.<br>
>>>> Fortunatelt, Mike seems to have a patch in place for Nova in order to<br>
>>>> fix this <a href="https://review.openstack.org/#/c/253859/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/253859/</a><br>
>>>><br>
>>>> I'd suggest an intensive review pass on that one to make sure it's OK.<br>
>>><br>
>>> do you folks have a best practice suggestion on this?  My patch kind of<br>
>>> stayed twisting in the wind for a week even though those who read it<br>
>>> would have seen "hey, this is going to break on Alembic's next minor<br>
>>> release!"    I pinged the important people and all on it, but it still<br>
>>> got no attention.<br>
>><br>
>> Which people were those? I guess none of us this morning knew this was<br>
>> going to be an issue and were surprised that 12 hours worth of patches<br>
>> had all failed.<br>
>><br>
>>      -Sean<br>
><br>
> Best practice is send an email to the openstack-dev list:<br>
><br>
> Subject: [all] the following test jobs will be broken by Alembic 0.8.4<br>
> release<br>
><br>
> The Alembic 0.8.4 release is scheduled on 12/15. When it comes out it<br>
> will break Nova unit tests on all branches.<br>
><br>
> The following patch will fix master - .....<br>
><br>
> You all will need to backport it as well to all branches.<br>
><br>
><br>
> Instead of just breaking the world, and burning 10s to 100 engineer<br>
> hours in redo tests and investigating and addressing the break after the<br>
> fact.<br>
<br>
</div></div>I was hoping to get a thanks for even *testing* unreleased versions of<br>
my entirely non-Openstack, upstream projects against Openstack itself.<br>
 If I did *less* effort here, and just didn't bother the way 100% of all<br>
other non-Openstack projects do, then I'd not have been scolded by you.<br>
<br>
<br>
<br>
<br>
<br>
<br>
><br>
>       -Sean<br>
<div class="HOEnZb"><div class="h5">><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><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div>Kevin Benton</div></div>
</div>