| aneeeshp1 | whoami-rajat comments in https://review.opendev.org/c/openstack/cinder/+/849143 are addressed. Fungible Storage CI passed. Can it be merged? | 00:57 |
|---|---|---|
| opendevreview | OpenRefactory Research proposed openstack/cinder master: Replace incorrect dictionary API usage with a correct one https://review.opendev.org/c/openstack/cinder/+/869285 | 05:28 |
| Kumar_Kanishka | Hi all, need a small help on my review request (https://review.opendev.org/c/openstack/cinder/+/869305), for which the zuul check is returning -1. It seems due to some pep8 issue with the release notes file but the same issue is not reproducible in our local environment. | 07:35 |
| *** dviroel|out is now known as dviroel | 11:18 | |
| opendevreview | Sathyanarayana R proposed openstack/cinder master: Allow Volume Migration with additional extra specs to enter driver code https://review.opendev.org/c/openstack/cinder/+/869999 | 11:26 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Fix Restore Full Backups https://review.opendev.org/c/openstack/cinder/+/750782 | 14:00 |
| akawai | whoami-rajat: Hi, if you have a time, please review "846977: Hitachi and OEM: Support multi pool | https://review.opendev.org/c/openstack/cinder/+/846977", this patch is depended on HPE-XP driver, whose CI ran successfully last Sunday. I've reviewed more 3 patches last Saturday. | 14:40 |
| opendevreview | Andrew Bogott proposed openstack/cinder master: cinder-backup: use the same backup backend host for incremental backups https://review.opendev.org/c/openstack/cinder/+/861566 | 15:10 |
| opendevreview | Andrew Bogott proposed openstack/cinder master: cinder-backup: use the same backup backend host for incremental backups https://review.opendev.org/c/openstack/cinder/+/861566 | 15:12 |
| Tony_Saad | Hello Everyone, I am looking for a couple more reviews on my first patch :) https://review.opendev.org/c/openstack/cinder/+/867753 | 15:24 |
| ganso | rosmaita, whoami-rajat: Hi! when you have some time, could you please review those 2 patches: https://review.opendev.org/c/openstack/cinder/+/812685 and https://review.opendev.org/c/openstack/cinder/+/868485 thanks in advance! | 15:49 |
| rosmaita | ganso: ack | 15:51 |
| *** dviroel is now known as dviroel|lunch | 15:53 | |
| sfv880_ | rosmaita: Hello Brian, I'm sorry to bother you! We are looking for a final +2 and +w on the Infinidat fix: https://review.opendev.org/c/openstack/cinder/+/864287 - thank you very much! | 15:54 |
| rosmaita | sfv880_: ok, will take a look | 15:54 |
| sfv880_ | thank you very much! | 15:55 |
| eharney | reviews on this simple image_utils cleanup would be appreciated: https://review.opendev.org/c/openstack/cinder/+/834920/ | 15:57 |
| opendevreview | Eric Harney proposed openstack/cinder master: [SVf] lsportip needs to fetch IPs with "host" flag https://review.opendev.org/c/openstack/cinder/+/869305 | 15:59 |
| abdi | whoami-rajat: Could you confirm https://review.opendev.org/c/openstack/cinder/+/815582 will be merged? Thanks again. | 16:05 |
| eharney | https://review.opendev.org/c/openstack/cinder/+/871284 is a simple tox.ini change that will let the pylint env run at all (currently it doesn't), please review | 16:20 |
| opendevreview | Sathyanarayana R proposed openstack/cinder master: [SVf] As part of Flashcopy 2.0 adding config parameter to support volumegroup https://review.opendev.org/c/openstack/cinder/+/871256 | 16:26 |
| opendevreview | Martin Kopec proposed openstack/devstack-plugin-nfs master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/871072 | 16:32 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Remove six from netapp unit tests https://review.opendev.org/c/openstack/cinder/+/871516 | 16:39 |
| *** dviroel|lunch is now known as dviroel | 16:51 | |
| opendevreview | Eric Harney proposed openstack/cinder master: Yadro tatlin_client: Fix bad message formatting https://review.opendev.org/c/openstack/cinder/+/871518 | 17:04 |
| happystacker | hello guys, we have several patches to get reviewed, https://etherpad.opendev.org/p/cinder-antelope-meetings#L144 thank you | 17:10 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: PowerMax Driver - Fix for force flag https://review.opendev.org/c/openstack/cinder/+/797970 | 17:34 |
| opendevreview | Merged openstack/devstack-plugin-nfs master: Remove outdated comment https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/735959 | 18:26 |
| enriquetaso | Please review https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/871072 | 18:33 |
| enriquetaso | ^ Switch NFS testing to Antelope testing runtime (currently is outdated) | 18:33 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: nvmeof: Support new connection properties https://review.opendev.org/c/openstack/cinder/+/836072 | 19:12 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: LVM: terminate_connection fails if no initiator https://review.opendev.org/c/openstack/cinder/+/836073 | 19:12 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: LVM nvmet: Add support for shared subsystems https://review.opendev.org/c/openstack/cinder/+/836074 | 19:12 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: LVM nvmet: Add support for multiple ip addresses https://review.opendev.org/c/openstack/cinder/+/839064 | 19:12 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Add missing extend_target driver method https://review.opendev.org/c/openstack/cinder/+/843493 | 19:12 |
| aneeeshp1 | Can someone suggest how to retrigger Zuul check on this patch? https://review.opendev.org/c/openstack/cinder/+/849143 Looks like it failed due to some error not caused by the patch. | 19:17 |
| opendevreview | Andrew Bogott proposed openstack/cinder master: cinder-backup: use the same backup backend host for incremental backups https://review.opendev.org/c/openstack/cinder/+/861566 | 19:31 |
| opendevreview | Martin Kopec proposed openstack/devstack-plugin-nfs master: Switch testing to Antelope testing runtime https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/871072 | 19:35 |
| opendevreview | Merged openstack/cinder master: S3 Backup: Warn if verify_ssl is true with no cert file https://review.opendev.org/c/openstack/cinder/+/866594 | 20:06 |
| *** dviroel is now known as dviroel|out | 21:25 | |
| opendevreview | Sofia Enriquez proposed openstack/cinder-tempest-plugin master: Add Backup Restore Multibackend Tests https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/835512 | 22:18 |
| opendevreview | Merged openstack/cinder master: Use new get_rpc_client API from oslo.messaging https://review.opendev.org/c/openstack/cinder/+/871158 | 23:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!