| opendevreview | Takashi Kajinami proposed openstack/ironic-python-agent master: Get rid of pkg_resources https://review.opendev.org/c/openstack/ironic-python-agent/+/931188 | 00:49 |
|---|---|---|
| rpittau | good morning ironic! o/ | 06:46 |
| iurygregory | good morning ironic o/ | 10:30 |
| opendevreview | Merged openstack/ironic-python-agent master: agent: make _find_routable_addr work with IPv4- and IPv6-only setups https://review.opendev.org/c/openstack/ironic-python-agent/+/925087 | 10:36 |
| TheJulia | good morning | 13:12 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 13:32 |
| rpittau | OpenInfra Live in 20 minutes :) | 13:40 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931277 | 13:57 |
| TheJulia | rpittau: good luck! | 13:58 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2024.2: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931278 | 13:59 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2024.1: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931279 | 13:59 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2023.2: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931280 | 14:00 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2023.1: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931281 | 14:00 |
| JayF | TheJulia: that isn't the patch we care about this morning, yeah? | 14:27 |
| TheJulia | no | 14:46 |
| TheJulia | sorry, laptop crashed | 14:46 |
| JayF | np | 14:47 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931293 | 14:50 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2024.2: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931294 | 14:50 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2024.1: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931295 | 14:51 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/26.0: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931296 | 14:51 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/25.0: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931297 | 14:51 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/24.0: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931298 | 14:51 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2023.2: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931299 | 14:52 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2023.1: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931300 | 14:52 |
| JayF | https://review.opendev.org/c/openstack/ossa/+/931307 | 14:56 |
| TheJulia | TYVM! | 15:01 |
| TheJulia | And the flood of email begins | 15:01 |
| JayF | pop pop pop pop pop | 15:01 |
| TheJulia | heh | 15:01 |
| JayF | maybe will be enough to finally get me to change the tbird incoming email sound | 15:02 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931293 | 15:05 |
| TheJulia | doh! | 15:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931293 | 15:06 |
| JayF | why is it depends-on the DIB patch? | 15:06 |
| TheJulia | no, that is my mistake | 15:06 |
| TheJulia | I'm trying to change gears already | 15:06 |
| TheJulia | fwiw, codespell doesn't catch it locally | 15:06 |
| rpittau | mmm which python version? codespell I mean | 15:07 |
| JayF | Did it get caught in CI? | 15:07 |
| JayF | I just noticed during review | 15:08 |
| TheJulia | rpittau: 3.11 locally | 15:08 |
| TheJulia | https://zuul.opendev.org/t/openstack/status?change=931293 | 15:09 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 15:10 |
| TheJulia | okay, that is the change which needed the depends-on | 15:10 |
| JayF | +2 on master patch | 15:11 |
| rpittau | looking at it now | 15:13 |
| TheJulia | so, now that I have a TON of stress off my plate, the 4k disk image dump truck is backing on to my plate | 15:21 |
| TheJulia | Wheeeeee | 15:22 |
| iurygregory | =( | 15:26 |
| JayF | https://security.openstack.org/ossa/OSSA-2024-004.html | 15:26 |
| JayF | thanks Julia | 15:26 |
| iurygregory | if you need review of something feel free to ping me o/, I'm looking at some cases but I can find time to review | 15:27 |
| JayF | just go down the stable set of patches for that one | 15:27 |
| JayF | the checksum changes | 15:27 |
| iurygregory | ack o/ | 15:27 |
| rpittau | good night! o/ | 15:50 |
| JayF | iurygregory: you approved the old stable patch before any of the current ones? | 15:52 |
| JayF | nevermind I don't know how to computer | 15:52 |
| iurygregory | I was literally following the list I saw on irc .-. | 15:52 |
| * JayF still reading '2023.2' where '2024.2' now exists | 15:52 | |
| JayF | iurygregory: this is old man without coffee problems, not yours, I shoulda taken a second look before bugging you ;( | 15:53 |
| iurygregory | JayF, no worries! | 15:53 |
| JayF | the other thing, we'll need to look as the stable branches land and do releases | 15:54 |
| JayF | especially for dalmation doing it asap is nice | 15:54 |
| iurygregory | ack | 15:54 |
| iurygregory | happy to +1 any release patches to help with that o/ | 15:55 |
| iurygregory | if Push the patch I don't think the +1 counts (not 100% sure) | 15:55 |
| JayF | if you are a release manager and you push the patch | 15:56 |
| JayF | it automatically does PTL+1 | 15:56 |
| JayF | they have some great automation over there :D | 15:57 |
| iurygregory | yeah, but i never saw the release team approving it (only after someone added a +1) | 15:57 |
| iurygregory | but happy to do the releases today after we have the patches merged | 15:57 |
| JayF | I let that team know to expect em | 15:58 |
| iurygregory | let me continue the review (they seem clean cherry-pick, but I reading the code in each patch) | 15:58 |
| JayF | I'm going to note here, I've probably said it before: I start a bit of a PTO stretch this afternoon, I'll be gone and nearly 100% away from a computer until next Weds. | 16:08 |
| TheJulia | Cool cool | 16:10 |
| TheJulia | This one is nowhere near as bad as the last | 16:11 |
| TheJulia | We shouldn’t stress out over it | 16:11 |
| TheJulia | I do enough of that automatically | 16:11 |
| JayF | generally agreed, I'm mainly only sweating about the release that literally was made official yesterday | 16:11 |
| TheJulia | Yeah | 16:11 |
| iurygregory | can I call it Friday already? please?! | 16:21 |
| iurygregory | The issue is that their Cisco policy does not allow mounting the ISO with a random name. They need to create a policy using the name of the ISO before mounting it. | 16:21 |
| iurygregory | I don't know how to react after reading this .-. | 16:21 |
| JayF | are you talking about for redfish vmedia support? | 16:24 |
| JayF | if so, that's not in the standard, right? can't you hit them over the head with the DMTF? | 16:24 |
| JayF | Cisco being well-known for listening to external engineers | 16:24 |
| iurygregory | yeah | 16:24 |
| iurygregory | like, I saw many different cases with redfish on Cisco | 16:25 |
| iurygregory | but requiring such thing is something I wasn't expecting | 16:25 |
| iurygregory | the funny thing is that from the logs, ironic was able to mount the iso | 16:26 |
| TheJulia | ... So, let me get this straightend out in my head | 16:27 |
| opendevreview | Merged openstack/ironic bugfix/26.0: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931296 | 16:27 |
| opendevreview | Merged openstack/ironic bugfix/25.0: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931297 | 16:27 |
| opendevreview | Takashi Kajinami proposed openstack/sushy master: Remove Python 3.8 support https://review.opendev.org/c/openstack/sushy/+/931321 | 16:38 |
| JayF | aight, master patch cleared check | 16:46 |
| TheJulia | Another stress relief | 16:47 |
| opendevreview | Merged openstack/ironic bugfix/24.0: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931298 | 16:53 |
| iurygregory | JayF, I will be submitting one patch with all releases | 16:57 |
| JayF | I am -1 to waiting for all the stable/* patches to land before we push the one for stable/2024.2 | 16:59 |
| JayF | but I guess we can see if that's a practical concern or not | 16:59 |
| JayF | yeah they are all V+1 and going thru gate | 17:00 |
| JayF | so maybe we just get lucky and my concern is impractical | 17:00 |
| iurygregory | I'm hopping they all will merge | 17:00 |
| iurygregory | XD | 17:01 |
| iurygregory | I have the feeling I will have to edit manually https://github.com/openstack/releases/blob/master/deliverables/dalmatian/ironic.yaml for the "bugfix/25.0" and for "bugfix/26.0" | 17:08 |
| TheJulia | bugfix releases don't get new releases | 17:16 |
| JayF | yeah, only release for stable/ stuff | 17:18 |
| JayF | https://security.openstack.org/ossa/OSSA-2024-004.html#notes | 17:18 |
| iurygregory | oh ok! | 17:19 |
| iurygregory | I was trying to figure out how to properly do for the bugfix XD | 17:19 |
| JayF | we tried to do that for a while, it wasn't really effective or useful | 17:20 |
| iurygregory | ++ | 17:21 |
| TheJulia | a stacking we go, a stacking we go, a rebuilding a devstack because 4k blocks | 17:36 |
| * TheJulia has lost her mind... | 17:37 | |
| JayF | https://review.opendev.org/c/openstack/ironic/+/931301 and the chain of UM security fixes still need review | 17:45 |
| JayF | I will single core land 'em as it's UM + security urgent if they aren't reviewed by the time the stable/ land | 17:45 |
| opendevreview | Merged openstack/ironic master: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931293 | 17:58 |
| JayF | everything green on the other jobs except stable/2023.1 has a failure | 17:59 |
| iurygregory | JayF, ack | 18:00 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931300 | 18:06 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931300 | 18:49 |
| opendevreview | Merged openstack/ironic stable/2023.2: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931299 | 19:11 |
| opendevreview | Merged openstack/ironic stable/2024.2: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931294 | 19:11 |
| opendevreview | Merged openstack/ironic stable/2024.1: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931295 | 19:11 |
| JayF | bam | 19:11 |
| JayF | iurygregory: maybe ^ hit all of them but stable/2023.1 since that's the only one that needed a recheck? | 19:11 |
| JayF | I can push the patch if you'd like | 19:11 |
| * JayF going to jfdi | 19:12 | |
| JayF | iurygregory: https://review.opendev.org/c/openstack/releases/+/931341 | 19:16 |
| iurygregory | JayF, i was eating , sorry! | 19:16 |
| JayF | those messages were pushed in like, 20 seconds :P | 19:17 |
| JayF | I didn't jfdi because you were gone, I did it b/c I remembered what you said about the PTL +1 sometimes not working when you push | 19:17 |
| JayF | so I push, you +1, all good :D | 19:17 |
| iurygregory | done o/ | 19:17 |
| JayF | I think release team might be asleep now anyway? I just wanted to make sure we weren't what was waiting on | 19:17 |
| iurygregory | maybe, ttx said we could ping him | 19:18 |
| JayF | like, 5 hours from post to merge for all but antelope is pretty stellar | 19:25 |
| iurygregory | yeah =( | 19:29 |
| iurygregory | now time to switch to golang mode D: | 19:29 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: CI: Change 4k job to use LVM https://review.opendev.org/c/openstack/ironic/+/931061 | 19:59 |
| opendevreview | Adam McArthur proposed openstack/ironic master: Added Localdev Docs on changing microversion https://review.opendev.org/c/openstack/ironic/+/931351 | 21:07 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931300 | 21:24 |
| opendevreview | Adam McArthur proposed openstack/ironic master: Added Localdev Docs on changing microversion https://review.opendev.org/c/openstack/ironic/+/931351 | 21:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!