| *** mdfr8 is now known as mdfr | 00:09 | |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Ensure NIC firmware version is cached https://review.opendev.org/c/openstack/ironic/+/982038 | 02:56 |
|---|---|---|
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 11:59 |
| *** dking is now known as Guest5857 | 12:08 | |
| *** Guest5857 is now known as dking | 12:19 | |
| opendevreview | Abhishek Bongale proposed openstack/ironic-tempest-plugin master: Add conditional graphical console validation https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 | 12:26 |
| clif | gm o/ | 14:19 |
| JayF | o/ | 14:23 |
| cardoe | JayF: so for the nova/ironic slowness... do they have a goal to land the fixes in nova? | 14:27 |
| cardoe | How do folks feel about adjusting the UX of the CLI? Like https://review.opendev.org/c/openstack/python-ironicclient/+/981805 ? | 14:41 |
| JayF | cardoe: we have a PTG session to chat about it, my personal goal is to have a solid plan in place. Sean's changes, so far, are mainly vibe coded pointers to things that need to be fixed. They need review and real testing. I suspect they are only a small % of the overall improvement we can gain | 14:41 |
| zigo | Hi team! | 14:41 |
| zigo | Building Ironic 35.0.0 in Debian Experimental, I get: | 14:41 |
| zigo | https://paste.opendev.org/show/bjgJNSlqrX4cjZqUgrpi/ | 14:41 |
| zigo | Does this ring a bell to someone? | 14:42 |
| JayF | cardoe: *plan is to have a solid plan in place by the end of "H" | 14:42 |
| JayF | zigo: py hlapi or snmp version mismatch | 14:42 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Soft power off before ejecting virtual media regardless of CONF.inspector.power_off https://review.opendev.org/c/openstack/ironic/+/981942 | 14:42 |
| JayF | zigo: that code is unlikely to work even if imported, I'd suggest patching out the irmc+snmp driver | 14:42 |
| JayF | We are removing it this cycle | 14:42 |
| JayF | wait, it's this cycle!!!!!!! | 14:43 |
| * JayF gets out the code-machete | 14:43 | |
| zigo | JayF: Ok, so I can just blacklist the 2 failing tests, then? | 14:44 |
| JayF | I would ensure the service starts up :) (I think in default config it would), but yes | 14:44 |
| JayF | I think some of that keys on if a package is installed | 14:44 |
| zigo | Thanks, that's enough for me! :P | 14:45 |
| zigo | FYI, we're on pysnmp 7.1.22 in Debian ATM. | 14:45 |
| TheJulia | cardoe: I could be good with that, I guess | 14:48 |
| cardoe | I've also wanted to alias "avail" to "provide" but I'm not necessarily as convinced. | 14:49 |
| JayF | cardoe: I am +1 to adding the "correct" ordering. I am -1 to deprecating the old way | 14:50 |
| JayF | it breaks scripts for .... no real good reason afaict? | 14:50 |
| TheJulia | queue "why not both" gif ? | 14:56 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/portgroup.py` https://review.opendev.org/c/openstack/python-ironicclient/+/981661 | 14:58 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/v1/resource_fields.py` https://review.opendev.org/c/openstack/python-ironicclient/+/981662 | 14:58 |
| frickler | JayF: oh, yes, please, that's essentially the only remaining u-c pin that we have, getting rid of that would be very cool https://review.opendev.org/c/openstack/requirements/+/977551 | 15:16 |
| JayF | frickler: yeah, I know, and I've felt bad about it that's why we're killing the driver | 15:17 |
| JayF | just had to have the deprecation timer run out | 15:17 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Soft power off before ejecting virtual media regardless of CONF.inspector.power_off https://review.opendev.org/c/openstack/ironic/+/981942 | 15:45 |
| JayF | it's christmas in claude-code! Although a lot more red decorations than green ones :P | 15:52 |
| * JayF is using it to whack the pysnmp dep | 15:52 | |
| TheJulia | oh, my. | 15:55 |
| opendevreview | Merged openstack/bifrost stable/2026.1: Update .gitreview for stable/2026.1 https://review.opendev.org/c/openstack/bifrost/+/981268 | 15:56 |
| opendevreview | Merged openstack/bifrost stable/2026.1: Update TOX_CONSTRAINTS_FILE for stable/2026.1 https://review.opendev.org/c/openstack/bifrost/+/981269 | 15:56 |
| JayF | we should mention deprecations in oif live, the big one and the ironic one | 15:56 |
| * JayF puts that here so it helps him remember | 15:57 | |
| TheJulia | the general session or the ironicy one? | 16:00 |
| JayF | yes | 16:00 |
| TheJulia | ack ack | 16:00 |
| TheJulia | clif: w/r/t https://docs.google.com/presentation/d/1SDNGtEYIP45AkjMxAiyBfTjJDO4yQQcjaDMjosm17aM/edit?slide=id.g3d7b9bf4b0b_1_36#slide=id.g3d7b9bf4b0b_1_36 you might want to split in to 2-3 slides. Try and start with the basic challenge which exists so we can convey across what we're actually solving. Also, if you see slide 5, its a blank slide, I'd honestly try to keep a similar format just so there is a clean handoff and | 16:02 |
| TheJulia | we're also making the top level mental impression of "oh, there is a thing, and this person is going to speak about it. | 16:02 |
| clif | TheJulia: Sure, that works. I'll split it up and elaborate some more. | 16:03 |
| cardoe | JayF: gimme a better way to convey why the docs will be updated to the new way? | 16:05 |
| JayF | "This usage pattern is more consistent with the CLI; even though the old method is remaining for compatability" | 16:06 |
| cardoe | Okay. I didn’t intend for us to remove them. Just not use them in docs and examples | 16:07 |
| cardoe | I’ll change the message | 16:07 |
| JayF | deprecated implies it'll be removed one day; I think if you just make a bigger comment it'd do the trick | 16:07 |
| JayF | I'm glad we agree just use different language | 16:08 |
| TheJulia | jay, stared a slide on the big deck | 16:09 |
| JayF | ack | 16:09 |
| JayF | wdyt about the other ones already in place | 16:10 |
| TheJulia | JayF: who is that directed to | 16:13 |
| JayF | you, re: the big deck ironic stuff I put in | 16:13 |
| JayF | for the original oi live | 16:13 |
| TheJulia | I don't think I have that link | 16:18 |
| gtfrans2re | Hello, I've task #3 on bifrost installation and usage. Anyone facing issues with it? | 16:19 |
| gtfrans2re | Or anythhing else I can help with? | 16:20 |
| JayitaPramanik[m] | gtfrans2re: what issue you are facing? | 16:36 |
| JayitaPramanik[m] | gtfrans2re: I also started exploring the documentation, what are the issues you are facing? | 16:38 |
| TheJulia | That is the critical question to ask and information to exchange. Maintainers and regular condtributors might not ever see the kind of issue someone who is coming into a project with a "fresh" perspective, so if you can paint the picture for us to understand, its a lot easier for us to help or provide insight. | 16:42 |
| JayitaPramanik[m] | Hi Ironic mentors!... (full message at <https://matrix.org/oftc/media/v1/media/download/AfOg5mm4OQMx6TFoJXC4Wl1M8cWGFf71VtaNI6JM4rdh-lY0GpQTvXcBk8NPxreT8YwrFtaGVAKZ74lxqwNfHZpCedbgc68gAG1hdHJpeC5vcmcvek1YUWtKVENRdHN4SXBpYWVPSGNsQW5m>) | 16:57 |
| JayitaPramanik[m] | * Hi Ironic mentors!... (full message at <https://matrix.org/oftc/media/v1/media/download/AW8rzs3W17sisinyalLLbm615tUU1ziFbCwUay7PEdC2uZkjC9DLNbKrhUK21OTvr1LHkYPOfHSnRMrp5PNa0FtCedbgxM0AAG1hdHJpeC5vcmcvSXdYdUpDWFpWZXNpaHpHeGtEdVNZVERi>) | 17:03 |
| Lizzy | Hi everyone I am Lizzy, I have recently started working with Bifrost and learning more about OpenStack Ironic. Looking forward to learning from you all and contributing where I can. | 17:12 |
| Priestess[m] | Lizzy: Welcome, Lizzy | 17:20 |
| Lizzy | Thank you for the warm welcome @Priestess | 17:22 |
| cardoe | I've passed https://bugs.launchpad.net/ironic/+bug/2076265 back over to Nova because before we can change anything, they need to change. I've proposed a fix on their side but have gotten zero feedback. | 17:33 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove the iRMC hardware type and all interfaces https://review.opendev.org/c/openstack/ironic/+/982145 | 19:17 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove the SNMP hardware type and power interface https://review.opendev.org/c/openstack/ironic/+/982146 | 19:17 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove pysnmp and related dependencies https://review.opendev.org/c/openstack/ironic/+/982147 | 19:17 |
| JayF | don't review those | 19:17 |
| TheJulia | toooooo late ;) | 19:20 |
| TheJulia | One thing stuck out | 19:21 |
| JayF | I know unit tests are failing on the first commit | 19:21 |
| JayF | but I can't figure out why so I wanted to look in gerrit lol | 19:21 |
| JayF | it's like I broke driver loading somehow | 19:21 |
| TheJulia | schweet | 19:46 |
| clif | in launchpad is it possible to add me as an ironic member? I guess this goes for any other related project as well? or do I have to be core in order to have that level of access? | 19:58 |
| TheJulia | it should totally be doable | 20:02 |
| TheJulia | uhhhh | 20:02 |
| TheJulia | https://launchpad.net/~ironic-bugs right ? | 20:03 |
| clif | I suppose? | 20:03 |
| clif | I joined the team | 20:04 |
| TheJulia | okay :) | 20:06 |
| TheJulia | if that doesn't give you what you need, just let me know | 20:06 |
| clif | rgr | 20:06 |
| cardoe | clif: you know the initiation rule right? You need to personally fix 10 bugs in 7 days. | 20:07 |
| TheJulia | cardoe: wow, your evil. | 20:07 |
| TheJulia | ;) | 20:07 |
| clif | ez, just merge 10 bugs and then fix them right after | 20:07 |
| TheJulia | lol | 20:08 |
| cardoe | It's been one of those days. | 20:08 |
| cardoe | clif: this is the way | 20:08 |
| TheJulia | cardoe: it is what it is some days | 20:09 |
| JayF | you know, clif brings up a reasonable question | 20:12 |
| clif | cardoe: are you actively working this issue? https://bugs.launchpad.net/ironic/+bug/2146355 would you assign yourself if so | 20:12 |
| JayF | given we no longer have a dashboard that properly identifies review quantities | 20:12 |
| JayF | how do we manage the core group? | 20:12 |
| JayF | lfx insights is not useful for managing review quantities and it's all we've got | 20:12 |
| cardoe | clif: done. | 20:13 |
| cardoe | Are folks okay with my proposed fix? | 20:13 |
| cardoe | And can I term this as a backportable fix so that I can have it in 2026.1? | 20:13 |
| JayF | so right now I'm asking clif to review enough to be a -reviewer and ... I can't even judge if he's on the right track with either a manager *or* DPL hat on | 20:13 |
| clif | It sounds reasonable to me cardoe, but I don't have a lot of context on the redfish driver | 20:13 |
| cardoe | clif: well if the BMC clock is too far off SSL stuff doesn't work. | 20:14 |
| TheJulia | oh wow, I'm a jerk according to: https://www.stackalytics.io/?release=all&module=ironic-group | 20:14 |
| clif | cardoe: yea that makes sense | 20:14 |
| clif | I'm kind of surprised there's no way to tell it to contact a time server | 20:15 |
| TheJulia | JayF: and that dashboard has data from today. | 20:15 |
| JayF | TheJulia: can I trust that data? It's unofficial and it's unclear to me how to get affiliation updates merged there. | 20:15 |
| JayF | TheJulia: last I checked, that was crediting some nontrivial % of GR work to others | 20:16 |
| JayF | Yeah, CID is not tracked as GR in that dash. | 20:16 |
| TheJulia | is there an entry in the stackalytics affiliation config for CID? | 20:17 |
| JayF | Where does that live now? I've updated it in 3 different places over my time here, and I directed him to update it as well | 20:17 |
| TheJulia | great questions | 20:18 |
| JayF | I'm going to email the list | 20:18 |
| JayF | honestly still leftover-annoyed from just everyone being broken on short notice | 20:18 |
| JayF | and nobody giving a damn that it's still broken months later | 20:18 |
| cid | erm,, nice we have a new dashboard. | 20:22 |
| cid | And I'm not on it :facepalm: | 20:23 |
| TheJulia | https://opendev.org/x/stackalytics/src/branch/master/etc/default_data.json | 20:23 |
| cid | JayF, this is what we were configuring for the other time? | 20:23 |
| cid | Might I have missed something? | 20:23 |
| JayF | cid: this is the unofficial dashboard everyone is using now because the official one is more broken than the unofficial one | 20:23 |
| cid | :D | 20:24 |
| JayF | cid: if you can make sure that file linked by julia is up to date for you, it'd be nice. I think you already proposed a change once, it might have not been merged? | 20:24 |
| cid | Let me check | 20:24 |
| cardoe | clif: you can tell the BMC to contact a time server... but you gotta be able to connect to the BMC to tell it to contact the timeserver... the only command Ironic ignores the SSL errors on is to set the BMC clock. | 20:56 |
| cardoe | I want to have a runbook for our operators to clear BMC jobs and set the clock back. And I wanna have that runbook ultimately be part of the automated cleaning via runbook stuff. | 20:58 |
| cardoe | Right now clear BMC jobs and set clock is "stuff done outside of Ironic" | 20:58 |
| cardoe | At least the way I've envisioned the runbooks API (and others can correct me if I'm wrong) is that it's a set of operations that are packaged up that's safe-ish for a DC tech todo or potentially a lessee to do on the box. | 20:59 |
| TheJulia | Clearly, presentation AI art is needed to remind ourselves that "bmcs are evil" | 21:00 |
| cardoe | So just thinking of all the out-of-ironic operations we do and seeing how we can implement them as runbooks. | 21:00 |
| TheJulia | +++++ | 21:00 |
| TheJulia | cardoe: Hi, is there an online whiskey store gift card I can get you for that? | 21:01 |
| cardoe | heh | 21:01 |
| TheJulia | Seriously, I mentioned that people "do physical fleet management activities with ironic" on a call today and a bunch of people just looked at me like I had grown an extra set of eyes and more arms | 21:01 |
| TheJulia | (in other words, more examples == better) | 21:04 |
| cardoe | We just got several cabs of hardware and my instructions to the person pulling them in was to write down every time you have to do something outside of the baremetal CLI command. And every time something seems awkward. | 21:04 |
| cardoe | She had some pretty interesting feedback as a hot take. | 21:05 |
| cardoe | The "baremetal node add trait" being different than other commands was one. | 21:05 |
| cardoe | "baremetal node provide" grammatically doesn't match the other verbs was another. | 21:06 |
| cid | re'stacklytics... took a bit of time trying to understand the ordering, figured it's by launchpad id. | 21:06 |
| cid | I have patch up now at https://review.opendev.org/c/x/stackalytics/+/982167 | 21:07 |
| TheJulia | cardoe: so her "not wrong" column I take it now has at least two checks ? | 21:07 |
| cardoe | Yeah I mean she's not wrong. | 21:07 |
| cardoe | All the other commands are "baremetal node <thing> add" except for trait. | 21:08 |
| cardoe | And stuff like "baremetal node manage -> manageable" or "baremetal node rescue -> rescue" or "baremetal node enroll -> enroll" but then "baremetal node provide -> available" | 21:09 |
| JayF | cardoe: the idea there is verb->noun | 21:39 |
| JayF | cardoe: the act of making something available is to provide is | 21:40 |
| JayF | *it | 21:40 |
| JayF | cardoe: I'm not saying it's good; I'm saying it follows a pattern. rescue/enroll are cleaner because they are both noun/verb depending on usage | 21:40 |
| cardoe | yeah makes sense. | 21:41 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Update auto-tls cert signing to SHA-512 https://review.opendev.org/c/openstack/ironic-python-agent/+/982173 | 22:04 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP: Enforce TLS 1.2 minimum across all connections https://review.opendev.org/c/openstack/ironic-python-agent/+/982174 | 22:04 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: WIP: Upgrade TLS certificates to P-384 curve with configurable support https://review.opendev.org/c/openstack/ironic-python-agent/+/982175 | 22:04 |
| * TheJulia gets out the CI popcorn | 22:04 | |
| JayF | are we really spending time on this post-quantum stuff? | 22:04 |
| TheJulia | eh, we've got some bad defaults in the agent, so makes sense to do some changes and provide some knobs | 22:05 |
| JayF | I only bristle because of the push at last PTG by a security person at redhat who had no interest in helping with active-on-fire security stuff | 22:05 |
| JayF | it feels like rewarding bad behavior to go along with that push (not that I would ever -1 or -2 out of spite) | 22:06 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove the SNMP hardware type and power interface https://review.opendev.org/c/openstack/ironic/+/982146 | 22:08 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove pysnmp and related dependencies https://review.opendev.org/c/openstack/ironic/+/982147 | 22:08 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove the iRMC hardware type and all interfaces https://review.opendev.org/c/openstack/ironic/+/982176 | 22:08 |
| TheJulia | I'll put it this way, we have an underlying mandate to improve the state on my team, so on the ironic side, we likely want to add a tls enforcement threshold knob as well. The person who your speaking of is a product manager, not an engineer and also not the person to make the $other_security_stuff business cases, albeit they should at least try and I've had some recent discussions with them trying to guide them in the | 22:09 |
| TheJulia | right direction. | 22:09 |
| TheJulia | Hence, also, why two of those are WIPs for now, I want to see how horrible it will be ;) | 22:09 |
| TheJulia | (and, have this discussion) | 22:10 |
| JayF | Project manager or not, it's difficult to not paint a picture with all the data even if that individual isn't responsible for it all :) | 22:11 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove pysnmp and related dependencies https://review.opendev.org/c/openstack/ironic/+/982147 | 22:11 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Remove pysnmp and related dependencies https://review.opendev.org/c/openstack/ironic/+/982147 | 22:12 |
| JayF | this stack should be happier now | 22:13 |
| JayF | it would've taken me a damn day to track that unit test failure w/o claude | 22:13 |
| JayF | tl;dr we were updating (in a test) `redfish.management` in a place that `redfish.management` existing relied on the iRMC driver to have already been imported | 22:13 |
| JayF | I'll h/t it once CI is proven happy | 22:13 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!