<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 9, 2018 at 10:56 AM 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">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 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 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>
>> > 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>
>> 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>
> <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>
> <a href="http://logs.openstack.org/21/607521/2/check/cross-glance-py35/e2161d7/testr_results.html.gz" rel="noreferrer" target="_blank">http://logs.openstack.org/21/607521/2/check/cross-glance-py35/e2161d7/testr_results.html.gz</a><br>
<br>
These failures should be fixed by releasing the patch that Mehdi<br>
provided that ensures there is a valid default transport configured.<br>
<br>
> <a href="http://logs.openstack.org/21/607521/2/check/cross-keystone-py35/908a1c2/testr_results.html.gz" rel="noreferrer" target="_blank">http://logs.openstack.org/21/607521/2/check/cross-keystone-py35/908a1c2/testr_results.html.gz</a><br>
<br>
Lance has already described these as mocking implementation details of<br>
the library. I expect we'll need someone with keystone experience to<br>
work out what the best solution is to do there.<br></blockquote><div><br></div><div>So - I think it's apparent there are two things to do to fix this for keystone, which could be true for other projects as well.<div><br></div><div>To recap, keystone has tests to assert the plumbing to send a notification was called, or not called, depending on configuration options in keystone (we allow operators to opt out of noisy notifications, like authenticate).</div><div><br></div><div>As noted earlier, we shouldn't be making these assertions using an internal method of oslo.messaging. I have a patch up to refactor that to use the public API instead [0]. Even with that fix [0], the tests mentioned by Matt still fail because there isn't a sane default. I have a separate patch up to make keystone's tests work by supplying the default introduced in version 9.0.1 [1], overriding the configuration option for transport_url. This got a bit hairy in a circular-dependency kind of way because get_notification_transport() [2] is what registers the default options, which is broken. I have a patch to keystone [3] showing how I worked around this, which might not be needed if we allow the constructor to accept an override for transport_url.</div><div><br></div><div>[0] <a href="https://review.openstack.org/#/c/609072/">https://review.openstack.org/#/c/609072/</a></div><div>[1] <a href="https://review.openstack.org/#/c/608196/3/oslo_messaging/transport.py">https://review.openstack.org/#/c/608196/3/oslo_messaging/transport.py</a></div><div>[2] <a href="https://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo_messaging/notify/notifier.py#n167">https://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo_messaging/notify/notifier.py#n167</a></div><div>[3] <a href="https://review.openstack.org/#/c/609106/">https://review.openstack.org/#/c/609106/</a></div></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>
><br>
> -- <br>
> Matthew Thode (prometheanfire)<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>
__________________________________________________________________________<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></div></div></div></div></div></div>