<div dir="ltr">Hi Cinder team,<div><br></div><div>The Kioxia Kumoscale NVMeOF driver is ready for reviews.</div><div>Gate tests are passing and "KIOXIA CI" is leaving comments on review threads. Here is an example of last response it posted: <a href="https://review.opendev.org/c/openstack/cinder/+/765304/2#message-6ca11d3b3c8a28666c74c65aeece090acfe26f9b">https://review.opendev.org/c/openstack/cinder/+/765304/2#message-6ca11d3b3c8a28666c74c65aeece090acfe26f9b</a></div><div>All changes that the CI responded to can be found here: <a href="http://104.254.65.37/">http://104.254.65.37/</a> (each will also have a gerrit_event.json which will have a ['change']['url'] with a link to the review page)</div><div><br></div><div>Part of this contribution is a major expansion to the nvmeof connector in os-brick.</div><div>This was initially discussed in the PTG meetings, and mentioned in some follow up IRC meetings.</div><div>The main change is adding (optional) support for mdraid replication of nvmeof volumes. Target connection sharing is also implemented (removing the need for a separate nvmeof target per volume.) Additionally, a concurrency / race condition issue in the existing nvmeof connector (nvmeof.py line 120) is resolved as part of these changes.</div><div><br></div><div>Though this change is a big rework, it is in line with the spec, which had an in depth review and was approved:</div><div><a href="https://review.opendev.org/c/openstack/cinder-specs/+/766730">https://review.opendev.org/c/openstack/cinder-specs/+/766730</a></div><div><br></div><div>We are dedicated to addressing possible regressions, however, following a review of existing CIs using or testing the nvmeof connector, none were found to be using it. All suspected CIs seem to pass or be using a different connector. This is to the best of our knowledge, we suspect that Kioxia may be the first CI to be testing with NVMeOF protocol and driver. If we missed anything, please let us know with a review comment.</div><div><br></div><div>Kioxia Kumoscale driver review:</div><div><a href="https://review.opendev.org/c/openstack/cinder/+/768574">https://review.opendev.org/c/openstack/cinder/+/768574</a><br></div><div><br></div><div>NVMeOF connector review:</div><div><a href="https://review.opendev.org/c/openstack/os-brick/+/768575">https://review.opendev.org/c/openstack/os-brick/+/768575</a><br></div><div><br></div><div>Thank you!</div><div><br></div><div>Zohar</div></div>