Thursday, 2023-02-02

*** blarnath is now known as d34dh0r5312:08
*** dasm|off is now known as dasm13:53
pdeore#startmeeting glance13:59
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:59
opendevmeetThe meeting name has been set to 'glance'13:59
pdeore#topic roll call13:59
pdeore#link https://etherpad.openstack.org/p/glance-team-meeting-agenda13:59
pdeoreo/13:59
abhishekko/14:00
pdeorelets wait few minutes for others to join14:00
jokke_o/14:00
mrjoshi__o/14:00
croelandto/14:01
pdeoreI think everyone is here, lets start :)14:01
pdeore#topic release/periodic jobs updates14:01
pdeoreMilestone 3 is in 2 weeks14:01
pdeoreSo it's review party time next week14:01
pdeoreand as usual, next week we will have video meeting instead of irc meeting on thursday at same time14:02
pdeoreI hope everyone is ok with the review party plan next week :)14:02
abhishekkwhen we decided to have video meeting?14:02
croelandtit's a review party14:03
abhishekkaah14:03
pdeorewhen meaning ? time ?14:03
pdeore:D14:03
croelandtthe only way to avoid the video meeting is to close *all* open patches 14:03
abhishekkmay be I will be out next week :/14:03
abhishekkhaha14:03
pdeoreohh14:03
rosmaitao/14:03
croelandtabhishekk: PTO request denied14:03
abhishekkBut I will try to join atleast for 30 mins14:04
abhishekkcroelandt, ack :D14:04
rosmaitacroelandt: i will endeavor to participate at my usual level at the review parties14:04
croelandtrosmaita++14:05
pdeoreok, let's move ahead14:06
pdeorePeriodic job all green except TIME_OUT for fips jobs 14:06
pdeore#topic Catching up with reviews14:06
pdeoreUpdate Migration Constant #link https://review.opendev.org/c/openstack/glance/+/861752/14:07
rosmaitadoes anyone know what's the deal with the FIPS jobs? i thought they were fixed14:07
croelandtSo is this our final approach for the migration constant?14:08
abhishekki don't recollect when they were working14:08
abhishekkcroelandt, there is no other way14:08
abhishekkunless we revisit entire glance migration stuff14:08
rosmaitame neither, but i thought they were going to run them on ubuntu and then everything would work14:08
abhishekkyeah, but we didn't got any update about it later14:08
rosmaitaok, thanks, sorry for interrupting14:09
croelandtabhishekk: ok will review the patch after the meeting then14:09
abhishekkpdeore, could you please try to reach out to concern person about fips?14:09
pdeoreI will try to get the update after the meeting14:09
abhishekkcroelandt, thanks14:09
pdeoreabhishekk, yes14:09
abhishekkcool14:09
abhishekkalso we still have some backports which needs to be merged14:10
abhishekkthose are failing with grenade/docs/infra related issue14:10
abhishekkwallaby/ussuri and train14:10
croelandtabhishekk: so I thikn the fixes by gmann are not backported all the way to train14:11
rosmaitai think wallaby gate may be cleard up?14:11
croelandtshould we backport these as well?14:11
jokke_My vote goes to EOL those branches as their gate seems to need way too much maintenance14:11
croelandtthey are already -em, can we EOL them even more? :)14:11
rosmaitayes, but i think we need to keep train open14: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
abhishekkcroelandt, I think there are still some failures in wallaby14:12
abhishekkgmann, has pointed those out on ML (docs job is failing on wallaby) and grenade job is still failing on ussuri14:13
rosmaitai must have slipped one past before the docs job broke14:13
abhishekkand for train neutron-grenade job is failing :.14:13
croelandtabhishekk: yeah but he's not patching stable/train14:14
croelandtso the  failures there are likely not going away, are they?14:14
abhishekkright14:14
rosmaitawell, there is no obligation to run grenade in EM branches14:14
abhishekkso can we drop that14:14
rosmaitaand actually, no obligation to have the gate working at all, unless the project team wants to merge stuff there14:14
abhishekkok14:15
rosmaitafrom the release docs: "Without regular comprehensive maintenance, it is quite possible that someone proposing a backport to an EM branch will find that tests14:15
rosmaitance, it is quite possible that someone proposing a backport to an EM branch will find that tests 14:16
rosmaitabad paste14:16
rosmaitalet me start over14: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 last14:16
rosmaitasuccessful 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
rosmaitaThe onus for that falls on the backporter or the group of people looking after a specific release."14:17
rosmaitahttps://docs.openstack.org/project-team-guide/stable-branches.html#extended-maintenance14:17
abhishekkso we can keep basic jobs like unit/functional and docs on em branches ?14:17
rosmaitaright14:17
abhishekkcool, that will sort out train issue14:18
abhishekkwe will wait for wallaby to be fixed properly14:18
croelandtso jokke_ has some complaints about removing jobs :)14:18
rosmaitafrom 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 them14:18
croelandtabhishekk: oh for wallaby it's the docs that fail now14:18
abhishekkcorrect14:19
croelandtalways a new failure14:19
abhishekk:P14:19
abhishekkI can push a patch to train to remove unnecessary jobs14:19
pdeoreabhishekk, Thanks !14:20
rosmaitai would keep at least one tempest job, though14:20
abhishekkack14:20
jokke_my ocncern is, why do we bother to keep the branches, if we don't test them14:20
jokke_concern14:20
rosmaitawell, we do test them14:21
rosmaitai think grenade doesn't make sense, but tempest does14: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:P14:21
rosmaitabut the main reason is that you can't post reviews of patches to a deleted branch in gerrit14:21
croelandtThis is probably something the whole OpenStack community should discuss with packagers14:21
abhishekk++14:22
croelandtWho uses these branches? How useful are they?14:22
croelandtGet people from Debian, Fedora, Ubuntu etc.14:22
croelandtsee how they use them14:22
rosmaitawell, zigo was on the ML, just repackaged rocky for debian to include a CVE fix14:22
rosmaitaand from comments on that bug, looks like a lot of interest in rocky currently14:23
croelandtdamn, Debian stable truly is in the past14:23
rosmaita"that bug" == https://bugs.launchpad.net/cinder/+bug/199618814:23
abhishekkno further arguments14: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 downstream14:24
jokke_They were supposed to be 0 effort for the project teams, which clearly is not the case14:25
rosmaitai don't think *anything* is supposed to be 0 effort for project teams!14:26
croelandtyeah so maybe we could discuss having a reduced set of tests14:26
croelandtthe CI is always failing for some reason14:26
croelandtand usually that reason is not an issue with the actual code14:26
croelandtbut rather infrastructure-related issues14:26
abhishekk+114:27
rosmaitareduced tests is fine, though eventually you get to the point where like jokke_ says, why bother14:27
rosmaitaso i think reduce tests, and when we get to that point, have the EOL discussion14:28
croelandtrosmaita: how would we start that conversation? I think it needs to be an Openstack-wide policy14:28
rosmaitait already is14:28
abhishekkI think its better to take this topic in upcoming PTG either in TC or glance schedule14:28
rosmaitaread 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 EM14:29
rosmaitahttps://docs.openstack.org/project-team-guide/stable-branches.html#extended-maintenance14:29
pdeoreabhishekk++14:29
abhishekkrosmaita, ack14:30
rosmaitacinder can EOL all the way up to wallaby if it wants to14:30
croelandtok14:30
rosmaitathough i think some peoples' employers would have issue with that 14:30
rosmaita(if you know what i mean)14:30
abhishekk:D14:30
abhishekkLet's move ahead, keep watch on ussuri and wallaby fixes and then get the patches merged14:32
pdeorethanks :) moving ahead14:32
abhishekkTrain we can decide what to drop and what to keep on the review itself14:32
pdeore#topic Glance cinder store changes14:32
pdeoreso glance store release is in a week (10th Feb last date), so kindly please have a look at below patches on priority14:33
pdeoreRefactor: https://review.opendev.org/c/openstack/glance_store/+/843103, this one is required for extending attched volumes support patch14:34
pdeoreExtend attached volumes: #link https://review.opendev.org/c/openstack/glance_store/+/86874214:34
pdeoreGlance test fix: #link https://review.opendev.org/c/openstack/glance/+/86902114:34
abhishekkack14:36
pdeoreThat's it from me for today, let's move to Open discussion14:36
pdeore#topic Open Discussions14:36
abhishekkI don't have anything14:36
abhishekkjust fyi, PTL nomination is open14:37
pdeoreyeah :)14:37
pdeoretill 15th feb14:37
croelandtpdeore! pdeore! pdeore!14:37
pdeoreno choice :P14:38
pdeoreanyone has anythhing else to discuss or we are good to wrap up early?14:39
abhishekkthank you all!14:39
jokke_I'm good, thanks!14:40
croelandtthanks!14:40
pdeorecool, thanks everyone !!14:40
pdeore#endmeeting14:40
opendevmeetMeeting ended Thu Feb  2 14:40:25 2023 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:40
opendevmeetMinutes:        https://meetings.opendev.org/meetings/glance/2023/glance.2023-02-02-13.59.html14:40
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/glance/2023/glance.2023-02-02-13.59.txt14:40
opendevmeetLog:            https://meetings.opendev.org/meetings/glance/2023/glance.2023-02-02-13.59.log.html14:40
*** dasm is now known as dasm|off22:42

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