<div dir="ltr"><div><div>add Ironic team  in the loop<br><br></div>the revert patch got -1 from  ironic folks , more details  please see the comments in <span class="gmail-m_-6331875189824115667gmail-im"><a target="_blank" href="https://review.openstack.org/534736">https://review.openstack.org/<wbr>534736</a><br></span></div><span class="gmail-m_-6331875189824115667gmail-im">The possible solution is to figure out why  the change break Glance's unit test.  which side should be fixed.  </span><br><div><div><span class="gmail-m_-6331875189824115667gmail-im"></span><div><br><br><div><div class="gmail_extra"><br><div class="gmail_quote">2018-01-17 20:14 GMT+08:00 ChangBo Guo <span dir="ltr"><<a target="_blank" href="mailto:glongwave@gmail.com">glongwave@gmail.com</a>></span>:<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><div dir="ltr"><div>I dig a little.  It shows success when updating constraint to 2.21.2 [1] but  failure when updating constraint to 2.22.0 [2].   according to release information [3].<br></div><div>It means 2.21.1 works with glance test but  2.21.2 doesn't work well with glance. The only issue patch <span class="gmail-m_-6331875189824115667gmail-im">is <a target="_blank" href="https://github.com/openstack/oslo.serialization/commit/c1a7079c26d27a2e46cca26963d3d9aa040bdbe8">https://github.com/openstack/<wbr>oslo.serialization/commit/<wbr>c1a7079c26d27a2e46cca26963d3d9<wbr>aa040bdbe8</a>.<br>    <br></span></div><div><br>[1] <a target="_blank" href="https://review.openstack.org/514833">https://review.openstack.org/<wbr>514833</a><br>[2] <a target="_blank" href="https://review.openstack.org/#/c/525136">https://review.openstack.org/#<wbr>/c/525136</a><br>[3] <a target="_blank" href="https://github.com/openstack/releases/blob/master/deliverables/queens/oslo.serialization.yaml">https://github.com/openstack/<wbr>releases/blob/master/<wbr>deliverables/queens/oslo.<wbr>serialization.yaml</a><br><br><br></div><div>Actions:<br><br></div><div>Block  oslo.serialization  version  2.21.2,  2.22.0, 2. 23.0    in  <a target="_blank" href="https://review.openstack.org/534739">https://review.openstack.org/<wbr>534739</a><br></div><div>Revert <span class="gmail-m_-6331875189824115667gmail-im">c1a7079c26d27a2e46cca26963d3d9<wbr>aa040bdbe8 in <a target="_blank" href="https://review.openstack.org/534736">https://review.openstack.org/<wbr>534736</a><br></span><br><br><br></div></div><div class="gmail_extra"><div><div class="gmail-h5"><br><div class="gmail_quote">2018-01-16 23:35 GMT+08:00 Matthew Thode <span dir="ltr"><<a target="_blank" href="mailto:prometheanfire@gentoo.org">prometheanfire@gentoo.org</a>></span>:<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span>On 18-01-16 19:12:16, ChangBo Guo wrote:<br>
> What's the issue for Glance,  any bug link ?<br>
><br>
> 2018-01-16 0:12 GMT+08:00 Matthew Thode <<a target="_blank" href="mailto:prometheanfire@gentoo.org">prometheanfire@gentoo.org</a>>:<br>
><br>
> > On 18-01-13 00:41:28, Matthew Thode wrote:<br>
> > > <a target="_blank" rel="noreferrer" href="https://review.openstack.org/531788">https://review.openstack.org/5<wbr>31788</a> is the review we are seeing it in,<br>
> > > but 2.22.0 failed as well.<br>
> > ><br>
> > > I'm guessing it was introduced in either<br>
> > ><br>
> > > <a target="_blank" rel="noreferrer" href="https://github.com/openstack/oslo.serialization/commit/">https://github.com/openstack/o<wbr>slo.serialization/commit/</a><br>
> > c1a7079c26d27a2e46cca26963d3d9<wbr>aa040bdbe8<br>
> > > or<br>
> > > <a target="_blank" rel="noreferrer" href="https://github.com/openstack/oslo.serialization/commit/">https://github.com/openstack/o<wbr>slo.serialization/commit/</a><br>
> > cdb2f60d26e3b65b6370f87b2e9864<wbr>045651c117<br>
> ><br>
> > bamp<br>
> ><br>
<br>
</span>The best bug for this is<br>
<a target="_blank" rel="noreferrer" href="https://bugs.launchpad.net/oslo.serialization/+bug/1728368">https://bugs.launchpad.net/osl<wbr>o.serialization/+bug/1728368</a> and we are<br>
currently getting test fails in <a target="_blank" rel="noreferrer" href="https://review.openstack.org/531788">https://review.openstack.org/5<wbr>31788</a><br>
<span class="gmail-m_-6331875189824115667HOEnZb"><font color="#888888"><br>
--<br>
Matthew Thode (prometheanfire)<br>
</font></span></blockquote></div><br><br clear="all"><br></div></div><span class="gmail-">-- <br><div class="gmail-m_-6331875189824115667gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>ChangBo Guo(gcb)<br></div><div>Community Director @EasyStack<br></div></div></div></div></div>
</span></div>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>ChangBo Guo(gcb)<br></div><div>Community Director @EasyStack<br></div></div></div></div></div>
</div></div></div></div></div></div>