| iurygregory | JayF, +W | 02:47 |
|---|---|---|
| JayF | TY | 02:47 |
| opendevreview | Merged openstack/ironic stable/2025.2: More RAM for Anaconda job https://review.opendev.org/c/openstack/ironic/+/964730 | 03:52 |
| rpittau | good morning ironic! happy friday! o/ | 07:31 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost stable/2025.2: [stable only] increase test VM mem size https://review.opendev.org/c/openstack/bifrost/+/964752 | 08:23 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Run metal3 integration job using UEFI boot (default) https://review.opendev.org/c/openstack/ironic/+/939694 | 08:45 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Run metal3 integration job using UEFI boot (default) https://review.opendev.org/c/openstack/ironic/+/939694 | 08:45 |
| opendevreview | Merged openstack/ironic-ui stable/2025.2: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-ui/+/964103 | 08:46 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix power sync timeouts after BMC firmware update https://review.opendev.org/c/openstack/ironic/+/962560 | 09:03 |
| opendevreview | Freerk-Ole Zakfeld proposed openstack/ironic stable/2024.2: Set upper_bound to bigint https://review.opendev.org/c/openstack/ironic/+/964771 | 12:18 |
| opendevreview | Freerk-Ole Zakfeld proposed openstack/ironic stable/2024.2: Set upper_bound column in bios_settings table to bigint https://review.opendev.org/c/openstack/ironic/+/964771 | 12:22 |
| *** AB is now known as Guest29808 | 13:33 | |
| JayF | I'm not working today, but taking time to push the anaconda ram fix back a little further. Please help me get it back if you have time. | 13:54 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2025.1: More RAM for Anaconda job https://review.opendev.org/c/openstack/ironic/+/964781 | 13:59 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.1: More RAM for Anaconda job https://review.opendev.org/c/openstack/ironic/+/964781 | 13:59 |
| JayF | jinx cardoe :( | 14:00 |
| JayF | I was rebasing lol | 14:00 |
| JayF | if yours looks the same as mine I'll land it | 14:00 |
| cardoe | I was trying to help ya! | 14:00 |
| JayF | yeah cardoe review mine, yours is needing the rebase :D | 14:00 |
| JayF | the +2 on mine is maximum help :D | 14:00 |
| JayF | and hopefully the next step is less rebasey | 14:00 |
| JayF | that was an annoying rebase for a ci config | 14:00 |
| cardoe | I just clicked the button and that's what mine did. | 14:01 |
| cardoe | it looks like I stomped over yours. | 14:01 |
| JayF | I don't know how to fix this directly | 14:02 |
| JayF | I'm going to do it the long way | 14:02 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2025.1: More RAM for Anaconda job https://review.opendev.org/c/openstack/ironic/+/964781 | 14:02 |
| JayF | just needed to amend commit to get a new commit timestamp for ability to push another patchset | 14:02 |
| JayF | that should be good now :) | 14:02 |
| cardoe | sorry. I was just trying to do this in a browser window while drinking my coffee. | 14:03 |
| JayF | I tried the same last night | 14:03 |
| JayF | I have my gerrit set to complain if a rebase will have conflicts | 14:03 |
| JayF | so I saw that, said nope, and went to bed lol | 14:03 |
| JayF | yeah just trying to get this fix back as it's blocking clif | 14:04 |
| JayF | +A on that change as a CI fix based on cardoe's review | 14:04 |
| cardoe | If ya got one for 2024.2.. I might +2 that as well. | 14:08 |
| cardoe | rpittau: For https://review.opendev.org/c/openstack/ironic/+/964771 can I just rebase it to master and would that switch it to master for the author? | 14:09 |
| cardoe | There's a bunch of what seems to be trivial +W's if someone wants to just smack em all. | 14:10 |
| cardoe | https://review.opendev.org/c/openstack/ironic/+/964450 https://review.opendev.org/c/openstack/ironic/+/964451 https://review.opendev.org/c/openstack/sushy-tools/+/964550 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/964446 https://review.opendev.org/c/openstack/ironic/+/939694 | 14:14 |
| JayF | I'll take a look while I'm waiting for my gentoo update to start (can't waste laptop downtime lol) | 14:15 |
| JayF | the sushy-tools change isn't something I'm familiar enough with for a quick review | 14:17 |
| cardoe | That's fair. It was a bigger one that required me to actually read through the intent. | 14:18 |
| JayF | also left a comment on the ipmi validation change; that 100% needs a release note | 14:19 |
| JayF | and now my laptop update is fully going BRRRRRR so I'm going to go away :D have a good one o/ | 14:20 |
| cardoe | Have a good weekend! | 14:28 |
| cardoe | clif: What would I need to change in https://review.opendev.org/c/openstack/ironic/+/964570 to work with your portgroup stuff? | 14:29 |
| rpittau | cardoe: not sure that's a good idea, you'll have to resolve a ton of conflicts | 14:49 |
| cardoe | okay | 14:55 |
| rpittau | cardoe: probably just apply the commit changes on top of master will work, no need to rebase | 14:56 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Publish CentOS Stream 10 IPA dib images https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/964446 | 15:15 |
| rpittau | \o/ | 15:18 |
| opendevreview | Merged openstack/sushy-tools master: novadriver: Fix race for instance task_state https://review.opendev.org/c/openstack/sushy-tools/+/964550 | 15:20 |
| opendevreview | Merged openstack/ironic master: Run metal3 integration job using UEFI boot (default) https://review.opendev.org/c/openstack/ironic/+/939694 | 15:25 |
| opendevreview | Merged openstack/ironic master: Document instance_uuid JSON PATCH non-compliance https://review.opendev.org/c/openstack/ironic/+/964450 | 15:25 |
| *** mdfr7 is now known as mdfr | 16:50 | |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix power sync timeouts after BMC firmware update https://review.opendev.org/c/openstack/ironic/+/962560 | 17:10 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Fix power sync timeouts after BMC firmware update https://review.opendev.org/c/openstack/ironic/+/962560 | 17:14 |
| opendevreview | Merged openstack/ironic stable/2025.1: More RAM for Anaconda job https://review.opendev.org/c/openstack/ironic/+/964781 | 17:34 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2024.2: More RAM for Anaconda job https://review.opendev.org/c/openstack/ironic/+/964810 | 18:45 |
| *** AB is now known as Guest29835 | 19:46 | |
| opendevreview | Freerk-Ole Zakfeld proposed openstack/ironic stable/2024.2: Set upper_bound column in bios_settings table to bigint https://review.opendev.org/c/openstack/ironic/+/964771 | 20:56 |
| opendevreview | Freerk-Ole Zakfeld proposed openstack/ironic stable/2024.2: Set upper_bound column in bios_settings table to bigint https://review.opendev.org/c/openstack/ironic/+/964771 | 20:57 |
| opendevreview | Freerk-Ole Zakfeld proposed openstack/ironic stable/2024.2: Set upper_bound column in bios_settings table to bigint https://review.opendev.org/c/openstack/ironic/+/964771 | 20:59 |
| opendevreview | Freerk-Ole Zakfeld proposed openstack/ironic stable/2024.2: Set upper_bound column in bios_settings table to bigint https://review.opendev.org/c/openstack/ironic/+/964771 | 21:01 |
| *** dmellado9 is now known as dmellado | 22:37 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!