<div dir="ltr"><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 9, 2018 at 12:30 PM Doug Hellmann <<a href="mailto:doug@doughellmann.com">doug@doughellmann.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Ken Giusti <<a href="mailto:kgiusti@gmail.com" target="_blank">kgiusti@gmail.com</a>> writes:<br>
<br>
> On Tue, Oct 9, 2018 at 11:56 AM Doug Hellmann <<a href="mailto:doug@doughellmann.com" target="_blank">doug@doughellmann.com</a>> wrote:<br>
><br>
>> Matthew Thode <<a href="mailto:prometheanfire@gentoo.org" target="_blank">prometheanfire@gentoo.org</a>> writes:<br>
>><br>
>> > On 18-10-09 11:12:30, Doug Hellmann wrote:<br>
>> >> Matthew Thode <<a href="mailto:prometheanfire@gentoo.org" target="_blank">prometheanfire@gentoo.org</a>> writes:<br>
>> >><br>
>> >> > several projects have had problems with the new release, some have<br>
>> ways<br>
>> >> > of working around it, and some do not.  I'm sending this just to raise<br>
>> >> > the issue and allow a place to discuss solutions.<br>
>> >> ><br>
>> >> > Currently there is a review proposed to blacklist 9.0.0, but if this<br>
>> is<br>
>> >> > going to still be an issue somehow in further releases we may need<br>
>> >> > another solution.<br>
>> >> ><br>
>> >> > <a href="https://review.openstack.org/#/c/608835/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/608835/</a><br>
>> >> ><br>
>> >> > --<br>
>> >> > Matthew Thode (prometheanfire)<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>
>> >> Do you have links to the failure logs or bug reports or something? If I<br>
>> >> wanted to help I wouldn't even know where to start.<br>
>> >><br>
>> ><br>
>> ><br>
>> <a href="http://logs.openstack.org/21/607521/2/check/cross-cinder-py35/e15722e/testr_results.html.gz" rel="noreferrer" target="_blank">http://logs.openstack.org/21/607521/2/check/cross-cinder-py35/e15722e/testr_results.html.gz</a><br>
>><br>
>> These failures look like we should add a proper API to oslo.messaging to<br>
>> set the notification and rpc backends for testing. The configuration<br>
>> options are *not* part of the API of the library.<br>
>><br>
>> There is already an oslo_messaging.conffixture module with a fixture<br>
>> class, but it looks like it defaults to rabbit. Maybe someone wants to<br>
>> propose a patch to make that a parameter to the constructor?<br>
>><br>
><br>
> oslo.messaging's conffixture uses whatever the config default for<br>
> transport_url is unless the test<br>
> specifically overrides it by setting the transport_url attribute.<br>
> The o.m. unit tests's base test class sets conffixture.transport_url to<br>
> "fake:/" to use the fake in memory driver.<br>
> That's the existing practice (I believe it's used like that outside of o.m.)<br>
<br>
OK, so it sounds like the fixture is relying on the configuration to be<br>
set up in advance, and that's the thing we need to change. We don't want<br>
users outside of the library to set up tests by using the configuration<br>
options, right?<br></blockquote><div><br></div><div>That's the intent of ConfFixture it seems - provide a wrapper API so tests don't have to monkey directly with the config.</div><div><br></div><div>How about this:</div><div><br></div><div>  <a href="https://review.openstack.org/609063">https://review.openstack.org/609063</a><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Doug<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">Ken Giusti  (<a href="mailto:kgiusti@gmail.com" target="_blank">kgiusti@gmail.com</a>)</div></div></div>