Tuesday, 2023-05-30

*** geguileo is now known as Guest171402:26
*** auniyal8 is now known as auniyal04:57
*** amoralej|off is now known as amoralej07:24
opendevreviewLuigi Toscano proposed openstack/cinder master: Revert "Add tempest integrated storage job to run on ubuntu focal"  https://review.opendev.org/c/openstack/cinder/+/88467109:47
*** Guest1714 is now known as geguileo09:50
toskythis ^^ may unblock the gates09:50
*** amoralej is now known as amoralej|lunch11:03
whoami-rajatjungleboyj, hey, i know it's pretty late but can you upload the recording of last month's video+irc meeting? https://drive.google.com/file/d/1YuQk3xOWExNMpQ0eRG9UTPbGL2cCUUrS/view?usp=sharing11:35
*** amoralej|lunch is now known as amoralej12:21
toskywhoami-rajat: hi, I believe https://review.opendev.org/c/openstack/cinder/+/884671 may fix the cinder gates, where that focal job is failing12:33
toskyI didn't spend too much time investigating the failure (something wrong in the keystone logs) when I realized this combination (master/2023.2 on focal) is not supposed to work12:33
opendevreviewRaghavendra Tilay proposed openstack/cinder master: DNM: WIP: HPE 3par: Unable to create clone of replicated vol  https://review.opendev.org/c/openstack/cinder/+/88459412:37
opendevreviewEric Harney proposed openstack/cinder master: Remove six from kaminario driver  https://review.opendev.org/c/openstack/cinder/+/88469313:14
opendevreviewYosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on backups  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/87858713:21
whoami-rajattosky, hey, thanks for looking into it, yeah i added that as a requirement from TC in 2023.1 but i agree it isn't required for 2023.213:40
*** amoralej is now known as amoralej|off16:02
opendevreviewMerged openstack/cinder-tempest-plugin master: [srbac] Testing Volumes  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/87867216:10
toskydansmith: hi, are those testing reviews still needed? https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882459  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882460  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882458  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882461 ?16:48
dansmithtosky: I was going to rebase/recheck them if we start seeing the kernel crashes after the cirros bump.. they're DNM so hopefully not confusing anyone, but if they need to be abandoned, I can do it in a different project. This was just a good job configuration mix to test with16:50
toskydansmith: oh, no, they can stay, it's just that they were in merge conflict state and I wasn't sure if they were still useful16:58
toskynow I know, thanks16:58
geguileorosmaita: eharney whoami-rajat did we agree that we would EOL anything older than Yoga or did I imagine that?16:58
rosmaitageguileo: i don't remember that, but i don't always pay attention17:15
geguileorosmaita: I thought we had talked about it when we decided not to backport the CVE beyond Yoga17:16
rosmaitawell, the VMT policy is that we are only obligated to provide fixes for the releaseable stable branches; EM branches (Xena and older) can be fixed as a courtesy, but aren't required17:17
opendevreviewDan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #1  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/88245817:31
opendevreviewDan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #2  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/88245917:31
opendevreviewDan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #3  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/88246017:31
opendevreviewDan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #4  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/88246117:31
opendevreviewDan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #1  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/88245817:36
opendevreviewDan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #2  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/88245917:36
opendevreviewDan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #3  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/88246017:36
opendevreviewDan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #4  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/88246117:36
geguileorosmaita: yeah, but if we don't backport this CVE, then we can consider that we are not supporting those branches18:01
geguileoat least that's how I see it18:01
rosmaitageguileo: good point18:01
rosmaitawe should discuss at the cinder meeting tomorrow18:02
geguileorosmaita: sounds good18:05
clarkbgeguileo: rosmaita EM means not supported fwiw18:07
rosmaitaclarkb: what's your opinion about keeping a branch EM and not EOL if we don't backport a CVE fix to that branch?18:08
clarkbrosmaita: my understanding is that is exactly what EM is for18:09
clarkbIts a best effort collabortation space for the people who do show up18:09
clarkbif you don't want to do the work then don't18:09
rosmaitaok, thanks, guess we need to decide if we want to do the work or not18:09
clarkbThe original idea behind EM was that the various distros were all needing to do backports themselves downstream because the centralize locaiton was getting shut down18:10
clarkbwith EM the idea was the project teams could stop caring about them but give space for people like that to centralize their work18:11
geguileorosmaita: I don't  ;-)18:24
rosmaita:D18:24
geguileoI would EOL the hell out of Xena the warrior and every earlier release18:25
dansmithyeah, recent backports and *not* the CVE fix is super confusing to people I think18:27
dansmithI lean towards culling the old ones that we're not going to fix, and maybe even updating the recommendations for reasons-to-EOL including "some major problem we can't/won't fix"18:28
geguileodansmith: +1  The CVE is the perfect excuse  ;-)18:29
dansmithyup, and nova's train branch has two.. the VMDK thing we haven't and won't be backporting18:29
clarkbdansmith: I agree it can be confusing but the whole point of EM was to give the distros (or anyone else not the project team) the opportunity to patch things like this18:31
clarkband if you delete the branch that opportunit goes away and that to me means EM isn't super useful18:31
dansmithyup, I know18:31
dansmithdata point being redhat (for one) isn't going to fix either of those CVEs on train18:32
dansmithtbh, I'm not a fan of deleting branches at all, but right now that's the "is it EM or EOL" flag and as such, I think people will assume the other trivial backports and the M in EM will mean that the most critical bug in the last decade will be fixed there, and it won't18:33
clarkbIf we think that project teams need to patch EM branches then we should either delete EM entirely or treat it all as supported for longer is my point. We've explicitly said we don't want to do those things so I don't see this as problematic with the rules we have18:35
clarkb(even if it is less than ideal)18:35
dansmithyep not strictly problematic, just MHO18:37
opendevreviewJakub Skunda proposed openstack/devstack-plugin-nfs master: Switch to bobcat testing runtime  https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/88325519:05
opendevreviewMerged openstack/cinder-tempest-plugin master: [srbac] Testing Volume Actions  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/88318220:03

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!