Monday, 2022-12-05

arne_wiebalckGood morning, Ironic!07:17
rpittaugood morning ironic! o/09:00
opendevreviewVanou Ishii proposed openstack/ironic master: Handle HTTP connection incompatibility in iRMC S6 2.x  https://review.opendev.org/c/openstack/ironic/+/86507409:11
opendevreviewVanou Ishii proposed openstack/ironic master: Handle IPMI incompatibility in iRMC S6 2.x  https://review.opendev.org/c/openstack/ironic/+/86507509:11
kubajjGood morning Ironic! o/09:13
dtantsurmorning folks09:29
dtantsurTheJulia: yeah, I think I even mention it in one of the comments: maybe it's easier to migrate to SA2.0 code right away09:30
arozmanHi, Ironic!09:30
dtantsurhi arozman 09:31
dtantsurkubajj: hey, did you get your questions answered last week?09:31
kubajjdtantsur: I think I did about the configuration of container name, but I am still not sure how to test whether the Swift contains what it is supposed to09:33
opendevreviewRiccardo Pittau proposed openstack/bifrost stable/xena: [DNM] test ci xena  https://review.opendev.org/c/openstack/bifrost/+/86655910:03
iurygregorymorning Ironic11:15
dtantsurkubajj: well, this probably involves mocking since you cannot have real swift in tests11:20
rpittauJayF dtantsur I think bifrost centos8 jobs in wallaby are foobar, and it probably depends on python3.6, do we want to spend time trying to fix them or just set them non-voting for the time being? maybe remove the integration job from ironic-prometheus-explorer?12:24
dtantsurle sigh...13:08
dtantsurrpittau: are they foobar because of CentOS vs Stream?13:09
rpittaudtantsur: I had the same doubt, but no, I've double-checked, it uses CS813:50
rpittauI'm comparing it to xena as I really don't know where to bump my head :/13:50
rpittauone big difference is the lbivrti-python version, but I think that's locked by upper constraints, also in focal the same version works fine13:53
opendevreviewVanou Ishii proposed openstack/ironic master: Handle IPMI incompatibility in iRMC S6 2.x  https://review.opendev.org/c/openstack/ironic/+/86507514:01
TheJuliablah14:56
TheJuliaCan it be friday now?14:56
iurygregory++14:57
iurygregorygood morning TheJulia 14:57
Kirill_Hi, i've written a new spec. can somebody review it?) https://review.opendev.org/c/openstack/ironic-specs/+/86653715:00
JayFGood morning o/ 15:01
JayF#startmeeting ironic15:01
opendevmeetMeeting started Mon Dec  5 15:01:20 2022 UTC and is due to finish in 60 minutes.  The chair is JayF. Information about MeetBot at http://wiki.debian.org/MeetBot.15:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:01
opendevmeetThe meeting name has been set to 'ironic'15:01
TheJuliao/15:01
Kirill_o/15:01
JayFGood morning; who all is here today15:01
ajyao/15:01
rlooo/15:02
JayF#topic Announcements/Reminder15:02
JayFAs always, hashtag patches that need review with #ironic-week-prio and please use our priorty review dashboard to guide your reviewing if you're a core.15:02
JayFI'm going to go hashtag the shard key spec; it's getting close and needs to land soon15:03
rpittauo/ (a bit alte)15:03
rpittauor even late15:03
JayF#topic Review action items from last meeting15:04
JayFI don't think we had any?15:04
TheJuliasame15:04
JayFYeah no actions15:04
JayF#topic Ironic CI Status + Update15:04
erbarro/15:04
JayFThanks to rpittau and iurygregory who have been looking at the last couple of stable branches to fix for Zuul config errors caused by the queue key.15:04
JayFI think we're down to two patches to land to finish it all up?15:05
JayFAny other observations about CI for the meeting?15:05
TheJuliarpittau did make a comment about wallaby and bifrost earlier15:05
rpittauah yes15:05
TheJuliasomething about it being fubar15:07
rpittauso bifrost centos8 jobs are foobar in wallaby15:07
rpittauas a consequence the integration job in ironic-prometheus-exporter is also not in a good shape15:08
TheJuliaDo we know if anyone is consuming i-p-e for wallaby?15:08
rpittauwondering if we want to spend time trying to fix them or just make them no-voting and remove it from i-p-e15:08
TheJuliaI'm +1 to removing from i-p-e15:09
JayFWallaby is EM. I'd suggest removing (don't NV them unless someone is going to fix some day) those jobs if they aren't possible to keep around anymore.15:09
TheJuliait is not great, but it is also a very simple plugin15:09
JayFWe also extremely rarely backport to IPE15:09
JayFI don't have a problem asking someone to manually test if we do one15:09
TheJulia++15:09
iurygregoryo/15:10
rpittausounds good15:10
JayF#action rpittau to remove centos8 jobs from bifrost/IPE wallaby15:10
* iurygregory is in two meetings at same time15:10
rpittauwill do :)15:10
JayFAnything else on ci?15:10
JayFMoving on...15:11
JayF#topic 2023.1 WIP15:11
JayF#link https://etherpad.opendev.org/p/IronicWorkstreams2023.115:11
JayFI updated this with something for shards; we really need that spec to land as implemnetation is 50%+ done and nova side implementation is going to be moving soon15:12
TheJuliaAgreed15:12
JayFPlease review the WIP etherpad for status and update it if you're working on something15:13
JayFmoving on15:13
JayF#topic Bugfix branch retirement15:13
TheJuliaWe could always count an external review/sanity from a known contributor as positive feedback to move it forward. FWIW15:13
JayFTheJulia: there's no acceptable reason why we don't have ironic core reviews on that; folks who are cores need to do the reviews they are signed up to do15:14
TheJuliaTrue15:14
JayFOK; bugfix release branch retirement15:14
JayFtalking to releases team; we'll have to get some ACLs added for us to be able tohandle that ourselves15:14
JayFthis will include ability to del bugfix branches and make new tags15:14
JayFI made this topic because I'm thinking; that might need to be a smaller group than ironic-core15:15
rloo(might be useful to add links to that IronicWokstreams2023.1 etherpad. I'll try to find time to find the shard thing and review ;) )15:15
JayFack 15:15
JayFWhat do folks think about that? Making a smaller group for bugfix branch mangement?15:15
rlooare those ACLs on a per-person basis?15:16
JayFit'd be on a per-group basis15:16
JayFe.g. ironic-core or $new-group-I-am-proposing-now15:16
rloosweet. i think that'll be great.15:16
rpittauJayF: sounds like a good idea15:17
JayFObjections to me making it a smaller group? Just go logistically there aren't a dozen creds out there which can destroy branches15:17
JayFI was going to suggest release liasons for that access? dtantsur rpittau iurygregory and me?15:17
rpittauworks for me15:17
dtantsurmakes sense15:17
TheJuliaSounds good to me15:17
JayF#action Jay to create new ironic-releasers group with him, dtantsur rpittau and iurygregory, and give it the access to manage bugfix releases.15:18
JayFThat also means I will act on retiring bugfix branches next week ifI get that in15:18
rloobtw -- how is that diff from ironic-stable-maint?15:18
JayFstable-maint is only for merging in changes to stable branches15:19
rloowhich, umm, is really old so maybe no one uses that: https://review.opendev.org/admin/groups/950,members15:19
JayFthis credential will be able to delete old bugfix branches and tag them as EOL15:19
JayFdirectly in git15:19
JayFnot just gerrit review15:19
JayFso it's a pretty big hammer15:19
* iurygregory reading now15:19
rlooin that case, makes sense to have a separate one/group.15:20
TheJuliastable-maint groups were largely changed to be the whole core group for most (all?) projects a few years back15:20
rlooah, thx TheJulia, it is coming back now!15:20
dtantsuryeah, I think it's legacy at this point15:21
JayF#action Jay will retire branches listed in here if ACLs get setup properly https://etherpad.opendev.org/p/IronicBugfixBranchCleanup15:21
JayFThis is what was already mentioned on the list, but please have a look.15:21
JayFMoving on if there are no further questions?15:21
JayF#topic RFE Review15:22
JayFI saw Kirill_ was talking about a new spec15:22
JayFthis is a good place to plug that 15:22
TheJulia++15:22
JayFWell, scroll up to just before the meeting if you wanna see it :D 15:25
JayF#topic Open Discussion15:25
JayFanything for open discussion before we call it quits?15:25
TheJuliayes15:25
TheJuliaSo, we are fast approaching the end of year, and traditionally, we've shut down meetings from mid-december through like the second week in January as a number of contributors need to use PTO or loose it.15:25
TheJuliaSo generally we've lacked quorum for meetings for a few weeks15:26
JayFSo maybe cancel 12/26, 1/2 ?  12/19 as well maybe?15:26
JayFI wouldn't be here for 12/26 myself anyway :) 15:26
TheJuliaYeah, likely15:26
JayFWDYT about 12/19?15:27
TheJuliaI might not be around personally15:27
TheJuliaI don't know about others15:27
JayFI'll be here, lets just have one 12/19 worst case I just talk to myself and the bot for 10 minutes and walk through it.15:27
TheJuliaor just go "well, no quorum, that's all folks!"15:28
TheJulia;)15:28
JayFEh; our meetings are mostly informational and rarely decision-y15:28
TheJuliaIt can vary, but yeah15:28
JayF#action JayF to cancel meetings 12/26 and 1/2 so people can worry about PTO and holidays and not Ironic15:28
rloodo we have quorum so we can vote for that? ha ha15:29
TheJulialol15:29
JayFI think I know personally enough contributors to have quorum that none of them would complain about a cancelled meeting lol15:29
JayFlike asking a kid on halloween if they want candy :D 15:29
opendevreviewMerged openstack/ironic-inspector bugfix/11.0: Fix compatibility with oslo.db 12.1.0  https://review.opendev.org/c/openstack/ironic-inspector/+/85650515:30
JayFAnything else for Open Discussion?15:30
TheJulianot from me15:31
JayFAight, thanks for coming all, see you next week o/15:31
JayF#endmeeting ironic15:31
opendevmeetMeeting ended Mon Dec  5 15:31:59 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:31
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-12-05-15.01.html15:31
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-12-05-15.01.txt15:31
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-12-05-15.01.log.html15:31
JayFOne note; I'm feeling pretty crummy this morning. I may be sicking out shortly.15:32
TheJulia:(15:32
TheJuliafeel better15:32
TheJuliaI'm feeling exhausted... flying, family, flying15:32
JayFYeah I think Vanessa gave me something from her students15:33
TheJuliaugh15:33
JayFwe've both felt kinda crappy all weekend; I think maybe we have flu-but-have-a-flu-shot15:34
JayFwhere you're not like SICK SICK but you're feeling bad and tired as your body fights off whatever15:34
TheJuliaInfluenza A is spreading like crazy here15:34
opendevreviewRiccardo Pittau proposed openstack/bifrost stable/wallaby: Remove CentOS Stream 8 based ci jobs  https://review.opendev.org/c/openstack/bifrost/+/86659615:48
opendevreviewRiccardo Pittau proposed openstack/ironic-prometheus-exporter stable/wallaby: Remove bifrost integration job  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86659715:49
opendevreviewRiccardo Pittau proposed openstack/bifrost stable/wallaby: Remove CentOS Stream 8 based ci jobs  https://review.opendev.org/c/openstack/bifrost/+/86659615:50
TheJulialooks like we have python 3.11 issues :(16:15
rpittau\o/16:27
rpittaugood night! o/17:01
opendevreviewDmitry Tantsur proposed openstack/sushy master: Handle a different error code for missing TransferProtocolType  https://review.opendev.org/c/openstack/sushy/+/86661217:30
dtantsurjanders: for when you're awake and ready to deal with more redfish ^^^17:33
opendevreviewRiccardo Pittau proposed openstack/ironic-prometheus-exporter stable/wallaby: Remove bifrost integration job  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86659719:01
stevebaker[m]good morning19:25
-opendevstatus- NOTICE: The lists.opendev.org and lists.zuul-ci.org sites will be offline briefly for migration to a new server20:01
opendevreviewSteve Baker proposed openstack/ironic stable/zed: Align iRMC driver with Ironic's default boot_mode  https://review.opendev.org/c/openstack/ironic/+/86662521:57
opendevreviewSteve Baker proposed openstack/ironic stable/yoga: Align iRMC driver with Ironic's default boot_mode  https://review.opendev.org/c/openstack/ironic/+/86662621:58
opendevreviewSteve Baker proposed openstack/ironic stable/xena: Align iRMC driver with Ironic's default boot_mode  https://review.opendev.org/c/openstack/ironic/+/86662721:59
opendevreviewSteve Baker proposed openstack/ironic stable/wallaby: Align iRMC driver with Ironic's default boot_mode  https://review.opendev.org/c/openstack/ironic/+/86662822:00

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