| *** JoyAkinyi is now known as Guest6254 | 08:57 | |
| *** JoyAkinyi is now known as Guest6255 | 08:57 | |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add port enable/disable support for Arista EOS https://review.opendev.org/c/openstack/networking-generic-switch/+/982755 | 09:23 |
|---|---|---|
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 10:22 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 10:25 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 10:31 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 13:05 |
| VictorAkoh[m] | hroy: Does this mean I have completed the initial task of installing and using bifrost? | 14:02 |
| TheJulia | Good morning folks! | 14:08 |
| edomfeh | Hi everyione. Outreachy applicant working on Bifrost install. I'm on ARM64 Ubuntu 24.04. `./bifrost-cli testenv --legacy-boot` works, but install fails trying to download grub-efi-amd64-signed (not available on ARM64). Tried uefi_boot_mode=false and download_packages=[]. Anyone run Bifrost on ARM64? Should I document the limitation for my report? | 14:18 |
| TheJulia | edomfeh: That would be a valid bug if your on arm64, you might want or may need to explicitly declare it only as needed for x86_64 hosts, patches welcome to that effect | 14:27 |
| dtantsur | I think arm support is still WIP in bifrost | 14:40 |
| TheJulia | To be run on arm, I suspect so | 14:52 |
| edomfeh | TheJulia: Thank you for confirming! I'll document the bug in my Outreachy report and will work on a patch to add the architecture condition. | 15:04 |
| TheJulia | Very cool, thanks! | 15:04 |
| dtantsur | https://review.opendev.org/c/openstack/bifrost/+/920057 is the WIP attempt | 15:04 |
| dtantsur | I'm afraid mnasiadka has run out of steam just like me before :) | 15:05 |
| mnasiadka | I’ll gather some steam tomorrow :) | 15:25 |
| mnasiadka | Some damn security bugs kept me busy | 15:25 |
| dtantsur | Security bugs can do that, yes | 15:34 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Migrate package metadata from setup.cfg to pyproject.toml https://review.opendev.org/c/openstack/ironic-python-agent/+/982864 | 15:43 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Update hacking to 7.0.0 https://review.opendev.org/c/openstack/ironic-python-agent/+/982867 | 15:49 |
| cardoe | If any core's have some cycles, can you peek at ironic-week-prio? | 15:59 |
| cardoe | There's a number of bug fixes that I'd like to work on backports for that are pending against master. | 16:00 |
| * dtantsur is buried under kubecon follow-ups | 16:00 | |
| cardoe | old branches of python-ironicclient fail cause of pkg_resources missing (2025.1 and older) we should likely fix that since its still supported no? | 16:05 |
| opendevreview | Syed Haseeb Ahmed proposed openstack/ironic stable/2025.2: make target_datetime optional for set_bmc_clock step https://review.opendev.org/c/openstack/ironic/+/982870 | 16:05 |
| TheJulia | I can try to cycle to reviews a little later. Honestly trying to avoid code reviews on cold mds | 16:08 |
| TheJulia | meds | 16:08 |
| cardoe | Your comments might be more fun than usual "ooooo look at all the red and then green text here!" | 16:13 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Enforce TLS 1.2 minimum across all connections https://review.opendev.org/c/openstack/ironic-python-agent/+/982174 | 16:14 |
| TheJulia | cardoe: more like "ooh... squirrel.... ZzZzZzZzzz" | 16:15 |
| cardoe | I like how reviews are bad on cold meds but new crypto code? YOLO! | 16:15 |
| TheJulia | claude is my friend | 16:21 |
| TheJulia | and I'm avoiding jira stuffs | 16:22 |
| cardoe | So I know we had some issue with utf8_collate_ci right? Is there some correct value to use? I'm trying to find it in the docs but striking out. | 16:35 |
| TheJulia | with column size? | 16:44 |
| TheJulia | in terms of byte width ? | 16:44 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Upgrade TLS certificates to P-384 curve with configurable support https://review.opendev.org/c/openstack/ironic-python-agent/+/982175 | 16:45 |
| TheJulia | cardoe: what repo are you seeking reviews on for stuff that might be backportable? | 16:47 |
| cardoe | ironic | 16:48 |
| cardoe | inspection bug, bmc_clock, and ramdisk driver bug | 16:49 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2026.1: Fix nova rebuilds w/ramdisk driver https://review.opendev.org/c/openstack/ironic/+/982877 | 17:01 |
| TheJulia | cardoe https://review.opendev.org/c/openstack/ironic/+/981942 ? | 17:02 |
| cardoe | That'd be another one | 17:02 |
| TheJulia | which one were you looking for? | 17:02 |
| JayF | https://us06web.zoom.us/j/84752772101?pwd=cWGuPg94g4fh4IiH7jXSUNqOh8ebHX.1&jst=2 GR-OSS team review is now() ish | 17:03 |
| JayF | if you wanna just jump in and point us at things | 17:03 |
| TheJulia | I'm unfortunately chatting with dmtf folk at the moment | 17:07 |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch stable/2026.1: Fix Arista EOS vxlan vlan unplug syntax https://review.opendev.org/c/openstack/networking-generic-switch/+/982881 | 17:22 |
| hroy | VictorAkoh[m], as the bifrost task states, you just need to get familiar, use it and share your experience ( that includes issues encountered, and what you did to resolve them ). We are not asking for advanced usage of bifrost, it's up to you as to what things you want to try there | 17:26 |
| hroy | just make sure to share your experience | 17:26 |
| hroy | as for your issues, I will try the steps myself and get back to you as soon as possible | 17:27 |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch stable/2026.1: Fix Arista EOS route-target auto syntax https://review.opendev.org/c/openstack/networking-generic-switch/+/982882 | 17:32 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/978789 and the parent need backporting? cc: cid | 17:41 |
| JayF | I'm thinking yes? | 17:41 |
| TheJulia | maybe, I think the thing to verify on backport that it doesne't inadvertently create a break or sudden thread launch change behavior | 17:45 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix idrac-redfish SKU→serial_number override to run before rule evaluation https://review.opendev.org/c/openstack/ironic/+/982478 | 17:46 |
| opendevreview | Jay Faulkner proposed openstack/networking-generic-switch stable/2026.1: arista: Added ERROR_MSG_PATTERNS https://review.opendev.org/c/openstack/networking-generic-switch/+/982888 | 17:47 |
| TheJulia | slight conundrum: looks like https://review.opendev.org/c/openstack/ironic/+/982478 has a unicode char in the commit message which raises the question was an ai agent used. How paranoid do we want to be is a super good question | 17:48 |
| TheJulia | thoughts? | 17:48 |
| TheJulia | cardoe: fyi ^ | 17:49 |
| cardoe | Tell him to update it. | 17:50 |
| cardoe | And I'll poke him with a stick. | 17:50 |
| cardoe | He should have put Assisted-by on it. | 17:50 |
| * TheJulia hands cardoe the appropriate stick with pre-sharpened point | 17:51 | |
| cardoe | If you give the review comment that'd help me out. | 17:52 |
| TheJulia | Just did | 17:52 |
| TheJulia | do I need to bring out the ai policy legal link as well ;) | 17:53 |
| JayF | Ah, I didn't realize that contributor was a racker | 17:56 |
| JayF | or I woulda DM'd you the WTF instead of Julia :) | 17:56 |
| TheJulia | I knew they were a racker which is why I brought it up | 17:56 |
| opendevreview | Syed Haseeb Ahmed proposed openstack/ironic master: Fix idrac-redfish SKU to serial_number override to run before rule evaluation https://review.opendev.org/c/openstack/ironic/+/982478 | 17:56 |
| TheJulia | I mean, otherwise it doesn't really read as it, but claude has gotten really really good at fitting into the context of what it is editing | 17:57 |
| TheJulia | but yeah, the unicode chars are a giant giveaway | 17:57 |
| TheJulia | single core approved with the updated commit message since it was previously approved | 17:58 |
| JayF | I don't LOVE trying to be the AI police | 18:02 |
| JayF | but someone either deserves kudos for using unicode well | 18:02 |
| JayF | or a redirection lol | 18:02 |
| JayF | at least claude doesn't drop curly-quotation-marks everywhere | 18:03 |
| TheJulia | heh | 18:08 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix nova rebuilds w/ramdisk driver https://review.opendev.org/c/openstack/ironic/+/982701 | 18:28 |
| opendevreview | Merged openstack/ironic-python-agent master: Add bootaa64.csv to BOOTLOADERS_EFI https://review.opendev.org/c/openstack/ironic-python-agent/+/982470 | 18:34 |
| opendevreview | Merged openstack/networking-generic-switch master: Fix Arista EOS vxlan vlan unplug syntax https://review.opendev.org/c/openstack/networking-generic-switch/+/982392 | 19:18 |
| opendevreview | Merged openstack/networking-generic-switch master: Fix Arista EOS route-target auto syntax https://review.opendev.org/c/openstack/networking-generic-switch/+/982393 | 19:18 |
| VictorAkoh[m] | hroy: Alright. Thank you | 19:20 |
| opendevreview | Merged openstack/networking-generic-switch master: Add redistribute learned to Arista EOS L2VNI conf https://review.opendev.org/c/openstack/networking-generic-switch/+/982424 | 19:41 |
| opendevreview | Merged openstack/networking-generic-switch master: arista: Added ERROR_MSG_PATTERNS https://review.opendev.org/c/openstack/networking-generic-switch/+/982339 | 19:41 |
| opendevreview | Verification of a change to openstack/networking-generic-switch master failed: Arista EOS: Create VLAN before VXLAN mapping https://review.opendev.org/c/openstack/networking-generic-switch/+/982400 | 19:41 |
| opendevreview | Merged openstack/ironic master: make target_datetime optional for set_bmc_clock step https://review.opendev.org/c/openstack/ironic/+/982261 | 19:51 |
| opendevreview | Merged openstack/ironic master: Soft power off before ejecting virtual media regardless of CONF.inspector.power_off https://review.opendev.org/c/openstack/ironic/+/981942 | 19:51 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix pickle error when oslo.service uses spawn mode https://review.opendev.org/c/openstack/ironic/+/977266 | 19:53 |
| opendevreview | Verification of a change to openstack/ironic master failed: Make CONF picklable for oslo.service's spawn probe https://review.opendev.org/c/openstack/ironic/+/978789 | 19:53 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 20:46 |
| opendevreview | Merged openstack/ironic master: Fix idrac-redfish SKU to serial_number override to run before rule evaluation https://review.opendev.org/c/openstack/ironic/+/982478 | 21:04 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Use startswith for interface validation in tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982960 | 21:13 |
| opendevreview | Abhishek Bongale proposed openstack/ironic master: Limit interfaces to fake-graphical for testing https://review.opendev.org/c/openstack/ironic/+/982270 | 21:16 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Arista EOS: Create VLAN before VXLAN mapping https://review.opendev.org/c/openstack/networking-generic-switch/+/982400 | 21:39 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add port enable/disable support for Arista EOS https://review.opendev.org/c/openstack/networking-generic-switch/+/982755 | 21:42 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Use startswith for interface validation in tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982960 | 21:50 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 21:50 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Use startswith for interface validation in tests https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982960 | 21:59 |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 21:59 |
| opendevreview | Harald Jensås proposed openstack/networking-baremetal master: Remove invalid requested-chassis opt for localnet ports https://review.opendev.org/c/openstack/networking-baremetal/+/981692 | 22:00 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Fix Arista EOS VLAN cleanup for VXLAN interfaces https://review.opendev.org/c/openstack/networking-generic-switch/+/982987 | 22:30 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2026.1: Fix idrac-redfish SKU to serial_number override to run before rule evaluation https://review.opendev.org/c/openstack/ironic/+/982990 | 23:52 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!