| opendevreview | cid proposed openstack/ironic master: Special treatment of .json is now disabled for nodes with .json extension. https://review.opendev.org/c/openstack/ironic/+/913467 | 00:05 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-inspector/+/906935 | 04:26 |
| rpittau | good morning ironic! o/ | 08:13 |
| opendevreview | Merged openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-inspector/+/906935 | 08:30 |
| drannou | Sandzwerg[m]: don't know if you had time to check, but I made a quick test on my side. Of course, checking if context is unset and returning True works perfectly | 10:30 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-inspector/+/913655 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-inspector/+/913656 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-inspector master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-inspector/+/913657 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/913658 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/913659 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/913660 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/913661 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/913662 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/913663 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/913664 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/913665 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/913666 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-ui/+/913667 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-ui/+/913668 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-ui/+/913669 | 11:10 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/913670 | 11:11 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/913671 | 11:11 |
| opendevreview | OpenStack Release Bot proposed openstack/networking-generic-switch master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/913672 | 11:11 |
| iurygregory | good morning | 11:19 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 11:35 |
| *** sfinucan is now known as stephenfin | 11:38 | |
| opendevreview | Merged openstack/networking-generic-switch stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/913670 | 11:44 |
| Sandzwerg[m] | drannou: I just briefly checked and I don't see any commits that touch this behavior at a glance. But I remember before we gave the ironic user image_admin rights for example the IPA images needed to be public | 11:46 |
| opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Add fake_ipa inspection, lookup and heartbeater to fake system https://review.opendev.org/c/openstack/sushy-tools/+/875366 | 12:01 |
| opendevreview | Merged openstack/networking-generic-switch master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/913672 | 12:01 |
| opendevreview | Merged openstack/networking-generic-switch stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/networking-generic-switch/+/913671 | 12:03 |
| opendevreview | Merged openstack/ironic-python-agent stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/913664 | 12:07 |
| opendevreview | Merged openstack/ironic-python-agent stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/913665 | 12:07 |
| opendevreview | Merged openstack/ironic-ui stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-ui/+/913667 | 12:07 |
| opendevreview | Merged openstack/ironic-python-agent master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent/+/913666 | 12:08 |
| opendevreview | Merged openstack/ironic-ui stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-ui/+/913668 | 12:10 |
| opendevreview | Merged openstack/ironic-ui master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-ui/+/913669 | 12:10 |
| opendevreview | Merged openstack/ironic-inspector master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-inspector/+/913657 | 12:10 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/913661 | 12:10 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/913662 | 12:10 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/913663 | 12:10 |
| opendevreview | Merged openstack/ironic-prometheus-exporter master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/913660 | 12:12 |
| opendevreview | Merged openstack/ironic-inspector stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-inspector/+/913655 | 12:17 |
| opendevreview | Merged openstack/ironic-inspector stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-inspector/+/913656 | 12:22 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 12:41 |
| mgoddard | Hi dtantsur, thanks for your (old) response on the mailing list about SLURP support for Bifrost. Would you be open to supporting it if we added a CI job? | 12:41 |
| mgoddard | We've added them for kolla and kayobe, in both cases no changes were required to the code | 12:41 |
| dtantsur | mgoddard: CI jobs are easy. I'm not sure I want to commit to extending deprecation period to 1+ year. We're stretched quite thin already. | 12:42 |
| dtantsur | A->C, for instance, features inspector disabling | 12:42 |
| mgoddard | dtantsur: ack. We're probably going to just do it anyway in kayobe, and handle the consequences | 12:45 |
| dtantsur | We can add a non-voting job and see how it behaves | 12:45 |
| mgoddard | that would be appreciated | 12:45 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/913658 | 12:58 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/913659 | 12:58 |
| dking | Good morning folks. Another unit testing question! Given that skip_list is a set here: https://opendev.org/openstack/ironic-python-agent/src/commit/783a0377ad2da5a486836edd54d3f623fc60751a/ironic_python_agent/hardware.py#L1547 is there a reason why it's mocked as a list here: https://opendev.org/openstack/ironic-python-agent/src/commit/783a0377ad2da5a486836edd54d3f623fc60751a/ironic_python_agent/tests/unit/test_hardware.py#L471 | 12:59 |
| dking | 5 | 12:59 |
| mgoddard | dtantsur: turns out bifrost is already testing slurp: upgrade_from: stable/2023.1 :D | 13:05 |
| dtantsur | oh LOL | 13:06 |
| dking | I just simply am replacing it with a set, and that's probably minor enough to not need a bug ticket, right? | 13:07 |
| dtantsur | dking: yeah, sounds really minor | 13:08 |
| opendevreview | Mark Goddard proposed openstack/bifrost master: CI: Modify upgrade jobs to upgrade from 2023.2 (Bobcat) https://review.opendev.org/c/openstack/bifrost/+/913679 | 13:23 |
| opendevreview | Mark Goddard proposed openstack/bifrost master: CI: Add a non-voting SLURP upgrade job for Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/913680 | 13:23 |
| zigo | Hi there! | 13:25 |
| zigo | Building the latest release for IPA (ie: 9.10.0), I'm getting this: | 13:25 |
| zigo | https://paste.opendev.org/show/bQOjIWNFKpGoXQgCcVnV/ | 13:25 |
| zigo | This is built with ironic-lib 6.0.0 (the latest). Am I missing something? | 13:25 |
| zigo | Ah no, it built with 5.5.0. | 13:26 |
| zigo | Anyways, the requirements.txt should say >= 6 ... | 13:26 |
| iurygregory | yeah agree, seems like ironic-lib had a major bump and we didn't update our requirements.. | 13:29 |
| * iurygregory checks why it was a major bump in ironic-lib | 13:29 | |
| dtantsur | The qemu_img module is new | 13:30 |
| zigo | Correct. tjat | 13:30 |
| zigo | That's what I just found out too. | 13:30 |
| iurygregory | seems like we need a new release for ipa | 13:31 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 13:34 |
| iurygregory | if someone can push the update in ipa I will review, a bit busy getting my new setup of machines to work | 13:35 |
| rpittau | I'll take care of both, I have 5 minutes now | 13:36 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Update ironic-lib min required version https://review.opendev.org/c/openstack/ironic-python-agent/+/913681 | 13:38 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Update ironic-lib min required version https://review.opendev.org/c/openstack/ironic-python-agent/+/913681 | 13:39 |
| rpittau | I'll propose the release once this ^ merges | 13:39 |
| iurygregory | rpittau, wondering now if we need a bump in ironic and inspector ... | 13:42 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 13:42 |
| rpittau | ironic looks good | 13:42 |
| iurygregory | ack | 13:42 |
| rpittau | I was looking at inspector now | 13:42 |
| rpittau | and yes we need it | 13:42 |
| rpittau | well | 13:42 |
| rpittau | wellll.... | 13:42 |
| rpittau | we don't "need" it | 13:43 |
| iurygregory | depends on the perspective =) | 13:43 |
| zigo | *I* don't even need it (I expressed the ironic-lib version concstraint in the package already). | 13:43 |
| zigo | Though it'd be nice to fix the requirements asap. | 13:43 |
| rpittau | zigo: I agree, IPA is on the way | 13:43 |
| zigo | (at least for other distros than Debian) | 13:43 |
| rpittau | inspector can work with the version it has currently, we haven't really touched anything there related to ironic-lib AFAICS | 13:44 |
| zigo | FYI, I got all the RC1s already uploaded! :P | 13:45 |
| * zigo is currently working on tempest package validation | 13:45 | |
| iurygregory | well, ironic hasn't released yet =) | 13:45 |
| rpittau | awesome | 13:45 |
| zigo | well, Ironic doesn't produce RCs ... :P | 13:46 |
| rpittau | no, we don't | 13:46 |
| iurygregory | truth =) | 13:46 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 13:47 |
| rpittau | I confirm inspector passes test with ironic-lib 4.3.0 so we're good | 13:48 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 13:51 |
| rpittau | dtantsur if you have a moment can you please approve https://review.opendev.org/c/openstack/ironic-python-agent/+/913681 ? | 14:08 |
| rpittau | or JayF ^ | 14:09 |
| dtantsur | If you remind me tomorrow.. | 14:23 |
| dtantsur | Folks, I'll shut down my IRC bouncer during KubeCon. I'll try to check here from time to time, but if you really need to reach me, use email. | 14:23 |
| dtantsur | See you on Monday otherwise. | 14:23 |
| JayF | rpittau: I thought we just cut that release? | 14:48 |
| rpittau | JayF: we did but we haven't updated ironic-lib in requirements | 14:48 |
| iurygregory | yeah | 14:49 |
| iurygregory | we only noticed after zigo comment | 14:49 |
| JayF | Okay so that's going to be backported and we'll get a point release | 14:49 |
| JayF | Cool | 14:49 |
| iurygregory | yup | 14:50 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic/+/913692 | 15:25 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic/+/913693 | 15:25 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic/+/913694 | 15:25 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent stable/2024.1: Update ironic-lib min required version https://review.opendev.org/c/openstack/ironic-python-agent/+/913510 | 16:19 |
| opendevreview | Merged openstack/ironic master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/ironic/+/913694 | 16:20 |
| rpittau | bye everyone, I'll be at KubeCon for the rest of the week, but I'll keep an eye here o/ | 16:56 |
| iurygregory | bye rpittau o/ enjoy the KubeCon! | 16:59 |
| opendevreview | Merged openstack/ironic stable/2024.1: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/ironic/+/913692 | 17:21 |
| opendevreview | Merged openstack/ironic stable/2024.1: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/ironic/+/913693 | 17:21 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Fix service holding https://review.opendev.org/c/openstack/ironic/+/913707 | 17:54 |
| opendevreview | Merged openstack/ironic-python-agent master: Update ironic-lib min required version https://review.opendev.org/c/openstack/ironic-python-agent/+/913681 | 18:12 |
| opendevreview | Merged openstack/ironic-python-agent stable/2024.1: Update ironic-lib min required version https://review.opendev.org/c/openstack/ironic-python-agent/+/913510 | 18:12 |
| *** jamesdenton_ is now known as jamesdenton | 18:41 | |
| zigo | Hi. I don't understand why: | 19:59 |
| zigo | -pyasn1-lextudio>=1.1.0 # BSD | 19:59 |
| zigo | -pyasn1-modules-lextudio>=0.2.0 # BSD | 19:59 |
| zigo | +pyasn1>=0.5.1 # BSD | 19:59 |
| zigo | +pyasn1-modules>=0.3.0 # BSD | 19:59 |
| zigo | Can someone enlight me why we're moving back away from lextudio implementation? | 19:59 |
| zigo | Is asn1 back to a maintained state? | 20:00 |
| zigo | I thought its author died... | 20:00 |
| TheJulia | zigo: the original author died, pyasn1 has been taken over successfully and lextudio nuked their fork from high orbit | 20:13 |
| jamesdenton | Does NGS support VLAN tagging on host switchports? | 20:16 |
| TheJulia | jamesdenton: what do you mean by host in that question | 20:16 |
| jamesdenton | baremetal node | 20:16 |
| TheJulia | and is the host the switch, or are you asking about the host attachment to the switchport of a switch? | 20:17 |
| jamesdenton | server<->switch connections versus switch uplinks | 20:17 |
| TheJulia | It focuses on the server<->switch conneciton, the uplinks are not focused on it | 20:17 |
| jamesdenton | i notice there's a feature related to 'trunks', but that seems to be actual switch<->switch uplinks versus a host's switchport as trunk vs access (in Cisco parlance) | 20:18 |
| TheJulia | Yeah, I do seem to remember there was a patch or three to support adding vlans to trunk ports as well | 20:19 |
| TheJulia | but was definitely not it's original function | 20:19 |
| jamesdenton | i'm curious about the ability to leverage Neutron 'trunk' functionality to allow a baremetal node to support multiple (tagged) networks on an interface versus a native (untagged) vlan | 20:21 |
| TheJulia | so it might just work in theory, I've never tried it | 20:22 |
| TheJulia | ... it might require tagging them as not native vlans as well, I just don't remember exactly what it is doign under the hood anymore since when I used it so many years ago it was just access ports | 20:22 |
| jamesdenton | i think it might be doable, too, but looking at the drivers in NGS none really seem suited for it - will have to see if i can get something going there | 20:23 |
| TheJulia | Also, in the grand scheme of things, it is a very simple tool with well defined patterns | 20:25 |
| jamesdenton | indeed | 20:28 |
| zigo | TheJulia: thanks for the info, I'll make sure to nuke lextudio in Debian as well then :-) | 20:37 |
| TheJulia | zigo: not to be confused with pysnmp of course :) | 20:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!