| harshailani | Hello reviewers, Please provide your inputs on the bugfixes for IBM SVf driver: | 08:13 |
|---|---|---|
| harshailani | https://review.opendev.org/c/openstack/cinder/+/850310 | 08:13 |
| harshailani | https://review.opendev.org/c/openstack/cinder/+/844140 | 08:13 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: support data deduplication and compression https://review.opendev.org/c/openstack/cinder/+/850831 | 09:57 |
| whoami-rajat | hi rosmaita , when you come online can you check my comment here? https://review.opendev.org/c/openstack/cinder/+/847864 | 10:16 |
| whoami-rajat | geguileo, hey, can you take a look at this? it is related to the new nvmeof format https://review.opendev.org/c/openstack/cinder/+/852261 | 10:20 |
| geguileo | whoami-rajat: I didn't see the ping notification, looking now | 10:45 |
| geguileo | whoami-rajat: new code looks good, the patch is such missing a unit test | 10:53 |
| whoami-rajat | geguileo, ack, thanks for taking a look | 10:55 |
| opendevreview | Merged openstack/cinder stable/ussuri: PowerMax Driver - Allow for case mismatch in SGs https://review.opendev.org/c/openstack/cinder/+/821749 | 11:23 |
| opendevreview | Merged openstack/cinder stable/train: PowerMax Driver - Fix for snapVx generations https://review.opendev.org/c/openstack/cinder/+/816705 | 11:23 |
| *** dviroel|out is now known as dviroel | 11:28 | |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: PowerMax Driver - Fix for renaming GVG https://review.opendev.org/c/openstack/cinder/+/801354 | 11:31 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: PowerMax Driver - Fix for renaming GVG https://review.opendev.org/c/openstack/cinder/+/801354 | 11:33 |
| opendevreview | Jan Hartkopf proposed openstack/cinder-tempest-plugin master: Add test for Ceph backup snapshot cleanup option https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/855876 | 12:17 |
| *** tosky_ is now known as tosky | 12:23 | |
| opendevreview | Jan Hartkopf proposed openstack/cinder-tempest-plugin master: Add test for Ceph backup snapshot cleanup option https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/855876 | 12:29 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Support retype to another pool for VSP driver https://review.opendev.org/c/openstack/cinder/+/842905 | 13:06 |
| yuval_ | hey | 14:40 |
| yuval_ | https://review.opendev.org/c/openstack/cinder/+/839768 | 14:40 |
| yuval_ | this has a +1 from zuul +1 from a reviewer and passed our third party ci | 14:40 |
| yuval_ | can a core reviewer check it? | 14:41 |
| opendevreview | Jan Hartkopf proposed openstack/cinder-tempest-plugin master: Add test for Ceph backup snapshot cleanup option https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/855876 | 15:11 |
| *** dviroel is now known as dviroel|lunch | 15:32 | |
| opendevreview | Danila Balagansky proposed openstack/cinder master: Resize volume after creating it from image-volume cache with NFS driver https://review.opendev.org/c/openstack/cinder/+/855964 | 16:18 |
| *** dviroel|lunch is now known as dviroel | 16:41 | |
| whoami-rajat | simondodsley, hey, I noticed the Pure NVMe driver isn't mentioned in our support matrix, can you do a followup to add it there? | 20:32 |
| simondodsley | whoami-rajat strange. It was in the patch. Let me check | 20:34 |
| whoami-rajat | simondodsley, yeah it's there, sorry my bad was looking at an old guide | 20:36 |
| whoami-rajat | simondodsley, what is interesting is we mention Pure mentions all of iSCSI, FC and NVMe drivers together and it mentions it supports replication | 20:37 |
| whoami-rajat | however IIUC, replication for nvme is enabled by this change https://review.opendev.org/c/openstack/cinder/+/855058 | 20:37 |
| simondodsley | Yeah. Luckily as the patch and the main NVMe driver are both only for Zed it doesn't matter now | 20:38 |
| simondodsley | whoami-rajat the driver did support replication. Async. The patch is for sync repl | 20:41 |
| whoami-rajat | ack, so partial support but now we support both types, just we should be careful what we mention as supported else people complain on ML saying something is mentioned as supported but doesn't work :) | 20:43 |
| simondodsley | whoami-rajat There are lots of drivers that say repl support but are only async. Maybe we should have a new line for sync repl support | 21:06 |
| *** dviroel is now known as dviroel|out | 21:29 | |
| opendevreview | Merged openstack/cinder master: [Pure Storage] Add replication support for NVMe driver https://review.opendev.org/c/openstack/cinder/+/855058 | 22:10 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!