[cinder] new driver freeze & exceptions
Hello Argonauts, The new driver merge deadline passed at 20:00 UTC today. I'm extending the new driver merge deadline to Friday 28 January at 20:00 UTC for two drivers: 1. Lightbits: It has both a cinder and os-brick patch, and I want the team to have more time to look at the os-brick patch. I think the driver patch is close to ready; the developers have been quick to respond to comments and make revisions. Also, the third-party CI is functioning and responding on patches. cinder: https://review.opendev.org/c/openstack/cinder/+/821602 os-brick: https://review.opendev.org/c/openstack/os-brick/+/821603 2. NEC Storage V Series: the driver patch has one +2 and the third party CI is functioning and responding on patches; I don't see any reason to make this one wait for the Z release. https://review.opendev.org/c/openstack/cinder/+/815614 Cinder core reviewers: please continue to review the above patches. With respect to the other proposed drivers: - Pure Storage NVMe-RoCE: vendor decided to hold it for Z - Dell EMC PowerStore NFS driver: vendor decided to hold it for Z - HPE XP Storage FC & iSCSI: the code is very straightforward, but the CI is not ready yet, so this will have to wait for Z - YADRO Tatlin.UNIFIED driver: initial patch arrived yesterday (and has a -1 from Zuul); the CI appears to be running, but the cinder team needs to move on to reviewing os-brick and Yoga feature patches, so this will have to wait for Z also cheers, brian
Update: The NEC Storage V Series driver has merged. The Lightbits driver has three +2s. The Lightbits os-brick connector it depends on has not yet merged, but has been actively revised to address comments. I am extending the new driver freeze exception until 20:00 Wednesday 2 February. On 1/21/22 5:25 PM, Brian Rosmaita wrote:
Hello Argonauts,
The new driver merge deadline passed at 20:00 UTC today.
I'm extending the new driver merge deadline to Friday 28 January at 20:00 UTC for two drivers:
1. Lightbits: It has both a cinder and os-brick patch, and I want the team to have more time to look at the os-brick patch. I think the driver patch is close to ready; the developers have been quick to respond to comments and make revisions. Also, the third-party CI is functioning and responding on patches. cinder: https://review.opendev.org/c/openstack/cinder/+/821602 os-brick: https://review.opendev.org/c/openstack/os-brick/+/821603
2. NEC Storage V Series: the driver patch has one +2 and the third party CI is functioning and responding on patches; I don't see any reason to make this one wait for the Z release. https://review.opendev.org/c/openstack/cinder/+/815614
Cinder core reviewers: please continue to review the above patches.
With respect to the other proposed drivers: - Pure Storage NVMe-RoCE: vendor decided to hold it for Z - Dell EMC PowerStore NFS driver: vendor decided to hold it for Z - HPE XP Storage FC & iSCSI: the code is very straightforward, but the CI is not ready yet, so this will have to wait for Z - YADRO Tatlin.UNIFIED driver: initial patch arrived yesterday (and has a -1 from Zuul); the CI appears to be running, but the cinder team needs to move on to reviewing os-brick and Yoga feature patches, so this will have to wait for Z also
cheers, brian
On 28/01, Brian Rosmaita wrote:
Update:
The NEC Storage V Series driver has merged.
The Lightbits driver has three +2s. The Lightbits os-brick connector it depends on has not yet merged, but has been actively revised to address comments. I am extending the new driver freeze exception until 20:00 Wednesday 2 February.
Update: The Lightbits cinder driver and os-brick connector patches have merged withing the freeze exception timeframe.
On 1/21/22 5:25 PM, Brian Rosmaita wrote:
Hello Argonauts,
The new driver merge deadline passed at 20:00 UTC today.
I'm extending the new driver merge deadline to Friday 28 January at 20:00 UTC for two drivers:
1. Lightbits: It has both a cinder and os-brick patch, and I want the team to have more time to look at the os-brick patch. I think the driver patch is close to ready; the developers have been quick to respond to comments and make revisions. Also, the third-party CI is functioning and responding on patches. cinder: https://review.opendev.org/c/openstack/cinder/+/821602 os-brick: https://review.opendev.org/c/openstack/os-brick/+/821603
2. NEC Storage V Series: the driver patch has one +2 and the third party CI is functioning and responding on patches; I don't see any reason to make this one wait for the Z release. https://review.opendev.org/c/openstack/cinder/+/815614
Cinder core reviewers: please continue to review the above patches.
With respect to the other proposed drivers: - Pure Storage NVMe-RoCE: vendor decided to hold it for Z - Dell EMC PowerStore NFS driver: vendor decided to hold it for Z - HPE XP Storage FC & iSCSI: the code is very straightforward, but the CI is not ready yet, so this will have to wait for Z - YADRO Tatlin.UNIFIED driver: initial patch arrived yesterday (and has a -1 from Zuul); the CI appears to be running, but the cinder team needs to move on to reviewing os-brick and Yoga feature patches, so this will have to wait for Z also
cheers, brian
On Thu, 2022-02-03 at 11:16 +0100, Gorka Eguileor wrote:
On 28/01, Brian Rosmaita wrote:
Update:
The NEC Storage V Series driver has merged.
The Lightbits driver has three +2s. The Lightbits os-brick connector it depends on has not yet merged, but has been actively revised to address comments. I am extending the new driver freeze exception until 20:00 Wednesday 2 February.
Update:
The Lightbits cinder driver and os-brick connector patches have merged withing the freeze exception timeframe. we were just talking about this on the nova channel. for it to be usable in nova an os-brick release needs to be done https://github.com/openstack/os-brick/compare/5.1.0...master
so we need a 5.2.0 release based on b29f1529228bb7a16c2373f7bbf1cf862438aa23 i guess https://github.com/openstack/releases/blob/master/deliverables/yoga/os-brick... then that will allow this to be useable in the zuul jobs ectra.
On 1/21/22 5:25 PM, Brian Rosmaita wrote:
Hello Argonauts,
The new driver merge deadline passed at 20:00 UTC today.
I'm extending the new driver merge deadline to Friday 28 January at 20:00 UTC for two drivers:
1. Lightbits: It has both a cinder and os-brick patch, and I want the team to have more time to look at the os-brick patch. I think the driver patch is close to ready; the developers have been quick to respond to comments and make revisions. Also, the third-party CI is functioning and responding on patches. cinder: https://review.opendev.org/c/openstack/cinder/+/821602 os-brick: https://review.opendev.org/c/openstack/os-brick/+/821603
2. NEC Storage V Series: the driver patch has one +2 and the third party CI is functioning and responding on patches; I don't see any reason to make this one wait for the Z release. https://review.opendev.org/c/openstack/cinder/+/815614
Cinder core reviewers: please continue to review the above patches.
With respect to the other proposed drivers: - Pure Storage NVMe-RoCE: vendor decided to hold it for Z - Dell EMC PowerStore NFS driver: vendor decided to hold it for Z - HPE XP Storage FC & iSCSI: the code is very straightforward, but the CI is not ready yet, so this will have to wait for Z - YADRO Tatlin.UNIFIED driver: initial patch arrived yesterday (and has a -1 from Zuul); the CI appears to be running, but the cinder team needs to move on to reviewing os-brick and Yoga feature patches, so this will have to wait for Z also
cheers, brian
On 2/3/22 10:06 AM, Sean Mooney wrote:
On Thu, 2022-02-03 at 11:16 +0100, Gorka Eguileor wrote:
On 28/01, Brian Rosmaita wrote:
Update:
The NEC Storage V Series driver has merged.
The Lightbits driver has three +2s. The Lightbits os-brick connector it depends on has not yet merged, but has been actively revised to address comments. I am extending the new driver freeze exception until 20:00 Wednesday 2 February.
Update:
The Lightbits cinder driver and os-brick connector patches have merged withing the freeze exception timeframe. we were just talking about this on the nova channel. for it to be usable in nova an os-brick release needs to be done https://github.com/openstack/os-brick/compare/5.1.0...master
so we need a 5.2.0 release based on b29f1529228bb7a16c2373f7bbf1cf862438aa23 i guess https://github.com/openstack/releases/blob/master/deliverables/yoga/os-brick...
then that will allow this to be useable in the zuul jobs ectra.
We're planning to release yoga os-brick next Thursday (10 Feb).
On 1/21/22 5:25 PM, Brian Rosmaita wrote:
Hello Argonauts,
The new driver merge deadline passed at 20:00 UTC today.
I'm extending the new driver merge deadline to Friday 28 January at 20:00 UTC for two drivers:
1. Lightbits: It has both a cinder and os-brick patch, and I want the team to have more time to look at the os-brick patch. I think the driver patch is close to ready; the developers have been quick to respond to comments and make revisions. Also, the third-party CI is functioning and responding on patches. cinder: https://review.opendev.org/c/openstack/cinder/+/821602 os-brick: https://review.opendev.org/c/openstack/os-brick/+/821603
2. NEC Storage V Series: the driver patch has one +2 and the third party CI is functioning and responding on patches; I don't see any reason to make this one wait for the Z release. https://review.opendev.org/c/openstack/cinder/+/815614
Cinder core reviewers: please continue to review the above patches.
With respect to the other proposed drivers: - Pure Storage NVMe-RoCE: vendor decided to hold it for Z - Dell EMC PowerStore NFS driver: vendor decided to hold it for Z - HPE XP Storage FC & iSCSI: the code is very straightforward, but the CI is not ready yet, so this will have to wait for Z - YADRO Tatlin.UNIFIED driver: initial patch arrived yesterday (and has a -1 from Zuul); the CI appears to be running, but the cinder team needs to move on to reviewing os-brick and Yoga feature patches, so this will have to wait for Z also
cheers, brian
participants (3)
-
Brian Rosmaita
-
Gorka Eguileor
-
Sean Mooney