| cardoe | iurygregory: yeah nice find. | 00:03 |
|---|---|---|
| cardoe | The job is there and it shows up. | 00:03 |
| cardoe | JayF: https://docs.openstack.org/hacking/latest/user/hacking.html H306 doesn’t seem to | 00:04 |
| cardoe | Have the special OpenStack but not us group. Which means I bet I can configure ruff to comply with H306 | 00:04 |
| JayF | ooh | 03:14 |
| cardoe | Just saying... I love sorting imports in 0.10 seconds vs 15 seconds. | 03:27 |
| opendevreview | Morten Stephansen proposed openstack/ironic-python-agent stable/2024.1: Fix for motherboards where efibootmgr returns UTF-8. https://review.opendev.org/c/openstack/ironic-python-agent/+/957948 | 08:12 |
| *** kubajj_ is now known as kubajj | 10:44 | |
| zigo | Who's taking care of networking-baremetal ? Is this under the Ironic team ? | 10:50 |
| zigo | Because version 7.0.0 of networking-baremetal doesn't work with Neutron 26.0.0 (ie: epoxy), and its requirement must be 27.0.0~rc1 (or maybe the b1, I didn't check for it). | 10:51 |
| opendevreview | Stephen Finucane proposed openstack/ironic master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/ironic/+/961788 | 11:23 |
| iurygregory | good morning ironic | 11:45 |
| iurygregory | dtantsur, TheJulia idrac10 https://paste.opendev.org/show/bvrvdimREnVRlM4VeADv/ | 12:12 |
| iurygregory | I think I'm almost there... | 12:12 |
| dtantsur | You sure the image is actually good and reachable? | 12:12 |
| iurygregory | dtantsur, yup, I can access it in other machines http://<ip>:8080/centos9-image.qcow2 | 12:13 |
| dtantsur | and the GPT-vs-MBR is also as expected? | 12:14 |
| iurygregory | inspection finished without problems, now it's a problem when i trigger ./bifrost-cli deploy -e @baremetal-install-env.json | 12:14 |
| iurygregory | not 100% sure, going to start debugging now, need more coffee :D | 12:15 |
| iurygregory | but at least I was able to enroll the node and it went to available https://paste.opendev.org/show/bFaLNVvDRvXIVNouG0pg/ | 12:15 |
| iurygregory | I think is a good sign at least... | 12:16 |
| dtantsur | indeed | 12:20 |
| iurygregory | this was with an ironic hack for slot1 + TheJulia patch https://review.opendev.org/c/openstack/sushy/+/961730 | 12:21 |
| iurygregory | and a fresh bifrost install with stable 2025.1 | 12:22 |
| opendevreview | Takashi Kajinami proposed openstack/sushy master: Restore flake8-import-order https://review.opendev.org/c/openstack/sushy/+/953124 | 12:54 |
| TheJulia | iurygregory: that ain't your BMC controller, that is the host has a 4k raid controller and you likely used a 512 byte block aligned image | 13:31 |
| TheJulia | they are mutually exclusive layouts | 13:31 |
| iurygregory | TheJulia, ohhhhhhhhh | 13:32 |
| iurygregory | so I need to do some magic for it to work? | 13:32 |
| TheJulia | We hit some of this on the openstack side with newer raid controllers based on the latest generation of Broadcom LSI Fusion MPT SAS controllers | 13:33 |
| iurygregory | dtantsur, in ocp do we handle such cases? <eyes> | 13:33 |
| TheJulia | you literally just have to use a 4k block aligned image | 13:33 |
| TheJulia | your not rolling an image out | 13:33 |
| TheJulia | in OCP, quite like this | 13:33 |
| TheJulia | uhh... stevebaker did some work on DIB to be able to convert images | 13:33 |
| TheJulia | but you can also pass DIB_BLOCK_SIZE=4096 to a diskimage-builder command as an environment variable and it will align the blocks properly for that hardware | 13:34 |
| TheJulia | The root issue, unfortunately, is the way the GPT partitioning standard is written | 13:34 |
| TheJulia | It doesn't say strucutres are positionally aligned by byte location, but by logical blocks based upon the device | 13:34 |
| TheJulia | So for 4k, your partitioning data starts at like block 2, for 512 byte blocks its like block 16 or 32 (offhand, I don't remember exactly), but if you stop and do the math, you'd realize that it leaves a ton of extra space which structurally makes it impossible to have a universal image | 13:36 |
| TheJulia | it sort of cascades from there, but hopefully you get the basic idea | 13:36 |
| * TheJulia sips coffee in a vain attempt to become human | 13:39 | |
| * iurygregory searches for DIB_BLOCK_SIZE | 13:45 | |
| TheJulia | zigo: it is ours... 7.0.0 is flamingo, not epoxy though, so I'm a little confused what the issue is | 13:46 |
| TheJulia | iurygregory: downstream, we have a kb, step 3 is along the lines of what you could do to convert an image; https://access.redhat.com/articles/7096924 | 13:52 |
| iurygregory | TheJulia, tks! | 13:52 |
| TheJulia | iurygregory: so, does that mean the target changes were not necessary for sushy? | 13:59 |
| TheJulia | I guess we need to get a change landed w/r/t slotting | 13:59 |
| iurygregory | TheJulia, they were necessary, without https://review.opendev.org/c/openstack/sushy/+/950694 it would fail | 14:00 |
| opendevreview | Merged openstack/sushy master: Improve Dell Asynchronous task handling https://review.opendev.org/c/openstack/sushy/+/961730 | 14:01 |
| TheJulia | with or without the retry logic change for async handling | 14:01 |
| TheJulia | ... ^ that one | 14:01 |
| TheJulia | (that was great timing!) | 14:01 |
| iurygregory | I've tested with it =) | 14:07 |
| iurygregory | TheJulia, by any chance you have the block-device-default.yaml ? seems like my centos9 machine is unable to find openstack-tripleo-image-elements | 14:08 |
| iurygregory | or can I just use https://opendev.org/openstack/diskimage-builder/src/branch/master/diskimage_builder/elements/block-device-mbr/block-device-default.yaml ? | 14:11 |
| TheJulia | iurygregory: uhh | 14:15 |
| TheJulia | you can use a default one from DIB | 14:16 |
| TheJulia | uhh | 14:16 |
| TheJulia | you *do not* want to use mbr | 14:16 |
| TheJulia | you want EFI | 14:16 |
| iurygregory | ok | 14:16 |
| iurygregory | https://opendev.org/openstack/diskimage-builder/src/branch/master/diskimage_builder/elements/block-device-efi-4k/block-device-default.yaml | 14:16 |
| iurygregory | this one | 14:16 |
| TheJulia | So, technically MBR booting might work on that kind of gear, but I think it would be insane | 14:17 |
| iurygregory | now I will map things for disk-image-create -x -a x86_64 -o overcloud-hardened-uefi-full-4k.qcow2 --image-size 6GiB replace-partition-redhat :D | 14:17 |
| TheJulia | I don't know what your source is, but should likely work | 14:20 |
| TheJulia | DIB_BLOCK_SIZE does work the same on a standard image build, though | 14:20 |
| iurygregory | image-size 6GiB seems a lot | 14:21 |
| iurygregory | O.o | 14:21 |
| iurygregory | I think I'm doing something wrong https://paste.opendev.org/show/bixHZsFDw4nAjOHGuKJT/ O.o | 15:12 |
| dtantsur | iurygregory: OCP does not use images | 15:14 |
| dtantsur | coreos-installer uses the content of the ISO | 15:14 |
| iurygregory | so we won't have the 4k problem? nice | 15:15 |
| dtantsur | yeah, I don't think we will | 15:15 |
| dtantsur | (except for BMaaS, of course) | 15:15 |
| iurygregory | I'm trying to spin an OCP cluster with some machines | 15:16 |
| iurygregory | will likely be able to test with the custom changes to be sure things work begin next week | 15:16 |
| iurygregory | fingers crossed | 15:16 |
| cardoe | Just wanna poke folks and say the ironic-week-prio is filling up (my definition is over 25) and there's a number of patches with one (sometimes three) +2 reviews. | 15:47 |
| cardoe | We can use the weekend lull for all the gate jobs. | 15:47 |
| TheJulia | iurygregory: that is the base image size to expect | 16:09 |
| TheJulia | dtantsur: yeah, and in that case, all you can really do is just make folks aware and point to tools/options to make a 4k image | 16:10 |
| * dtantsur nods | 16:13 | |
| iurygregory | cardoe, I will take a look at some today | 16:17 |
| opendevreview | Julia Kreger proposed openstack/sushy stable/2025.2: Improve Dell Asynchronous task handling https://review.opendev.org/c/openstack/sushy/+/961818 | 16:41 |
| -opendevstatus- NOTICE: All hosted mailing lists are undergoing maintenance for the next hour: https://lists.opendev.org/archives/list/service-announce@lists.opendev.org/message/UTMXRWWTE5WA3IF6WS3BIEJAORI2D62V/ | 20:01 | |
| zigo | TheJulia: Its unit tests are failing if I don't put python3-neutron (>= 2:27.0.0~rc1) in the build-depends (ie: it fails with the epoxy version). I'm telling you so you should add such thing in the requirements of networking-baremetal for Flamingo, IMO. | 21:52 |
| *** dmellado9 is now known as dmellado | 22:01 | |
| JayF | Ideally, yes, but upper-constraints.txt enforces the right version, too. | 22:36 |
| JayF | If you put in the merge request I'd review it | 22:37 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!