| *** dviroel|out is now known as dviroel | 11:35 | |
| whoami-rajat | #startmeeting cinder | 14:00 |
|---|---|---|
| opendevmeet | Meeting started Wed Jul 27 14:00:07 2022 UTC and is due to finish in 60 minutes. The chair is whoami-rajat. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:00 |
| whoami-rajat | #topic roll call | 14:00 |
| TusharTgite | hi | 14:00 |
| simondodsley | here - but can't join on video | 14:00 |
| Alexander | hi! | 14:00 |
| felipe_rodrigues | o/ | 14:00 |
| whoami-rajat | today's meeting is in video+IRC format so join here https://redhat.bluejeans.com/556681290 | 14:00 |
| luizsantos[m] | o/ | 14:00 |
| whoami-rajat | simondodsley, ack np | 14:01 |
| caiquemello[m] | o/ | 14:01 |
| raffaelacunha[m] | Hi | 14:01 |
| nahimsouza[m] | o/ | 14:01 |
| aneeshp1 | o/ | 14:01 |
| *** aneeshp1 is now known as aneeeshp1 | 14:02 | |
| whoami-rajat | #link https://etherpad.openstack.org/p/cinder-zed-meetings | 14:02 |
| enriquetaso_ | hi | 14:04 |
| HappyStacker | hi | 14:05 |
| HappyStacker | Is Antelope, OR or is this the animal? | 14:06 |
| whoami-rajat | #topic announcements | 14:07 |
| whoami-rajat | OpenStack next release name is out and it's Antelope | 14:07 |
| whoami-rajat | HappyStacker, i think it is | 14:07 |
| whoami-rajat | next, Driver merge deadline this week | 14:07 |
| HappyStacker | oh great | 14:08 |
| simondodsley | is the pure driver being looked at with the associated dependencies? | 14:13 |
| whoami-rajat | simondodsley, do you mean the os-brick dependencies? | 14:14 |
| simondodsley | there is also a tempest patch dependency which in itself is dependent on a small cinder patch | 14:14 |
| simondodsley | added those to the etherpad | 14:16 |
| whoami-rajat | #link https://review.opendev.org/c/openstack/tempest/+/849841 | 14:17 |
| whoami-rajat | simondodsley, will take a look at the tempest patch but QA team can merge it so good to ask them for reviews | 14:17 |
| whoami-rajat | and for the driver, geguileo and I have agreed to review it | 14:17 |
| simondodsley | thery have already seen it and are waiting on the cinder patch | 14:18 |
| whoami-rajat | so we need the cinder driver In first and tempest is dependent on it? | 14:18 |
| simondodsley | cinder patch, then tempest, then os-brick then the driver | 14:19 |
| whoami-rajat | ack | 14:19 |
| simondodsley | sorry for the deep dependencies, but this is a new protocol we are messing with here... | 14:19 |
| whoami-rajat | 1) https://review.opendev.org/c/openstack/cinder/+/849690 | 14:19 |
| whoami-rajat | 2) https://review.opendev.org/c/openstack/tempest/+/849841 | 14:20 |
| whoami-rajat | 3) https://review.opendev.org/c/openstack/cinder/+/799871 | 14:20 |
| whoami-rajat | sorry, 3) os-brick patches (MERGED) | 14:20 |
| whoami-rajat | 4) https://review.opendev.org/c/openstack/cinder/+/799871 | 14:20 |
| simondodsley | yep | 14:20 |
| whoami-rajat | we're discussing it right now | 14:21 |
| whoami-rajat | next, Cinder midcycle-2 planning | 14:30 |
| whoami-rajat | simondodsley, not sure i remember correctly but did you have a conflict on 10th august -- related to midcycle-2 | 14:31 |
| whoami-rajat | #topic os-brick CI follow-up | 14:34 |
| whoami-rajat | #topic Ceph: add option to keep only last n snapshots per backup | 14:36 |
| whoami-rajat | jhartkopf that's you | 14:36 |
| jhartkopf | yes | 14:36 |
| jhartkopf | So there was an issue with incremental backups when base backups were deleted by the new feature, but this should be resolved now. I pushed that patch some months ago. | 14:37 |
| jhartkopf | Is it realistic to get this merged before Zed releases? | 14:37 |
| eharney | do we have any tempest tests that exercise this functionality? (just for risk assessment) | 14:37 |
| whoami-rajat | #link https://review.opendev.org/c/openstack/cinder/+/810457 | 14:37 |
| geguileo | jhartkopf: there's still time to merge it | 14:38 |
| HappyStacker | I don't see any link to the blueprint, as it seems to be a new feature | 14:38 |
| jhartkopf | Well, maybe more of a refinement instead of a new feature | 14:38 |
| HappyStacker | ok, so we're good if no blueprint is provided? | 14:39 |
| enriquetaso_ | it has a related bug linked https://bugs.launchpad.net/cinder/+bug/1703011 | 14:39 |
| geguileo | jhartkopf: we definitely don't have this covaered by tempest | 14:39 |
| HappyStacker | enriquetaso: Yep I saw it. thanks | 14:41 |
| whoami-rajat | jhartkopf, the releasenote mentions it's a new feature, we need some clarity on it since deadlines for features and bug fixes are different | 14:42 |
| *** dasm|off is now known as dasm | 14:42 | |
| whoami-rajat | jhartkopf, we discussed that a tempest test is needed, even if it doesn't cover the whole feature, it will make sure we're not breaking anything | 14:42 |
| jhartkopf | whoami-rajat: Ok, so should I add another test to tempest? Or what would be the best way going forward with this? | 14:44 |
| geguileo | jhartkopf: probably best to add it to the cinder tempest tests and not tempest itself | 14:45 |
| geguileo | (that's what I think) | 14:45 |
| enriquetaso_ | i.e https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/791921 | 14:46 |
| geguileo | enriquetaso_: thanks for the link :-) | 14:47 |
| whoami-rajat | let's move to open discussion | 14:48 |
| whoami-rajat | #topic open discussion | 14:48 |
| aneeeshp1 | Hi, i have created a third party CI setup for Fungible driver. Would like to request permission to test against the Cinder repo. Shall I enable commenting on Cinder patches? The setup is validated using ci-sandbox and look good. | 14:48 |
| eharney | cinder-mypy (non-voting) is failing on all patches until we merge https://review.opendev.org/c/openstack/cinder/+/851032 | 14:48 |
| whoami-rajat | aneeeshp1, we allow third party CIs to comment on patches but it should be non voting | 14:48 |
| jhartkopf | geguileo, enriquetaso_: Thanks, will look into this | 14:49 |
| aneeeshp1 | whoami-rajat okay thank you. I will enable it only after removing the voting. | 14:49 |
| geguileo | eharney: I added a comment | 14:49 |
| geguileo | jhartkopf: thank you for working on it | 14:50 |
| whoami-rajat | aneeeshp1, thanks, but I'm afraid we're too close to the deadline that fungible driver might not be able to make it into this release | 14:50 |
| aneeeshp1 | whoami-rajat I understand that. I too think it will slip to the next release. | 14:51 |
| whoami-rajat | aneeeshp1, ack, good to know, we can start reviewing it early next release | 14:52 |
| aneeeshp1 | whoami-rajat great thank you | 14:52 |
| whoami-rajat | thanks! | 14:52 |
| whoami-rajat | thanks everyone for attending | 14:53 |
| whoami-rajat | #endmeeting | 14:53 |
| opendevmeet | Meeting ended Wed Jul 27 14:53:48 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:53 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-07-27-14.00.html | 14:53 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-07-27-14.00.txt | 14:53 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-07-27-14.00.log.html | 14:53 |
| *** dviroel is now known as dviroel|lunch | 14:57 | |
| *** dviroel|lunch is now known as dviroel | 15:52 | |
| *** dasm is now known as dasm|off | 21:02 | |
| *** dviroel is now known as dviroel|afk | 22:36 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!