| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NetconfOpenConfigSwitch driver https://review.opendev.org/c/openstack/networking-generic-switch/+/990061 | 00:22 |
|---|---|---|
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NETCONF OpenConfig driver documentation https://review.opendev.org/c/openstack/networking-generic-switch/+/990062 | 00:22 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in MTU management for switch ports https://review.opendev.org/c/openstack/networking-generic-switch/+/984832 | 03:07 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in port carrier bounce on bind https://review.opendev.org/c/openstack/networking-generic-switch/+/984833 | 03:07 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in STP edge port and BPDU guard support https://review.opendev.org/c/openstack/networking-generic-switch/+/984847 | 03:07 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in port carrier bounce on bind https://review.opendev.org/c/openstack/networking-generic-switch/+/984833 | 03:32 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add opt-in STP edge port and BPDU guard support https://review.opendev.org/c/openstack/networking-generic-switch/+/984847 | 03:33 |
| opendevreview | OpenStack Proposal Bot proposed openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/990418 | 03:53 |
| *** hroy_ is now known as hroy | 06:23 | |
| opendevreview | Merged openstack/ironic master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic/+/990418 | 08:19 |
| opendevreview | Himanshu Roy proposed openstack/ironic master: Add operator-configurable step disallow lists https://review.opendev.org/c/openstack/ironic/+/989017 | 09:01 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NetconfOpenConfigSwitch driver https://review.opendev.org/c/openstack/networking-generic-switch/+/990061 | 09:48 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NETCONF OpenConfig driver documentation https://review.opendev.org/c/openstack/networking-generic-switch/+/990062 | 09:48 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NetconfOpenConfigSwitch driver https://review.opendev.org/c/openstack/networking-generic-switch/+/990061 | 10:34 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add NETCONF OpenConfig driver documentation https://review.opendev.org/c/openstack/networking-generic-switch/+/990062 | 10:34 |
| opendevreview | Mithun Krishnan Umesan proposed openstack/ironic-python-agent master: Apply TLS configuration from Ironic lookup response https://review.opendev.org/c/openstack/ironic-python-agent/+/988150 | 10:56 |
| mnasiadka | JayF: I don’t know how much Ironic likes having tenks in it’s governance, but if that’s a burden - I think we could move that to Kolla (given that majority of reviewers and usage are from Kolla world anyway) | 11:20 |
| dtantsur | mnasiadka: I don't think any of us use tenks, so it may make sense. Not that it's a burden, just why keep it under ironic? | 12:16 |
| mnasiadka | dtantsur: if you don’t use it - then it doesn’t make sense for you to maintain it :-) | 12:59 |
| JayF | I sort of feel like that project is more of a roommate. I have no objection if it wants to move to another program in openstack | 13:01 |
| TheJulia | That has been my view as well. | 13:20 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add SONiC multicast BUM configuration validation https://review.opendev.org/c/openstack/networking-generic-switch/+/981193 | 13:51 |
| opendevreview | Himanshu Roy proposed openstack/ironic master: Add operator-configurable step disallow lists https://review.opendev.org/c/openstack/ironic/+/989017 | 13:52 |
| opendevreview | David Nwosu proposed openstack/ironic master: Add project field to node history and record deployment transitions https://review.opendev.org/c/openstack/ironic/+/983349 | 14:39 |
| mnasiadka | Ok, thanks JayF, TheJulia and dtantsur - I’ll discuss that on tomorrow’s Kolla meeting and raise a governance patch if we get to that decision | 14:39 |
| opendevreview | David Nwosu proposed openstack/ironic master: Add project field to node history https://review.opendev.org/c/openstack/ironic/+/983349 | 14:49 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/987391 | 14:59 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Align development tooling with Ironic https://review.opendev.org/c/openstack/bifrost/+/991128 | 15:12 |
| opendevreview | Clif Houck proposed openstack/ironic master: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/990724 | 15:52 |
| opendevreview | Merged openstack/ironic master: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/990724 | 18:54 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2026.1: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/991179 | 19:03 |
| JayF | thejulia: clif: ^ please review my modifications to the stable/2026.1 version of that change to flip the default | 19:03 |
| opendevreview | Ghanshyam Maan proposed openstack/ironic-tempest-plugin master: Drop python 3.10 https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/991186 | 19:19 |
| clif | JayF: LGTM, do you want me to backport the rest with this change or do you want o? | 19:28 |
| clif | to* | 19:28 |
| JayF | you can as long as it passes review, ideally we get the second +2 before you do all that work | 19:28 |
| clif | sure | 19:29 |
| JayF | Is CI broken on stable branches? I can't get lint to pass on 2025.2, for instance. It's upset about bandit versions. I thought we had a backport around that, but I can't find the patch | 19:42 |
| JayF | Ah, I think it's due to the python version it's running under being too new | 19:52 |
| opendevreview | Verification of a change to openstack/ironic-python-agent master failed: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/987391 | 19:54 |
| JayF | that's just the bugged notification on the first arm test | 19:56 |
| JayF | I'll note I'm nervous this is still gonna bail due to the depends-on existing even though it's merged, but science :) | 19:56 |
| JayF | cardoe: pre-commit is NOT respecting tox python versions. This makes it nearly impossible to use it locally on stable branches :| | 19:57 |
| JayF | > /home/jay/.cache/pre-commit/repok6kntvlx/py_env-python3/lib/python3.14/site-packages/hacking/checks/except_checks.py | 19:57 |
| JayF | errors are coming that point to 3.14 even though I set basepython to 3.12 | 19:57 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2025.2: [stable-only][ci] Fix bandit inside pre-commit https://review.opendev.org/c/openstack/ironic/+/991214 | 20:00 |
| JayF | I think ^ all older branches will need that | 20:01 |
| JayF | but I can't explain why right now | 20:01 |
| TheJulia | afaik, bandit is one of the items which can move and evolve and b104 was removed at some point afaik. I had to go hunt that fairly recently and yeah. :\ | 20:23 |
| dansmith | does anyone know if a fix for this: https://bugs.launchpad.net/ironic/+bug/2155052 is in progress? If not, I've almost got something ready to go for nova that can be lifted back to ironic | 20:36 |
| JayF | I think the answer to that is yes | 20:38 |
| JayF | but I'm not 100% sure, TheJulia do you remember? Were you or dtantsur going to work that? | 20:38 |
| TheJulia | in a meeting atm | 20:39 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/990458 | 20:41 |
| dansmith | hmm, no test for that? | 20:42 |
| dansmith | I had tests for mine, | 20:42 |
| TheJulia | test ++ | 20:42 |
| TheJulia | err, tests++ | 20:42 |
| dansmith | and I also didn't want to expose to the client why that happened, | 20:42 |
| dansmith | because if someone is working on exploiting they're going to have access to the client side of that pipeline | 20:43 |
| TheJulia | toats fair | 20:43 |
| TheJulia | ++ | 20:43 |
| dansmith | I'll comment and point at mine, even if they just copy the salient bits | 20:43 |
| JayF | That contribution was from someone outside Ironic, and I should've remembered this was in the pile of security-hardening fixes that got stealth-dropped on us | 20:43 |
| TheJulia | As the great and wise Teal'c once said, "Indeed." | 20:44 |
| TheJulia | dansmith: thanks! | 20:44 |
| dansmith | there are other problems with that implementation too.. anyway, I commented and linked | 20:49 |
| TheJulia | Much appreciated, thanks again! | 20:58 |
| TheJulia | clif: the release note on https://review.opendev.org/c/openstack/ironic/+/991179 should be "false" | 21:00 |
| JayF | ah, I screwed it up | 21:01 |
| JayF | good thing we got hte second review | 21:01 |
| JayF | I'll edit it r/n | 21:01 |
| TheJulia | k | 21:01 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/2026.1: Add an agent flag to disable installing boatloaders https://review.opendev.org/c/openstack/ironic/+/991179 | 21:01 |
| TheJulia | Anything else you need my eyes on before EOD? | 21:02 |
| TheJulia | asking since an exercise break sounds like a ++good idea | 21:02 |
| JayF | I have my dog's food pouch on about to walk him | 21:03 |
| JayF | he doesn't wanna go though but I'm going to find vitamin D either way :D | 21:03 |
| * dansmith wonders if JayF is wearing the food pouch, or the dog | 21:03 | |
| TheJulia | Excellent plan! Mine is looking at himself in the mirror and it is... disturbing. | 21:03 |
| TheJulia | dansmith: why not both?!? | 21:04 |
| dansmith | idk, I'm a cat person, dog people do weird stuff... | 21:05 |
| TheJulia | Well, my wife is not around today and twice now our void has decided that she must use her claws to climb into my lap.... so.... yeah. | 21:06 |
| opendevreview | Julia Kreger proposed openstack/networking-generic-switch master: Add ngs_ssh_allowed_algorithms for SSH cipher control https://review.opendev.org/c/openstack/networking-generic-switch/+/991225 | 21:07 |
| JayF | dansmith: imagine if you could double the uncoolness of a fanny pouch, then fill it with dog treats. That's what I had on, but the dog doesn't wanna go | 21:08 |
| dansmith | sexy. | 21:10 |
| dansmith | this rfb.py feels like a terrible thing to replicate between projects for exactly this reason, | 21:12 |
| dansmith | but it also is like just a tad under the size threshold for being worth putting in a central location :/ | 21:12 |
| opendevreview | Merged openstack/ironic-python-agent master: Add a flag to disable installing bootloaders https://review.opendev.org/c/openstack/ironic-python-agent/+/987391 | 21:35 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!