| opendevreview | Merged openstack/cinder master: Remove Block Storage API v2 https://review.opendev.org/c/openstack/cinder/+/792299 | 05:18 |
|---|---|---|
| opendevreview | Merged openstack/cinder master: Update Block Storage API v2 api-ref https://review.opendev.org/c/openstack/cinder/+/793244 | 05:18 |
| *** bhagyashris_ is now known as bhagyashris | 08:05 | |
| opendevreview | Jun Chen proposed openstack/cinder master: Add a compression policy for backup chunked driver https://review.opendev.org/c/openstack/cinder/+/801717 | 08:22 |
| opendevreview | Jun Chen proposed openstack/cinder master: Add a compression policy for backup chunked driver https://review.opendev.org/c/openstack/cinder/+/801717 | 08:53 |
| stephenfin | whoami-rajat: Sorry, I was out sick post-COVID jab yesterday. I think rosmaita covered it though | 09:14 |
| stephenfin | tl;dr: review dem patches :D | 09:15 |
| *** bhagyashris_ is now known as bhagyashris | 09:35 | |
| opendevreview | Luigi Toscano proposed openstack/cinder master: Native multibackend-matrix Zuul v3 job https://review.opendev.org/c/openstack/cinder/+/751579 | 10:52 |
| *** mabrams is now known as mabrams|afk | 11:42 | |
| *** mabrams|afk is now known as mabrams | 11:43 | |
| whoami-rajat | stephenfin, ack | 12:41 |
| *** elodilles is now known as elodilles_afk | 12:48 | |
| gibi | hi! I see that openstacksdk-functional-devstack job started to fail with "openstack.exceptions.NotSupported: The block-storage service for devstack:RegionOne exists but does not have any supported versions." around when https://review.opendev.org/c/openstack/cinder/+/792299 was merged | 14:10 |
| tosky | gibi: but isn't it a problem on openstacksdk then? That patch can't be reverted | 14:14 |
| gibi | tosky: I'm not promoting any revert I'm just noting that this job failure is now blocking the gate | 14:15 |
| gibi | it seems these are v2 tests in the sdk | 14:17 |
| tosky | the openstacksdk gate I guesS? | 14:17 |
| gibi | so probably can be deleted | 14:17 |
| gibi | tosky: the nova gate too | 14:17 |
| gibi | I haven't checked how many other projects run the sdk jobs in their gate | 14:17 |
| gibi | cinder isn't | 14:17 |
| tosky | the old tests could be run against an older version of openstack, like wallaby (that's how tempest solved the problem with v2 tests) | 14:18 |
| gibi | sdk is branched so we can drop the tests from the master and keep it on stable branches | 14:19 |
| tosky | oh, ok | 14:22 |
| gmann | yeah I do not think we need to keep them as sdk is branched and does not need to test after wallaby | 14:27 |
| gmann | but we need to check if that is extra v2 tests or cinder tests still using v2(no other correspondent v3 test) | 14:28 |
| gmann | ah we have v3 test coverage separate. | 14:32 |
| *** elodilles_afk is now known as elodilles | 14:34 | |
| gibi | proposed the func test remove from the sdk https://review.opendev.org/c/openstack/openstacksdk/+/801864 | 14:38 |
| opendevreview | Merged openstack/cinder master: PowerMax Driver - Allow for case mismatch in SGs https://review.opendev.org/c/openstack/cinder/+/796256 | 15:23 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Use oslo_utils.uuidutils to manipulate UUID Strings: Part 1 https://review.opendev.org/c/openstack/cinder/+/801868 | 15:23 |
| opendevreview | Ghanshyam proposed openstack/cinder master: Add openstacksdk-functional-devstack job in cinder gate https://review.opendev.org/c/openstack/cinder/+/801875 | 15:47 |
| gmann | dansmith: jungleboyj gibi whoami-rajat ^^ | 15:48 |
| dansmith | ++ | 15:48 |
| jungleboyj | gmann: ++ Thank you for proposing. | 15:56 |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Fix for group snapshot deletion https://review.opendev.org/c/openstack/cinder/+/801902 | 16:07 |
| opendevreview | Helen Walsh proposed openstack/cinder master: Cinder matrix - Clarification on Snapshot Attachment https://review.opendev.org/c/openstack/cinder/+/800015 | 16:20 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Use oslo_utils.uuidutils to manipulate UUID Strings: Part 1 https://review.opendev.org/c/openstack/cinder/+/801868 | 17:26 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix detach notification https://review.opendev.org/c/openstack/cinder/+/778534 | 17:49 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Expose volume_attachments in Volume OVO https://review.opendev.org/c/openstack/cinder/+/779576 | 17:49 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Delete attachment on remove_export failure https://review.opendev.org/c/openstack/cinder/+/801912 | 17:49 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix: Nova thinks deleted volume is attached https://review.opendev.org/c/openstack/cinder/+/801913 | 17:49 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix detach notification https://review.opendev.org/c/openstack/cinder/+/778534 | 17:57 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Delete attachment on remove_export failure https://review.opendev.org/c/openstack/cinder/+/801912 | 17:57 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Expose volume_attachments in Volume OVO https://review.opendev.org/c/openstack/cinder/+/779576 | 17:57 |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: Fix: Nova thinks deleted volume is attached https://review.opendev.org/c/openstack/cinder/+/801913 | 17:57 |
| geguileo | mmmm, I only changed the last patch adding the release note, don't know why git review submitted everything again | 17:58 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: NetApp: Use valid UUIDs for fake ids https://review.opendev.org/c/openstack/cinder/+/801914 | 18:08 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Replace deprecated tenacity.Retrying.call https://review.opendev.org/c/openstack/cinder/+/801916 | 18:11 |
| opendevreview | Takashi Kajinami proposed openstack/os-brick master: Replace deprecated tenacity.Retrying.call https://review.opendev.org/c/openstack/os-brick/+/801917 | 18:13 |
| opendevreview | Ghanshyam proposed openstack/cinder master: Add openstacksdk-functional-devstack job in cinder gate https://review.opendev.org/c/openstack/cinder/+/801875 | 18:18 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Use valid UUId in unit tests https://review.opendev.org/c/openstack/cinder/+/801923 | 19:34 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: Fix deprecation warnings caused by invalid UUIDs https://review.opendev.org/c/openstack/cinder/+/801923 | 19:42 |
| opendevreview | Merged openstack/cinder master: NFS: Update connection info on online snap create https://review.opendev.org/c/openstack/cinder/+/799843 | 20:40 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!