<div dir="ltr">From a release point of view this RFE LGTM. <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 29 mars 2021 à 15:23, Brian Rosmaita <<a href="mailto:rosmaita.fossdev@gmail.com">rosmaita.fossdev@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 3/29/21 8:50 AM, Brian Rosmaita wrote:<br>
> Hello Requirements Team,<br>
> <br>
> The Cinder team recently became aware of a potential data-loss bug [0] <br>
> that has been fixed in os-brick master [1] and backported to os-brick <br>
> stable/wallaby [2].  We've proposed a release of os-brick 4.4.0 from <br>
> stable/wallaby [3] and are petitioning for an RFE to include 4.4.0 in <br>
> the wallaby release.<br>
<br>
I had a quick discussion with Hervé and he prefers 4.3.1 as the version <br>
number for the new release.  Except for the version number, everything <br>
else in this email still applies.<br>
<br>
> <br>
> We have three jobs running tempest with os-brick source in master that <br>
> have passed with [1]: os-brick-src-devstack-plugin-ceph [4], <br>
> os-brick-src-tempest-lvm-lio-barbican [5],and os-brick-src-tempest-nfs <br>
> [6].  The difference between os-brick master (at the time the tests were <br>
> run) and stable/wallaby since the 4.3.0 tag is as follows:<br>
> <br>
> master:<br>
> d4205bd   3 days ago  iSCSI: Fix flushing after multipath cfg change <br>
> (Gorka Eguileor)<br>
> 0e63fe8  2 weeks ago  Merge "RBD: catch read exceptions prior to <br>
> modifying offset" (Zuul)<br>
> 28545c7 4 months ago  RBD: catch read exceptions prior to modifying <br>
> offset (Jon Bernard)<br>
> 99b2c60  2 weeks ago  Merge "Dropping explicit unicode literal" (Zuul)<br>
> 7cfdb76  6 weeks ago  Dropping explicit unicode literal (tushargite96)<br>
> 9afa1a0  3 weeks ago  Add Python3 xena unit tests (OpenStack Release Bot)<br>
> ab57392  3 weeks ago  Update master for stable/wallaby (OpenStack <br>
> Release Bot)<br>
> 91a1cca  3 weeks ago  (tag: 4.3.0) Merge "NVMeOF connector driver <br>
> connection information compatibility fix" (Zuul)<br>
> <br>
> stable/wallaby:<br>
> f86944b   3 days ago  Add release note prelude for os-brick 4.4.0 (Brian <br>
> Rosmaita)<br>
> c70d70b   3 days ago  iSCSI: Fix flushing after multipath cfg change <br>
> (Gorka Eguileor)<br>
> 6649b8d  3 weeks ago  Update TOX_CONSTRAINTS_FILE for stable/wallaby <br>
> (OpenStack Release Bot)<br>
> f3f93dc  3 weeks ago  Update .gitreview for stable/wallaby (OpenStack <br>
> Release Bot)<br>
> 91a1cca  3 weeks ago  (tag: 4.3.0) Merge "NVMeOF connector driver <br>
> connection information compatibility fix" (Zuul)<br>
> <br>
> This gives us very high confidence that the results of the tests run <br>
> against master also apply to stable/wallaby at f86944b.<br>
> <br>
> Thank you for considering this request.<br>
> <br>
> (I've included Nova here because the bug occurs when the configuration <br>
> option that enables multipath connections on a compute is changed while <br>
> volumes are attached, so if this RFE is approved, nova might want to <br>
> raise the minimum version of os-brick in wallaby to 4.4.0.)<br>
> <br>
> <br>
> [0] <a href="https://launchpad.net/bugs/1921381" rel="noreferrer" target="_blank">https://launchpad.net/bugs/1921381</a><br>
> [1] <a href="https://review.opendev.org/c/openstack/os-brick/+/782992" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/os-brick/+/782992</a><br>
> [2] <a href="https://review.opendev.org/c/openstack/os-brick/+/783207" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/os-brick/+/783207</a><br>
> [3] <a href="https://review.opendev.org/c/openstack/releases/+/783641" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/releases/+/783641</a><br>
> [4] <br>
> <a href="https://zuul.opendev.org/t/openstack/build/30a103668e4c4a8cb6f1ef907ef3edcb" rel="noreferrer" target="_blank">https://zuul.opendev.org/t/openstack/build/30a103668e4c4a8cb6f1ef907ef3edcb</a><br>
> [5] <br>
> <a href="https://zuul.opendev.org/t/openstack/build/bb11eef737d34c41bb4a52f8433850b0" rel="noreferrer" target="_blank">https://zuul.opendev.org/t/openstack/build/bb11eef737d34c41bb4a52f8433850b0</a><br>
> [6] <br>
> <a href="https://zuul.opendev.org/t/openstack/build/3ad3359ca712432d9ef4261d72c787fa" rel="noreferrer" target="_blank">https://zuul.opendev.org/t/openstack/build/3ad3359ca712432d9ef4261d72c787fa</a><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>