| opendevreview | Alexon Ferreira de Oliveira proposed openstack/sushy master: trivial: fix variable in log message https://review.opendev.org/c/openstack/sushy/+/940757 | 01:36 |
|---|---|---|
| opendevreview | Merged openstack/ironic stable/2024.2: Fix redfish session cache on missing password https://review.opendev.org/c/openstack/ironic/+/940634 | 01:54 |
| opendevreview | Merged openstack/ironic stable/2024.2: Fix agent from being locked out with complex steps https://review.opendev.org/c/openstack/ironic/+/940471 | 02:11 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2024.1: Fix agent from being locked out with complex steps https://review.opendev.org/c/openstack/ironic/+/940759 | 03:48 |
| rpittau | good morning ironic! o/ | 07:47 |
| opendevreview | Vasyl Saienko proposed openstack/networking-generic-switch master: Remove unneeded hasattr check for bond methods https://review.opendev.org/c/openstack/networking-generic-switch/+/940296 | 08:56 |
| opendevreview | Merged openstack/sushy master: trivial: fix variable in log message https://review.opendev.org/c/openstack/sushy/+/940757 | 10:31 |
| opendevreview | Merged openstack/ironic master: Fix invalid command in grenade destroy https://review.opendev.org/c/openstack/ironic/+/940745 | 11:01 |
| opendevreview | Merged openstack/ironic stable/2023.2: Fix redfish session cache on missing password https://review.opendev.org/c/openstack/ironic/+/940636 | 11:01 |
| opendevreview | Kaifeng Wang proposed openstack/ironic master: Fixes the issue of flat network provision when porgroup is configured https://review.opendev.org/c/openstack/ironic/+/934433 | 11:15 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic master: Fixes upstream table width on cleaning docs https://review.opendev.org/c/openstack/ironic/+/940779 | 12:01 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic master: Fixes upstream table width on cleaning docs https://review.opendev.org/c/openstack/ironic/+/940779 | 12:13 |
| vsaienko_ | JayF: please set vote one more time https://review.opendev.org/c/openstack/networking-generic-switch/+/940296 there was a trivial rebase | 12:54 |
| opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent stable/2024.1: Fix errors in the function erase_devices_express https://review.opendev.org/c/openstack/ironic-python-agent/+/940782 | 13:14 |
| opendevreview | Jakub Jelinek proposed openstack/ironic-python-agent stable/2023.2: Fix errors in the function erase_devices_express https://review.opendev.org/c/openstack/ironic-python-agent/+/940783 | 13:14 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Document metal3-integration job https://review.opendev.org/c/openstack/ironic/+/940800 | 15:28 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: [WIP] Document metal3-integration job https://review.opendev.org/c/openstack/ironic/+/940800 | 15:33 |
| opendevreview | Merged openstack/networking-generic-switch master: Remove unneeded hasattr check for bond methods https://review.opendev.org/c/openstack/networking-generic-switch/+/940296 | 15:58 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic master: Create a reference section Closes-Bug: #2072354 https://review.opendev.org/c/openstack/ironic/+/940811 | 16:22 |
| JayF | Whoa, hello Satoshi! | 16:25 |
| JayF | I haven't even said hello and pushing a patch already 👀 | 16:26 |
| TheJulia | nice | 16:29 |
| JayF | He's going to be working on the IPA hardware manager to run podman/docker containers as steps | 16:30 |
| JayF | I think we're going to start it out of tree to proof of concept then propose it in | 16:31 |
| TheJulia | It... really shouldn't be that complicated, but I could also see wanting to try and get the conductor to mirror containers | 16:42 |
| TheJulia | ... which... might be a bit of work in itself | 16:43 |
| JayF | our MVP is IPA connecting directly to the container registry | 16:43 |
| JayF | with that registry url being configurable | 16:43 |
| JayF | (but not runtime-configurable) | 16:43 |
| JayF | I wanted to be able to make an MVP that was ... kinda to a lower bar than I think an official IPA feature would need? | 16:44 |
| JayF | Once we get there, we will spec it upstream and adapt it | 16:44 |
| JayF | admittedly though, a nonzero part of that was so that I could punt on teaching gerrit, and clearly that's not a need :D | 16:44 |
| TheJulia | sort of easy to go "okay, you have a github PR on a branch or a branch in github, we're going to show you how to prepare and upload that!" | 16:54 |
| kubajj | JayF: hi, I saw that you have change https://review.opendev.org/c/openstack/ironic-python-agent/+/934091 which also fails the pipeline in Bobcat. Do you have any suggestion on how to fix it (conflicting dependency) or is it ok not to backport to Bobcat anymore? | 17:03 |
| JayF | I am completely out of context on those CI failures; I dug superficially but didn't dedicate a large amount of time to it. | 17:03 |
| JayF | I strongly suggest an approach of: 1) ensure CI failures are not actually failures in the code and 2) disabling jobs that are failing as long as we retain basic coverage | 17:04 |
| TheJulia | so when do we want a bootc demo | 17:05 |
| TheJulia | ? | 17:05 |
| rpittau | good night! o/ | 17:06 |
| JayF | I have a meeting at 10:30 pst thru about noon, outside of that I'm open today | 17:06 |
| kubajj | JayF: I think it shouldn't be a problem in the code, it's a conflicting dependency of pbr for sushy-tools, I think | 17:06 |
| JayF | Might be worth looking to see if that branch in Ironic got a fix; like pinning sushy-tools version | 17:07 |
| kubajj | TheJulia: I had a look into bootc and looks very interesting | 17:07 |
| kubajj | JayF: will have a look, thanks | 17:07 |
| JayF | Otherwise given that's IPA I'm slightly tempted to say just nuke redfish coverage and use ipmi, it *shouldn't* matter what protocol we use for BMCs ... | 17:07 |
| JayF | I'm not 100% sure that's okay, but it's also not okay for us to all spend our lives toiling against CI :) | 17:08 |
| TheJulia | for deploy, yeah, seems reasonable on older branches | 17:08 |
| TheJulia | I was thinking, based upon the discussion, that I could demo it tomorrow, but we could likely do something and record it today | 17:09 |
| JayF | I have only 30m of meeting tomorrow so again pretty open | 17:10 |
| TheJulia | Might just be best to do a call and record it | 17:13 |
| TheJulia | I'm mid-practice run right now | 17:13 |
| JayF | happy to play the part of inquisitive viewer if you want lol | 17:14 |
| TheJulia | ooohhh | 17:15 |
| TheJulia | yes, it should be done shortly and I can reset | 17:15 |
| TheJulia | if that works for you | 17:15 |
| JayF | as long as we're done by 1030 | 17:16 |
| TheJulia | should be | 17:16 |
| JayF | good stuff, lmk when you're ready and I'll relocate to my camera | 17:16 |
| TheJulia | it is presently unpacking to disk | 17:17 |
| TheJulia | almost done... | 17:19 |
| TheJulia | looks like I have a call at the top of the hour as well | 17:23 |
| TheJulia | okay, its shutting down it looks like | 17:23 |
| TheJulia | JayF and anyone else interested in bootc: https://meet.google.com/bph-yvys-yya | 17:25 |
| JayF | omw | 17:26 |
| cardoe | Joining 30 minutes late :( | 17:59 |
| TheJulia | sorry cardoe :( | 18:03 |
| cardoe | eh I missed your convo on it cause I had lunch with my wife so I didn't know. | 18:04 |
| cardoe | free internet points to anyone that reviews and +W's https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/940574 to save my butt from the rebase I did unnecessarily. | 18:06 |
| TheJulia | lunch with wife is waaaay more important | 18:07 |
| TheJulia | Hmm... I'm worried it might have eaten the recording | 18:23 |
| TheJulia | I guess I'll give it a little bit | 18:23 |
| TheJulia | and worst comes to worst, cardoe can get a demo | 18:23 |
| cardoe | yay | 18:25 |
| cardoe | I've got Zoom and not Google Meet. But sometimes it's like "what meeting and recording? You didn't have one." And then magically it appears after a bit. | 18:26 |
| TheJulia | Yeah, it finally just appeared a little bit ago | 18:40 |
| cardoe | There's etherpads from our ironic-networking calls right? | 18:41 |
| cardoe | also when is the next one? | 18:41 |
| TheJulia | Well, the last discussion was to move it to irc, and folks didn't show so... | 18:42 |
| TheJulia | dunno | 18:42 |
| TheJulia | I've got a lot going on right now | 18:42 |
| cardoe | I'll get you the Taylor Swift shirt. | 18:44 |
| TheJulia | cardoe: I should have a video link for you shortly | 18:44 |
| cardoe | I don't even fully know the joke. All I know is that Taylor Swift said something like "not a lot going on right now" when in fact she was super busy. So it's a thing to wear a shirt that says "not a lot going on" when you're in fact busy. I don't social media so I rely on my wife for these pop culture things. Boy I really do sound like the guy that yells at kids from his lawn. | 18:46 |
| TheJulia | TIL | 18:46 |
| cardoe | Really just looking for the old notes if ya got a pointer. | 18:47 |
| TheJulia | the old old notes from long ago? | 18:48 |
| TheJulia | https://etherpad.opendev.org/p/ironic-networking <-- recent notes | 18:48 |
| TheJulia | cardoe: https://drive.google.com/file/d/1MKC9SPIqnoFsziDeXu0oRdMDSa0CI2ol/view?usp=sharing | 18:49 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/940574 | 18:49 |
| TheJulia | JayF: your *super* quiet in the recording :( | 18:50 |
| opendevreview | Satoshi Shirosaka proposed openstack/ironic master: Create a reference section https://review.opendev.org/c/openstack/ironic/+/940811 | 19:30 |
| cardoe | TheJulia: Thank you! | 19:30 |
| opendevreview | Merged openstack/ironic master: Create a reference section https://review.opendev.org/c/openstack/ironic/+/940811 | 20:34 |
| TheJulia | cardoe: any questions regarding that | 20:51 |
| TheJulia | ? | 20:51 |
| cardoe | So trying to work further on the trunking bits and I swear there was some doc somewhere that said "for VMs the VLAN for cloud-init is known to be broken" | 21:05 |
| cardoe | wanted to fix the doc but can't find it and I was hoping there was a bread crumb in there. | 21:05 |
| cardoe | Also trying to land https://review.opendev.org/c/openstack/nova/+/940836 and hoping that works. | 21:05 |
| cardoe | As soon as I can get off of calls I'm gonna watch the video. | 21:05 |
| opendevreview | cid proposed openstack/ironic master: Make port binding failure (configurably) fatal https://review.opendev.org/c/openstack/ironic/+/699953 | 22:10 |
| JayF | I can attest that vlans work with cloud init and that vlans on top of bonds work in cloud init. The caveat is if you're using the metadata service, it doesn't work with those. Config drive it does. | 22:54 |
| JayF | I believe that is the entirety of the exceptions, but I could always be wrong | 22:54 |
| cardoe | It doesn’t work with cloud-init at all. In neither config drive or metadata is it populated. | 23:08 |
| JayF | cardoe: I was reading on mobile and completely glazed over the "for VMs" part of that message | 23:10 |
| JayF | cardoe: https://opendev.org/openstack/nova/src/commit/e27bbe72e0d293e55c30d4f90ca0afcf47427419/nova/virt/ironic/driver.py#L1045 is pretty self-explanatory as to why (the configdrive implementation for writing out bond+vlan is in our virt driver) | 23:11 |
| JayF | I would be a fan of an attempt to close this gap by making the official nova network-data generator look like Ironic's; I don't recall why we did it separately now as it was literally almost a decade ago | 23:11 |
| JayF | cardoe: perhaps that link would suggest to you a patch that could fix that downstream for you :-D | 23:11 |
| cardoe | Lemme follow up in a bit. Going on a bike ride with kids. | 23:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!