rpittau | good morning ironic! o/ | 07:35 |
---|---|---|
arne_wiebalck | Good morning rpittau and Ironic! | 07:45 |
rpittau | hey arne_wiebalck :) | 07:45 |
rpittau | arne_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 Friday | 09:00 |
arne_wiebalck | rpittau: thanks! | 09:00 |
MahnoorAsghar | Good morning o/ | 09:25 |
opendevreview | Merged openstack/tenks master: [TrivialFix] Use correct project badge in README https://review.opendev.org/c/openstack/tenks/+/832157 | 09:26 |
iurygregory | good morning Ironic | 10:39 |
dtantsur | morning folks | 10:40 |
dtantsur | could I get a 2nd +2 on the bifrost CI fix? https://review.opendev.org/c/openstack/bifrost/+/831771 | 10:41 |
iurygregory | looking now | 10:42 |
rpittau | hey iurygregory dtantsur :) | 11:04 |
iurygregory | rpittau, o/ | 11:06 |
ajya | Hi 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 |
dtantsur | yeah, thanks, we'll pass it on. janders ^^ | 11:42 |
janders | good morning / evening Ironic o/ | 11:44 |
janders | dtantsur 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 |
janders | interestingly I crashed it while uploading iDRAC firmware to a lab server :D | 11:45 |
janders | I suppose I gotta be easy on it till I figure it out another day | 11:45 |
opendevreview | Merged openstack/bifrost master: Fix the CI on CentOS 9 https://review.opendev.org/c/openstack/bifrost/+/831771 | 11:53 |
TheJulia | good morning | 13:49 |
rpittau | good morning TheJulia :) | 13:50 |
MahnoorAsghar | Good morning o/ | 14:04 |
opendevreview | OpenStack Release Bot proposed openstack/ironic-lib stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/ironic-lib/+/832231 | 14:07 |
opendevreview | OpenStack Release Bot proposed openstack/ironic-lib stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/ironic-lib/+/832232 | 14:07 |
opendevreview | OpenStack Release Bot proposed openstack/ironic-lib master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/ironic-lib/+/832233 | 14:07 |
opendevreview | OpenStack Release Bot proposed openstack/metalsmith stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/metalsmith/+/832234 | 14:07 |
opendevreview | OpenStack Release Bot proposed openstack/metalsmith stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/metalsmith/+/832235 | 14:07 |
opendevreview | OpenStack Release Bot proposed openstack/metalsmith master: Update master for stable/yoga https://review.opendev.org/c/openstack/metalsmith/+/832236 | 14:07 |
opendevreview | OpenStack Release Bot proposed openstack/metalsmith master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/metalsmith/+/832237 | 14:07 |
opendevreview | OpenStack 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/+/832238 | 14:08 |
opendevreview | OpenStack 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/+/832239 | 14:08 |
opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Update master for stable/yoga https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/832240 | 14:08 |
opendevreview | OpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/832241 | 14:08 |
opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/python-ironicclient/+/832242 | 14:08 |
opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/python-ironicclient/+/832243 | 14:08 |
opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient master: Update master for stable/yoga https://review.opendev.org/c/openstack/python-ironicclient/+/832244 | 14:08 |
opendevreview | OpenStack Release Bot proposed openstack/python-ironicclient master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/python-ironicclient/+/832245 | 14:09 |
opendevreview | OpenStack Release Bot proposed openstack/sushy stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/sushy/+/832246 | 14:09 |
opendevreview | OpenStack Release Bot proposed openstack/sushy stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/sushy/+/832247 | 14:09 |
opendevreview | OpenStack Release Bot proposed openstack/sushy master: Update master for stable/yoga https://review.opendev.org/c/openstack/sushy/+/832248 | 14:09 |
opendevreview | OpenStack Release Bot proposed openstack/sushy master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/sushy/+/832249 | 14:09 |
iurygregory | spam time =) | 14:17 |
dtantsur | yeah :) | 14:17 |
opendevreview | Ruby Loo proposed openstack/ironic bugfix/20.0: More fixes for anaconda deploy interface https://review.opendev.org/c/openstack/ironic/+/831865 | 14:30 |
rloo | good 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 |
dtantsur | rloo: at least the last one, preferably the last two | 14:36 |
dtantsur | some of us have vested interest in more branches, but these are optional | 14:36 |
rloo | thx dtantsur! i'll do the last two :) | 14:37 |
opendevreview | Ruby Loo proposed openstack/ironic bugfix/19.0: More fixes for anaconda deploy interface https://review.opendev.org/c/openstack/ironic/+/831867 | 14:38 |
rloo | (incentive to get your PRs landed b4 moar bugfix branches are created! ha ha) | 14:39 |
TheJulia | sooo many emails this morning :( | 14:49 |
iurygregory | yeah =( | 14:52 |
iurygregory | I'm trying to not go crazy with the amount of emails and docs I need to read today XD | 14:53 |
TheJulia | heh | 14:59 |
opendevreview | Merged openstack/metalsmith master: Update master for stable/yoga https://review.opendev.org/c/openstack/metalsmith/+/832236 | 15:18 |
opendevreview | Merged openstack/python-ironic-inspector-client master: Update master for stable/yoga https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/832240 | 15:18 |
opendevreview | Merged openstack/sushy master: Update master for stable/yoga https://review.opendev.org/c/openstack/sushy/+/832248 | 15:20 |
opendevreview | Merged openstack/bifrost master: Fetch uuid if it is not present in the inventory https://review.opendev.org/c/openstack/bifrost/+/829554 | 15:26 |
opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/xena: Fetch uuid if it is not present in the inventory https://review.opendev.org/c/openstack/bifrost/+/831869 | 15:29 |
dtantsur | rpittau: we didn't have stream 9 in xena, did we? | 15:30 |
dtantsur | morning TheJulia, rloo, MahnoorAsghar | 15:30 |
rloo | gooood morning dtantsur :) | 15:31 |
MahnoorAsghar | morning o/ | 15:31 |
rpittau | dtantsur: we didn't | 15:31 |
opendevreview | Harald Jensås proposed openstack/networking-baremetal master: WIP - OpenConfig YANG and Netconf https://review.opendev.org/c/openstack/networking-baremetal/+/832268 | 15:35 |
dtantsur | k | 15:41 |
opendevreview | Mahnoor Asghar proposed openstack/ironic master: [WIP] Create API documentation from docstrings https://review.opendev.org/c/openstack/ironic/+/827200 | 15:45 |
opendevreview | Merged openstack/ironic bugfix/19.0: Set correct initrd_filename for iPXE when using Swift https://review.opendev.org/c/openstack/ironic/+/829187 | 15:47 |
opendevreview | Merged openstack/ironic bugfix/19.0: Inspector: better error message on DiscoveryFailure https://review.opendev.org/c/openstack/ironic/+/827335 | 15:47 |
opendevreview | Merged openstack/ironic-lib stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/ironic-lib/+/832231 | 15:47 |
opendevreview | Merged openstack/ironic-lib stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/ironic-lib/+/832232 | 15:47 |
* iurygregory has the feeling we will have a merge spam during the meeting :D | 15:55 | |
TheJulia | in accordance with the prophecy | 15:57 |
rpittau | #startmeeting ironic | 16:00 |
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
opendevmeet | The meeting name has been set to 'ironic' | 16:00 |
TheJulia | o/ | 16:00 |
rloo | o/ | 16:00 |
rpioso | o/ | 16:00 |
erbarr | o/ | 16:00 |
iurygregory | o/ | 16:00 |
ajya | o/ | 16:00 |
rpittau | Hello ironicers, welcome to our weekly meeting! | 16:00 |
rpittau | The meeting agenda can be found here: | 16:01 |
rpittau | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 16:01 |
arne_wiebalck | o/ | 16:01 |
rpittau | #topic Announcements / Reminders | 16:02 |
rpittau | First news is old news, Yoga is due to be released at the end of March, in 3 weeks! | 16:02 |
iurygregory | I think the first RC is next week | 16:02 |
rloo | (time flies...) | 16:03 |
* iurygregory checks | 16:03 | |
rpittau | iurygregory: I think you're right :) | 16:03 |
rpittau | Right after that we'll have the Zed PTG, at the beginning of April | 16:03 |
iurygregory | ops, it's this week the RC-1 | 16:03 |
rpittau | ah, well! | 16:03 |
iurygregory | Mar 07 - Mar 11 | 16:03 |
rpittau | Zed PTG April 4 - 8 | 16:03 |
rpittau | we also have as shiny etherpad for the occasion https://etherpad.opendev.org/p/ironic-zed-ptg | 16:04 |
iurygregory | #info Zed PTG April 4 - 8 | 16:04 |
iurygregory | #link https://etherpad.opendev.org/p/ironic-zed-ptg | 16:04 |
rpittau | add your topics there and don't forget to register if you haven't done it yet | 16:04 |
opendevreview | Merged openstack/python-ironic-inspector-client master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/832241 | 16:04 |
rpittau | does anyone have anything else to announce/remind us of ? | 16:05 |
TheJulia | hjensas: fyi, I added yang/netconf to the etherpad last week | 16:06 |
rpittau | alright, we don't have any action times from last time, so moving on to next topic | 16:07 |
rpittau | #topic Review subteam status reports | 16:07 |
rpittau | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 16:07 |
rpittau | around L62 | 16:07 |
rpittau | stendulker: for CS9 support, when you have a minute please check my comments in https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/830909 | 16:08 |
rpittau | nope | 16:08 |
rpittau | stevebaker[m]: ^^ | 16:09 |
opendevreview | Mahnoor Asghar proposed openstack/ironic master: [WIP] Create API documentation from docstrings https://review.opendev.org/c/openstack/ironic/+/827200 | 16:09 |
opendevreview | Merged openstack/sushy master: Fix session authentication issues https://review.opendev.org/c/openstack/sushy/+/821324 | 16:14 |
TheJulia | *gasp* | 16:14 |
TheJulia | \o/ | 16:14 |
* TheJulia dances | 16:14 | |
rpittau | :) | 16:14 |
rpittau | looks like we're good with updates | 16:14 |
iurygregory | now we will need a backport XD | 16:15 |
rpittau | moving on | 16:15 |
rpittau | #topic Deciding on priorities for the coming week | 16:15 |
rpittau | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 16:15 |
rpittau | if anyone has a moment this week please check https://review.opendev.org/c/openstack/ironic/+/819121 | 16:16 |
TheJulia | iurygregory: yeah... I guess I know what is next after some code reviews today :) | 16:16 |
iurygregory | XD | 16:16 |
TheJulia | If there are no objections, I'll tag the sushy backports with the tag once I create them | 16:16 |
rpittau | TheJulia: sounds good :) | 16:16 |
iurygregory | TheJulia, ++ | 16:16 |
dtantsur | I'd like to get https://review.opendev.org/c/openstack/bifrost/+/829517 and the following 2 patches on the list | 16:17 |
rpittau | dtantsur: yup | 16:17 |
iurygregory | are we ok with cutting stable/yoga for projects like ironic-ui, IPE, ngs, networking-baremetal this week? | 16:18 |
rloo | I'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 |
rpittau | iurygregory: I don't see outsstanding patches there, so yes :) | 16:18 |
iurygregory | rloo, ++ please add the tag :D | 16:18 |
iurygregory | rpittau, yup, I don't think we will be able to introduce https://review.opendev.org/c/openstack/networking-baremetal/+/832268 in Yoga | 16:20 |
TheJulia | iurygregory: I think so (w/r/t projects noted) | 16:20 |
rpittau | iurygregory: yeah that's for Zed | 16:20 |
iurygregory | cool! I will proceed with the releases today =) | 16:21 |
TheJulia | yeah, hjensas just started looking at that really recently | 16:21 |
rpittau | we should be good with the list for this week, last rush before the final release | 16:21 |
TheJulia | I would say, the ideas are still brewing | 16:21 |
TheJulia | At some point, we will have glorious mead | 16:21 |
iurygregory | \o/ | 16:21 |
rpittau | ok, moving on | 16:23 |
rpittau | #topic Discussion | 16:23 |
rpittau | #info Zed PTG slots | 16:23 |
rpittau | #link http://lists.openstack.org/pipermail/openstack-discuss/2022-March/027523.html | 16:23 |
rpittau | iurygregory: mic is yours :) | 16:24 |
iurygregory | rpittau, ack =) | 16:24 |
opendevreview | Merged openstack/sushy master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/sushy/+/832249 | 16:24 |
iurygregory | I've sent this email last week with some ideas on how we should have the slots for the Zed PTG | 16:25 |
* dtantsur is curious how many active APAC contributors we have left | 16:25 | |
TheJulia | fwiw, option 3 is what we've traditionally ended up with in the past, but we've often seen low APAC attendance | 16:25 |
iurygregory | yeah | 16:25 |
opendevreview | Merged openstack/sushy stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/sushy/+/832246 | 16:25 |
rpittau | if we include Australia and NZ, at least a couple :) | 16:25 |
dtantsur | another data point: Friday is also problematic for Israel | 16:26 |
dtantsur | yeah, I'm not saying we don't have anyone :) | 16:26 |
iurygregory | dtantsur, right good point! | 16:26 |
arne_wiebalck | what about monday being a day for cross project and/or TC discussions ... was that an issue in the past? | 16:26 |
dtantsur | iurygregory: do you count as NA or APAC nowadays? :) | 16:26 |
dtantsur | arne_wiebalck: yeah, last time we had a conflict | 16:27 |
* TheJulia suspects s/NA/Americas/ | 16:27 | |
arne_wiebalck | dtantsur: I think TC is planning on Monday afternoon atm | 16:27 |
iurygregory | dtantsur, NA =) | 16:27 |
dtantsur | do we have any specific hours for APAC morning? | 16:28 |
dtantsur | I remember meeting stevebaker[m] online quite a few times when I stayed late | 16:28 |
dtantsur | (will be harder for janders though) | 16:28 |
TheJulia | I 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 wise | 16:28 |
TheJulia | Steve is also, in particular, mostly a morning person | 16:29 |
iurygregory | dtantsur, it should be someting like 8am/9am for EMEA (based on what I talked with janders ) | 16:29 |
TheJulia | at least, that is what I have gathered | 16:29 |
dtantsur | iurygregory: you're talking about the evening slot (where stevebaker[m] will have tighter limitations than janders IIRC) | 16:29 |
dtantsur | * their evening slot | 16:29 |
iurygregory | right | 16:29 |
TheJulia | dtantsur: I believe that is correct | 16:30 |
dtantsur | anyway, I sorta think #3 is fair enough, and maybe some EMEA people will even be able to participate in both slots | 16:30 |
opendevreview | Merged openstack/sushy stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/sushy/+/832247 | 16:31 |
dtantsur | not that I sleep well nowadays anyway..... | 16:31 |
TheJulia | what is this sleep word? | 16:31 |
opendevreview | Merged openstack/python-ironic-inspector-client stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/832238 | 16:32 |
opendevreview | Merged openstack/python-ironic-inspector-client stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/832239 | 16:32 |
iurygregory | it's a magic word with a very complicated definition | 16:32 |
dtantsur | so, Queensland has 9 hours more than central europe, it seems | 16:32 |
dtantsur | (although it may change in the end of March?) | 16:32 |
dtantsur | but at least now, their 7am is what, 10pm for me? still doable a few times. | 16:33 |
rpittau | there's maybe one things to consider also, on March 27th Europe changes to CEST | 16:33 |
dtantsur | yep, but we'll get closer to Australia this way? | 16:34 |
rpioso | The US springs ahead next Sunday. | 16:34 |
TheJulia | April 3rd, AU moves their clock back one hour | 16:35 |
dtantsur | hmm, I've just realized NZ is further way to the east. damn. | 16:35 |
dtantsur | TheJulia: https://www.timeanddate.com/time/zone/australia/queensland tells me Queensland doesn't have DST | 16:35 |
TheJulia | https://www.timeanddate.com/time/change/australia | 16:35 |
TheJulia | oh, good | 16:35 |
dtantsur | anyway. if my migraine-haunted head calculates right, we should have 1 hour that is reasonable for APAC and NA, and somewhat acceptable for EMEA | 16:36 |
rloo | we should confirm that steve and anders want to attend... | 16:37 |
rpittau | mmmm | 16:37 |
rloo | (or anyone else in APAC) | 16:37 |
iurygregory | janders, and stevebaker[m] I think they will (not sure about other APAC contributors =( ) | 16:38 |
dtantsur | I think we should at least leave the opportunity. maybe not every day, maybe just Tue-Thu | 16:38 |
iurygregory | sounds like a plan | 16:39 |
dtantsur | worst case, some spec reviews will be done in that slot :) | 16:39 |
iurygregory | something like Option #3 right? | 16:39 |
rpittau | we're talking about late at night or early in the morning anyway | 16:39 |
rpittau | option 3 sounds right | 16:40 |
iurygregory | rpittau, late at night for EMEA based on what dtantsur said (/me waits for confirmation) | 16:40 |
dtantsur | ignore EMEA, I guess. if we can make it - we can. yep, option 3 | 16:40 |
rpittau | ok | 16:41 |
iurygregory | yeah | 16:41 |
iurygregory | cool, I will talk later today with stevebaker[m] and janders to check their timezones and tomorrow I will provide some information in the thread | 16:42 |
rpittau | great | 16:43 |
iurygregory | I think we can move on now o/ | 16:43 |
rpittau | 2nd topic of discussion is connected: | 16:43 |
rpittau | #info March 27th time change in EU, move the meeting 1 hour earlier? | 16:43 |
TheJulia | I'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 |
rpittau | otherwise the meeting moves to 18:00 CEST | 16:44 |
arne_wiebalck | rpittau: I'd be in favor to move it. | 16:45 |
iurygregory | I think it's ok to move back to 15 UTC if people are ok with it | 16:46 |
rpittau | alright, unless there are objections at this point, or before next meeting, we will move the meeting at 15:00 UTC starting March 28th | 16:47 |
rpittau | thanks! | 16:47 |
rpittau | let's move on | 16:47 |
rpittau | #topic Baremetal SIG | 16:47 |
rpittau | #link https://etherpad.opendev.org/p/bare-metal-sig | 16:47 |
arne_wiebalck | We do not have a speaker for tomorrow, short-term change of plans. | 16:48 |
arne_wiebalck | So, unless we have someone spontaneously stepping in, I suggest to cancel. | 16:48 |
TheJulia | seems reasonable to do so at this point | 16:49 |
TheJulia | April's sounds really interesting | 16:49 |
dtantsur | rpittau: sorry, responding late. whatever happens to the meeting, we should probably ask on the ML first. | 16:49 |
arne_wiebalck | That was the March one :) | 16:49 |
opendevreview | Merged openstack/ironic stable/xena: Inspector: better error message on DiscoveryFailure https://review.opendev.org/c/openstack/ironic/+/827334 | 16:49 |
opendevreview | Merged openstack/python-ironicclient master: Update master for stable/yoga https://review.opendev.org/c/openstack/python-ironicclient/+/832244 | 16:49 |
rpittau | dtantsur: absolutely | 16:49 |
rpittau | I'll send a mail | 16:49 |
arne_wiebalck | One suggested topic was an Ironic retrospective, given its 8th birthday. | 16:50 |
TheJulia | That would be fun actually | 16:51 |
arne_wiebalck | Like from people who were around and still remember :-D | 16:51 |
arne_wiebalck | ... or still want to remember. | 16:51 |
TheJulia | lol | 16:51 |
arne_wiebalck | Anyway, April is taken, but we could aim for May. | 16:51 |
arne_wiebalck | As I am still an Ironic youngster, I would rely on others to make suggesions. | 16:52 |
TheJulia | it is all a blur for me | 16:52 |
arne_wiebalck | *suggestions | 16:52 |
arne_wiebalck | heh | 16:52 |
arne_wiebalck | I think that is it for the SIG | 16:53 |
rpittau | thanks arne_wiebalck | 16:53 |
rpittau | unless 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 |
iurygregory | o/ | 16:54 |
MahnoorAsghar | one item from me | 16:55 |
rpittau | sure 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 RFE | 16:55 | |
TheJulia | #link https://storyboard.openstack.org/#!/story/2009785 | 16:56 |
TheJulia | #link https://review.opendev.org/c/openstack/ironic/+/827200/ | 16:56 |
* MahnoorAsghar whoops, sorry | 16:56 | |
* MahnoorAsghar I posted the wrong link | 16:56 | |
TheJulia | No 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 output | 16:57 | |
TheJulia | I 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 RFE | 16:57 |
iurygregory | yeah, makes sense | 16:58 |
iurygregory | we can probably add to the list of reviews so people can provide feedback | 16:58 |
* MahnoorAsghar Yes, I will be adding more of input now | 16:58 | |
* MahnoorAsghar okay :) | 16:59 | |
TheJulia | overall, 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 view | 16:59 |
* MahnoorAsghar okay | 16:59 | |
* MahnoorAsghar what did you originally have in mind? | 16:59 | |
rpittau | we're at the top of the hour, I'm going to end the meeting and we can keep discussing :) | 17:00 |
rpittau | thanks everyone! | 17:00 |
rpittau | #endmeeting | 17:00 |
opendevmeet | Meeting ended Mon Mar 7 17:00:18 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 17:00 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-03-07-16.00.html | 17:00 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-03-07-16.00.txt | 17:00 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-03-07-16.00.log.html | 17:00 |
* MahnoorAsghar okay :) | 17:00 | |
TheJulia | Kind 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 reference | 17:00 |
TheJulia | and kind of gives us a path to maybe bridge that more as time moves forward | 17:00 |
MahnoorAsghar | I see, okay | 17:01 |
MahnoorAsghar | So you would want to keep both of them for a while? | 17:01 |
TheJulia | I think so | 17:01 |
TheJulia | It is all about stepwise evolution. Take a step, evaluate, ponder, take an additional step | 17:02 |
MahnoorAsghar | Makes sense | 17:02 |
MahnoorAsghar | But wouldn't it lead to some confusion? | 17:02 |
MahnoorAsghar | or maybe the WIP can remain unpublished... | 17:03 |
MahnoorAsghar | whichever is more outdated | 17:03 |
TheJulia | They kind of serve different purposes | 17:04 |
MahnoorAsghar | ah, okay | 17:04 |
TheJulia | but 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 |
TheJulia | at least in programming terms | 17:05 |
MahnoorAsghar | it helps with adding the exception information, perhaps | 17:06 |
TheJulia | exactly | 17:06 |
MahnoorAsghar | okay :) | 17:07 |
TheJulia | We're good at creating layers of indirection and sometimes they can be a little difficult to trace back without context | 17:07 |
MahnoorAsghar | I see | 17:07 |
TheJulia | This is also kind of why I want to make ironic easier to troubleshoot | 17:08 |
TheJulia | time/energy of course permitting :) | 17:08 |
MahnoorAsghar | makes sense | 17:08 |
MahnoorAsghar | You're thinking more than adding more logging, I guess | 17:09 |
MahnoorAsghar | And/or more verbose exception output | 17:09 |
MahnoorAsghar | ...or is that exactly what you mean | 17:10 |
TheJulia | unrelated to outreachy items, one of the hardest things for operators to determine is why did a deployment fail deep into the process | 17:11 |
TheJulia | it typically requires access to logs which requires an elevated level of access to get the log files | 17:11 |
MahnoorAsghar | ah okay | 17:11 |
TheJulia | and that is not really ideal | 17:12 |
MahnoorAsghar | security problem? | 17:12 |
TheJulia | More so complexity | 17:12 |
MahnoorAsghar | ooh okay | 17:12 |
MahnoorAsghar | (one question - by operators do you mean kubernetes operators in particular, or just users of ironic?) | 17:13 |
TheJulia | infrastructure operators, as in humans basically | 17:15 |
TheJulia | regardless of platform or tooling I guess | 17:15 |
MahnoorAsghar | okay :D | 17:15 |
MahnoorAsghar | soo we could help with this problem by increasing verbosity in places other than log files | 17:16 |
TheJulia | not really | 17:17 |
TheJulia | not what I would like to eventually do | 17:17 |
dtantsur | see you tomorrow folks o/ | 17:18 |
TheJulia | I started exploring us adding some way to retrieve the log files for the agent | 17:18 |
TheJulia | and return them to the API consumer | 17:18 |
TheJulia | so they could examine them | 17:18 |
MahnoorAsghar | bye dtantsur o/ | 17:18 |
MahnoorAsghar | oh I see! | 17:18 |
MahnoorAsghar | TheJulia: did that exploration lead to something? | 17:19 |
opendevreview | Merged openstack/metalsmith master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/metalsmith/+/832237 | 17:21 |
opendevreview | Merged openstack/metalsmith stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/metalsmith/+/832234 | 17:21 |
opendevreview | Merged openstack/metalsmith stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/metalsmith/+/832235 | 17:21 |
TheJulia | MahnoorAsghar: I've got some ideas, I started a specification document but haven't had time to revisit it unfortunately | 17:27 |
* MahnoorAsghar I see ^-^ | 17:27 | |
rpittau | good night! o/ | 17:32 |
* MahnoorAsghar o/ | 17:33 | |
* TheJulia takes migraine meds | 17:37 | |
MahnoorAsghar | Ah, get well soon :) | 17:39 |
opendevreview | Merged openstack/ironic stable/wallaby: Set correct initrd_filename for iPXE when using Swift https://review.opendev.org/c/openstack/ironic/+/829188 | 17:58 |
opendevreview | Merged openstack/python-ironicclient stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/python-ironicclient/+/832242 | 17:58 |
opendevreview | Merged openstack/python-ironicclient stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/python-ironicclient/+/832243 | 17:58 |
TheJulia | iurygregory: fyi https://review.opendev.org/c/openstack/releases/+/830695 | 18:22 |
TheJulia | had to change the version # | 18:22 |
iurygregory | TheJulia, ack let me check | 18:22 |
opendevreview | Merged openstack/bifrost master: Dynamic inventory: do not set network_interface https://review.opendev.org/c/openstack/bifrost/+/829562 | 18:24 |
opendevreview | Merged openstack/ironic-python-agent master: Rescan device after filesystem creation https://review.opendev.org/c/openstack/ironic-python-agent/+/827936 | 18:37 |
TheJulia | hmm, what is going on with grenade... | 18:58 |
arne_wiebalck | bye everyone o/ | 19:00 |
TheJulia | goodnight arne_wiebalck | 19:00 |
TheJulia | MahnoorAsghar: so your change failed ci due to grenade. The grenade failure looks *entirely* unrelated, fwiw. | 19:09 |
TheJulia | MahnoorAsghar: also, bandit is failing for unrelated reasons as well | 19:10 |
MahnoorAsghar | Uh oh | 19:12 |
opendevreview | Julia Kreger proposed openstack/ironic master: Suppress Bandit B509 in snmp.py https://review.opendev.org/c/openstack/ironic/+/832306 | 19:57 |
TheJulia | MahnoorAsghar: ^^ | 19:57 |
TheJulia | also iurygregory ^^ fyi | 19:57 |
iurygregory | oh interesting | 20:19 |
iurygregory | I will keep an eye on it | 20:21 |
TheJulia | iurygregory: fwiw, passes locally | 21:36 |
iurygregory | https://zuul.opendev.org/t/openstack/status#832306 CI seems happy :D | 21:37 |
iurygregory | TheJulia, you ok if I just +W? | 21:41 |
opendevreview | Steve 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/+/830909 | 22:11 |
TheJulia | iurygregory: sure | 22:18 |
iurygregory | already did =) | 22:20 |
iurygregory | going to drop now, bbl o/ | 22:20 |
opendevreview | Merged openstack/ironic master: Suppress Bandit B509 in snmp.py https://review.opendev.org/c/openstack/ironic/+/832306 | 23:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!