<div dir="ltr"><br><div class="gmail_quote"><div dir="ltr">On Tue, Oct 9, 2018 at 10:31 AM Ben Nemec <<a href="mailto:openstack@nemebean.com">openstack@nemebean.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On 10/9/18 9:06 AM, Lance Bragstad wrote:<br>
> Keystone is failing because it's missing a fix from oslo.messaging [0]. <br>
> That said, keystone is also relying on an internal implementation detail <br>
> in oslo.messaging by mocking it in tests [1]. The notification work has <br>
> been around in keystone for a *long* time, but it's apparent that we <br>
> should revisit these tests to make sure we aren't testing something that <br>
> is already tested by oslo.messaging if we're mocking internal <br>
> implementation details of a library.<br>
<br>
This is actually the same problem Cinder and Glance had, it's just being <br>
hidden because there is an exception handler in Keystone that buried the <br>
original exception message in log output. 9.0.1 will get Keystone <br>
working too.<br>
<br>
But mocking library internals is still naughty and you should stop that. :-P<br></blockquote><div><br></div><div>Agreed. I have a note to investigate and see if I can rip those bits out or rewrite them.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> <br>
> Regardless, blacklisting version 9.0.0 will work for keystone, but we <br>
> can work around it another way by either rewriting the tests to not care <br>
> about oslo.messaging specifics, or removing them if they're obsolete.<br>
> <br>
> [0] <a href="https://review.openstack.org/#/c/608196/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/608196/</a><br>
> [1] <br>
> <a href="https://git.openstack.org/cgit/openstack/keystone/tree/keystone/tests/unit/common/test_notifications.py#n1343" rel="noreferrer" target="_blank">https://git.openstack.org/cgit/openstack/keystone/tree/keystone/tests/unit/common/test_notifications.py#n1343</a><br>
> <br>
> On Mon, Oct 8, 2018 at 10:59 PM Matthew Thode <<a href="mailto:prometheanfire@gentoo.org" target="_blank">prometheanfire@gentoo.org</a> <br>
> <mailto:<a href="mailto:prometheanfire@gentoo.org" target="_blank">prometheanfire@gentoo.org</a>>> wrote:<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:<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://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>
> 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></div>