| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove remaining referemce to python-dracclient https://review.opendev.org/c/openstack/ironic/+/998257 | 03:29 |
|---|---|---|
| opendevreview | Merged openstack/networking-baremetal stable/2026.1: router_ha_binding: Skip networks without external ports https://review.opendev.org/c/openstack/networking-baremetal/+/998076 | 09:05 |
| opendevreview | Merged openstack/networking-baremetal master: Remove Python 3.10 support https://review.opendev.org/c/openstack/networking-baremetal/+/998179 | 09:12 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Refactor: extract OpenConfigModelMixin https://review.opendev.org/c/openstack/networking-generic-switch/+/992884 | 10:36 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add to_restconf_dict() methods to OpenConfig model https://review.opendev.org/c/openstack/networking-generic-switch/+/992885 | 10:36 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: restconf: add path helpers and resource_path util https://review.opendev.org/c/openstack/networking-generic-switch/+/992886 | 10:36 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add RestconfSwitch base class for HTTP transport https://review.opendev.org/c/openstack/networking-generic-switch/+/992887 | 10:36 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add RestconfOpenConfigSwitch driver https://review.opendev.org/c/openstack/networking-generic-switch/+/992888 | 10:36 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Add RESTCONF documentation https://review.opendev.org/c/openstack/networking-generic-switch/+/992889 | 10:36 |
| opendevreview | Harald Jensås proposed openstack/networking-generic-switch master: Rename netconf_models to yang_models https://review.opendev.org/c/openstack/networking-generic-switch/+/993090 | 10:36 |
| dtantsur | TheJulia: I assume we'll need to wire https://review.opendev.org/c/openstack/networking-generic-switch/+/984832 in standalone networking too? | 12:01 |
| dtantsur | or is it too exotic to bother? | 12:01 |
| opendevreview | Verification of a change to openstack/ironic master failed: Remove remaining referemce to python-dracclient https://review.opendev.org/c/openstack/ironic/+/998257 | 12:54 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Switch Metal3 functional jobs to 16GiB nodeset https://review.opendev.org/c/openstack/ironic/+/998299 | 12:56 |
| alegacy | dtantsur: i think i've put enough code in place to handle that mtu stuff automatically. | 13:02 |
| dtantsur | ah, cool! | 13:02 |
| alegacy | it was first on my follow-up todo list but TheJulia beat me to it. | 13:03 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Drop nonexistent irrelevant-files; update the list in metal3-jobs https://review.opendev.org/c/openstack/ironic/+/998301 | 13:14 |
| TheJulia | heh, I should have a queued up response for that, but I'm still waking up | 13:56 |
| dtantsur | TheJulia: good morning! If you have any cycles this week, https://review.opendev.org/c/openstack/ironic/+/997998 could use your input | 14:04 |
| dtantsur | (it's not a lot of text, just opinionated text) | 14:04 |
| TheJulia | Queue joke about free cycles | 14:11 |
| TheJulia | anyhow, you nerd sniped me | 14:11 |
| TheJulia | A few comments to consider | 14:11 |
| dtantsur | \o/ success! | 14:11 |
| TheJulia | Speaking of reviews, for your consideration: https://review.opendev.org/c/openstack/ironic-specs/+/997653 | 14:18 |
| TheJulia | alegacy: you may want to review https://review.opendev.org/c/openstack/ironic-specs/+/997653 as well | 14:18 |
| opendevreview | Doug Szumski proposed openstack/ironic-python-agent-builder stable/2025.1: Fixed rescue ConditionPathExists flag https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998318 | 14:59 |
| TheJulia | dtantsur: replied to your comemnts. I guess a follow-on spec is a g-rpc based config model as well so we have an alternate path including Sonic and NVUe | 15:12 |
| dtantsur | Simple patch, should reduce the pressure on our jobs a bit: https://review.opendev.org/c/openstack/ironic/+/998299 | 15:28 |
| opendevreview | Doug Szumski proposed openstack/ironic-python-agent-builder stable/2026.1: Fix IPA service not starting on CentOS 10 (systemd 257) https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998331 | 16:10 |
| opendevreview | Doug Szumski proposed openstack/ironic-python-agent-builder stable/2025.2: Fix IPA service not starting on CentOS 10 (systemd 257) https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998332 | 16:10 |
| opendevreview | Doug Szumski proposed openstack/ironic-python-agent-builder stable/2025.1: Fix IPA service not starting on CentOS 10 (systemd 257) https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/998333 | 16:10 |
| opendevreview | Merged openstack/ironic master: Switch Metal3 functional jobs to 16GiB nodeset https://review.opendev.org/c/openstack/ironic/+/998299 | 16:42 |
| TheJulia | Quick switch poll: Who has Aruba in your environment. | 16:50 |
| TheJulia | Furtherore, Who has SONiC or NVUE ? | 16:50 |
| TheJulia | err, furthermore | 16:50 |
| opendevreview | Merged openstack/ironic master: Remove remaining referemce to python-dracclient https://review.opendev.org/c/openstack/ironic/+/998257 | 17:35 |
| mnasiadka | TheJulia: we have NVUE in a number of environments | 19:40 |
| TheJulia | mnasiadka: is that code basically the same as what is in upstream? | 19:46 |
| TheJulia | mnasiadka: I ask because I might look at making a paramiko-less path for engaging with such devices | 19:46 |
| TheJulia | well, I am looking at, but not acting yet | 19:49 |
| mnasiadka | TheJulia: yes, we use upstream | 19:50 |
| mnasiadka | NGS | 19:50 |
| mnasiadka | I was thinking of writing a driver that uses API, but it fell to the bottom of my neverending todo list | 19:50 |
| TheJulia | I have a spec for netconf, but it looks like its either their API or ssh commands basically | 19:51 |
| mnasiadka | Netconf is not popular in NVIDIA ;) | 19:51 |
| * TheJulia shrugs | 19:52 | |
| TheJulia | Yeah, its not in nvue at all :) | 19:52 |
| TheJulia | but netconf is a model we can take for most other modern vendors, the only odd-balls are sonic and aruba beyond that | 19:52 |
| TheJulia | sonic... if we just use commands, should be fine | 19:52 |
| mnasiadka | Well, depends which sonic cli (klish, click or however these are called) - but sonic does rest api as well | 19:54 |
| TheJulia | true, I think we're going for the standard config one | 19:55 |
| TheJulia | I don't remember what the current driver uses under the hood | 19:55 |
| TheJulia | config and vtysh invocations it looks like | 19:59 |
| mnasiadka | Current driver uses click, which IIRC is incompatible with Cisco-like klish (so if you use the latter for majority of your config you end up with some cross-cli sync issues for ngs managed ports) - but that’s my vague memory of it | 20:00 |
| TheJulia | ugh | 20:01 |
| TheJulia | fun | 20:01 |
| mnasiadka | But IIRC the config invocations (click) is the one designed for automation - so that’s just a choice | 20:01 |
| TheJulia | that is a good data point | 20:02 |
| opendevreview | Julia Kreger proposed openstack/ironic-specs master: Add spec for sonic and nvue driver replacements. https://review.opendev.org/c/openstack/ironic-specs/+/998363 | 20:12 |
| TheJulia | dtantsur: hjensas ^^^ | 20:18 |
| TheJulia | mnasiadka: you may also be interested in ^^^ | 20:18 |
| TheJulia | of course linters fail | 20:48 |
| opendevreview | Merged openstack/networking-generic-switch master: Refactor: extract OpenConfigModelMixin https://review.opendev.org/c/openstack/networking-generic-switch/+/992884 | 21:33 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Move container execution into a single method https://review.opendev.org/c/openstack/ironic-python-agent/+/998374 | 23:28 |
| opendevreview | cid proposed openstack/ironic-python-agent master: security: Fix the container allowlist not being enforced https://review.opendev.org/c/openstack/ironic-python-agent/+/998375 | 23:28 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Prevent container steps from shadowing existing steps https://review.opendev.org/c/openstack/ironic-python-agent/+/998376 | 23:28 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Only honor container step priorities while cleaning https://review.opendev.org/c/openstack/ironic-python-agent/+/998377 | 23:28 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Accept whitespace separated container options https://review.opendev.org/c/openstack/ironic-python-agent/+/998378 | 23:28 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Check that the configured container runtime is present https://review.opendev.org/c/openstack/ironic-python-agent/+/998379 | 23:28 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Report errors reading the container steps file https://review.opendev.org/c/openstack/ironic-python-agent/+/998380 | 23:28 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Ignore the transport prefix when matching allowed containers https://review.opendev.org/c/openstack/ironic-python-agent/+/998381 | 23:28 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Drop the unused container_conf_file option https://review.opendev.org/c/openstack/ironic-python-agent/+/998382 | 23:28 |
| opendevreview | cid proposed openstack/ironic-python-agent master: Rename container_clean_step to generic_container_step https://review.opendev.org/c/openstack/ironic-python-agent/+/998383 | 23:28 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!