<div dir="ltr">Thanks Gorka for the detailed description.<div>The LVM+nvmet changes certainly are a good candidate for FFE and adds value to our testing framework.</div><div>Hence, I would like to grant FFE for the LVM+nvmet patches.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 20, 2023 at 3:38 PM Gorka Eguileor <<a href="mailto:geguileo@redhat.com">geguileo@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 20/02, Rajat Dhasmana wrote:<br>
> Hello Argonauts,<br>
><br>
> We are past the feature freeze deadline i.e. 17 Feb, 2023[1]. If you have a<br>
> feature which you would like to get it into the 2023.1 (Antelope) release,<br>
> please request a feature freeze exception on the OpenStack discuss mailing<br>
> list with the following subject: *[cinder] Request for Feature Freeze<br>
> Exception*<br>
> Include the patch details, reason for missing the feature freeze<br>
> deadline and current status in the email.<br>
><br>
> The feature freeze exception will be granted for 1 week i.e. till 24th Feb,<br>
> 2023<br>
> and no features will be accepted after that.<br>
><br>
> Following are the targeted features and their current status:<br>
><br>
> *Hitachi: support data deduplication and compression*<br>
> Patch: <a href="https://review.opendev.org/c/openstack/cinder/+/850831" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/cinder/+/850831</a><br>
> Status: Fix merge conflict and add unit test<br>
><br>
><br>
> *Add iSCSI export support to the StorPool driver*<br>
> Patch: <a href="https://review.opendev.org/c/openstack/cinder/+/847536" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/cinder/+/847536</a><br>
> Status: Has multiple negative votes with feedback that needs to be addressed<br>
><br>
><br>
> *[SVf] Adding support of clone operation for volumegroup*<br>
> Patch: <a href="https://review.opendev.org/c/openstack/cinder/+/871257" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/cinder/+/871257</a><br>
> Status: Has negative votes with feedback that needs to be addressed<br>
><br>
> *LVM NVMe Target*<br>
> Topic: <a href="https://review.opendev.org/q/topic:lvm-nvmet-shared" rel="noreferrer" target="_blank">https://review.opendev.org/q/topic:lvm-nvmet-shared</a><br>
> Status: Need confirmation that the patches are in a reviewable state<br>
<br>
Hi Rajat,<br>
<br>
Thanks for bringing this up.<br>
<br>
I would like to request a FFE for the LVM+nvmet patches.<br>
<br>
They had a small bug that was detected by the WIP tempest CI job which<br>
got fixed a while back, but I've been fighting the normal CI job with<br>
spurious errors since then.<br>
<br>
Now they all have +1 from CI and are ready to merge.<br>
<br>
These patches add new features to the LVM driver over NVMe-oF, like<br>
native multipathing and sharing targets. Thanks to those any developer<br>
will be able to work on the NVMe-oF os-brick code without needing a high<br>
end storage array, and we'll be able to have CI jobs that can exercise<br>
those code paths as well.<br>
<br>
Cheers,<br>
Gorka.<br>
<br>
><br>
> [1] <a href="https://releases.openstack.org/antelope/schedule.html#a-ff" rel="noreferrer" target="_blank">https://releases.openstack.org/antelope/schedule.html#a-ff</a><br>
><br>
> Thanks<br>
> Rajat Dhasmana<br>
<br>
</blockquote></div>