| opendevreview | Merged openstack/bifrost master: Release note for ironic-inspector deprecation https://review.opendev.org/c/openstack/bifrost/+/954877 | 01:18 |
|---|---|---|
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] Skip initial reboot to IPA when updating firmware out-of-band https://review.opendev.org/c/openstack/ironic/+/954311 | 04:36 |
| queensly[m] | Good morning ironic o/ | 09:09 |
| abongale | good morning! | 09:09 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/ironic master: Add manual clean and automated verify steps to set BMC clock via Redfish Manager https://review.opendev.org/c/openstack/ironic/+/953477 | 09:54 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/sushy-tools master: Validate JSON content type before parsing manager PATCH requests https://review.opendev.org/c/openstack/sushy-tools/+/954945 | 12:50 |
| opendevreview | Queensly Kyerewaa Acheampongmaa proposed openstack/ironic master: Add manual clean and automated verify steps to set BMC clock via Redfish Manager https://review.opendev.org/c/openstack/ironic/+/953477 | 12:56 |
| TheJulia | good morning! | 13:03 |
| clif | o/ | 15:03 |
| alegacy | o/ | 15:03 |
| queensly[m] | o/ | 15:06 |
| cid | One of these days, I would love to run the meeting. I just need the template :D | 15:07 |
| * cid notes no quorum for today's meeting. | 15:08 | |
| TheJulia | o/ | 15:10 |
| TheJulia | Its july, vacations and such. Do we want to try and hold a meeting? | 15:10 |
| cid | I don't think there's much to talk about. s/much == anything :( | 15:13 |
| * cid vacations ++ | 15:13 | |
| TheJulia | Yeah, I'm past due for a vacatoin | 15:14 |
| alegacy | On the standalone networking front I have been making good progress. Hoping that by end of this week I sort of have things roughed in end to end. | 15:14 |
| alegacy | Will polish that a bit and then propose giving a bit of a demo before opening reviews. | 15:15 |
| TheJulia | alegacy: cool, do you need anything from the community | 15:15 |
| alegacy | not for now, things are going fairly well so far. Might need to discuss some of the details a bit further into it. | 15:15 |
| TheJulia | cool cool, about to loose connectivity and moving, bbiam | 15:17 |
| * TheJulia got kicked out of the RV | 15:17 | |
| * TheJulia is back... at the customer lounge | 15:20 | |
| * TheJulia may finally get to leave today | 15:21 | |
| opendevreview | Clif Houck proposed openstack/ironic master: Add a new 'vendor' field to the Port object https://review.opendev.org/c/openstack/ironic/+/954966 | 15:42 |
| TheJulia | dtantsur: stacked patches, minor merge conflict but nothing awful for none. haven't exercised it locally as I'm just watching power sync loop behavior trying to get a rough idea of how much time/overhead/etc. | 18:04 |
| opendevreview | Merged openstack/ironic master: Remove focus from ironic-inspector in the documentation https://review.opendev.org/c/openstack/ironic/+/954879 | 18:11 |
| TheJulia | whoops, didn't modify single process to be threaded, doh. | 18:12 |
| TheJulia | gah, my install didn't update | 18:17 |
| TheJulia | Yeah, we should tune the thread stack size down | 18:33 |
| stevebaker[m] | kubajj_ TheJulia: Hey I hear there might be hardware storing efi data as utf-8. I was looking at this area a while ago and came to the conclusion that efibootmgr converts to utf-8 so we should just consume that and tolerate errors. Here is the change https://review.opendev.org/c/openstack/ironic-python-agent/+/902214 | 20:31 |
| TheJulia | Could it all be in different versions of efibootmgr? | 20:32 |
| TheJulia | I guess that might require some digging of the code versions and whatnot | 20:32 |
| stevebaker[m] | If there really is utf-8 in efi data causing problems then possibly the fix should be here https://github.com/rhboot/efivar/blob/main/src/loadopt.c#L362 but yeah we need some solid data on what is actually going on | 20:32 |
| stevebaker[m] | https://github.com/rhboot/efibootmgr has been converting to utf-8 for 9 years. I don't know if other distros fork this | 20:35 |
| TheJulia | ... | 21:07 |
| TheJulia | could it be a lack of the byte marker? | 21:07 |
| TheJulia | when we call a decode, we explicitly declare what we think | 21:08 |
| TheJulia | (thinking outloud, isent't there a "give me a hexdump?" version | 21:09 |
| TheJulia | (there is, the actual file on sysfs I think, but its all super blurry for my brain right now) | 21:14 |
| TheJulia | dtantsur: so, your patch all layered does work, nothing really screams. We might have some weirdness with early periodic registration which seems unrelated | 21:15 |
| TheJulia | (the use jsonrpc patch) | 21:15 |
| TheJulia | Threading on the other hand, hmmmmmm | 21:15 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!