Monday, 2022-10-03

arne_wiebalckGood morning, Ironic!06:58
rpittaugood morning ironic! o/07:02
arne_wiebalckhey rpittau o/07:03
rpittauhey arne_wiebalck :)07:03
rpittauJayF: metal3 does not use bugfix branches, it gets RPMs directly from RDO, on the other hand though Openshift does use bugfix branches -> https://pastebin.com/raw/j6FG10QE07:09
arozmanGood morning Ironic!07:15
opendevreviewWill Szumski proposed openstack/networking-generic-switch master: add DellOS10 device type with updated commands  https://review.opendev.org/c/openstack/networking-generic-switch/+/86006708:50
Nisha_AgarwalJayF, hi09:16
opendevreviewWill Szumski proposed openstack/networking-generic-switch master: add DellOS10 device type with updated commands  https://review.opendev.org/c/openstack/networking-generic-switch/+/86006709:38
opendevreviewWill Szumski proposed openstack/networking-generic-switch master: add DellOS10 device type with updated commands  https://review.opendev.org/c/openstack/networking-generic-switch/+/86006709:52
opendevreviewWill Szumski proposed openstack/networking-generic-switch master: Add support for DellOS10  https://review.opendev.org/c/openstack/networking-generic-switch/+/86006710:00
opendevreviewWill Szumski proposed openstack/networking-generic-switch master: Add support for DellOS10  https://review.opendev.org/c/openstack/networking-generic-switch/+/86006710:03
opendevreviewWill Szumski proposed openstack/networking-generic-switch master: Add support for Dell OS10  https://review.opendev.org/c/openstack/networking-generic-switch/+/86006710:03
arozmaniurygregory: Hi could you please re-review this https://review.opendev.org/c/openstack/ironic-python-agent/+/855866 for me? (I have added the release notes that you have asked for)11:16
iurygregoryarozman, tks I will try to take a look during this week11:21
arozman iurygregory: thanks a lot!11:22
rpittauarozman: I left a comment on the reno11:22
arozmanrpittau: thanks11:23
opendevreviewAdam Rozman proposed openstack/ironic-python-agent master: prioritize lsblk as a source of device serials  https://review.opendev.org/c/openstack/ironic-python-agent/+/85586611:27
opendevreviewMerged openstack/ironic master: Imported Translations from Zanata  https://review.opendev.org/c/openstack/ironic/+/86004711:36
opendevreviewMerged openstack/bifrost master: Use openstacksdk<0.99.0 and a-c-o<2.0.0 on bifrost-*collections-* jobs  https://review.opendev.org/c/openstack/bifrost/+/85998412:36
TheJuliagood morning13:03
*** rcastillo is now known as rcastillo|ruck13:29
iurygregorymorning 13:51
TheJuliaoh shoot... 13:57
TheJulianame is 255 chars in nodes... WUT13:58
iurygregorythis is bad?13:59
TheJuliawell...14:02
TheJuliawhy does allocations hate us then14:02
TheJuliabahahahahahaha14:05
TheJuliaI *see* what is going on14:06
TheJuliaI think.14:06
TheJuliashoot...14:06
TheJuliawelp, down to one error14:21
iurygregoryyay14:21
TheJuliabut... same error, not same root it *seems*14:24
TheJuliaalthough I think this is worse...14:24
iurygregory=(14:29
TheJuliablarg14:30
rpittaubye everyone o/14:31
Nisha_AgarwalTheJulia, hi14:33
Nisha_AgarwalTheJulia, i proposed even https://review.opendev.org/c/openstack/ironic/+/860055 for PXE boot to work with url based Anaconda deploy14:34
TheJuliahttps://www.irccloud.com/pastebin/cLIILJnl/We%20missed%20the%20charset%20upfront%2C%20so%20the%20default%20charset%20on%20the%20table%20is%20whatever%20the%20db%20default%20is%20%20%F0%9F%98%AD%F0%9F%98%AD%F0%9F%98%AD%F0%9F%98%AD14:37
TheJuliaNisha_Agarwal: OH! Grub based!14:37
Nisha_AgarwalTheJulia, yes plus there is a issue with hosting the kickstart file in the httproot directory for the kickstart to be downloadable by the baremetal 14:39
TheJuliawith tftp using grub14:39
TheJuliayeah, that makes sense now14:39
JayF#startmeeting ironic15:00
opendevmeetMeeting started Mon Oct  3 15:00:05 2022 UTC and is due to finish in 60 minutes.  The chair is JayF. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'ironic'15:00
JayFGood morning! Who all is here o/15:00
ajyao/15:00
rlooo/15:00
hjensaso/15:00
iurygregoryo/15:01
JayF#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting 15:01
JayFlooks like a very light agenda for today; going to try and get us through it quickly15:01
JayF#topic Announcements/Reminder15:01
TheJuliao/15:01
JayF#note PTG is in two weeks, schedule is at https://ptg.opendev.org/ptg.html15:02
JayF#note There is one remaining session to be scheduled: a joint session about the Nova->Ironic driver, which will be scheduled tomorrow at Nova's meeting15:02
erbarro/15:02
JayFwhen that's done; I'll email the list with an ICS file you all can import to your calendars to make sure nobody misses it15:02
JayFdoes anyone have an announcement or reminder that was not on the agenda?15:02
iurygregoryJayF, it would be good to add the schedule in the etherpad we will use for the PTG + email to the ML o/15:03
JayF#action JayF to add the PTG schedule to the etherpad when sending out final schedule to list15:03
iurygregoryThis week is week the Zed release will be announced if I recall correctly 15:03
JayFiurygregory: good suggestion; I've marked that here and will put it on my list15:03
JayF#note Zed release is being cut this week. There are no outstanding action items for Ironic for Zed release afaik.15:04
JayFiurygregory: that reminded me about something else too15:04
iurygregorynot sure if last week I've mentioned this in the meeting, but probably good to15:05
iurygregoryhttps://review.opendev.org/c/openstack/governance/+/85946415:05
JayF#note Thursday, 1400 UTC, there will be an OpenInfra live highlighting features from Zed. I'll be there representing Ironic and invite anyone else who wants to join to attend. 15:05
JayFiurygregory: ack; that can merge today15:06
iurygregory\o/15:06
JayF#note A resolution is expected to pass the TC to dedicate the Zed release to Ilya Etingof.15:06
JayFIf there is nothing else; lets move on?15:06
TheJuliahttps://www.youtube.com/watch?v=MSbB3L9_MeY <--  Link for that stream15:06
JayF#link OpenInfra Live at https://www.youtube.com/watch?v=MSbB3L9_MeY15:07
JayF#topic Review action items from previous meeting15:07
JayFSadly, it looks like the bot didn't get all the items from our last meeting. I haven't checked raw IRC logs to confirm; but I don't recall any actions from last week?15:07
JayFGoing to take that as a no and move on. 15:08
JayF#topic Subteam status report review15:08
iurygregoryafaik no action items required15:08
JayF#link https://etherpad.opendev.org/p/IronicWhiteBoard15:09
JayFI request that anyone who has a subteam status report on here; please clean up any items that are only relevant for Zed release. I anticipate we'll have more items come up once PTG is over.15:09
JayFI see no updates from this week on the whiteboard; understandable because of release activities. 15:10
JayFI will move on if there is no comment?15:10
TheJuliasounds good15:10
JayF#topic  Deciding on priorities for the coming wee'k15:10
JayFIf you have any patches that need attention, please be sure to add hashtag: #ironic-week-prio 15:11
TheJuliaI'd like to get eyes on our multibyte db issues, but that likely won't be until later in the week and will add the hashtag then15:11
JayFAgain, I don't expect much movement here due to most contributors working on release activities.15:11
TheJuliaSeems it is more... complex... than shortening the field.15:11
JayFI appreciate that you're digging in :) I look forward to reading that change15:11
JayFMoving on...15:12
JayF#topic Baremetal SIG15:12
JayF#note the next Baremetal SIG meeting will be an Operator Hour at the PTG. I'll send out official notice and an ICS file to the list this week. 15:12
JayFThat should be it for the BM SIG, moving on.15:13
JayF#topic RFE Review15:13
JayFNo RFEs listed here for review; please remember to add any RFE bugs you want approved to the agenda for review.15:13
JayF#topic Open Discussion15:13
JayFAnything else from anyone?15:13
JayFI'll note that generally I'm going to try to run these meetings pretty quickly in the future so we waste as little time as possible. Please ensure you've got agenda items up, or attention on the meeting, if you have something to bring up :).15:15
JayFI'd rather have us full-attention pointed this way for 15-30 minutes than have it spread over almost the full hour for the same content15:15
TheJuliaSounds good to me!15:15
iurygregory++15:16
JayFLast call for open discussion, or meeting items of any kind15:16
JayF#endmeeting15:17
opendevmeetMeeting ended Mon Oct  3 15:17:53 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:17
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-10-03-15.00.html15:17
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-10-03-15.00.txt15:17
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-10-03-15.00.log.html15:17
hjensasThanks JayF!15:18
JayFno problem, thanks for coming15:18
TheJuliaokay, I *might* have found the other issue, running tests now15:30
opendevreviewNisha Agarwal proposed openstack/ironic master: Fixes anaconda deploy for PXE boot  https://review.opendev.org/c/openstack/ironic/+/86005515:34
TheJuliaeureka16:04
iurygregory\o/16:10
opendevreviewAdam Rozman proposed openstack/ironic-python-agent master: prioritize lsblk as a source of device serials  https://review.opendev.org/c/openstack/ironic-python-agent/+/85586616:26
JayFGenerally taking the temp: what do folks think of retiring all EM branches older than train? (queens/rocky/stein)21:36
TheJulia+100021:39
opendevreviewJay Faulkner proposed openstack/ironic stable/stein: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic/+/86015821:39
opendevreviewJay Faulkner proposed openstack/ironic stable/rocky: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic/+/86015921:40
opendevreviewJay Faulkner proposed openstack/ironic stable/queens: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic/+/86016021:40
JayFWe also created all these bugfix/ branches21:41
JayFand don't seem to have a documented way to take them out of service21:41
JayFso we have literally a dozen+ in some projects of broken branches21:41
TheJulia:(21:41
JayFCI on ironic-lib broken everywhere, CI on IPA older branches broken21:41
TheJuliaugh :(21:41
JayFbefore we even get to the bugfix/* branches showing as broken in zuul configs, but we treat as retired21:41
JayFthis is untenable21:42
TheJuliawe need to just kill the un-needed branches at this point, this is the third consistent discussion in a year21:42
opendevreviewJay Faulkner proposed openstack/ironic-inspector stable/yoga: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86016121:46
opendevreviewJay Faulkner proposed openstack/ironic-inspector stable/xena: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86016221:47
opendevreviewJay Faulkner proposed openstack/ironic-inspector stable/wallaby: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86016321:47
opendevreviewJay Faulkner proposed openstack/ironic-inspector stable/victoria: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86016421:48
opendevreviewJay Faulkner proposed openstack/ironic-inspector stable/ussuri: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86016521:48
opendevreviewJay Faulkner proposed openstack/ironic-inspector stable/train: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86016621:48
opendevreviewJay Faulkner proposed openstack/ironic-inspector stable/stein: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86016721:48
opendevreviewJay Faulkner proposed openstack/ironic-inspector stable/rocky: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86016821:49
opendevreviewJay Faulkner proposed openstack/ironic-inspector stable/queens: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86016921:49
opendevreviewJay Faulkner proposed openstack/ironic-inspector bugfix/10.9: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86017021:49
opendevreviewJay Faulkner proposed openstack/ironic-inspector bugfix/10.7: CI: Zuul no longer respects queue  https://review.opendev.org/c/openstack/ironic-inspector/+/86017121:49
opendevreviewJay Faulkner proposed openstack/ironic-lib stable/yoga: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-lib/+/86017221:51
opendevreviewJay Faulkner proposed openstack/ironic-lib stable/xena: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-lib/+/86017321:51
opendevreviewJay Faulkner proposed openstack/ironic-lib stable/wallaby: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-lib/+/86017421:51
opendevreviewJay Faulkner proposed openstack/ironic-lib stable/victoria: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-lib/+/86017521:52
opendevreviewJay Faulkner proposed openstack/ironic-lib stable/ussuri: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-lib/+/86017621:52
opendevreviewJay Faulkner proposed openstack/ironic-lib stable/stein: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-lib/+/86017721:53
opendevreviewJay Faulkner proposed openstack/ironic-lib stable/rocky: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-lib/+/86017821:53
JayFTheJulia: I'm going to check for existing PRs before I go further21:53
JayFTheJulia: strangely; train was landed in ironic-lib but nothing else was21:53
JayFwhich is really weird21:53
JayFwell, ironic-prometheus-exporter has none open so that is ez to check lol21:54
opendevreviewJay Faulkner proposed openstack/ironic-prometheus-exporter stable/yoga: CI: Zuul no longer respects queue key  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86017921:56
opendevreviewJay Faulkner proposed openstack/ironic-prometheus-exporter stable/xena: CI: Zuul no longer respects queue key  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86018021:56
TheJuliathe odds that jobs on the super old branches work (like the ones which should have been closed out) working is kin dof low... fwiw21:56
opendevreviewJay Faulkner proposed openstack/ironic-prometheus-exporter stable/wallaby: CI: Zuul no longer respects queue key  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86018121:56
opendevreviewJay Faulkner proposed openstack/ironic-prometheus-exporter stable/victoria: CI: Zuul no longer respects queue key  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86018221:56
jandersdtantsur JayF thank you for sorting out the backports of https://review.opendev.org/c/openstack/ironic/+/859916 - these fell through the cracks somehow when I was working on this fix21:57
JayFTheJulia: a PR proving that is the best ammo for a ML list to retire it21:57
opendevreviewJay Faulkner proposed openstack/ironic-prometheus-exporter stable/ussuri: CI: Zuul no longer respects queue key  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86018321:57
opendevreviewJay Faulkner proposed openstack/ironic-prometheus-exporter stable/train: CI: Zuul no longer respects queue key  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86018421:58
opendevreviewJay Faulkner proposed openstack/ironic-python-agent stable/yoga: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86018522:08
opendevreviewJay Faulkner proposed openstack/ironic-python-agent stable/xena: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86018622:09
opendevreviewJay Faulkner proposed openstack/ironic-python-agent stable/wallaby: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86018722:09
opendevreviewJay Faulkner proposed openstack/ironic-python-agent stable/victoria: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86018822:09
opendevreviewJay Faulkner proposed openstack/ironic-python-agent stable/ussuri: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86018922:09
opendevreviewJay Faulkner proposed openstack/ironic-python-agent stable/train: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86019022:09
opendevreviewJay Faulkner proposed openstack/ironic-python-agent stable/stein: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86019122:09
opendevreviewJay Faulkner proposed openstack/ironic-python-agent stable/rocky: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86019222:09
opendevreviewJay Faulkner proposed openstack/ironic-python-agent stable/queens: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86019322:10
opendevreviewJay Faulkner proposed openstack/ironic-python-agent bugfix/8.1: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86019422:13
opendevreviewJay Faulkner proposed openstack/ironic-python-agent bugfix/8.3: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86019522:13
opendevreviewJay Faulkner proposed openstack/ironic-python-agent bugfix/8.6: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86019622:13
* TheJulia is starting to wonder if the original report was truncated22:26
TheJuliathen again, a bunch of these branches are the ones I thought we were deleting a while back22:29
TheJuliaso likely just me22:29
JayFIt also seems from chatting w/clarkb22:38
JayFthere is no process for retiring the bugfix/x branches that are too old to be maintained22:39
JayFthose are also in the list of broken CI stuff; and I have that list locally to investigate how to eol22:39
JayFprobably just a tag? IDK22:39
opendevreviewMerged openstack/ironic-python-agent bugfix/8.6: CI: Zuul no longer respects queue param  https://review.opendev.org/c/openstack/ironic-python-agent/+/86019622:46
TheJuliayeah, I think that was what we discussed22:58
opendevreviewMerged openstack/ironic-prometheus-exporter stable/yoga: CI: Zuul no longer respects queue key  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86017923:17
opendevreviewMerged openstack/ironic-prometheus-exporter stable/xena: CI: Zuul no longer respects queue key  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86018023:20
opendevreviewMerged openstack/ironic-prometheus-exporter stable/victoria: CI: Zuul no longer respects queue key  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/86018223:33
opendevreviewJulia Kreger proposed openstack/ironic master: Fix allocations default table type  https://review.opendev.org/c/openstack/ironic/+/86019823:34
opendevreviewJulia Kreger proposed openstack/ironic master: Fix allocations default table type  https://review.opendev.org/c/openstack/ironic/+/86019823:34

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