Monday, 2022-03-07

rpittaugood morning ironic! o/07:35
arne_wiebalckGood morning rpittau and Ironic!07:45
rpittauhey arne_wiebalck :)07:45
rpittauarne_wiebalck: I added your announcement (postponing the talk)  to the baremetal-sig part of the ironic meeting for today in case we want to expand the discussion that happened on Friday09:00
arne_wiebalckrpittau: thanks!09:00
MahnoorAsgharGood morning o/09:25
opendevreviewMerged openstack/tenks master: [TrivialFix] Use correct project badge in README  https://review.opendev.org/c/openstack/tenks/+/83215709:26
iurygregorygood morning Ironic10:39
dtantsurmorning folks10:40
dtantsurcould I get a 2nd +2 on the bifrost CI fix? https://review.opendev.org/c/openstack/bifrost/+/83177110:41
iurygregorylooking now10:42
rpittauhey iurygregory dtantsur :)11:04
iurygregoryrpittau, o/11:06
ajyaHi dtantsur , still can't reproduce the failure on my 13G system. I suggest to contact dell support directly. It's odd that issue does not go away after rebooting idrac, even for a while.11:41
dtantsuryeah, thanks, we'll pass it on. janders ^^11:42
jandersgood morning / evening Ironic o/11:44
jandersdtantsur ajya noted!11:44
janders(sorry comms problems tonight, been trying to firmware-update my backup router several times to no avail - unfortunately it's out of warranty)11:45
jandersinterestingly I crashed it while uploading iDRAC firmware to a lab server :D11:45
jandersI suppose I gotta be easy on it till I figure it out another day11:45
opendevreviewMerged openstack/bifrost master: Fix the CI on CentOS 9  https://review.opendev.org/c/openstack/bifrost/+/83177111:53
TheJuliagood morning13:49
rpittaugood morning TheJulia :)13:50
MahnoorAsgharGood morning o/14:04
opendevreviewOpenStack Release Bot proposed openstack/ironic-lib stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/ironic-lib/+/83223114:07
opendevreviewOpenStack Release Bot proposed openstack/ironic-lib stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/ironic-lib/+/83223214:07
opendevreviewOpenStack Release Bot proposed openstack/ironic-lib master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/ironic-lib/+/83223314:07
opendevreviewOpenStack Release Bot proposed openstack/metalsmith stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/metalsmith/+/83223414:07
opendevreviewOpenStack Release Bot proposed openstack/metalsmith stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/metalsmith/+/83223514:07
opendevreviewOpenStack Release Bot proposed openstack/metalsmith master: Update master for stable/yoga  https://review.opendev.org/c/openstack/metalsmith/+/83223614:07
opendevreviewOpenStack Release Bot proposed openstack/metalsmith master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/metalsmith/+/83223714:07
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/83223814:08
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/83223914:08
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Update master for stable/yoga  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/83224014:08
opendevreviewOpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/83224114:08
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/python-ironicclient/+/83224214:08
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/python-ironicclient/+/83224314:08
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient master: Update master for stable/yoga  https://review.opendev.org/c/openstack/python-ironicclient/+/83224414:08
opendevreviewOpenStack Release Bot proposed openstack/python-ironicclient master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/python-ironicclient/+/83224514:09
opendevreviewOpenStack Release Bot proposed openstack/sushy stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/sushy/+/83224614:09
opendevreviewOpenStack Release Bot proposed openstack/sushy stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/sushy/+/83224714:09
opendevreviewOpenStack Release Bot proposed openstack/sushy master: Update master for stable/yoga  https://review.opendev.org/c/openstack/sushy/+/83224814:09
opendevreviewOpenStack Release Bot proposed openstack/sushy master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/sushy/+/83224914:09
iurygregoryspam time =)14:17
dtantsuryeah :)14:17
opendevreviewRuby Loo proposed openstack/ironic bugfix/20.0: More fixes for anaconda deploy interface  https://review.opendev.org/c/openstack/ironic/+/83186514:30
rloogood morning :) Sorry if this is a dumb question, I've forgotten how we do things. If something is on master and we backport, do we backport to bugfix 20.0 and 19.0 or just 20.0 ?14:32
dtantsurrloo: at least the last one, preferably the last two14:36
dtantsursome of us have vested interest in more branches, but these are optional14:36
rloothx dtantsur! i'll do the last two :)14:37
opendevreviewRuby Loo proposed openstack/ironic bugfix/19.0: More fixes for anaconda deploy interface  https://review.opendev.org/c/openstack/ironic/+/83186714:38
rloo(incentive to get your PRs landed b4 moar bugfix branches are created! ha ha)14:39
TheJuliasooo many emails this morning :(14:49
iurygregoryyeah =(14:52
iurygregoryI'm trying to not go crazy with the amount of emails and docs I need to read today XD14:53
TheJuliaheh14:59
opendevreviewMerged openstack/metalsmith master: Update master for stable/yoga  https://review.opendev.org/c/openstack/metalsmith/+/83223615:18
opendevreviewMerged openstack/python-ironic-inspector-client master: Update master for stable/yoga  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/83224015:18
opendevreviewMerged openstack/sushy master: Update master for stable/yoga  https://review.opendev.org/c/openstack/sushy/+/83224815:20
opendevreviewMerged openstack/bifrost master: Fetch uuid if it is not present in the inventory  https://review.opendev.org/c/openstack/bifrost/+/82955415:26
opendevreviewDmitry Tantsur proposed openstack/bifrost stable/xena: Fetch uuid if it is not present in the inventory  https://review.opendev.org/c/openstack/bifrost/+/83186915:29
dtantsurrpittau: we didn't have stream 9 in xena, did we?15:30
dtantsurmorning TheJulia, rloo, MahnoorAsghar 15:30
rloogooood morning dtantsur :)15:31
MahnoorAsgharmorning o/15:31
rpittaudtantsur: we didn't15:31
opendevreviewHarald JensÃ¥s proposed openstack/networking-baremetal master: WIP - OpenConfig YANG and Netconf  https://review.opendev.org/c/openstack/networking-baremetal/+/83226815:35
dtantsurk15:41
opendevreviewMahnoor Asghar proposed openstack/ironic master: [WIP] Create API documentation from docstrings  https://review.opendev.org/c/openstack/ironic/+/82720015:45
opendevreviewMerged openstack/ironic bugfix/19.0: Set correct initrd_filename for iPXE when using Swift  https://review.opendev.org/c/openstack/ironic/+/82918715:47
opendevreviewMerged openstack/ironic bugfix/19.0: Inspector: better error message on DiscoveryFailure  https://review.opendev.org/c/openstack/ironic/+/82733515:47
opendevreviewMerged openstack/ironic-lib stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/ironic-lib/+/83223115:47
opendevreviewMerged openstack/ironic-lib stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/ironic-lib/+/83223215:47
* iurygregory has the feeling we will have a merge spam during the meeting :D15:55
TheJuliain accordance with the prophecy15:57
rpittau#startmeeting ironic16:00
opendevmeetMeeting started Mon Mar  7 16:00:09 2022 UTC and is due to finish in 60 minutes.  The chair is rpittau. Information about MeetBot at http://wiki.debian.org/MeetBot.16:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
opendevmeetThe meeting name has been set to 'ironic'16:00
TheJuliao/16:00
rlooo/16:00
rpiosoo/16:00
erbarro/16:00
iurygregoryo/16:00
ajyao/16:00
rpittauHello ironicers, welcome to our weekly meeting!16:00
rpittauThe meeting agenda can be found here:16:01
rpittau#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting16:01
arne_wiebalcko/16:01
rpittau#topic Announcements / Reminders16:02
rpittauFirst news is old news, Yoga is due to be released at the end of March, in 3 weeks!16:02
iurygregoryI think the first RC is next week16:02
rloo(time flies...)16:03
* iurygregory checks16:03
rpittauiurygregory: I think you're right :)16:03
rpittauRight after that we'll have the Zed PTG, at the beginning of April16:03
iurygregoryops, it's this week the RC-116:03
rpittauah, well!16:03
iurygregoryMar 07 - Mar 1116:03
rpittauZed PTG April 4 - 816:03
rpittauwe also have as shiny etherpad for the occasion https://etherpad.opendev.org/p/ironic-zed-ptg16:04
iurygregory#info Zed PTG April 4 - 816:04
iurygregory#link https://etherpad.opendev.org/p/ironic-zed-ptg16:04
rpittauadd your topics there and don't forget to register if you haven't done it yet16:04
opendevreviewMerged openstack/python-ironic-inspector-client master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/83224116:04
rpittaudoes anyone have anything else to announce/remind us of ?16:05
TheJuliahjensas: fyi, I added yang/netconf to the etherpad last week16:06
rpittaualright, we don't have any action times from last time, so moving on to next topic16:07
rpittau#topic Review subteam status reports16:07
rpittau#link https://etherpad.opendev.org/p/IronicWhiteBoard16:07
rpittauaround L6216:07
rpittaustendulker: for CS9 support, when you  have a minute please check my comments in https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/83090916:08
rpittaunope16:08
rpittaustevebaker[m]: ^^16:09
opendevreviewMahnoor Asghar proposed openstack/ironic master: [WIP] Create API documentation from docstrings  https://review.opendev.org/c/openstack/ironic/+/82720016:09
opendevreviewMerged openstack/sushy master: Fix session authentication issues  https://review.opendev.org/c/openstack/sushy/+/82132416:14
TheJulia*gasp*16:14
TheJulia\o/16:14
* TheJulia dances16:14
rpittau:)16:14
rpittaulooks like we're good with updates16:14
iurygregorynow we will need a backport XD16:15
rpittaumoving on16:15
rpittau#topic Deciding on priorities for the coming week16:15
rpittau#link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio16:15
rpittauif anyone has a moment this week please check https://review.opendev.org/c/openstack/ironic/+/819121 16:16
TheJuliaiurygregory: yeah... I guess I know what is next after some code reviews today :)16:16
iurygregoryXD16:16
TheJuliaIf there are no objections, I'll tag the sushy backports with the tag once I create them16:16
rpittauTheJulia: sounds good :)16:16
iurygregoryTheJulia, ++ 16:16
dtantsurI'd like to get https://review.opendev.org/c/openstack/bifrost/+/829517 and the following 2 patches on the list16:17
rpittaudtantsur: yup16:17
iurygregoryare we ok with cutting stable/yoga for projects like ironic-ui, IPE, ngs, networking-baremetal this week?16:18
rlooI've got two anaconda-related PRs https://review.opendev.org/c/openstack/ironic/+/827933/ (fix configdrive) and https://review.opendev.org/c/openstack/ironic/+/831964 (fix rebuilds). Would be good to get that into yoga.16:18
rpittauiurygregory: I don't see outsstanding patches there, so yes :)16:18
iurygregoryrloo, ++ please add the tag :D16:18
iurygregoryrpittau, yup, I don't think we will be able to introduce https://review.opendev.org/c/openstack/networking-baremetal/+/832268 in Yoga16:20
TheJuliaiurygregory: I think so (w/r/t projects noted)16:20
rpittauiurygregory: yeah that's for Zed16:20
iurygregorycool! I will proceed with the releases today =)16:21
TheJuliayeah, hjensas just started looking at that really recently16:21
rpittauwe should be good with the list for this week, last rush before the final release16:21
TheJuliaI would say, the ideas are still brewing16:21
TheJuliaAt some point, we will have glorious mead16:21
iurygregory\o/16:21
rpittauok, moving on16:23
rpittau#topic Discussion16:23
rpittau#info Zed PTG slots16:23
rpittau#link http://lists.openstack.org/pipermail/openstack-discuss/2022-March/027523.html16:23
rpittauiurygregory: mic is yours :)16:24
iurygregoryrpittau, ack =)16:24
opendevreviewMerged openstack/sushy master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/sushy/+/83224916:24
iurygregoryI've sent this email last week with some ideas on how we should have the slots for the Zed PTG16:25
* dtantsur is curious how many active APAC contributors we have left16:25
TheJuliafwiw, option 3 is what we've traditionally ended up with in the past, but we've often seen low APAC attendance16:25
iurygregoryyeah16:25
opendevreviewMerged openstack/sushy stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/sushy/+/83224616:25
rpittauif we include Australia and NZ, at least a couple :)16:25
dtantsuranother data point: Friday is also problematic for Israel16:26
dtantsuryeah, I'm not saying we don't have anyone :)16:26
iurygregorydtantsur, right good point!16:26
arne_wiebalckwhat about monday being a day for cross project and/or TC discussions ... was that an issue in the past? 16:26
dtantsuriurygregory: do you count as NA or APAC nowadays? :)16:26
dtantsurarne_wiebalck: yeah, last time we had a conflict16:27
* TheJulia suspects s/NA/Americas/16:27
arne_wiebalckdtantsur: I think TC is planning on Monday afternoon atm16:27
iurygregorydtantsur, NA =)16:27
dtantsurdo we have any specific hours for APAC morning?16:28
dtantsurI remember meeting stevebaker[m] online quite a few times when I stayed late16:28
dtantsur(will be harder for janders though)16:28
TheJuliaI  know I'm the most westerly person in our regular Americas contingent (afaik, we don't have any contributors in Alaska...), so I'm fine as long as I have advance notice scheduling wise16:28
TheJuliaSteve is also, in particular, mostly a morning person16:29
iurygregorydtantsur, it should be someting like 8am/9am for EMEA (based on what I talked with janders )16:29
TheJuliaat least, that is what I have gathered16:29
dtantsuriurygregory: you're talking about the evening slot (where stevebaker[m] will have tighter limitations than janders IIRC)16:29
dtantsur* their evening slot16:29
iurygregoryright16:29
TheJuliadtantsur: I believe that is correct16:30
dtantsuranyway, I sorta think #3 is fair enough, and maybe some EMEA people will even be able to participate in both slots16:30
opendevreviewMerged openstack/sushy stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/sushy/+/83224716:31
dtantsurnot that I sleep well nowadays anyway.....16:31
TheJuliawhat is this sleep word?16:31
opendevreviewMerged openstack/python-ironic-inspector-client stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/83223816:32
opendevreviewMerged openstack/python-ironic-inspector-client stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/83223916:32
iurygregoryit's a magic word with a very complicated definition 16:32
dtantsurso, Queensland has 9 hours more than central europe, it seems16:32
dtantsur(although it may change in the end of March?)16:32
dtantsurbut at least now, their 7am is what, 10pm for me? still doable a few times.16:33
rpittauthere's maybe one things to consider also, on March 27th Europe changes to CEST16:33
dtantsuryep, but we'll get closer to Australia this way?16:34
rpiosoThe US springs ahead next Sunday.16:34
TheJuliaApril 3rd, AU moves their clock back one hour16:35
dtantsurhmm, I've just realized NZ is further way to the east. damn.16:35
dtantsurTheJulia: https://www.timeanddate.com/time/zone/australia/queensland tells me Queensland doesn't have DST16:35
TheJuliahttps://www.timeanddate.com/time/change/australia16:35
TheJuliaoh, good16:35
dtantsuranyway. if my migraine-haunted head calculates right, we should have 1 hour that is reasonable for APAC and NA, and somewhat acceptable for EMEA16:36
rloowe should confirm that steve and anders want to attend...16:37
rpittaummmm16:37
rloo(or anyone else in APAC)16:37
iurygregoryjanders, and stevebaker[m] I think they will (not sure about other APAC contributors =( )16:38
dtantsurI think we should at least leave the opportunity. maybe not every day, maybe just Tue-Thu16:38
iurygregorysounds like a plan16:39
dtantsurworst case, some spec reviews will be done in that slot :)16:39
iurygregorysomething like Option #3 right?16:39
rpittauwe're talking about late at night or early in the morning anyway16:39
rpittauoption 3 sounds right16:40
iurygregoryrpittau, late at night for EMEA based on what dtantsur said (/me waits for confirmation)16:40
dtantsurignore EMEA, I guess. if we can make it - we can. yep, option 316:40
rpittauok16:41
iurygregoryyeah 16:41
iurygregorycool, I will talk later today with stevebaker[m] and janders to check their timezones and tomorrow I will provide some information in the thread16:42
rpittaugreat16:43
iurygregoryI think we can move on now o/16:43
rpittau2nd topic of discussion is connected:16:43
rpittau#info March 27th time change in EU, move the meeting 1 hour earlier?16:43
TheJuliaI've generally been waking up before 6 AM local, and moving the meeting one hour earlier wouldn't *really* impact me unless I want to sleep in.16:44
rpittauotherwise the meeting moves to 18:00 CEST16:44
arne_wiebalckrpittau: I'd be in favor to move it.16:45
iurygregoryI think it's ok to move back to 15 UTC if people are ok with it16:46
rpittaualright, unless there are objections at this point, or before next meeting, we will move the meeting at 15:00 UTC starting March 28th16:47
rpittauthanks!16:47
rpittaulet's move on16:47
rpittau#topic Baremetal SIG16:47
rpittau#link https://etherpad.opendev.org/p/bare-metal-sig16:47
arne_wiebalckWe do not have a speaker for tomorrow, short-term change of plans.16:48
arne_wiebalckSo, unless we have someone spontaneously stepping in, I suggest to cancel.16:48
TheJuliaseems reasonable to do so at this point16:49
TheJuliaApril's sounds really interesting16:49
dtantsurrpittau: sorry, responding late. whatever happens to the meeting, we should probably ask on the ML first.16:49
arne_wiebalckThat was the March one :)16:49
opendevreviewMerged openstack/ironic stable/xena: Inspector: better error message on DiscoveryFailure  https://review.opendev.org/c/openstack/ironic/+/82733416:49
opendevreviewMerged openstack/python-ironicclient master: Update master for stable/yoga  https://review.opendev.org/c/openstack/python-ironicclient/+/83224416:49
rpittaudtantsur: absolutely16:49
rpittauI'll send a mail16:49
arne_wiebalckOne suggested topic was an Ironic retrospective, given its 8th birthday.16:50
TheJuliaThat would be fun actually16:51
arne_wiebalckLike from people who were around and still remember :-D16:51
arne_wiebalck... or still want to remember.16:51
TheJulialol16:51
arne_wiebalckAnyway, April is taken, but we could aim for May.16:51
arne_wiebalckAs I am still an Ironic youngster, I would rely on others to make suggesions.16:52
TheJuliait is all a blur for me16:52
arne_wiebalck*suggestions16:52
arne_wiebalckheh16:52
arne_wiebalckI think that is it for the SIG16:53
rpittauthanks arne_wiebalck 16:53
rpittauunless we have something else to discuss, we can ask the final question:16:54
rpittau#topic Who is going to run the next meeting?16:54
iurygregoryo/16:54
MahnoorAsgharone item from me16:55
rpittausure MahnoorAsghar 16:55
rpittau(and thanks iurygregory :)16:55
* MahnoorAsghar for my outreachy project, I was working on https://review.opendev.org/c/openstack/ironic/+/827200/16:55
* MahnoorAsghar an implementation for this RFE16:55
TheJulia#link https://storyboard.openstack.org/#!/story/200978516:56
TheJulia#link https://review.opendev.org/c/openstack/ironic/+/827200/16:56
* MahnoorAsghar whoops, sorry16:56
* MahnoorAsghar I posted the wrong link16:56
TheJuliaNo worries.16:57
* MahnoorAsghar thanks Julia :)16:57
* MahnoorAsghar The implementation is fairly ready - i.e. the Sphinx extension that processes docstrings and renders documentation output16:57
TheJuliaI believe the ask is for input given you'e made some progress to try and bridge the gap for the documentation in the original theme of the RFE16:57
iurygregoryyeah, makes sense16:58
iurygregorywe can probably add to the list of reviews so people can provide feedback16:58
* MahnoorAsghar Yes, I will be adding more of input now16:58
* MahnoorAsghar okay :)16:59
TheJuliaoverall, not exactly what was being thought of, but I do like the direction given it does help bridge the gap where it is somewhat critical, at least from a developer point of view16:59
* MahnoorAsghar okay16:59
* MahnoorAsghar what did you originally have in mind?16:59
rpittauwe're at the top of the hour, I'm going to end the meeting and we can keep discussing :)17:00
rpittauthanks everyone!17:00
rpittau#endmeeting17:00
opendevmeetMeeting ended Mon Mar  7 17:00:18 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)17:00
opendevmeetMinutes:        https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-03-07-16.00.html17:00
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-03-07-16.00.txt17:00
opendevmeetLog:            https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-03-07-16.00.log.html17:00
* MahnoorAsghar okay :)17:00
TheJuliaKind of the inverse in terms of path to data discovery, but your approach is actually superior from a development standpoint as it ties together the information without impacting the existing API reference17:00
TheJuliaand kind of gives us a path to maybe bridge that more as time moves forward17:00
MahnoorAsgharI see, okay17:01
MahnoorAsgharSo you would want to keep both of them for a while?17:01
TheJuliaI think so17:01
TheJuliaIt is all about stepwise evolution. Take a step, evaluate, ponder, take an additional step17:02
MahnoorAsgharMakes sense17:02
MahnoorAsgharBut wouldn't it lead to some confusion?17:02
MahnoorAsgharor maybe the WIP can remain unpublished...17:03
MahnoorAsgharwhichever is more outdated 17:03
TheJuliaThey kind of serve different purposes17:04
MahnoorAsgharah, okay17:04
TheJuliabut the gap end up with, which your change kind of highlighted, is we don't have a clean alignment/understanding in terms of an easy way to go from error code to actual root cause and your change provides that path foward, I think.17:05
TheJuliaat least in programming terms17:05
MahnoorAsgharit helps with adding the exception information, perhaps17:06
TheJuliaexactly17:06
MahnoorAsgharokay :)17:07
TheJuliaWe're good at creating layers of indirection and sometimes they can be a little difficult to trace back without context17:07
MahnoorAsgharI see17:07
TheJuliaThis is also kind of why I want to make ironic easier to troubleshoot17:08
TheJuliatime/energy of course permitting :)17:08
MahnoorAsgharmakes sense17:08
MahnoorAsgharYou're thinking more than adding more logging, I guess 17:09
MahnoorAsgharAnd/or more verbose exception output17:09
MahnoorAsghar...or is that exactly what you mean 17:10
TheJuliaunrelated to outreachy items, one of the hardest things for operators to determine is why did a deployment fail deep into the process17:11
TheJuliait typically requires access to logs which requires an elevated level of access to get the log files17:11
MahnoorAsgharah okay17:11
TheJuliaand that is not really ideal17:12
MahnoorAsgharsecurity problem?17:12
TheJuliaMore so complexity17:12
MahnoorAsgharooh okay17:12
MahnoorAsghar(one question - by operators do you mean kubernetes operators in particular, or just users of ironic?)17:13
TheJuliainfrastructure operators, as in humans basically17:15
TheJuliaregardless of platform or tooling I guess17:15
MahnoorAsgharokay :D17:15
MahnoorAsgharsoo we could help with this problem by increasing verbosity in places other than log files17:16
TheJulianot really17:17
TheJulianot what I would like to eventually do17:17
dtantsursee you tomorrow folks o/17:18
TheJuliaI started exploring us adding some way to retrieve the log files for the agent17:18
TheJuliaand return them to the API consumer17:18
TheJuliaso they could examine them17:18
MahnoorAsgharbye dtantsur o/17:18
MahnoorAsgharoh I see!17:18
MahnoorAsgharTheJulia: did that exploration lead to something?17:19
opendevreviewMerged openstack/metalsmith master: Add Python3 zed unit tests  https://review.opendev.org/c/openstack/metalsmith/+/83223717:21
opendevreviewMerged openstack/metalsmith stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/metalsmith/+/83223417:21
opendevreviewMerged openstack/metalsmith stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/metalsmith/+/83223517:21
TheJuliaMahnoorAsghar: I've got some ideas, I started a specification document but haven't had time to revisit it unfortunately17:27
* MahnoorAsghar I see ^-^17:27
rpittaugood night! o/17:32
* MahnoorAsghar o/17:33
* TheJulia takes migraine meds17:37
MahnoorAsgharAh, get well soon :)17:39
opendevreviewMerged openstack/ironic stable/wallaby: Set correct initrd_filename for iPXE when using Swift  https://review.opendev.org/c/openstack/ironic/+/82918817:58
opendevreviewMerged openstack/python-ironicclient stable/yoga: Update .gitreview for stable/yoga  https://review.opendev.org/c/openstack/python-ironicclient/+/83224217:58
opendevreviewMerged openstack/python-ironicclient stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga  https://review.opendev.org/c/openstack/python-ironicclient/+/83224317:58
TheJuliaiurygregory: fyi https://review.opendev.org/c/openstack/releases/+/83069518:22
TheJuliahad to change the version #18:22
iurygregoryTheJulia, ack let me check18:22
opendevreviewMerged openstack/bifrost master: Dynamic inventory: do not set network_interface  https://review.opendev.org/c/openstack/bifrost/+/82956218:24
opendevreviewMerged openstack/ironic-python-agent master: Rescan device after filesystem creation  https://review.opendev.org/c/openstack/ironic-python-agent/+/82793618:37
TheJuliahmm, what is going on with grenade...18:58
arne_wiebalckbye everyone o/19:00
TheJuliagoodnight arne_wiebalck 19:00
TheJuliaMahnoorAsghar: so your change failed ci due to grenade. The grenade failure looks *entirely* unrelated, fwiw.19:09
TheJuliaMahnoorAsghar: also, bandit is failing for unrelated reasons as well19:10
MahnoorAsgharUh oh19:12
opendevreviewJulia Kreger proposed openstack/ironic master: Suppress Bandit B509 in snmp.py  https://review.opendev.org/c/openstack/ironic/+/83230619:57
TheJuliaMahnoorAsghar: ^^19:57
TheJuliaalso iurygregory ^^ fyi19:57
iurygregoryoh interesting20:19
iurygregoryI will keep an eye on it20:21
TheJuliaiurygregory: fwiw, passes locally21:36
iurygregoryhttps://zuul.opendev.org/t/openstack/status#832306 CI seems happy :D21:37
iurygregoryTheJulia, you ok if I just +W?21:41
opendevreviewSteve Baker proposed openstack/ironic-python-agent-builder master: Make centos9 jobs voting, add to gate  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/83090922:11
TheJuliaiurygregory: sure22:18
iurygregoryalready did =)22:20
iurygregorygoing to drop now, bbl o/22:20
opendevreviewMerged openstack/ironic master: Suppress Bandit B509 in snmp.py  https://review.opendev.org/c/openstack/ironic/+/83230623:50

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