| cardoe | So why is firmware update via management and via firmware interface? A legacy thing? | 00:08 |
|---|---|---|
| JayF | yes | 00:10 |
| JayF | firmware update as a first-class ironic concept is ... 18-ish months old? | 00:10 |
| JayF | and was really enabled by redfish firmware stuff | 00:11 |
| JayF | I think we'll end up with all the management-style firmware updates dying out as the older drivers do (right?) | 00:11 |
| cardoe | Well I ask cause I squirreled in the specs and saw one for Dell that’s not implemented according to it for management firmware updates. | 00:12 |
| JayF | yeah, the dell one was in the idrac-wsman driver iirc | 00:13 |
| JayF | but I'm not super familiar with that side of the codebase | 00:13 |
| JayF | I've personally provisioned 0 dell servers with Ironic, and have never used a dell-specific driver | 00:13 |
| cardoe | https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/drac-firmware-update-spec.html | 00:14 |
| cardoe | That's for redfish and wsman. | 00:15 |
| cardoe | And that works today. | 00:15 |
| JayF | this seems done | 00:15 |
| JayF | yeah | 00:15 |
| JayF | I'd be +1 to you symlinking that to whatever ironic release it was we released the redfish firmware driver | 00:16 |
| JayF | (and removing the one from not-implemented) | 00:16 |
| iurygregory | more than 18 months =) | 00:16 |
| iurygregory | JayF, ack I will check about bugfix9.9 | 00:16 |
| JayF | 18 months is my default for "less than 5 years more than 1" | 00:16 |
| JayF | lol | 00:16 |
| iurygregory | oh ok! | 00:16 |
| JayF | time is a mystery | 00:16 |
| cardoe | I'll check it out | 00:16 |
| JayF | at least, perception of it by me :D | 00:17 |
| iurygregory | 2016 Mitaka | 00:18 |
| iurygregory | update_firmware for ilo | 00:18 |
| iurygregory | 2018 Queens a new step update_firmware_sum for ilo | 00:18 |
| iurygregory | 2020 update_firmware via management interface for Redfish | 00:18 |
| cardoe | https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/httpboot.html that's implemented as well. | 00:19 |
| cardoe | So it's 2020 that was done so just gotta figure out the release. | 00:19 |
| cardoe | Trying to find the spec for the child nodes | 00:20 |
| iurygregory | 2020 Victoria | 00:21 |
| iurygregory | sorry forgot to mention =) | 00:21 |
| JayF | holy hell | 00:22 |
| JayF | time flies when you're ... building clouds? | 00:22 |
| iurygregory | yes, for sure | 00:22 |
| iurygregory | I'm working with OpenStack things since 2015 =) | 00:22 |
| JayF | I checked at one point, I think my earliest commit was to teeth-agent (original IPA) back in like april of 2014 | 00:23 |
| iurygregory | nice! | 00:23 |
| opendevreview | Adam McArthur proposed openstack/ironic-tempest-plugin master: WIP: Microversion handling test https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/933970 | 00:50 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Correct invalid docstrings; s/Found/Error/ https://review.opendev.org/c/openstack/ironic-python-agent/+/911598 | 01:55 |
| cardoe | ^ this aint gonna pass | 01:56 |
| TheJulia | So, I have gained access to the ironic-bugs group and have granted permission to Jay and Dmitry. I think the thing to do is set an expiration on the group to let folks expire out | 02:34 |
| TheJulia | alternatively (and maybe still do this, go through and remove folks we know have left the community | 02:34 |
| rpittau | good morning ironic! o/ | 08:28 |
| rpittau | JayF: re bugfix/9.9 or bugfix branches in general: if you're ever in doubt if we support a branch justcheck https://etherpad.opendev.org/p/IronicWhiteBoard around line 51, I usually keep that up-to-date | 08:29 |
| rpittau | in this specific case on L55: "We keep 3 bugfix branches at all times, the latest 2 are actively supported, the third one is for safety." | 08:29 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use linux instead of linuxefi in grub config https://review.opendev.org/c/openstack/ironic/+/933631 | 08:41 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add support for Ubuntu 24.04 https://review.opendev.org/c/openstack/ironic/+/934150 | 08:41 |
| opendevreview | Merged openstack/ironic master: add qemu-img to necessary dependency list https://review.opendev.org/c/openstack/ironic/+/934262 | 09:20 |
| opendevreview | Verification of a change to openstack/ironic master failed: Migrate lint to pre-commit https://review.opendev.org/c/openstack/ironic/+/933686 | 09:33 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: Add vlan aware VMs support https://review.opendev.org/c/openstack/networking-generic-switch/+/928490 | 10:20 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add support for Ubuntu 24.04 https://review.opendev.org/c/openstack/ironic/+/934150 | 10:26 |
| opendevreview | Verification of a change to openstack/ironic master failed: Migrate lint to pre-commit https://review.opendev.org/c/openstack/ironic/+/933686 | 11:13 |
| *** mraineri_ is now known as mraineri | 11:39 | |
| dtantsur | folks, looking for a 2nd +2 here https://review.opendev.org/c/openstack/ironic/+/932720 https://review.opendev.org/c/openstack/ironic/+/932610 https://review.opendev.org/c/openstack/ironic-python-agent/+/931017 (all related to NC-SI) | 12:59 |
| dtantsur | also this refactoring https://review.opendev.org/c/openstack/ironic-python-agent/+/928779 | 13:00 |
| opendevreview | Merged openstack/ironic master: Migrate lint to pre-commit https://review.opendev.org/c/openstack/ironic/+/933686 | 13:07 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add a command to lock down the agent https://review.opendev.org/c/openstack/ironic-python-agent/+/934234 | 13:14 |
| cardoe | trade ya dtantsur https://review.opendev.org/c/openstack/sushy/+/932096 https://review.opendev.org/c/openstack/ironic/+/927544 | 13:52 |
| dtantsur | looking | 13:59 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Add support for Ubuntu 24.04 https://review.opendev.org/c/openstack/ironic/+/934150 | 14:04 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Inventoried MAC address for only ipv6 addresses https://review.opendev.org/c/openstack/ironic-python-agent/+/934330 | 14:42 |
| opendevreview | Verification of a change to openstack/ironic master failed: Redfish power: account for disable_power_off https://review.opendev.org/c/openstack/ironic/+/932610 | 14:46 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Add a command to lock down the agent https://review.opendev.org/c/openstack/ironic-python-agent/+/934234 | 14:50 |
| TheJulia | dtantsur: would you be able to do 1500 on Wednesdays? | 14:52 |
| TheJulia | For a networking working group? | 14:52 |
| dtantsur | TheJulia: will be a bit touch but doable (in winter at least) | 14:53 |
| TheJulia | We can always adjust after we get started | 14:54 |
| dtantsur | 1500 UTC, I assume? | 14:56 |
| TheJulia | yes | 15:00 |
| opendevreview | Merged openstack/sushy master: Add Port resource https://review.opendev.org/c/openstack/sushy/+/932096 | 15:01 |
| opendevreview | Merged openstack/ironic-python-agent master: Migrate more trivial code from ironic-lib https://review.opendev.org/c/openstack/ironic-python-agent/+/928779 | 15:07 |
| opendevreview | Merged openstack/ironic-python-agent master: A hardware manager call for a full sync before shutdown https://review.opendev.org/c/openstack/ironic-python-agent/+/931017 | 15:07 |
| cardoe | TheJulia: so just my stupid question on https://review.opendev.org/c/openstack/ironic/+/896570 and if my ask is wrong I'll +2 it | 16:05 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2024.2: add qemu-img to necessary dependency list https://review.opendev.org/c/openstack/ironic/+/934336 | 16:08 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: dedup reboot request in redfish bios path https://review.opendev.org/c/openstack/ironic/+/933020 | 16:13 |
| TheJulia | cardoe: replied | 16:20 |
| dtantsur | TheJulia: would it be safe to disable image format checks for TFTPImageCache? | 16:25 |
| dtantsur | I think it's only used for kernel/initramfs, but I'm afraid of missing something | 16:25 |
| opendevreview | Verification of a change to openstack/ironic master failed: Actually ignore [inspector]power_off with fast track https://review.opendev.org/c/openstack/ironic/+/932720 | 16:27 |
| TheJulia | dtantsur: errr.. hmmmmmm | 16:38 |
| TheJulia | dtantsur: and I take it the 10,000ft or (something like 3048 meter view) is something like they added a check which flags on raw kernels/ramdisks? | 16:40 |
| JayF | Should we adopt a philosophy similar to IPA? Only security check when/if you have to pass it to qemu-img/convert it? | 16:41 |
| TheJulia | and in knowing the type of artifacts, I guess it might be okay... dunno. | 16:41 |
| JayF | I guess image cache in Ironic might be more decoupled than the code in IPA though | 16:42 |
| TheJulia | Well, *ideally* we should be doing something with anything we pull down but if checks are reaching into unreasonable/not applicable, yeah... | 16:42 |
| TheJulia | a kernel artifact is a bit distinctly different, because it is also a bootable executable artifact | 16:43 |
| dtantsur | TheJulia: I don't know why kernels seem to have MBR and thus fall under GPTInspector | 16:51 |
| TheJulia | Because they are bios bootable | 16:51 |
| dtantsur | rly? | 16:51 |
| dtantsur | Oo | 16:51 |
| TheJulia | ya'rly | 16:51 |
| TheJulia | when we did rabid squirrel linux, we made a boot floppy (yes, seriously, I was the one doing this), which was a concatonated kernel and initrd | 16:52 |
| TheJulia | no bootloader required | 16:52 |
| dtantsur | Wow, TIL! | 16:52 |
| TheJulia | yeah, so the kernel look right after the end of itself for the initial ramdisk contents | 16:52 |
| opendevreview | Verification of a change to openstack/ironic master failed: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/ironic/+/927544 | 16:53 |
| dtantsur | TheJulia: this is actually pretty neat.. but it does cause us headache now :D | 16:56 |
| dtantsur | hmm, a lot of assorted gate failures today, it seems | 16:57 |
| rpittau | looks like the jobs don't even start | 16:57 |
| * dtantsur understands the jobs | 16:57 | |
| rpittau | lol | 16:57 |
| rpittau | good night! o/ | 17:00 |
| TheJulia | ugh | 17:04 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Inventoried MAC address for only ipv6 addresses https://review.opendev.org/c/openstack/ironic-python-agent/+/934330 | 17:13 |
| opendevreview | Doug Goldstein proposed openstack/sushy master: switch to pre-commit https://review.opendev.org/c/openstack/sushy/+/929704 | 17:18 |
| cardoe | JayF: the finally only walks the list of things that have been added | 17:19 |
| cardoe | JayF: btw ^ is my attempt to bring over your pre-commit changes. | 17:20 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: OCI Container Registry Image Source https://review.opendev.org/c/openstack/ironic-specs/+/933612 | 17:20 |
| * TheJulia dashes hopes and dreams | 17:20 | |
| dtantsur | TheJulia: another prior art for distributing disk images via OCI: https://kubevirt.io/user-guide/storage/disks_and_volumes/#containerdisk-workflow-example | 17:32 |
| TheJulia | dtantsur: oooh, yeah, similar, just a disk file inside of a container in that example | 17:45 |
| dtantsur | TheJulia: yep, the simplest possible way - exactly the way we wanted to do it | 17:45 |
| dtantsur | I have some high hopes for the meeting next week though | 17:45 |
| TheJulia | For rhoso, we're a little more complicated, we have a file in a container and we expect a script on container launch to copy the artifact to a shared folder which is then used | 17:46 |
| TheJulia | kind of... weird | 17:46 |
| dtantsur | We do something similar for our handling of CoreOS images | 17:47 |
| TheJulia | Yeah, the kind of annoying thing with the overall format modeling is the tooling is going to have to do base data extraction along the way no matter *what* we do | 17:47 |
| TheJulia | unless someone creats a clean shortcut | 17:47 |
| TheJulia | or an upper level manifest modeling, and maybe overall that can be iterated towards | 17:47 |
| dtantsur | Maybe? That's something I want to learn next week | 17:47 |
| TheJulia | ++ | 17:47 |
| TheJulia | Yeah, best thing to do is make them aware that "there is a thing people are doing" | 17:48 |
| dtantsur | At least, podman is doing the exactly thing I'm hoping for: qcow2 images shipped via OCI | 17:48 |
| dtantsur | as artifacts, not just files | 17:48 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Correct invalid docstrings; s/Found/Error/ https://review.opendev.org/c/openstack/ironic-python-agent/+/911598 | 17:49 |
| opendevreview | Maximilian Brandt proposed openstack/ironic-python-agent master: Inventoried MAC address for only ipv6 addresses https://review.opendev.org/c/openstack/ironic-python-agent/+/934330 | 17:53 |
| opendevreview | Verification of a change to openstack/ironic master failed: Actually ignore [inspector]power_off with fast track https://review.opendev.org/c/openstack/ironic/+/932720 | 18:08 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Support for bootable containers https://review.opendev.org/c/openstack/ironic-specs/+/933620 | 18:34 |
| opendevreview | Verification of a change to openstack/ironic master failed: add pyproject.toml to support pip 23.1 https://review.opendev.org/c/openstack/ironic/+/927544 | 18:43 |
| TheJulia | dtantsur: well, the way I read it, it is still a file in a container, your even able to provide an override path | 19:15 |
| TheJulia | the entire container becomes an artifact at that point | 19:16 |
| opendevreview | Julia Kreger proposed openstack/ironic-tempest-plugin master: Fix failure cases around test_ports https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/934387 | 22:26 |
| shermanm | another question, is there a straightforward way to have neutron bind a port to a specific baremetal port? e.g. if we have a node with both a 1g and a 10g interface, how does a user make sure the 10g interface is the one configured | 22:53 |
| shermanm | if we don't specify, it *seems* to just depend on the order that ports are returned from `baremetal port list` | 22:54 |
| JayF | I assume a lot of folks have this solved; but this is a review dash I put up that includes *every* ironic-related repo | 22:59 |
| JayF | https://review.opendev.org/q/status:open+and+((project:%5Eopenstack/ironi.*)+or+(project:%5Eopenstack/sush.*)+or+(project:openstack/virtualpdu)+or+(project:openstack/virtualbmc)+or+(project:openstack/metalsmith)+or+(project:openstack/molteniron)+or+(project:openstack/networking-baremetal)+or+(project:openstack/networking-generic-switch)+or+(project:%5Eopenstack/python-ironi.*)+or+(project:openstack/tenks))+and+is:mergeable | 22:59 |
| JayF | shermanm: I'm not sure; that might be a good one for the mailing list if nobody answers it soon | 23:07 |
| shermanm | fair! | 23:14 |
| cardoe | I need to add that as a button. | 23:37 |
| JayF | I spent some time today organizing my review dashboards, I now can just open em all up in one shot | 23:38 |
| JayF | I do a decent job staying on top of Ironic ad-hoc, but as I'm trying to review more oslo/dib I needed a system | 23:39 |
| JayF | but I already improved that url | 23:39 |
| JayF | well, it's too long for one message not, but I added and -owner:self and -reviewedby:self | 23:39 |
| *** tosky_ is now known as tosky | 23:40 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!