<div dir="ltr"><div>Hello Balazs,</div><div><br></div><div>Now that the os-brick changes on nova are merged do you plan to propose a RC2?<br></div><div><br></div><div><a href="https://review.opendev.org/c/openstack/nova/+/783674" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/nova/+/783674</a></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 29 mars 2021 à 17:43, Balazs Gibizer <balazs.gibizer@est.tech> 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"><br>
<br>
On Mon, Mar 29, 2021 at 16:05, Balazs Gibizer <balazs.gibizer@est.tech> <br>
wrote:<br>
> <br>
> <br>
> On Mon, Mar 29, 2021 at 08:50, Brian Rosmaita <br>
> <<a href="mailto:rosmaita.fossdev@gmail.com" target="_blank">rosmaita.fossdev@gmail.com</a>> wrote:<br>
>> Hello Requirements Team,<br>
>> <br>
>> The Cinder team recently became aware of a potential data-loss bug <br>
>>  [0] that has been fixed in os-brick master [1] and backported to <br>
>>  os-brick stable/wallaby [2].  We've proposed a release of os-brick <br>
>>  4.4.0 from stable/wallaby [3] and are petitioning for an RFE to <br>
>>  include 4.4.0 in the wallaby release.<br>
>> <br>
>> We have three jobs running tempest with os-brick source in master <br>
>>  that have passed with [1]: os-brick-src-devstack-plugin-ceph [4], <br>
>>  os-brick-src-tempest-lvm-lio-barbican [5],and <br>
>>  os-brick-src-tempest-nfs [6].  The difference between os-brick <br>
>> master  (at the time the tests were run) and stable/wallaby since <br>
>> 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" <br>
>> (Zuul)<br>
>> 7cfdb76  6 weeks ago  Dropping explicit unicode literal <br>
>> (tushargite96)<br>
>> 9afa1a0  3 weeks ago  Add Python3 xena unit tests (OpenStack Release <br>
>>  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 <br>
>>  (Brian 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 <br>
>> (OpenStack  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 <br>
>>  configuration option that enables multipath connections on a <br>
>> compute  is changed while volumes are attached, so if this RFE is <br>
>> approved,  nova might want to raise the minimum version of os-brick <br>
>> in wallaby  to 4.4.0.)<br>
>> <br>
> <br>
> Thanks for the heads up. After the new os-brick version is released I <br>
> will prepare a version bump patch in nova on master and <br>
> stable/wallaby. This also means that nova will release an RC2.<br>
<br>
I've proposed the nova patch on master to bump min os-brick to 4.3.1 in <br>
nova[1]<br>
<br>
[1] <a href="https://review.opendev.org/c/openstack/nova/+/783674" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/nova/+/783674</a><br>
<br>
> <br>
> Cheers,<br>
> gibi<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>
> <br>
> <br>
<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>