opendevreview | Merged openstack/cinder master: Pylint: add nvmet to ignored-modules https://review.opendev.org/c/openstack/cinder/+/871289 | 00:01 |
---|---|---|
TusharTgite | whoami-rajat: updated patch as per your review take a look https://review.opendev.org/c/openstack/cinder/+/820994/8 | 04:04 |
whoami-rajat | TusharTgite, done | 06:29 |
whoami-rajat | few minor comments | 06:29 |
opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf] : Enable support for mirror-pool option for replication volume-type https://review.opendev.org/c/openstack/cinder/+/870006 | 07:03 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Configure a storage node in cinder https://review.opendev.org/c/openstack/cinder/+/820994 | 08:28 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Remove multiatttach request parameter https://review.opendev.org/c/openstack/cinder/+/874865 | 09:09 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Remove multiatttach request parameter https://review.opendev.org/c/openstack/cinder/+/874865 | 09:42 |
ade_lee_ | abishop_, eharney - hey guys - we finally have a fips enabled job running on ubuntu and its failing volume related tests. Can you guys take a look and see whats wrong? | 10:43 |
ade_lee_ | https://zuul.opendev.org/t/openstack/build/44e7d0b4a565456893f1c096f6b9da61/logs | 10:43 |
ade_lee_ | abdi, eharney maybe we need some iscsi.conf magic or similar .. | 10:44 |
ade_lee_ | sorry abishop_ ^^ | 10:44 |
ade_lee_ | tosky, ^^ | 10:45 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: [WIP]Deleting a volume in 'downloading' state https://review.opendev.org/c/openstack/cinder/+/826607 | 10:56 |
TusharTgite | whoami-rajat: updated patch take a look https://review.opendev.org/c/openstack/cinder/+/820994 , and for this https://review.opendev.org/c/openstack/cinder/+/860209 IBM Storage CI still not start i've already tried this one pkvm- recheck | 11:17 |
opendevreview | Rajat Dhasmana proposed openstack/cinder master: Configure a storage node in cinder https://review.opendev.org/c/openstack/cinder/+/820994 | 11:28 |
whoami-rajat | TusharTgite, then you should contact IBM storage driver team, any change in the driver code should pass the third party CI else it could be problematic | 11:30 |
TusharTgite | whoami-rajat: ok | 11:31 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Configure a storage node in cinder https://review.opendev.org/c/openstack/cinder/+/820994 | 11:31 |
whoami-rajat | geguileo, hey, finished reviewing your patch, one issue noted in documentation, else it looks good https://review.opendev.org/c/openstack/cinder/+/819669 | 12:48 |
opendevreview | Gorka Eguileor proposed openstack/cinder master: Improve resource listing efficiency https://review.opendev.org/c/openstack/cinder/+/819669 | 13:29 |
geguileo | whoami-rajat: done | 13:29 |
geguileo | good catch | 13:30 |
geguileo | wait, what did I just do... | 13:31 |
geguileo | I should remove the cherry picked message in the commit message | 13:32 |
opendevreview | Gorka Eguileor proposed openstack/cinder master: Improve resource listing efficiency https://review.opendev.org/c/openstack/cinder/+/819669 | 13:33 |
*** abishop_ is now known as abishop | 13:38 | |
whoami-rajat | geguileo, great thanks | 14:53 |
opendevreview | Merged openstack/cinder-tempest-plugin master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/854989 | 14:57 |
whoami-rajat | rosmaita, jungleboyj eharney geguileo hey, can you take a look at these 3 hitachi patches? they're pretty small and straightforward https://review.opendev.org/c/openstack/cinder/+/871269/5 | 14:57 |
rosmaita | whoami-rajat: ack | 15:05 |
whoami-rajat | rosmaita, thanks | 15:19 |
opendevreview | Merged openstack/cinder master: RBD: Fix total_capacity https://review.opendev.org/c/openstack/cinder/+/829565 | 15:58 |
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Configure a storage node in cinder https://review.opendev.org/c/openstack/cinder/+/820994 | 16:48 |
TusharTgite | whoami-rajat: CI job run now https://review.opendev.org/c/openstack/cinder/+/860209 | 16:54 |
opendevreview | Merged openstack/cinder master: Hitachi: Fix key error when backend is down https://review.opendev.org/c/openstack/cinder/+/871269 | 17:14 |
opendevreview | Brian Rosmaita proposed openstack/cinder master: Correct release note formatting https://review.opendev.org/c/openstack/cinder/+/876413 | 17:52 |
opendevreview | OpenStack Release Bot proposed openstack/devstack-plugin-ceph stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876416 | 18:15 |
opendevreview | OpenStack Release Bot proposed openstack/devstack-plugin-ceph stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876417 | 18:15 |
opendevreview | Mohammed Naser proposed openstack/cinder stable/zed: Dell PowerStore: Volume caching exception support https://review.opendev.org/c/openstack/cinder/+/876226 | 18:48 |
opendevreview | Merged openstack/devstack-plugin-ceph stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876416 | 18:52 |
opendevreview | Merged openstack/devstack-plugin-ceph stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/876417 | 18:53 |
opendevreview | Merged openstack/cinder master: Update IBM Storwize drivers https://review.opendev.org/c/openstack/cinder/+/860209 | 20:47 |
opendevreview | Merged openstack/cinder master: Correct release note formatting https://review.opendev.org/c/openstack/cinder/+/876413 | 20:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!