<div dir="ltr"><div dir="ltr"><div>I hope the issue with Octavia was resolved by <a href="https://review.opendev.org/c/openstack/octavia/+/801860">https://review.opendev.org/c/openstack/octavia/+/801860</a> .</div><div>(Thank you Michael for merging this !)</div><div><br></div><div>AFAIK Mistral is still dependent on the old arguments and we need</div><div> <a href="https://review.opendev.org/c/openstack/mistral/+/801853">https://review.opendev.org/c/openstack/mistral/+/801853</a></div><div>to fix it. It'd be nice if somebody from Mistral team can merge this</div><div>(and the series of clean up patches)<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 1, 2022 at 3:47 AM Sean Mooney <<a href="mailto:smooney@redhat.com">smooney@redhat.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">On Mon, 2022-02-28 at 18:37 +0000, Sean Mooney wrote:<br>
> On Mon, 2022-02-28 at 10:09 -0800, Michael Johnson wrote:<br>
> > This is a legit API change in oslo.context (Thus the major version<br>
> > bump). It's been a long time coming too.<br>
> we are currently pass m3 in the rc period.<br>
> i dont know if we can do a breaking change like this in yoga at this point depending on how invaise it is.<br>
> we might need to hold this until after RC1 is out ideally this type of change would happen very erarly in the cycle before the ptg or m1<br>
> not at the very end on the nova side there is only one failing unit test<br>
> <a href="https://6b96a36d22f450141095-9e4eb9f05513870f0e5456f29474f6b2.ssl.cf5.rackcdn.com/829599/1/check/cross-nova-py38/60113ea/testr_results.html" rel="noreferrer" target="_blank">https://6b96a36d22f450141095-9e4eb9f05513870f0e5456f29474f6b2.ssl.cf5.rackcdn.com/829599/1/check/cross-nova-py38/60113ea/testr_results.html</a><br>
> but i need to also run the func tests to see if there is other impact<br>
> tempest-full-py3 passed so nova iteslf is likely fine.<br>
> <br>
> ill try running it locally and see if that is the only impact or not.<br>
<br>
gmann beat me too it.<br>
<a href="https://review.opendev.org/c/openstack/nova/+/831244" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/nova/+/831244</a><br>
so ya it is apprently just the one unit test so i dont think this should be an issue for nova<br>
<br>
> <br>
> > <br>
> > I will get started on the required code changes to Designate. Maybe<br>
> > Octavia as well if someone else doesn't get to it before I do.<br>
> > <br>
> > Michael<br>
> > <br>
> > On Mon, Feb 28, 2022 at 9:04 AM Matthew Thode <<a href="mailto:mthode@mthode.org" target="_blank">mthode@mthode.org</a>> wrote:<br>
> > > <br>
> > > The update to see what's breaking is here<br>
> > > <a href="https://review.opendev.org/829599" rel="noreferrer" target="_blank">https://review.opendev.org/829599</a><br>
> > > <br>
> > > Feel free to make that patch depend on your fix to test (with rechecks)<br>
> > > <br>
> > > --<br>
> > > Matthew Thode<br>
> > <br>
> <br>
<br>
<br>
</blockquote></div></div>