*** blarnath is now known as d34dh0r53 | 12:08 | |
*** dasm|off is now known as dasm | 13:53 | |
pdeore | #startmeeting glance | 13:59 |
---|---|---|
opendevmeet | Meeting started Thu Feb 2 13:59:47 2023 UTC and is due to finish in 60 minutes. The chair is pdeore. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:59 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:59 |
opendevmeet | The meeting name has been set to 'glance' | 13:59 |
pdeore | #topic roll call | 13:59 |
pdeore | #link https://etherpad.openstack.org/p/glance-team-meeting-agenda | 13:59 |
pdeore | o/ | 13:59 |
abhishekk | o/ | 14:00 |
pdeore | lets wait few minutes for others to join | 14:00 |
jokke_ | o/ | 14:00 |
mrjoshi__ | o/ | 14:00 |
croelandt | o/ | 14:01 |
pdeore | I think everyone is here, lets start :) | 14:01 |
pdeore | #topic release/periodic jobs updates | 14:01 |
pdeore | Milestone 3 is in 2 weeks | 14:01 |
pdeore | So it's review party time next week | 14:01 |
pdeore | and as usual, next week we will have video meeting instead of irc meeting on thursday at same time | 14:02 |
pdeore | I hope everyone is ok with the review party plan next week :) | 14:02 |
abhishekk | when we decided to have video meeting? | 14:02 |
croelandt | it's a review party | 14:03 |
abhishekk | aah | 14:03 |
pdeore | when meaning ? time ? | 14:03 |
pdeore | :D | 14:03 |
croelandt | the only way to avoid the video meeting is to close *all* open patches | 14:03 |
abhishekk | may be I will be out next week :/ | 14:03 |
abhishekk | haha | 14:03 |
pdeore | ohh | 14:03 |
rosmaita | o/ | 14:03 |
croelandt | abhishekk: PTO request denied | 14:03 |
abhishekk | But I will try to join atleast for 30 mins | 14:04 |
abhishekk | croelandt, ack :D | 14:04 |
rosmaita | croelandt: i will endeavor to participate at my usual level at the review parties | 14:04 |
croelandt | rosmaita++ | 14:05 |
pdeore | ok, let's move ahead | 14:06 |
pdeore | Periodic job all green except TIME_OUT for fips jobs | 14:06 |
pdeore | #topic Catching up with reviews | 14:06 |
pdeore | Update Migration Constant #link https://review.opendev.org/c/openstack/glance/+/861752/ | 14:07 |
rosmaita | does anyone know what's the deal with the FIPS jobs? i thought they were fixed | 14:07 |
croelandt | So is this our final approach for the migration constant? | 14:08 |
abhishekk | i don't recollect when they were working | 14:08 |
abhishekk | croelandt, there is no other way | 14:08 |
abhishekk | unless we revisit entire glance migration stuff | 14:08 |
rosmaita | me neither, but i thought they were going to run them on ubuntu and then everything would work | 14:08 |
abhishekk | yeah, but we didn't got any update about it later | 14:08 |
rosmaita | ok, thanks, sorry for interrupting | 14:09 |
croelandt | abhishekk: ok will review the patch after the meeting then | 14:09 |
abhishekk | pdeore, could you please try to reach out to concern person about fips? | 14:09 |
pdeore | I will try to get the update after the meeting | 14:09 |
abhishekk | croelandt, thanks | 14:09 |
pdeore | abhishekk, yes | 14:09 |
abhishekk | cool | 14:09 |
abhishekk | also we still have some backports which needs to be merged | 14:10 |
abhishekk | those are failing with grenade/docs/infra related issue | 14:10 |
abhishekk | wallaby/ussuri and train | 14:10 |
croelandt | abhishekk: so I thikn the fixes by gmann are not backported all the way to train | 14:11 |
rosmaita | i think wallaby gate may be cleard up? | 14:11 |
croelandt | should we backport these as well? | 14:11 |
jokke_ | My vote goes to EOL those branches as their gate seems to need way too much maintenance | 14:11 |
croelandt | they are already -em, can we EOL them even more? :) | 14:11 |
rosmaita | yes, but i think we need to keep train open | 14:12 |
jokke_ | croelandt: yes, EOL would be tagging the current state of the branch and then removing it killin all the jobs etc. | 14:12 |
abhishekk | croelandt, I think there are still some failures in wallaby | 14:12 |
abhishekk | gmann, has pointed those out on ML (docs job is failing on wallaby) and grenade job is still failing on ussuri | 14:13 |
rosmaita | i must have slipped one past before the docs job broke | 14:13 |
abhishekk | and for train neutron-grenade job is failing :. | 14:13 |
croelandt | abhishekk: yeah but he's not patching stable/train | 14:14 |
croelandt | so the failures there are likely not going away, are they? | 14:14 |
abhishekk | right | 14:14 |
rosmaita | well, there is no obligation to run grenade in EM branches | 14:14 |
abhishekk | so can we drop that | 14:14 |
rosmaita | and actually, no obligation to have the gate working at all, unless the project team wants to merge stuff there | 14:14 |
abhishekk | ok | 14:15 |
rosmaita | from the release docs: "Without regular comprehensive maintenance, it is quite possible that someone proposing a backport to an EM branch will find that tests | 14:15 |
rosmaita | nce, it is quite possible that someone proposing a backport to an EM branch will find that tests | 14:16 |
rosmaita | bad paste | 14:16 |
rosmaita | let me start over | 14:16 |
rosmaita | "Without regular comprehensive maintenance, it is quite possible that someone proposing a backport to an EM branch will find that tests have broken since the last | 14:16 |
rosmaita | successful merge. This means that tests (or test configuration) might need to be fixed, reduced, or reconfigured before the backport itself can be evaluated and merged. | 14:16 |
rosmaita | The onus for that falls on the backporter or the group of people looking after a specific release." | 14:17 |
rosmaita | https://docs.openstack.org/project-team-guide/stable-branches.html#extended-maintenance | 14:17 |
abhishekk | so we can keep basic jobs like unit/functional and docs on em branches ? | 14:17 |
rosmaita | right | 14:17 |
abhishekk | cool, that will sort out train issue | 14:18 |
abhishekk | we will wait for wallaby to be fixed properly | 14:18 |
croelandt | so jokke_ has some complaints about removing jobs :) | 14:18 |
rosmaita | from the TC meeting yesterday, a primary reason to keep EM branches non-deleted is just so the community can post patches and have a public place to review them | 14:18 |
croelandt | abhishekk: oh for wallaby it's the docs that fail now | 14:18 |
abhishekk | correct | 14:19 |
croelandt | always a new failure | 14:19 |
abhishekk | :P | 14:19 |
abhishekk | I can push a patch to train to remove unnecessary jobs | 14:19 |
pdeore | abhishekk, Thanks ! | 14:20 |
rosmaita | i would keep at least one tempest job, though | 14:20 |
abhishekk | ack | 14:20 |
jokke_ | my ocncern is, why do we bother to keep the branches, if we don't test them | 14:20 |
jokke_ | concern | 14:20 |
rosmaita | well, we do test them | 14:21 |
rosmaita | i think grenade doesn't make sense, but tempest does | 14:21 |
rosmaita | (if it wouldn't break so much) | 14:21 |
jokke_ | Ok so why don't we remove those excessive jobs from all branches then? ;) | 14:21 |
abhishekk | :P | 14:21 |
rosmaita | but the main reason is that you can't post reviews of patches to a deleted branch in gerrit | 14:21 |
croelandt | This is probably something the whole OpenStack community should discuss with packagers | 14:21 |
abhishekk | ++ | 14:22 |
croelandt | Who uses these branches? How useful are they? | 14:22 |
croelandt | Get people from Debian, Fedora, Ubuntu etc. | 14:22 |
croelandt | see how they use them | 14:22 |
rosmaita | well, zigo was on the ML, just repackaged rocky for debian to include a CVE fix | 14:22 |
rosmaita | and from comments on that bug, looks like a lot of interest in rocky currently | 14:23 |
croelandt | damn, Debian stable truly is in the past | 14:23 |
rosmaita | "that bug" == https://bugs.launchpad.net/cinder/+bug/1996188 | 14:23 |
abhishekk | no further arguments | 14:24 |
jokke_ | I'm not saying that the -em branches are not useful for anyone, but the deal was that those folks takes the responsibility of keeping the tests etc running and if that does not happen it's better to just EOL the branch and let them backport in downstream | 14:24 |
jokke_ | They were supposed to be 0 effort for the project teams, which clearly is not the case | 14:25 |
rosmaita | i don't think *anything* is supposed to be 0 effort for project teams! | 14:26 |
croelandt | yeah so maybe we could discuss having a reduced set of tests | 14:26 |
croelandt | the CI is always failing for some reason | 14:26 |
croelandt | and usually that reason is not an issue with the actual code | 14:26 |
croelandt | but rather infrastructure-related issues | 14:26 |
abhishekk | +1 | 14:27 |
rosmaita | reduced tests is fine, though eventually you get to the point where like jokke_ says, why bother | 14:27 |
rosmaita | so i think reduce tests, and when we get to that point, have the EOL discussion | 14:28 |
croelandt | rosmaita: how would we start that conversation? I think it needs to be an Openstack-wide policy | 14:28 |
rosmaita | it already is | 14:28 |
abhishekk | I think its better to take this topic in upcoming PTG either in TC or glance schedule | 14:28 |
rosmaita | read the EM policy ... it is clear that it's up to the project team when to go EOL and what the team's obligations are while a branch is in EM | 14:29 |
rosmaita | https://docs.openstack.org/project-team-guide/stable-branches.html#extended-maintenance | 14:29 |
pdeore | abhishekk++ | 14:29 |
abhishekk | rosmaita, ack | 14:30 |
rosmaita | cinder can EOL all the way up to wallaby if it wants to | 14:30 |
croelandt | ok | 14:30 |
rosmaita | though i think some peoples' employers would have issue with that | 14:30 |
rosmaita | (if you know what i mean) | 14:30 |
abhishekk | :D | 14:30 |
abhishekk | Let's move ahead, keep watch on ussuri and wallaby fixes and then get the patches merged | 14:32 |
pdeore | thanks :) moving ahead | 14:32 |
abhishekk | Train we can decide what to drop and what to keep on the review itself | 14:32 |
pdeore | #topic Glance cinder store changes | 14:32 |
pdeore | so glance store release is in a week (10th Feb last date), so kindly please have a look at below patches on priority | 14:33 |
pdeore | Refactor: https://review.opendev.org/c/openstack/glance_store/+/843103, this one is required for extending attched volumes support patch | 14:34 |
pdeore | Extend attached volumes: #link https://review.opendev.org/c/openstack/glance_store/+/868742 | 14:34 |
pdeore | Glance test fix: #link https://review.opendev.org/c/openstack/glance/+/869021 | 14:34 |
abhishekk | ack | 14:36 |
pdeore | That's it from me for today, let's move to Open discussion | 14:36 |
pdeore | #topic Open Discussions | 14:36 |
abhishekk | I don't have anything | 14:36 |
abhishekk | just fyi, PTL nomination is open | 14:37 |
pdeore | yeah :) | 14:37 |
pdeore | till 15th feb | 14:37 |
croelandt | pdeore! pdeore! pdeore! | 14:37 |
pdeore | no choice :P | 14:38 |
pdeore | anyone has anythhing else to discuss or we are good to wrap up early? | 14:39 |
abhishekk | thank you all! | 14:39 |
jokke_ | I'm good, thanks! | 14:40 |
croelandt | thanks! | 14:40 |
pdeore | cool, thanks everyone !! | 14:40 |
pdeore | #endmeeting | 14:40 |
opendevmeet | Meeting ended Thu Feb 2 14:40:25 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:40 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/glance/2023/glance.2023-02-02-13.59.html | 14:40 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/glance/2023/glance.2023-02-02-13.59.txt | 14:40 |
opendevmeet | Log: https://meetings.opendev.org/meetings/glance/2023/glance.2023-02-02-13.59.log.html | 14:40 |
*** dasm is now known as dasm|off | 22:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!