opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent-builder master: Create Podman DIB Element for container-based cleaning https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/943348 | 00:29 |
---|---|---|
freemanboss[m] | <Annosha> "Hello everyone, I am Annosha..." <- You're welcome. yeah that's how it's, you can't view previous messages before you join the channel. | 02:52 |
cardoe | So for the outreachy folks an idea that JayF had was around type annotations and checking. I’d be glad to help. | 03:19 |
JayF | They already have a project this cycle I believe. My suggestion was more future looking | 03:33 |
opendevreview | Kaifeng Wang proposed openstack/ironic master: Allow a node in service fail state to be unprovisioned https://review.opendev.org/c/openstack/ironic/+/944966 | 07:06 |
dtantsur | JayF, cardoe, chances are rather high that we'll have outreachy funding the next round too, so mentors are welcome ;) | 07:29 |
rpittau | good morning ironic! o/ | 07:40 |
rpittau | ManideepMaddileti[m], freemanboss[m]: welcome to the ironic community! After completing the essay please send it or share it to masghar and myself by email, you can find our addresses in the outreachy project page | 07:47 |
ManideepMaddileti[m] | rpittau: Sure! | 07:48 |
opendevreview | Merged openstack/ironic-ui stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-ui/+/944909 | 08:35 |
opendevreview | Merged openstack/ironic-ui stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-ui/+/944910 | 08:41 |
opendevreview | Merged openstack/ironic-ui master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-ui/+/944911 | 08:41 |
opendevreview | Merged openstack/networking-generic-switch stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/944894 | 08:46 |
opendevreview | Merged openstack/ironic-python-agent stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/944897 | 08:47 |
opendevreview | Merged openstack/ironic-python-agent stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/944898 | 08:47 |
opendevreview | Merged openstack/ironic-python-agent master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/944899 | 08:47 |
opendevreview | Merged openstack/networking-generic-switch stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/944895 | 08:48 |
opendevreview | Merged openstack/networking-generic-switch master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/944896 | 08:48 |
opendevreview | Merged openstack/networking-baremetal stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/networking-baremetal/+/944912 | 08:48 |
opendevreview | Merged openstack/networking-baremetal stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/networking-baremetal/+/944913 | 08:48 |
opendevreview | Merged openstack/networking-baremetal master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/networking-baremetal/+/944914 | 08:48 |
opendevreview | Merged openstack/ironic-python-agent-builder stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/944906 | 08:51 |
opendevreview | Merged openstack/ironic-python-agent-builder stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/944907 | 08:51 |
opendevreview | Merged openstack/ironic-python-agent-builder master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/944908 | 08:51 |
opendevreview | Merged openstack/ironic-prometheus-exporter stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/944903 | 09:11 |
opendevreview | Merged openstack/ironic-prometheus-exporter stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/944904 | 09:11 |
opendevreview | Merged openstack/ironic-prometheus-exporter master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/944905 | 09:11 |
opendevreview | Merged openstack/virtualpdu master: Remove translation sections from setup.cfg https://review.opendev.org/c/openstack/virtualpdu/+/906219 | 09:13 |
opendevreview | Riccardo Pittau proposed openstack/ironic master: Update release mappings for epoxy https://review.opendev.org/c/openstack/ironic/+/944973 | 09:26 |
rpittau | TheJulia, dtantsur, JayF when you have a moment please check ^ then we can release | 09:26 |
freemanboss[m] | <rpittau> "Manideep Maddileti, freemanboss..." <- Alright rpittau: I'll do that once mine is ready | 09:39 |
opendevreview | Merged openstack/bifrost stable/2025.1: Update .gitreview for stable/2025.1 https://review.opendev.org/c/openstack/bifrost/+/944900 | 09:55 |
opendevreview | Merged openstack/bifrost stable/2025.1: Update TOX_CONSTRAINTS_FILE for stable/2025.1 https://review.opendev.org/c/openstack/bifrost/+/944901 | 09:55 |
opendevreview | Merged openstack/bifrost master: Update master for stable/2025.1 https://review.opendev.org/c/openstack/bifrost/+/944902 | 09:55 |
opendevreview | cid proposed openstack/ironic master: A new 'description' field to the port object https://review.opendev.org/c/openstack/ironic/+/944769 | 10:47 |
opendevreview | cid proposed openstack/python-ironicclient master: A new 'description' field to the port object https://review.opendev.org/c/openstack/python-ironicclient/+/944770 | 10:47 |
opendevreview | cid proposed openstack/ironic master: Allow special characters in patch field keys https://review.opendev.org/c/openstack/ironic/+/933743 | 10:50 |
opendevreview | cid proposed openstack/ironic master: A new 'description' field to the port object https://review.opendev.org/c/openstack/ironic/+/944769 | 10:58 |
opendevreview | Kaifeng Wang proposed openstack/ironic master: Allow a node in service fail state to be unprovisioned https://review.opendev.org/c/openstack/ironic/+/944966 | 12:06 |
dtantsur | janders: this ^^ is a good point | 12:48 |
TheJulia | Oh yeah, that does sort of make sense | 13:55 |
TheJulia | rpittau: It can be okay to cut the release as long as the very next release has such a patch even if backported | 14:14 |
TheJulia | fwiw | 14:14 |
rpittau | TheJulia: sounds good to me, I just wanted to avoid backporting or worse forgetting as it happened before :D | 14:16 |
rpittau | we still have time, at least today | 14:16 |
rpittau | I will release the kraken before going offline later, then and backport that if not merged yet | 14:16 |
TheJulia | The recheck was already triggered for it, so... its a matter of time at the moment | 14:17 |
opendevreview | Kaifeng Wang proposed openstack/ironic master: Allow a node in service fail state to be unprovisioned https://review.opendev.org/c/openstack/ironic/+/944966 | 14:56 |
JayF | https://review.opendev.org/c/openstack/ironic/+/944769 do we want this approved before release (port.description) | 16:01 |
TheJulia | Has anyone seen ilos fail at https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ipmitool.py#L1219-L1220 when using ipmi ? | 16:03 |
JayF | nope | 16:06 |
TheJulia | I'm basically telling my customer we're at the end of commercial reasonable support if the bmc does something funky erroring on that call given it is ipmi and they really shouldn't be using it when redfish is a thing | 16:07 |
TheJulia | (and they have to explicitly turn IPMI on those machines when the vendor doesn't want you using it) | 16:08 |
rpittau | JayF: I think we'll wait for https://review.opendev.org/c/openstack/ironic/+/944966 and https://review.opendev.org/c/openstack/ironic/+/944973 to merge and that's all for epoxy | 16:38 |
JayF | that's what I thought | 16:49 |
opendevreview | Julia Kreger proposed openstack/ironic master: network testing: hooking in an external network simulator https://review.opendev.org/c/openstack/ironic/+/942298 | 16:52 |
opendevreview | Julia Kreger proposed openstack/ironic master: Add network simulator support for force10 OS 10 https://review.opendev.org/c/openstack/ironic/+/943345 | 16:52 |
opendevreview | Julia Kreger proposed openstack/ironic master: docs: detail network switch simulator support https://review.opendev.org/c/openstack/ironic/+/944139 | 16:52 |
TheJulia | vsaienko_: I added you as a reviewer to https://review.opendev.org/c/openstack/ironic/+/942298 specifically because of the recent tap change. I added a comment on it as well so you have background. | 16:58 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: network simulator support for Cisco Nexus 9k https://review.opendev.org/c/openstack/ironic/+/944873 | 17:04 |
opendevreview | Merged openstack/ironic master: Update release mappings for epoxy https://review.opendev.org/c/openstack/ironic/+/944973 | 17:07 |
TheJulia | rpittau: ^ | 17:09 |
TheJulia | JayF: w/r/t port.description, I think it can wait, can't be used right now without client updates which can't ship at this point | 17:11 |
JayF | still waiting on https://review.opendev.org/c/openstack/ironic/+/944966 for the release | 17:11 |
JayF | TheJulia: yeah, rpittau's answer matched my aassumption to wait :) | 17:11 |
TheJulia | k | 17:12 |
TheJulia | worst comes to worst, I'd be okay with backporting that since realistically it could be viewed as a bugfix as well | 17:12 |
JayF | I don't have a need for it in a specific release; I just naturally was the second +2 on it | 17:13 |
JayF | and figured we'd not want to approve something like that minutes before release :) | 17:13 |
TheJulia | its a semi-logical progression and I could see someone trying it | 17:13 |
JayF | I think making the port object more fully fleshed out is going to end up a focus | 17:14 |
JayF | with as much network stuff in the pipeline to architect as we have :D | 17:14 |
TheJulia | I concur. I had a discussion recently at acutally provisioning switch OSes with ironic as nodes, so I sort of wonder if we could end up in some crazy weird data model which actually makes sense if ironic owns both sides of the cable | 17:15 |
JayF | I'm less interested in making Ironic do more crazy network tricks than I am wiring through APIs to make the stuff we have more flexible | 17:15 |
JayF | but there's room at the party for everyone :D | 17:15 |
TheJulia | Yeah, there was an admission that "basically, its like the old snmp hardware type but where a specific pxe interface is needed which rips the PXE config out completely because it doesn't need to PXE boot the switch again | 17:16 |
TheJulia | when they explained it I basically said "that is actually super reasonable" | 17:16 |
JayF | time to use servicing to update cisco iOS versions :) | 17:17 |
* TheJulia twitches | 17:17 | |
JayF | I'm going to focus today on stable/UM branch stuff | 17:17 |
JayF | we have a lot of patches that are failing due to CI issues | 17:17 |
TheJulia | :( | 17:18 |
TheJulia | hmm, there are some weird failures in the logs | 17:22 |
TheJulia | also random post_failure | 17:22 |
JayF | yeah I think recheck will fix 944966 | 17:22 |
JayF | the stable stuff is all too old to have results | 17:23 |
JayF | but unless there's objection: for any UM branches, I'm going to chop down integration test jobs aggressively | 17:23 |
TheJulia | I think it makes sense | 17:29 |
TheJulia | This is a thing we need to do on a regular basis | 17:30 |
freemanboss[m] | rpittau: masghar: please is it advisable for me to test out deploying ironic using devstack before writing my 1/2 pages writeup? | 17:54 |
freemanboss[m] | Thank you | 17:54 |
opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: DNM SCIENCE: Do we need greendns patching? https://review.opendev.org/c/openstack/ironic-python-agent/+/945043 | 18:23 |
opendevreview | Merged openstack/ironic stable/2024.2: Make floppy images more floppy https://review.opendev.org/c/openstack/ironic/+/943396 | 18:36 |
opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: DNM SCIENCE: Do we need greendns patching? https://review.opendev.org/c/openstack/ironic-python-agent/+/945043 | 18:57 |
opendevreview | Jay Faulkner proposed openstack/ironic stable/2024.1: Make floppy images more floppy https://review.opendev.org/c/openstack/ironic/+/945046 | 18:57 |
JayF | freemanboss[m]: if you have the resources, a working devstack can be helpful to understand how Ironic works.. We have docs in Ironic about how to configure one; but you need a virtual machine (DO NOT DO THIS LOCALLY) with at least 8gb of ram. | 18:58 |
dtantsur | ... and quite a bit of time and brain capacity | 18:59 |
JayF | Honestly I think my team has it mostly down to a science now | 18:59 |
JayF | we've squeezed most of the unreliability out of the documented configs | 19:00 |
dtantsur | \o/ | 19:00 |
* dtantsur will stick to bifrost nonetheless | 19:00 | |
JayF | and we finally tracked down the reason the devstacks were dying inexplicably for us ... and it involves AWS security groups being too open + not changing *_PASSWORD in local.conf :) | 19:00 |
dtantsur | lol | 19:00 |
JayF | cardoe: Were you going to continue to pursue making OEM endpoint stuff accessible in Ironic? | 19:10 |
JayF | cardoe: I'm not sure I'll have many resources to help implement, but we have general interest | 19:10 |
* JayF just noticed it's absence from PTG topics | 19:15 | |
opendevreview | Merged openstack/python-ironic-inspector-client unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/936691 | 19:23 |
freemanboss[m] | <JayF> "Freeman Boss: if you have the..." <- Alright thank you. I'll set up VM soon then. I was planning to do it locally earlier. | 19:24 |
freemanboss[m] | But I've seen the docs in the ironic documentation on how to go about it | 19:24 |
JayF | other optino is the ironic local dev env stuff | 19:25 |
JayF | which stands up basically an empty shell of an ironic-api to test clients against | 19:25 |
freemanboss[m] | JayF: Oh I'll have to look for this | 19:25 |
JayF | https://docs.openstack.org/ironic/latest/contributor/local-dev-guide.html use the tox instructions | 19:26 |
JayF | this should run on any linux machine; unsure about mac | 19:26 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: network simulator support for Cisco Nexus 9k https://review.opendev.org/c/openstack/ironic/+/944873 | 19:27 |
opendevreview | Merged openstack/python-ironicclient unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/python-ironicclient/+/936693 | 19:30 |
JayF | My fiber just went dark. I'll be on my phone only for the forseeable future :| | 20:24 |
freemanboss[m] | <JayF> "https://docs.openstack.org/..." <- Alright thank you very much | 20:34 |
opendevreview | Merged openstack/ironic-python-agent unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/936679 | 20:39 |
TheJulia | yeouch :( | 20:41 |
JayF | it's always sad when you check, no outage, call in, no outage, and they book a tech | 20:42 |
JayF | that means you're going to be out for a while | 20:42 |
TheJulia | a localized fiber cut which yeah... means they need to roll a human to you locally | 20:42 |
JayF | I mainly am curious how "local" a cut has to be. There's construction down the road from me that's pretty invasive but it's on the main road, not here | 20:45 |
JayF | I assume if the **ont has no optical signal** that's probably a failure closer to me | 20:45 |
cardoe | JayF: yeah I wanted to work on the OEM bits..... just time | 20:48 |
JayF | I was thinking of ways to speed up CI and reduce failures | 20:50 |
JayF | I wonder if in the actual IPA -src jobs if it'd be more efficient to like, have a published "empty" ramdisk then jam IPA inside in the devstack build | 20:51 |
JayF | especially for tinyipa that'd be a boon | 20:51 |
opendevreview | Merged openstack/ironic master: Allow a node in service fail state to be unprovisioned https://review.opendev.org/c/openstack/ironic/+/944966 | 21:03 |
opendevreview | Merged openstack/networking-baremetal unmaintained/2023.1: Update .gitreview for unmaintained/2023.1 https://review.opendev.org/c/openstack/networking-baremetal/+/936687 | 21:17 |
opendevreview | Julia Kreger proposed openstack/ironic master: network simulator support for Cisco Nexus 9k https://review.opendev.org/c/openstack/ironic/+/944873 | 21:21 |
freemanboss[m] | rpittau: masghar: I just submitted my write up to your mail. I'll be expecting review and feedback. | 22:09 |
freemanboss[m] | Thank you | 22:09 |
freemanboss[m] | Also I'll be expecting feedback on the way forward too. | 22:10 |
freemanboss[m] | JayF: please are you our mentor too sir? | 22:10 |
freemanboss[m] | JayF: cardoe: please are you our mentors too? | 22:10 |
JayF | I'm a member of the community generally, been working on ironic for a little over 10 years -- I'm not an outreachy mentor but I've mentored MLH fellows multiple times so I have the docs at hand | 22:10 |
JayF | just a random person helping where possible | 22:11 |
masghar | freemanboss, ack, will take a look! | 22:13 |
freemanboss[m] | masghar: Alright masghar: thank you | 22:15 |
freemanboss[m] | <JayF> "I'm a member of the community..." <- Wow. I understand. | 22:16 |
freemanboss[m] | Thank you very much | 22:16 |
opendevreview | Satoshi Shirosaka proposed openstack/ironic-python-agent master: WIP Ironic Config Intergration for IPA https://review.opendev.org/c/openstack/ironic-python-agent/+/944951 | 23:02 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!