| opendevreview | Merged openstack/ironic master: remove sushy_oem_idrac usage https://review.opendev.org/c/openstack/ironic/+/970273 | 01:11 |
|---|---|---|
| opendevreview | Merged openstack/ironic master: doc: trivial: Quick revision of README https://review.opendev.org/c/openstack/ironic/+/968056 | 01:11 |
| opendevreview | Merged openstack/ironic stable/2025.1: fix: ensure that the bootloader path exists https://review.opendev.org/c/openstack/ironic/+/969122 | 01:11 |
| opendevreview | Merged openstack/ironic master: Replace remaining usage of policy.json https://review.opendev.org/c/openstack/ironic/+/970650 | 01:11 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Make post-firmware-update reboot conditional on component https://review.opendev.org/c/openstack/ironic/+/966344 | 04:04 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix firmware update failure when task remains in RUNNING state https://review.opendev.org/c/openstack/ironic/+/970998 | 04:06 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix BIOS firmware not flashing when task completes before first poll https://review.opendev.org/c/openstack/ironic/+/970999 | 04:15 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix BMC resource validation to catch all sushy exceptions https://review.opendev.org/c/openstack/ironic/+/971000 | 04:19 |
| rpittau | good morning ironic! o/ | 08:05 |
| *** ubuntu is now known as Guest33655 | 09:36 | |
| opendevreview | Merged openstack/ironic master: Use native override of RequestContext.from_environ https://review.opendev.org/c/openstack/ironic/+/970970 | 10:15 |
| opendevreview | Merged openstack/ironic master: fix: agent inspection hooks failure does not clean up https://review.opendev.org/c/openstack/ironic/+/970922 | 10:17 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Use common request_id middleware https://review.opendev.org/c/openstack/ironic/+/970978 | 11:15 |
| iurygregory | good morning ironic | 12:03 |
| iurygregory | does anyone notice a problem with releasenote jobs? | 12:04 |
| iurygregory | s/does/did | 12:04 |
| iurygregory | https://zuul.opendev.org/t/openstack/build/c39749889c3b48f5a00a819c705a022f | 12:04 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.2: fix: agent inspection hooks failure does not clean up https://review.opendev.org/c/openstack/ironic/+/971031 | 12:31 |
| opendevreview | Doug Goldstein proposed openstack/ironic bugfix/33.0: fix: agent inspection hooks failure does not clean up https://review.opendev.org/c/openstack/ironic/+/971032 | 12:32 |
| frickler | iurygregory: looks to me like an issue in that change, commented | 12:48 |
| iurygregory | frickler, oh tks! | 12:49 |
| iurygregory | I couldn't really find the error when looking at it .-. | 12:49 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Make post-firmware-update reboot conditional on component https://review.opendev.org/c/openstack/ironic/+/966344 | 13:35 |
| opendevreview | Doug Goldstein proposed openstack/ironic stable/2025.1: fix: agent inspection hooks failure does not clean up https://review.opendev.org/c/openstack/ironic/+/971043 | 13:40 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix firmware update failure when task remains in RUNNING state https://review.opendev.org/c/openstack/ironic/+/970998 | 14:03 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix BIOS firmware not flashing when task completes before first poll https://review.opendev.org/c/openstack/ironic/+/970999 | 14:06 |
| cardoe | Good morning ironic | 14:06 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix BMC resource validation to catch all sushy exceptions https://review.opendev.org/c/openstack/ironic/+/971000 | 14:06 |
| cardoe | Since I'm always in a work call during the ironic meeting and I struggle to keep up... wanted to throw https://review.opendev.org/c/openstack/ironic/+/957613 out here as one to have folks decide on. | 14:09 |
| womax | hi ironic team, a few weeks ago I came here to discuss temp URL and adding s3 support for such url. After a bit of discutions, I decided to go to Glance and propose direct support in Glance. | 14:12 |
| cardoe | dtantsur: what do we gotta do to https://review.opendev.org/c/openstack/ironic/+/969277 to get it over the line? | 14:12 |
| cardoe | womax: so updating glance to support the temp url style? or updating ironic's glance support? | 14:13 |
| womax | updating glance to support temp url | 14:13 |
| womax | more precisely temp url generation | 14:13 |
| womax | I came up with a spec I proposed recently (https://review.opendev.org/c/openstack/glance-specs/+/970214). In it I mentioned Ironic already use Swift URL and it could be useful to have then managed at glance level. And I am now facing the question of why are they used in Ironic and wanted to make sure I understand it correctly | 14:14 |
| womax | My current understanding is: it is python-ironic-agent that need to download image on the bm server, but deploying credentials to access either glance or storage backend would be tricky / not very secure. So ironic-conductor generates a temp URL that it passes down to python-ironic-agent that can then download the image without any credentials required. is that it? | 14:16 |
| opendevreview | Merged openstack/ironic bugfix/33.0: fix: agent inspection hooks failure does not clean up https://review.opendev.org/c/openstack/ironic/+/971032 | 14:18 |
| dtantsur | cardoe: I'd prefer my comments addressed; not sure if TheJulia has time though | 14:18 |
| cardoe | ah I didn't look at resolved comments so that's why I didn't see any | 14:19 |
| cardoe | I dunno when TheJulia is back from PTO. | 14:19 |
| cardoe | womax: that's correct | 14:19 |
| dtantsur | cardoe: I'm honestly not sure why they show as resolved Oo | 14:19 |
| cardoe | Just trying to triage the ironic-week-prio a little before my meetings start. | 14:20 |
| womax | cardoe: thanks | 14:25 |
| womax | cardoe: and to be a bit more precise (it is mostly curiosity at this point), the fact that there is no credentials deploy with ironic-python-agent is more linked to a security reason or feasibilty reason | 14:39 |
| womax | my guess is security but i'm a bit curious to know for sure | 14:40 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Fix handling of unknown metric keys in ironic parser https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/970627 | 14:49 |
| TheJulia | good morning | 14:52 |
| TheJulia | I'm back, but doing a lot of administrivia at the moment | 14:52 |
| kubajj | o/ | 15:01 |
| janders | o/ | 15:02 |
| dtantsur | Could someone who is not me drive the meeting this time? | 15:02 |
| TheJulia | The meeting has been cancelled | 15:03 |
| TheJulia | It was cancelled weeks ago | 15:03 |
| dtantsur | ah, fair enough | 15:03 |
| kubajj | I thought it's next week's that's cancelled | 15:03 |
| TheJulia | ... at least | 15:03 |
| kubajj | but fair | 15:03 |
| TheJulia | I think it was | 15:03 |
| TheJulia | uhhhh | 15:03 |
| * TheJulia looks | 15:03 | |
| kubajj | | <TheJulia> Any thoughts on skipping 12/22 and 12/29 ? | 15:04 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add retry logic for transient BMC errors during system reset https://review.opendev.org/c/openstack/ironic/+/971058 | 15:04 |
| kubajj | but anyway, don't know how many people are around | 15:04 |
| TheJulia | 22nd and 29th | 15:04 |
| TheJulia | Yeah, seems like we're below or right out qorum | 15:04 |
| TheJulia | Anyone have anything to discuss and I can fire up the meeting | 15:04 |
| janders | I have two-liner update from raising multi-system-systems topic with DMTF folks, not sure if it sanctions a meeting | 15:05 |
| * TheJulia takes silence as "nope" | 15:05 | |
| * TheJulia shrugs | 15:05 | |
| janders | (and I don't think I see Jay here) | 15:06 |
| janders | if no one is particularly interested, we can discuss this later | 15:06 |
| dtantsur | I don't have anything, and I'm debugging why my vim is misbehaving | 15:06 |
| TheJulia | Yeah, he has been the real driver there. I get where he is coming from albeit... part of it is the spec itself's modeling | 15:07 |
| TheJulia | dtantsur: maybe you need to install emacs just to cause vim to behave? :) | 15:07 |
| dtantsur | as a threat? yeah, may work | 15:11 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter stable/2025.2: Fix handling of unknown metric keys in ironic parser https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/971072 | 15:45 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-prometheus-exporter stable/2025.1: Fix handling of unknown metric keys in ironic parser https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/971073 | 15:46 |
| dtantsur | Well, this was a fun one: https://github.com/ycm-core/ycmd/issues/1800 | 16:03 |
| JayF | janders: yeah, thanks, I'm out today but can talk later this week | 16:11 |
| janders | JayF sounds good. I will be back from PTO Wednesday-ish so things will get easier | 16:12 |
| alegacy | has anything changed recently with how things work in bifrost or how configdrives are handled in any way? I'll explain... | 16:24 |
| alegacy | for the past months... while testing the standalone networking changes, I would use bifrost to do some of the testing. I would create the testenv, do the install, do the enroll, manually inspect, provide, deploy, undeploy, manage, inspect, delete... rinse -> repeat. | 16:26 |
| alegacy | all this was done with inspection on vlan10, cleaning on vlan20, and tenant network on vlan30. | 16:26 |
| alegacy | until late last week I have not had a problem with that workflow.... at least not that I noticed or remember. | 16:27 |
| alegacy | now what seems to be happening is that everything works fine up to the after the deploy finishes... but then when I do an undeploy which triggers a clean that times out. | 16:28 |
| alegacy | what appears to be happening is that the configdrive from the deploy step is still kicking around an so when the node boots to clean it uses the IP address that was used for the deployed node. | 16:28 |
| alegacy | that IP address won't work on the vlan that I have set aside for cleaning. | 16:29 |
| dtantsur | Uuuugh | 16:29 |
| alegacy | is this new behaviour? have I been missing something all this time? | 16:29 |
| dtantsur | I bet it's related to https://opendev.org/openstack/ironic-python-agent-builder/commit/3f83c9ccb56200c26c21f33d656bfbdbe467ee2d (cc TheJulia) | 16:29 |
| dtantsur | Haven't we solved that issue though? I was sure we have safeguards in place | 16:30 |
| alegacy | i had added that "force_dhcp" flag... but after some digging it looks like maybe that only works for virtual media booting and not PXE like in the bifrost case. ...and of course only for inspection. | 16:32 |
| TheJulia | Hhmm, I thought we fixed that | 16:32 |
| alegacy | in my current state even subsequent inspections won't work because it still seems to rely on that config drive. | 16:32 |
| opendevreview | Merged openstack/ironic stable/2025.2: fix: agent inspection hooks failure does not clean up https://review.opendev.org/c/openstack/ironic/+/971031 | 16:33 |
| alegacy | Maybe I'm missing a fix? ... I could try rebasing ironic and bifrost to latest. I thought I did that last week though so I should be fairly close to the tip | 16:33 |
| TheJulia | that would lend creedence to the other report of issues because it should be locking it out but maybe the order is not being honored | 16:33 |
| TheJulia | alegacy: latest bifrost with a cleared out image/artifact cache (or just "rm -r -f /httpboot" and reinstall) would be ideal | 16:34 |
| alegacy | k, I'll refresh things and report back. | 16:34 |
| TheJulia | Thanks | 16:34 |
| * TheJulia goes back to administrivia | 16:34 | |
| alegacy | TheJulia: for /httpboot are you referring to /var/lib/ironic/httpboot? | 16:37 |
| TheJulia | yes, yes I am | 16:37 |
| alegacy | k | 16:37 |
| opendevreview | Merged openstack/ironic stable/2025.1: fix: agent inspection hooks failure does not clean up https://review.opendev.org/c/openstack/ironic/+/971043 | 17:34 |
| alegacy | TheJulia: removed that directory and re-ran the bifrost install. Same result. Config drive seems to used still. | 18:03 |
| TheJulia | ugggh | 18:03 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Revert "Add simple-init by default" https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/971110 | 18:06 |
| dtantsur | alegacy: do you think you can fetch the entire journal from the running ramdisk? | 18:07 |
| TheJulia | yeah, that would likely help. I've gone ahead and reverted the change | 18:08 |
| alegacy | dtantsur: you mean the console log of the VM? | 18:08 |
| TheJulia | I'll have to dig up the CI change and get it running again. I bet order or process has changed silently somewhere along the way | 18:08 |
| TheJulia | alegacy: console is at least a solid data point | 18:09 |
| dtantsur | yep | 18:09 |
| dtantsur | I wonder if we inject the SSH key by default in bifrost... | 18:09 |
| dtantsur | mm, yes, but not by default for some reason.. ipa_add_ssh_key is set to false | 18:10 |
| alegacy | k, i can collect the console file... is there a standard way of sharing that here? | 18:10 |
| dtantsur | alegacy: https://paste.opendev.org/ | 18:11 |
| alegacy | perfect... k, if I can't do it before I need to skip out to the dentist shortly I'll do it when I get back. | 18:11 |
| TheJulia | dtantsur: fwiw, I +2+A'ed the revert I generated in gerrit. | 18:13 |
| dtantsur | ack | 18:14 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add Redfish LLDP data collection support to the Redfish inspection interface. https://review.opendev.org/c/openstack/ironic/+/967841 | 19:59 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Revert "Add simple-init by default" https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/971110 | 20:22 |
| opendevreview | Verification of a change to openstack/ironic master failed: Add Redfish LLDP data collection support to the Redfish inspection interface. https://review.opendev.org/c/openstack/ironic/+/967841 | 20:51 |
| opendevreview | Julia Kreger proposed openstack/ironic master: OCI: Add an option to fallback to HTTP https://review.opendev.org/c/openstack/ironic/+/969277 | 21:14 |
| opendevreview | Julia Kreger proposed openstack/ironic master: OCI Basic Auth/Tag access https://review.opendev.org/c/openstack/ironic/+/969314 | 21:14 |
| alegacy | dtantsur: TheJulia: paste.opendev.org only supports 64K chunks... so split the file into 4 chunks. First one is https://paste.opendev.org/show/829913/ ...and the other 3 are sequentially numbered ...914...915...916. | 21:38 |
| TheJulia | Err, I think I see what is going on | 21:42 |
| TheJulia | I should be able to make progress on it tomorrow | 21:42 |
| opendevreview | Merged openstack/ironic-python-agent master: Remove unicode literal from code https://review.opendev.org/c/openstack/ironic-python-agent/+/852512 | 21:45 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix redfish inspect system product name https://review.opendev.org/c/openstack/ironic/+/971142 | 22:41 |
| opendevreview | Doug Goldstein proposed openstack/ironic master: fix: allow system owner to view and manipulate inspection rules https://review.opendev.org/c/openstack/ironic/+/971146 | 23:43 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!