*** geguileo is now known as Guest1714 | 02:26 | |
*** auniyal8 is now known as auniyal | 04:57 | |
*** amoralej|off is now known as amoralej | 07:24 | |
opendevreview | Luigi Toscano proposed openstack/cinder master: Revert "Add tempest integrated storage job to run on ubuntu focal" https://review.opendev.org/c/openstack/cinder/+/884671 | 09:47 |
---|---|---|
*** Guest1714 is now known as geguileo | 09:50 | |
tosky | this ^^ may unblock the gates | 09:50 |
*** amoralej is now known as amoralej|lunch | 11:03 | |
whoami-rajat | jungleboyj, 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=sharing | 11:35 |
*** amoralej|lunch is now known as amoralej | 12:21 | |
tosky | whoami-rajat: hi, I believe https://review.opendev.org/c/openstack/cinder/+/884671 may fix the cinder gates, where that focal job is failing | 12:33 |
tosky | I 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 work | 12:33 |
opendevreview | Raghavendra Tilay proposed openstack/cinder master: DNM: WIP: HPE 3par: Unable to create clone of replicated vol https://review.opendev.org/c/openstack/cinder/+/884594 | 12:37 |
opendevreview | Eric Harney proposed openstack/cinder master: Remove six from kaminario driver https://review.opendev.org/c/openstack/cinder/+/884693 | 13:14 |
opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on backups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/878587 | 13:21 |
whoami-rajat | tosky, 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.2 | 13:40 |
*** amoralej is now known as amoralej|off | 16:02 | |
opendevreview | Merged openstack/cinder-tempest-plugin master: [srbac] Testing Volumes https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/878672 | 16:10 |
tosky | dansmith: 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 |
dansmith | tosky: 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 with | 16:50 |
tosky | dansmith: oh, no, they can stay, it's just that they were in merge conflict state and I wasn't sure if they were still useful | 16:58 |
tosky | now I know, thanks | 16:58 |
geguileo | rosmaita: eharney whoami-rajat did we agree that we would EOL anything older than Yoga or did I imagine that? | 16:58 |
rosmaita | geguileo: i don't remember that, but i don't always pay attention | 17:15 |
geguileo | rosmaita: I thought we had talked about it when we decided not to backport the CVE beyond Yoga | 17:16 |
rosmaita | well, 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 required | 17:17 |
opendevreview | Dan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #1 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882458 | 17:31 |
opendevreview | Dan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #2 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882459 | 17:31 |
opendevreview | Dan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #3 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882460 | 17:31 |
opendevreview | Dan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #4 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882461 | 17:31 |
opendevreview | Dan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #1 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882458 | 17:36 |
opendevreview | Dan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #2 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882459 | 17:36 |
opendevreview | Dan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #3 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882460 | 17:36 |
opendevreview | Dan Smith proposed openstack/cinder-tempest-plugin master: DNM: Testing for bug 2018612 #4 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/882461 | 17:36 |
geguileo | rosmaita: yeah, but if we don't backport this CVE, then we can consider that we are not supporting those branches | 18:01 |
geguileo | at least that's how I see it | 18:01 |
rosmaita | geguileo: good point | 18:01 |
rosmaita | we should discuss at the cinder meeting tomorrow | 18:02 |
geguileo | rosmaita: sounds good | 18:05 |
clarkb | geguileo: rosmaita EM means not supported fwiw | 18:07 |
rosmaita | clarkb: 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 |
clarkb | rosmaita: my understanding is that is exactly what EM is for | 18:09 |
clarkb | Its a best effort collabortation space for the people who do show up | 18:09 |
clarkb | if you don't want to do the work then don't | 18:09 |
rosmaita | ok, thanks, guess we need to decide if we want to do the work or not | 18:09 |
clarkb | The original idea behind EM was that the various distros were all needing to do backports themselves downstream because the centralize locaiton was getting shut down | 18:10 |
clarkb | with EM the idea was the project teams could stop caring about them but give space for people like that to centralize their work | 18:11 |
geguileo | rosmaita: I don't ;-) | 18:24 |
rosmaita | :D | 18:24 |
geguileo | I would EOL the hell out of Xena the warrior and every earlier release | 18:25 |
dansmith | yeah, recent backports and *not* the CVE fix is super confusing to people I think | 18:27 |
dansmith | I 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 |
geguileo | dansmith: +1 The CVE is the perfect excuse ;-) | 18:29 |
dansmith | yup, and nova's train branch has two.. the VMDK thing we haven't and won't be backporting | 18:29 |
clarkb | dansmith: 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 this | 18:31 |
clarkb | and if you delete the branch that opportunit goes away and that to me means EM isn't super useful | 18:31 |
dansmith | yup, I know | 18:31 |
dansmith | data point being redhat (for one) isn't going to fix either of those CVEs on train | 18:32 |
dansmith | tbh, 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't | 18:33 |
clarkb | If 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 have | 18:35 |
clarkb | (even if it is less than ideal) | 18:35 |
dansmith | yep not strictly problematic, just MHO | 18:37 |
opendevreview | Jakub Skunda proposed openstack/devstack-plugin-nfs master: Switch to bobcat testing runtime https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/883255 | 19:05 |
opendevreview | Merged openstack/cinder-tempest-plugin master: [srbac] Testing Volume Actions https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/883182 | 20:03 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!