| TheJulia | JayF: I my spoon canister has no spoons for imparting that reality right now | 00:17 |
|---|---|---|
| TheJulia | so, dtantsur it looks like cloud-init does have support, not enabeld by default, we likely need some new dib stuffs to aid the case. I'll see if I can hammer some stuff out tomorrow since it looks like glean *is* doing what we expect and cloud-init smashes it | 00:18 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent master: Test coverage for efi_utils.get_boot_record https://review.opendev.org/c/openstack/ironic-python-agent/+/899774 | 02:53 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent master: Parse efibootmgr type and details https://review.opendev.org/c/openstack/ironic-python-agent/+/899775 | 02:53 |
| *** tonyb_ is now known as tonyb | 08:57 | |
| *** erbarr_ is now known as erbarr | 08:57 | |
| *** TheJulia_ is now known as TheJulia | 08:57 | |
| *** v1k0d3n_ is now known as v1k0d3n | 08:57 | |
| *** buhman_ is now known as buhman | 08:57 | |
| *** mnasiadka_ is now known as mnasiadka | 08:57 | |
| *** clarkb is now known as Guest5393 | 08:59 | |
| *** ravlew is now known as Guest5411 | 09:00 | |
| *** clarkb is now known as Guest5418 | 10:45 | |
| iurygregory | good morning Ironic | 11:22 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent stable/2023.2: Conditional creation of RAIDed ESP for UEFI Software RAID https://review.opendev.org/c/openstack/ironic-python-agent/+/899325 | 11:49 |
| arne_wiebalck | hmm | 11:51 |
| arne_wiebalck | didn't realise there was a backport already ... anything I should do now? It is the same code. | 11:52 |
| iurygregory | a bit weird that the backport doesn't have the cherry-pick info O.o | 11:55 |
| iurygregory | oh, in your commit it removed XD | 11:56 |
| arne_wiebalck | iurygregory: '-x' ? | 12:00 |
| iurygregory | arne_wiebalck, probably you missed it =) (I would push a new one with the information) | 12:00 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent stable/2023.2: Conditional creation of RAIDed ESP for UEFI Software RAID https://review.opendev.org/c/openstack/ironic-python-agent/+/899325 | 12:01 |
| arne_wiebalck | iurygregory: done | 12:02 |
| iurygregory | arne_wiebalck, ty! | 12:02 |
| arne_wiebalck | let's see if I need cookies for zuul like in the old days :-D | 12:02 |
| iurygregory | XD | 12:02 |
| iurygregory | sometimes you will need | 12:03 |
| TheJulia | good morning | 13:16 |
| arne_wiebalck | Good morning TheJulia o/ | 13:25 |
| TheJulia | dtantsur: https://review.opendev.org/c/openstack/ironic-python-agent/+/899774/1/ironic_python_agent/tests/unit/test_efi_utils.py#226 <-- do you happen to have the efibootmgr output from cases where the machine has been configured to boot to CD ? | 13:36 |
| iurygregory | I think he will only answer tomorrow | 13:38 |
| *** dansmith_ is now known as dansmith | 13:40 | |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent stable/2023.1: Conditional creation of RAIDed ESP for UEFI Software RAID https://review.opendev.org/c/openstack/ironic-python-agent/+/899859 | 13:49 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent stable/zed: Conditional creation of RAIDed ESP for UEFI Software RAID https://review.opendev.org/c/openstack/ironic-python-agent/+/899860 | 13:51 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/899512 this fixes our devstack doc to actually work and is reasonably trivial | 14:42 |
| TheJulia | iurygregory: likely fine in the grand scheme of things | 14:56 |
| iurygregory | yes =) | 14:57 |
| *** Guest5418 is now known as clarkb | 15:03 | |
| opendevreview | Merged openstack/ironic master: Update devstack guide for image id; network always needed https://review.opendev.org/c/openstack/ironic/+/899512 | 15:11 |
| JayF | ty | 15:12 |
| opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent stable/yoga: Conditional creation of RAIDed ESP for UEFI Software RAID https://review.opendev.org/c/openstack/ironic-python-agent/+/899883 | 15:59 |
| *** sfinucan is now known as stephenfin | 16:15 | |
| *** promethe- is now known as prometheanfire | 16:49 | |
| *** prometheanfire is now known as Guest5452 | 16:49 | |
| *** Guest5452 is now known as prometheanfire | 16:51 | |
| JayF | janders: I have some downstream folks who might be interested in that repository-of-redfish-bmc-definitions stuff you were talking about at the PTG; can you pull me into any meaningful discussions about that? | 16:52 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP/DNM: Advanced vmedia deployment test ops https://review.opendev.org/c/openstack/ironic/+/898010 | 17:14 |
| TheJulia | muahahahaha I've reproduced the vmedia issue in CI, and looks like we're not far from having a fix in the current patch sets | 18:58 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP/DNM: Get rid of simple-init (but keep glean) https://review.opendev.org/c/openstack/ironic-python-agent/+/895519 | 19:37 |
| * TheJulia longingly awaits the CI job to complete | 19:51 | |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent master: Test coverage for efi_utils.get_boot_record https://review.opendev.org/c/openstack/ironic-python-agent/+/899774 | 19:57 |
| opendevreview | Steve Baker proposed openstack/ironic-python-agent master: Parse efibootmgr type and details https://review.opendev.org/c/openstack/ironic-python-agent/+/899775 | 19:57 |
| stevebaker[m] | good morning | 19:58 |
| TheJulia | good morning | 19:58 |
| stevebaker[m] | If anyone has "efibootmgr -v" entries that don't look like any of these then I can add them to the test https://review.opendev.org/c/openstack/ironic-python-agent/+/899774/2/ironic_python_agent/tests/unit/test_efi_utils.py#211 | 20:01 |
| TheJulia | stevebaker[m]: awesome you have the Fvfile stuffs :) | 20:03 |
| TheJulia | I'm starting to wonder if we should remove records serviced over USB interfaces as well | 20:03 |
| JayF | I met the right folks downstream to get that for me, too | 20:04 |
| JayF | so I have an ask out for an efibootmgr -v | 20:04 |
| stevebaker[m] | TheJulia: The step could have an argument for what to delete, defaulting to only HD? | 20:06 |
| TheJulia | I think under a default case, usb devices are where we would end up hiding orphaned boot records for virtual media CDs which is another case where we can sort of hit this weird case of bad records starting to build up | 20:07 |
| TheJulia | I'm sort of hopeful dtantsur has a efibootmgr -v output from a case where it has been reported to him | 20:08 |
| TheJulia | I guess another though is the usb devices are a generic carrier/transport | 20:17 |
| JayF | should we do something gross like, hash and store each line somewhere? | 20:33 |
| TheJulia | what would that gain us? | 20:39 |
| JayF | we could clean the diff. | 20:39 |
| TheJulia | what would we do with it? | 20:39 |
| TheJulia | A diff doesn't matter as much, really, and it is going to get changed by every OS | 20:40 |
| JayF | At first deploy/clean/enroll/some-starting-point | 20:40 |
| JayF | you can get a list of all the lines, hash each one | 20:40 |
| JayF | ones added/changed post-ironic-provisioning are cleaned | 20:40 |
| JayF | that has an assumption that, e.g. we can normalize the entries enough to remove things that would generate noise (like ordering) | 20:40 |
| TheJulia | but then we risk removing entries which are firmware added if something changes there | 20:41 |
| JayF | and that any ones created by the system we'd want to preserve are relatively-static | 20:41 |
| JayF | oooh that's a very compelling counterargument | 20:41 |
| JayF | especially given how many people would firmware update out of ironic's knowledge | 20:41 |
| * TheJulia looks at Emulex HBAs much like one looks at vermooth | 20:41 | |
| JayF | ight, I have a headache and a workday on Saturday (SeaGL conference prezi), so I'm going to stop staring at the screen o/ | 20:42 |
| TheJulia | Goodnight | 20:46 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP/DNM: Navigate re-loading simple-init configuration https://review.opendev.org/c/openstack/ironic-python-agent/+/895519 | 20:58 |
| opendevreview | Julia Kreger proposed openstack/sushy master: Adds an option for setting the http boot uri https://review.opendev.org/c/openstack/sushy/+/718276 | 21:22 |
| opendevreview | Julia Kreger proposed openstack/sushy-tools master: WIP: Support testing for HttpBootUri https://review.opendev.org/c/openstack/sushy-tools/+/896963 | 21:35 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Add test for dhcp-less vmedia based deployment https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/898006 | 22:53 |
| * TheJulia goes and exercises | 22:56 | |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: WIP: Add test for dhcp-less vmedia based deployment https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/898006 | 23:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!