| opendevreview | Jacob Anders proposed openstack/ironic master: fix redfish BIOS settings update to not require IPA https://review.opendev.org/c/openstack/ironic/+/977569 | 04:36 |
|---|---|---|
| opendevreview | Jacob Anders proposed openstack/ironic master: cleaning: auto-detect ramdisk need from step metadata https://review.opendev.org/c/openstack/ironic/+/978844 | 05:24 |
| opendevreview | Merged openstack/python-ironicclient master: Add type annotations to `ironicclient/common/base.py` https://review.opendev.org/c/openstack/python-ironicclient/+/978335 | 06:27 |
| rpittau | good morning ironic! o/ | 07:33 |
| FreemanBoss[m] | Good morning ironic | 09:49 |
| opendevreview | Merged openstack/bifrost master: Fix the dnsmasq.leases location on Debian systems https://review.opendev.org/c/openstack/bifrost/+/958873 | 11:04 |
| rpittau | TheJulia: when you have a moment pleas rebase https://review.opendev.org/c/openstack/bifrost/+/968417 would love to get that into bifrost release :) | 11:51 |
| alegacy_ | Will NGS get released to a new version at the end of this cycle along with Ironic? | 13:49 |
| rpittau | alegacy_: ngs has been released already https://review.opendev.org/c/openstack/releases/+/978471 | 13:52 |
| alegacy_ | rpittau: great, thank you! | 13:54 |
| * TheJulia takes a big sip of coffee | 14:12 | |
| opendevreview | Julia Kreger proposed openstack/bifrost master: Upload to and use artifact from OCI registry https://review.opendev.org/c/openstack/bifrost/+/968417 | 14:14 |
| cardoe | So how do we propose updates to https://ironicbaremetal.org/ ? | 14:41 |
| TheJulia | github! https://github.com/openstackweb/ironic-website | 15:09 |
| TheJulia | It is all netlify+SaSS. That being said, claude knows how to take text and make it a blog post | 15:09 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Add project badge https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/972354 | 15:09 |
| TheJulia | And it knows all the formatting pitfalls which bring madness | 15:09 |
| * TheJulia stresses... the bring madness | 15:09 | |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/exc.py` https://review.opendev.org/c/openstack/python-ironicclient/+/978336 | 17:49 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/common/utils.py` https://review.opendev.org/c/openstack/python-ironicclient/+/978337 | 17:49 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/common/http.py` https://review.opendev.org/c/openstack/python-ironicclient/+/978917 | 17:49 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a software upgrade | 18:11 | |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/exc.py` https://review.opendev.org/c/openstack/python-ironicclient/+/978336 | 18:23 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/common/utils.py` https://review.opendev.org/c/openstack/python-ironicclient/+/978337 | 18:23 |
| opendevreview | Karan Anand proposed openstack/python-ironicclient master: Add type annotations to `ironicclient/common/http.py` https://review.opendev.org/c/openstack/python-ironicclient/+/978917 | 18:23 |
| opendevreview | cid proposed openstack/ironic master: Abort now always aborts https://review.opendev.org/c/openstack/ironic/+/978953 | 18:27 |
| opendevreview | cid proposed openstack/ironic master: Abort now always aborts https://review.opendev.org/c/openstack/ironic/+/978953 | 18:31 |
| opendevreview | cid proposed openstack/ironic master: Abort now always aborts https://review.opendev.org/c/openstack/ironic/+/978953 | 18:32 |
| JayF | Has anyone created the etherpad for the H ptg? It's not on ptg.opendev.org or in the whiteboard | 19:24 |
| JayF | https://etherpad.opendev.org/p/ironic-ptg-2026.2 | 19:24 |
| JayF | I put it on the whiteboard, will upadte the ptg site when I get outta meetings | 19:30 |
| shermanm | I wanted to ask an opinion/approach question w.r.t. adding non-netmiko interfaces to ngs. We've got an internal branch of the cumulus_nvue http interface working now (to be updated on https://review.opendev.org/c/openstack/networking-generic-switch/+/946630), but it's also similar to https://review.opendev.org/c/openstack/networking-generic-switch/+/968484 | 20:27 |
| shermanm | so... where should we be putting things, and should we have a base class for things like that? or just ignore the problem until things become more reviewable? | 20:28 |
| JayF | I know my name is attached to the group review on that, but I'm going to let people better versed on NGS answer you :) | 20:29 |
| TheJulia | I was just glancing at the upgrade status check, but thinking maybe we might need to check some other drivers or "known removed drivers" in case someone is doing a giant version jump https://github.com/openstack/ironic/blob/master/ironic/command/status.py | 20:32 |
| TheJulia | shermanm: I think there is likely agreement on proceeding with logical non-netmiko interfaces, just need to figure them out and overall comfort level for landing them | 20:33 |
| TheJulia | in large part, because they are so much *more* code than "netmiko is a shim" | 20:33 |
| TheJulia | and "we know our commands!" | 20:33 |
| JayF | clif and I are going to push to get milestone 2 of TBN in this cycle, I'm trying to get data model stuff posted today, but quick reviews are greatly appreciated | 20:35 |
| JayF | it's within reach but we'll need quick turnaround on reviews | 20:35 |
| TheJulia | k | 20:36 |
| TheJulia | shermanm: a good example of that is in ndfc, to make it consistent, it looks like we have to poll state out first then update it | 20:42 |
| TheJulia | when doing the action on the switch is an entirely different flow that doesn't exist quite the same way | 20:42 |
| TheJulia | since you don't update a port, you update a network with the additional ports | 20:46 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add dynamic portgroup fields (API 1.111) https://review.opendev.org/c/openstack/ironic/+/978964 | 21:00 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Add dynamic portgroup fields (API 1.111) https://review.opendev.org/c/openstack/ironic/+/978964 | 21:05 |
| JayF | need to get CI feedback on it, but it passes my code review now \o/ | 21:06 |
| cardoe | TheJulia: you again have an overall view of how you expect your fabric to behave and you push that out. So there’s more to the sync that needs to happen. Refreshing to edit gets expensive so you need a cache. | 22:19 |
| TheJulia | Yeah, but that needs to be really heavily balanced | 22:34 |
| TheJulia | I haven’t thought of all the edges on that | 22:34 |
| TheJulia | Feels like a triangular blade | 22:34 |
| TheJulia | Hmm, seems like we wouldn't really be able to support vlan like we do natively with apstra :( | 22:53 |
| TheJulia | basically, crossing a switch is expected to occur on vxlan. Maybe it makes sense to do, dunno | 22:59 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!