<div dir="ltr"><div>You're welcome :)<br></div><div><br></div><div>Thank you for your commitment and for your investment around this topic!</div><div><br></div><div>I approved your patch and I'll ask another review to my comrades.<br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mer. 17 mars 2021 à 06:42, Goutham Pacha Ravi <<a href="mailto:gouthampravi@gmail.com">gouthampravi@gmail.com</a>> a écrit :<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 Tue, Mar 16, 2021 at 1:30 AM Herve Beraud <<a href="mailto:hberaud@redhat.com" target="_blank">hberaud@redhat.com</a>> wrote:<br>
><br>
> As already discussed with you on IRC if these changes are merged ASAP and if Matthew is Ok from a requirements perspective then I'm ok to proceed.<br>
><br>
> A version of python-manilaclient client have been frozen yesterday so now we can take advantage of the RFE mechanismes to land these changes that face a flaky CI.<br>
<br>
Thank you so much for your patience, Hervé. We found a bug on the<br>
server side that was causing the client CI to be flaky.<br>
It's fixed now, and all changes have merged.<br>
<br>
I requested a release: <a href="https://review.opendev.org/c/openstack/releases/+/780999" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/releases/+/780999</a><br>
<br>
<br>
><br>
> Le lun. 15 mars 2021 à 23:59, Goutham Pacha Ravi <<a href="mailto:gouthampravi@gmail.com" target="_blank">gouthampravi@gmail.com</a>> a écrit :<br>
>><br>
>> On Mon, Mar 15, 2021 at 3:47 PM Matthew Thode <<a href="mailto:mthode@mthode.org" target="_blank">mthode@mthode.org</a>> wrote:<br>
>> ><br>
>> > On 21-03-15 14:48:12, Goutham Pacha Ravi wrote:<br>
>> > > Hello,<br>
>> > ><br>
>> > > We were tracking one last feature inclusion into python-manilaclient<br>
>> > > for the wallaby release:<br>
>> > > <a href="https://review.opendev.org/c/openstack/python-manilaclient/+/775020" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/python-manilaclient/+/775020</a><br>
>> > ><br>
>> > > The change does not introduce any new requirements, but adds new SDK<br>
>> > > methods and shell implementations for end users to harness a feature<br>
>> > > addition in manila to be able to update their security services.<br>
>> > ><br>
>> > > The release team was very gracious in allowing us to wrap up code<br>
>> > > reviews pertaining to the above change over the weekend; however, the<br>
>> > > gate was uncooperative this morning by pointing us to lingering<br>
>> > > sporadic failures that are a combination of bad test case logic as<br>
>> > > well as occasional test environment setup issues. (Zuul is awesome,<br>
>> > > but it has a way to call out our procrastinations)<br>
>> > ><br>
>> > > No projects that depend on python-manilaclient need this change in<br>
>> > > their current form because it is net new feature functionality<br>
>> > > (existing SDK/CLI functionality is unaltered). However, I thought i<br>
>> > > should check here before requesting a re-release of<br>
>> > > python-manilaclient under the Wallaby cycle.<br>
>> > ><br>
>> > > Thanks,<br>
>> > > Goutham<br>
>> > ><br>
>> ><br>
>> > Fine by me from a requirements perspective.  Thanks.  I imagine this<br>
>> > will be a version bump on top of <a href="https://review.opendev.org/780667" rel="noreferrer" target="_blank">https://review.opendev.org/780667</a> then?<br>
>><br>
>> Yes. I worked with Hervé Beraud to get that release in, and pursue<br>
>> this FFE/RFE in parallel to include<br>
>> <a href="https://review.opendev.org/c/openstack/python-manilaclient/+/775020" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/python-manilaclient/+/775020</a><br>
>> into 2.6.0.<br>
>><br>
>><br>
>> ><br>
>> > --<br>
>> > Matthew Thode<br>
>><br>
><br>
><br>
> --<br>
> Hervé Beraud<br>
> Senior Software Engineer at Red Hat<br>
> irc: hberaud<br>
> <a href="https://github.com/4383/" rel="noreferrer" target="_blank">https://github.com/4383/</a><br>
> <a href="https://twitter.com/4383hberaud" rel="noreferrer" target="_blank">https://twitter.com/4383hberaud</a><br>
> -----BEGIN PGP SIGNATURE-----<br>
><br>
> wsFcBAABCAAQBQJb4AwCCRAHwXRBNkGNegAALSkQAHrotwCiL3VMwDR0vcja10Q+<br>
> Kf31yCutl5bAlS7tOKpPQ9XN4oC0ZSThyNNFVrg8ail0SczHXsC4rOrsPblgGRN+<br>
> RQLoCm2eO1AkB0ubCYLaq0XqSaO+Uk81QxAPkyPCEGT6SRxXr2lhADK0T86kBnMP<br>
> F8RvGolu3EFjlqCVgeOZaR51PqwUlEhZXZuuNKrWZXg/oRiY4811GmnvzmUhgK5G<br>
> 5+f8mUg74hfjDbR2VhjTeaLKp0PhskjOIKY3vqHXofLuaqFDD+WrAy/NgDGvN22g<br>
> glGfj472T3xyHnUzM8ILgAGSghfzZF5Skj2qEeci9cB6K3Hm3osj+PbvfsXE/7Kw<br>
> m/xtm+FjnaywZEv54uCmVIzQsRIm1qJscu20Qw6Q0UiPpDFqD7O6tWSRKdX11UTZ<br>
> hwVQTMh9AKQDBEh2W9nnFi9kzSSNu4OQ1dRMcYHWfd9BEkccezxHwUM4Xyov5Fe0<br>
> qnbfzTB1tYkjU78loMWFaLa00ftSxP/DtQ//iYVyfVNfcCwfDszXLOqlkvGmY1/Y<br>
> F1ON0ONekDZkGJsDoS6QdiUSn8RZ2mHArGEWMV00EV5DCIbCXRvywXV43ckx8Z+3<br>
> B8qUJhBqJ8RS2F+vTs3DTaXqcktgJ4UkhYC2c1gImcPRyGrK9VY0sCT+1iA+wp/O<br>
> v6rDpkeNksZ9fFSyoY2o<br>
> =ECSj<br>
> -----END PGP SIGNATURE-----<br>
><br>
<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Hervé Beraud</div><div>Senior Software Engineer at Red Hat</div><div>irc: hberaud</div><div><a href="https://github.com/4383/" target="_blank">https://github.com/4383/</a></div><div><a href="https://twitter.com/4383hberaud" target="_blank">https://twitter.com/4383hberaud</a><br></div><div>-----BEGIN PGP SIGNATURE-----<br><br>wsFcBAABCAAQBQJb4AwCCRAHwXRBNkGNegAALSkQAHrotwCiL3VMwDR0vcja10Q+<br>Kf31yCutl5bAlS7tOKpPQ9XN4oC0ZSThyNNFVrg8ail0SczHXsC4rOrsPblgGRN+<br>RQLoCm2eO1AkB0ubCYLaq0XqSaO+Uk81QxAPkyPCEGT6SRxXr2lhADK0T86kBnMP<br>F8RvGolu3EFjlqCVgeOZaR51PqwUlEhZXZuuNKrWZXg/oRiY4811GmnvzmUhgK5G<br>5+f8mUg74hfjDbR2VhjTeaLKp0PhskjOIKY3vqHXofLuaqFDD+WrAy/NgDGvN22g<br>glGfj472T3xyHnUzM8ILgAGSghfzZF5Skj2qEeci9cB6K3Hm3osj+PbvfsXE/7Kw<br>m/xtm+FjnaywZEv54uCmVIzQsRIm1qJscu20Qw6Q0UiPpDFqD7O6tWSRKdX11UTZ<br>hwVQTMh9AKQDBEh2W9nnFi9kzSSNu4OQ1dRMcYHWfd9BEkccezxHwUM4Xyov5Fe0<br>qnbfzTB1tYkjU78loMWFaLa00ftSxP/DtQ//iYVyfVNfcCwfDszXLOqlkvGmY1/Y<br>F1ON0ONekDZkGJsDoS6QdiUSn8RZ2mHArGEWMV00EV5DCIbCXRvywXV43ckx8Z+3<br>B8qUJhBqJ8RS2F+vTs3DTaXqcktgJ4UkhYC2c1gImcPRyGrK9VY0sCT+1iA+wp/O<br>v6rDpkeNksZ9fFSyoY2o<br>=ECSj<br>-----END PGP SIGNATURE-----<br><br></div></div></div></div></div></div></div></div></div></div></div></div></div>