| Nisha_Agarwal | dtantsur, Hi | 06:54 |
|---|---|---|
| Nisha_Agarwal | JayF, ping | 06:55 |
| arne_wiebalck | Good morning, Ironic! | 07:24 |
| dtantsur | morning folks | 07:53 |
| kaloyankotlarski | hello everyone | 07:59 |
| rpittau | good morning ironic! o/ | 08:06 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/21.3: Wipe Agent Token when cleaning timeout occcurs https://review.opendev.org/c/openstack/ironic/+/877402 | 08:31 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/21.0: Wipe Agent Token when cleaning timeout occcurs https://review.opendev.org/c/openstack/ironic/+/877403 | 08:31 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/20.2: Wipe Agent Token when cleaning timeout occcurs https://review.opendev.org/c/openstack/ironic/+/877404 | 08:31 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/19.0: Wipe Agent Token when cleaning timeout occcurs https://review.opendev.org/c/openstack/ironic/+/877405 | 08:32 |
| rpittau | a quick approval if someone has a moment during the day :) https://review.opendev.org/c/openstack/sushy/+/877077 | 08:33 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/21.3: Clean out agent token even if power is already off https://review.opendev.org/c/openstack/ironic/+/877406 | 08:33 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/21.0: Clean out agent token even if power is already off https://review.opendev.org/c/openstack/ironic/+/877407 | 08:34 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/20.2: Clean out agent token even if power is already off https://review.opendev.org/c/openstack/ironic/+/877408 | 08:34 |
| opendevreview | Merged openstack/ironic stable/2023.1: Clean out agent token even if power is already off https://review.opendev.org/c/openstack/ironic/+/877391 | 08:44 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Fixes Secureboot with Anaconda deploy https://review.opendev.org/c/openstack/ironic/+/860820 | 09:55 |
| opendevreview | Merged openstack/ironic stable/zed: Clean out agent token even if power is already off https://review.opendev.org/c/openstack/ironic/+/877392 | 10:14 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: Wipe Agent Token when cleaning timeout occcurs https://review.opendev.org/c/openstack/ironic/+/877394 | 10:14 |
| opendevreview | Merged openstack/ironic stable/zed: Wipe Agent Token when cleaning timeout occcurs https://review.opendev.org/c/openstack/ironic/+/877395 | 10:14 |
| opendevreview | Merged openstack/ironic stable/yoga: Wipe Agent Token when cleaning timeout occcurs https://review.opendev.org/c/openstack/ironic/+/877396 | 10:44 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 10:59 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Enables boot modes switching with Anaconda deploy for ilo driver https://review.opendev.org/c/openstack/ironic/+/860821 | 11:12 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Fixes Secureboot with Anaconda deploy https://review.opendev.org/c/openstack/ironic/+/860820 | 11:23 |
| iurygregory | good morning Ironic | 11:31 |
| opendevreview | Nisha Agarwal proposed openstack/ironic master: Enables boot modes switching with Anaconda deploy for ilo driver https://review.opendev.org/c/openstack/ironic/+/860821 | 11:43 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Set [inspector]require_managed_boot explicitly https://review.opendev.org/c/openstack/bifrost/+/877469 | 12:19 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Prepare [inspector]require_managed_boot to change to True in the future https://review.opendev.org/c/openstack/ironic/+/877470 | 12:19 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: [Trivial] Fix typo in efi_utils https://review.opendev.org/c/openstack/ironic-python-agent/+/877478 | 13:26 |
| opendevreview | Matt Crees proposed openstack/ironic stable/zed: Fix online upgrades for Bios/Traits https://review.opendev.org/c/openstack/ironic/+/877409 | 14:13 |
| opendevreview | Mark Goddard proposed openstack/ironic stable/zed: Fix online upgrades for Bios/Traits https://review.opendev.org/c/openstack/ironic/+/877409 | 14:21 |
| opendevreview | Mark Goddard proposed openstack/ironic stable/yoga: Fix online upgrades for Bios/Traits https://review.opendev.org/c/openstack/ironic/+/877490 | 14:22 |
| opendevreview | Mark Goddard proposed openstack/ironic stable/xena: Fix online upgrades for Bios/Traits https://review.opendev.org/c/openstack/ironic/+/877491 | 14:23 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 15:10 |
| dtantsur | rpittau: hey, do you remember if the shim.signed issue got backported? | 15:11 |
| dtantsur | bifrost-upgrade-ubuntu-focal fails with it | 15:12 |
| dtantsur | oh, and the regular focal job as well. so it's probably not a question of backporting.. | 15:13 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/2023.1: chore: allow ironic-inspector to work with IPv6 disabled https://review.opendev.org/c/openstack/bifrost/+/877494 | 15:14 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/2023.1: Update git_branch to stable/2023.1 https://review.opendev.org/c/openstack/bifrost/+/877513 | 15:19 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/zed: Update git_branch to stable/zed https://review.opendev.org/c/openstack/bifrost/+/877514 | 15:20 |
| rpittau | dtantsur: probably not | 15:20 |
| dtantsur | rpittau: I suspect we need the fix for all ubuntu's, not just jammy | 15:21 |
| dtantsur | (LTS is such LTS) | 15:21 |
| rpittau | long term sadness | 15:22 |
| rpittau | alright, let me check | 15:22 |
| dtantsur | rpittau: I'm preparing a patch, no worries | 15:22 |
| rpittau | ack | 15:22 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Use the jammy shimx64.efi.signed fix also on focal https://review.opendev.org/c/openstack/bifrost/+/877515 | 15:22 |
| dtantsur | let's see if this ^^ is just enough | 15:22 |
| dtantsur | wweelll.. since it's an upgrade job, we may be forced to do the reverse thing | 15:23 |
| rpittau | mmmm yeah | 15:23 |
| dtantsur | okay, let me propose a cherry-pick as well | 15:23 |
| dtantsur | ... and see what we're upgrading from even... | 15:23 |
| TheJulia | wheeeeee | 15:24 |
| rpittau | by the tentacles of cthulhu, the name is indeed changed in focal.... https://packages.ubuntu.com/focal-updates/amd64/shim-signed/filelist | 15:25 |
| dtantsur | such LTS very stable | 15:25 |
| rpittau | no, wit, nvm, read wrong, it seems the same | 15:25 |
| dtantsur | also, hmm, sorry, but rant incoming | 15:26 |
| * rpittau clean his glasses | 15:26 | |
| dtantsur | rpittau, TheJulia, JayF, could we please ensure we never EVER, unless to fix a horrible CI issue, merge things like https://review.opendev.org/c/openstack/bifrost/+/848828/10/zuul.d/project.yaml | 15:26 |
| dtantsur | please PLEASE | 15:26 |
| dtantsur | we cannot just drop one release and add another. we have upgrades. they're expected to work. | 15:27 |
| dtantsur | (let alone the fact that ironic still runs the focal job and probably other projects as well) | 15:27 |
| dtantsur | rant over, thank you for your attention | 15:27 |
| dtantsur | (I know, I was also guilty for not catching this on this patch) | 15:28 |
| rpittau | dtantsur: will do, although I remember that was discussed widely | 15:28 |
| dtantsur | yeah, as I said, I must have raised it much earlier. Appologies. It was not until this breakage when I realized that we left upgrades in a weird state. | 15:29 |
| dtantsur | on the other hand, we have the upgrade job and it highlighted the issue \o/ | 15:29 |
| dtantsur | okay, plan: 1) move the 2023.1->master upgrade job to jammy, but still fix focal. | 15:30 |
| dtantsur | 2) update stable/2023.1 with the fix | 15:30 |
| dtantsur | I guess this will help us avoid landing the stable change before master? | 15:30 |
| TheJulia | noted | 15:31 |
| * TheJulia tries to find where her brain was at to resume() | 15:31 | |
| * dtantsur would love to do this dance around ubuntu somewhat less frequent | 15:32 | |
| rpittau | dtantsur: for the future, do we want to keep at least voting job with the old relesae, besides the upgrade job? | 15:32 |
| rpittau | *one voting | 15:33 |
| dtantsur | as much as I hate it, probably a whole set of jobs.. normal, keystone and virtual media | 15:33 |
| rpittau | yay | 15:33 |
| rpittau | ok, it unfortunately makes sense :) | 15:34 |
| dtantsur | in our defences, we definitely did not expect ubuntu to start renaming files when we made the decision to switch over to jammy | 15:34 |
| dtantsur | lesson learned | 15:34 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Upgrade from 2023.1 and use Jammy for the upgrade job https://review.opendev.org/c/openstack/bifrost/+/877515 | 15:35 |
| dtantsur | part 1 incoming | 15:35 |
| dtantsur | okay, now for 2023.1 to work we need to patch zed first, right? | 15:36 |
| rpittau | for the upgrade job, yes | 15:37 |
| dtantsur | yeeeahhh... and zed probably does not have the jammy fix at all | 15:37 |
| TheJulia | unfortunately we've found ubuntu renaming files in a stable release before, so it is just not surprising | 15:37 |
| dtantsur | ..... | 15:37 |
| * TheJulia slides a tasty beer over to dtantsur | 15:38 | |
| dtantsur | and may fix probably breaks debian. aaaaaaaaaaaaa | 15:38 |
| dtantsur | TheJulia: thank you, a bouldering session first, just survive 30 minutes more | 15:38 |
| * TheJulia only has ephemeral virtual beer for these moments | 15:38 | |
| * TheJulia needs mystic arts level beer supply skills should Thor appear. | 15:39 | |
| rpittau | lol | 15:39 |
| rpittau | that reminds of a What If episode | 15:39 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Upgrade from 2023.1 and use Jammy for the upgrade job https://review.opendev.org/c/openstack/bifrost/+/877515 | 15:40 |
| opendevreview | Mark Goddard proposed openstack/ironic stable/zed: Fix online upgrades for Bios/Traits https://review.opendev.org/c/openstack/ironic/+/877409 | 15:41 |
| opendevreview | Mark Goddard proposed openstack/ironic stable/yoga: Fix online upgrades for Bios/Traits https://review.opendev.org/c/openstack/ironic/+/877490 | 15:42 |
| opendevreview | Mark Goddard proposed openstack/ironic stable/xena: Fix online upgrades for Bios/Traits https://review.opendev.org/c/openstack/ironic/+/877491 | 15:42 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/2023.1: Repair Focal support and bring back the main voting jobs https://review.opendev.org/c/openstack/bifrost/+/877513 | 15:44 |
| dtantsur | so, this should pass everything, except for upgrades ^^ | 15:45 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/zed: Repair Focal support https://review.opendev.org/c/openstack/bifrost/+/877514 | 15:46 |
| dtantsur | and this should probably do it ^^ | 15:46 |
| JayF | dtantsur: I saw your message, ack and ACK! :D | 15:49 |
| dtantsur | :D | 15:49 |
| dtantsur | now, I wonder if the zed patch will fail because of the upgrade from yoga... | 15:49 |
| JayF | I think we've got a few things coming down for fix; once the upgrade fix lands and Nisha_Agarwal's fixes for secure boot we should cut a release | 15:50 |
| JayF | maybe have the .1 out before the integrated release is announced | 15:50 |
| rpittau | dtantsur: are you sure the name change happened for focal? I don't see the change there, so it may break zed | 15:52 |
| dtantsur | huh | 15:53 |
| dtantsur | then why on earth... | 15:53 |
| dtantsur | okay, let's see what the CI says | 15:53 |
| rpittau | let me triple check | 15:53 |
| rpittau | so this is the content of the package in focal-updates, with the old name https://packages.ubuntu.com/focal-updates/amd64/shim-signed/filelist | 15:54 |
| rpittau | this is jammy-updates, with the new name https://packages.ubuntu.com/jammy-updates/amd64/shim-signed/filelist | 15:54 |
| dtantsur | "Source /tmp/ansible.fcx96n5i/shim-signed//usr/lib/shim/shimx64.efi.signed not found" | 15:54 |
| rpittau | ..... | 15:54 |
| dtantsur | from https://zuul.opendev.org/t/openstack/build/2e26c8d2ce3b47ef8eaf6bb2fc0a68bf/artifacts | 15:55 |
| rpittau | the version checks, so I don't understand | 15:56 |
| dtantsur | "/tmp/ansible.fcx96n5i/shim-signed/usr/lib/shim/shimx64.efi.signed.latest", | 15:57 |
| * dtantsur is slowly going even more insane than usual | 15:57 | |
| dtantsur | the package was shim-signed_1.40.9+15.7-0ubuntu1_amd64.deb | 15:57 |
| rpittau | yeah | 15:57 |
| rpittau | so the site is wrong | 15:57 |
| rpittau | fantastic | 15:57 |
| dtantsur | mmm, maybe the "normal" version is a symlink to the ".latest" one? | 15:58 |
| dtantsur | and this symlink is incorrect on unpacking? | 15:58 |
| rpittau | no no, if you check on the site the actual content does not show any .latest file | 15:58 |
| rpittau | just the old one | 15:58 |
| rpittau | Package: shim-signed (1.40.9+15.7-0ubuntu1) | 15:59 |
| dtantsur | ....... | 15:59 |
| rpittau | oh well | 15:59 |
| rpittau | (╯°□°)╯︵ ┻━┻ | 16:00 |
| dtantsur | know what, I'm gonna do some bouldering, maybe by tomorrow this magic explains itself.. | 16:00 |
| rpittau | wise decision | 16:01 |
| * TheJulia is unsure we can go any *more* insane | 16:04 | |
| * TheJulia suspects "PTO" is needed | 16:05 | |
| * TheJulia needs PTO | 16:05 | |
| dtantsur | I'll have a break around Easter. hopefully there is still enough snow in the remote areas of the mountains.. | 16:05 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/21.3: Fix online upgrades for Bios/Traits https://review.opendev.org/c/openstack/ironic/+/877495 | 16:05 |
| opendevreview | Julia Kreger proposed openstack/ironic bugfix/21.2: Fix online upgrades for Bios/Traits https://review.opendev.org/c/openstack/ironic/+/877496 | 16:06 |
| rpittau | good night! o/ | 16:43 |
| opendevreview | Merged openstack/ironic-python-agent master: [Trivial] Fix typo in efi_utils https://review.opendev.org/c/openstack/ironic-python-agent/+/877478 | 16:54 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 18:00 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 18:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!