| rpittau | good morning ironic! o/ | 06:48 |
|---|---|---|
| rpittau | where did my gerrit dark theme go?!? :/ | 06:48 |
| rpittau | TheJulia: re "no space left" issue: yep, going to add that, it will be good anyway | 06:57 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Collect disk usage info post CI jobs run https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/879826 | 07:17 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Collect disk usage info post CI jobs run https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/879826 | 07:20 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Move ubuntu jobs to jammy https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/879538 | 07:21 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Collect disk usage info post CI jobs run https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/879826 | 12:27 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Move ubuntu jobs to jammy https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/879538 | 12:28 |
| TheJulia | good morning | 13:17 |
| TheJulia | rpittau: looks like they did a version upgrade | 13:19 |
| rpittau | TheJulia: hi! for gerrit? yeah, I ahve my dark theme again :) | 13:20 |
| TheJulia | I selected it again and no dice for me, but *shrugs* | 13:21 |
| rpittau | :/ | 13:24 |
| JayF | TheJulia: I wonder if it keys off if system dark mode is enabled | 13:40 |
| JayF | mine is still happy and dark | 13:40 |
| TheJulia | dunno, i tried change it because I used dark mode for a long long time | 13:42 |
| JayF | https://imgur.com/Wxvwhnw is what I see | 13:44 |
| JayF | I changed no settings, just logged-in post upgrade | 13:44 |
| * TheJulia shrugs | 13:47 | |
| * JayF only here until noonish, then off on vaca \o/ | 13:48 | |
| TheJulia | I may only be here until wifey wakes up | 13:54 |
| rpittau | TheJulia: would this work to get the disk space? https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/879826 | 14:02 |
| rpittau | JayF, TheJulia, if you have a moment today please review https://review.opendev.org/c/openstack/ironic/+/879162, metal3 integration job now builds the ironic container with the actual patch code :) | 14:05 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP: Test utf-16 compatability change https://review.opendev.org/c/openstack/ironic-python-agent/+/879897 | 14:33 |
| TheJulia | rpittau: seems reasonable | 14:33 |
| TheJulia | Given we don’t know where is out of space | 14:33 |
| TheJulia | Trying to save the output to a file might fail at the moment | 14:34 |
| TheJulia | Well, that saves it | 14:34 |
| TheJulia | It might work, if it doesn’t that is a data point | 14:34 |
| TheJulia | so turns out people have started putting UTF-16 characters in the efi nvram data set | 14:41 |
| TheJulia | .... and guess what by default tried to force to UTF-8! | 14:41 |
| TheJulia | :* | 14:41 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Collect disk usage info post CI jobs run https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/879826 | 15:21 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent-builder master: Move ubuntu jobs to jammy https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/879538 | 15:22 |
| rpittau | it can't execute the command in run because the plabooks fails before :/ | 15:22 |
| rpittau | moved to a different playbook | 15:22 |
| rpittau | anyway, time to go | 15:23 |
| rpittau | have a great weekend everyone! o/ | 15:23 |
| opendevreview | Merged openstack/ironic master: Run metal3 integration with ironic-image from source https://review.opendev.org/c/openstack/ironic/+/879162 | 15:53 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/sushy master: Handle TransferMethod in vmedia insertion https://review.opendev.org/c/openstack/sushy/+/879903 | 16:11 |
| iurygregory | TheJulia, fyi ^ | 16:24 |
| iurygregory | I will check things back on monday since today is holiday here o/ | 16:25 |
| iurygregory | have a great easter Ironic o/ | 16:25 |
| TheJulia | o/ | 16:27 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP: Test utf-16 compatability change https://review.opendev.org/c/openstack/ironic-python-agent/+/879897 | 16:49 |
| JayF | TheJulia: iurygregory: dtantsur: Would one of you mind running the Monday, April 17th meeting? | 17:09 |
| JayF | I'll be here, but have a 30-minute-overlapping meeting if it goes into the last half hour, and want to ensure I can give proper attention to my other meeting | 17:09 |
| TheJulia | I can | 17:09 |
| JayF | wunderbar | 17:09 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Add sleep step https://review.opendev.org/c/openstack/ironic/+/879061 | 18:36 |
| TheJulia | still sort of musing ^ | 18:36 |
| TheJulia | maybe sleep is better than pause | 18:36 |
| JayF | I sorta talked myself out of calling it sleep when reviewing your spec | 18:36 |
| JayF | b/c if we're introducing "power actions" to cleaning | 18:36 |
| JayF | /sleep is a power action/ | 18:36 |
| TheJulia | yeah.... the whole flow that kind of results in my musing kind of makes things slightly interchangable too, but fundimentally the same basic pattern, just slight differences in state changes and updates | 18:40 |
| TheJulia | likely need to add a "file on conductor" thing into the agent or something | 18:41 |
| TheJulia | err, file on agent into the agent and have the conductor call it | 18:41 |
| JayF | the file on disk thing, I still struggle with | 18:41 |
| TheJulia | just a musing idea *really* | 18:42 |
| TheJulia | but the upper bound on sleep thing makes tons of sense to me | 18:42 |
| JayF | there's no case I can think of that leads to a file being put on the disk that isn't just "someone took a workflow and implemented it outside of ironic in-band" | 18:42 |
| JayF | which is not really a use case we should be embracing | 18:42 |
| TheJulia | I can agree with that | 18:42 |
| TheJulia | and even then, that is what hold is for | 18:42 |
| TheJulia | at *best* | 18:42 |
| JayF | well, that's why the "file on disk" is a bigger concern | 18:43 |
| JayF | hold makes sense because you can have external systems you can't control you need to integrate | 18:43 |
| TheJulia | yeah | 18:43 |
| JayF | file-on-node-disk means "you ran something on that node, with ironic, that instead of properly signalling 'done', or waiting to exit, put a file on disk instead" | 18:43 |
| TheJulia | agreed... I think the file on disk was your idea, was it not? | 18:44 |
| JayF | tbh, I expected the answer to my use case question to be "on $obscureHardware the BMC firmware update is confirmed when the BMC puts a file on the EFI parition" or something nuts like that lol | 18:44 |
| TheJulia | oh, jeeze | 18:44 |
| JayF | if it was my idea, then past me was wrong lol | 18:44 |
| TheJulia | yeah, no, they can go write python code on that level | 18:44 |
| TheJulia | the thing that someone pointed out to me was they have gear that has to be power cycled in rapid succession several times to trigger network booting | 18:44 |
| TheJulia | which seems... weird, but it is a thing. most of own such devices in various ways/shapes/forms | 18:45 |
| TheJulia | I guess the next step is map power on, power off to states as well, combined with the vendor interfaces in another patch and that seems like some of the base building blocks needed to do things like "I need to update a dpu" | 18:47 |
| TheJulia | JayF: if you wouldn't mind a quick review on https://review.opendev.org/c/openstack/sushy/+/879903 would likely be appreciated by Iury | 18:49 |
| JayF | you caught me with 11 minutes to go :) | 18:49 |
| TheJulia | it *is* quick :) | 18:49 |
| JayF | I'll take a look, I don't always +2 sushy stuff of significant substance | 18:49 |
| TheJulia | he can always workflow it himself on Monday if you feel it is good | 18:50 |
| TheJulia | It is relatively simple, hopefully it is the last thing to make a particular hardware vendor's hardware "just work" | 18:51 |
| JayF | we should name names around these workarounds imo | 18:51 |
| TheJulia | it... unfortunately got added to the dmtf standard | 18:52 |
| JayF | "BobCorp BMC Firmware 2.x changed behavior so that we now need to send this. Thanks for the breaking change!" | 18:52 |
| TheJulia | version 1.6 :\ | 18:52 |
| JayF | +3a | 18:54 |
| JayF | +2a * | 18:54 |
| TheJulia | that will make Iury happy on Monday :) | 18:55 |
| JayF | hopefully Tuesday ;) | 18:57 |
| TheJulia | today is the holiday there | 18:57 |
| TheJulia | :) | 18:57 |
| JayF | ah, I thought it was both | 18:57 |
| TheJulia | Anyhow, have a great time on PTO | 18:57 |
| JayF | yup, I'm about to shutdown o/ | 18:57 |
| TheJulia | Friday/Monday is many EU countries afaik | 18:57 |
| * TheJulia tosses things at unicode | 19:05 | |
| TheJulia | UnicodeDecodeError: 'utf-16-le' codec can't decode byte 0x0a in position 468: truncated data | 19:05 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP: Test utf-16 compatability change https://review.opendev.org/c/openstack/ironic-python-agent/+/879897 | 19:28 |
| opendevreview | Merged openstack/sushy master: Handle TransferMethod in vmedia insertion https://review.opendev.org/c/openstack/sushy/+/879903 | 19:44 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Fix Bandit errors https://review.opendev.org/c/openstack/ironic-python-agent/+/879912 | 20:41 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Fix UTF-16 result handling for efibootmgr https://review.opendev.org/c/openstack/ironic-python-agent/+/879897 | 21:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!