| *** tkajinam is now known as Guest299 | 04:46 | |
| ard__ | hi, I have a question regarding sushy_tool. Hope this is the right place to ask a question, ironic team seems to maintain this tool. | 07:49 |
|---|---|---|
| ard__ | In libvirt driver function set_boot_device sets only one boot source and just removes all other, Instead of figuring out the current boot order and just changing a new source in the first place. | 07:50 |
| TheJulia | ard__: the intent is one time override, for the most part | 07:51 |
| ard__ | If Pxe is used only for setup and it is not supposed to answer during normal work, then such virtual machines will fail to boot until hdd is set as a boot source. | 07:52 |
| TheJulia | But, we don’t save enough information to just do boot order list modification after the fact | 07:53 |
| TheJulia | Ironic specifically sets and the. Resets | 07:53 |
| TheJulia | Err, then resets | 07:53 |
| TheJulia | And UEFI boot loader install inserts a new first entry in the list as well | 07:54 |
| ard__ | https://github.com/openstack/sushy-tools/blob/master/sushy_tools/emulator/resources/systems/libvirtdriver.py#L386 Here we can take in account os section (to get all device types in boot order list) and count all previously devices with boot | 07:55 |
| TheJulia | UEFI specifically has to, for the most part be set from inside the OS context. Nvram entry editing is not generally available via BMCs nor is cleanly standardized in a cross vendor fashion in redfish | 07:55 |
| TheJulia | Patches welcome, just outside of the use pattern which the sushy-tools was developed to support ironic CI testing | 07:57 |
| ard__ | OK, I can create a patch for review. Just wanted to check if this behavior was intended as it differs from baremetal. | 08:00 |
| opendevreview | kamlesh chauvhan proposed openstack/sushy stable/zed: Fix volume deletion on newer iDRACs https://review.opendev.org/c/openstack/sushy/+/869311 | 08:01 |
| opendevreview | kamlesh chauvhan proposed openstack/sushy stable/yoga: Fix volume deletion on newer iDRACs https://review.opendev.org/c/openstack/sushy/+/869312 | 08:01 |
| TheJulia | So bare metal behavior of BMCs is generally inconsistent, for what it is worth. We do tend to signal onetime or not, I’d have to look at the code again without a migraine headache | 08:01 |
| opendevreview | kamlesh chauvhan proposed openstack/sushy stable/xena: Fix volume deletion on newer iDRACs https://review.opendev.org/c/openstack/sushy/+/869313 | 08:01 |
| opendevreview | kamlesh chauvhan proposed openstack/sushy stable/xena: Fix volume deletion on newer iDRACs https://review.opendev.org/c/openstack/sushy/+/869313 | 08:02 |
| TheJulia | We would be most inclined to match DMTF specified behavior, so it is also going to depend on your patch too | 08:03 |
| TheJulia | Anyhow, I’m going to try and get some sleep. Have a wonderful day! | 08:03 |
| kubajj | Good morning Ironic! | 08:21 |
| rpittau | good morning ironic! o/ | 08:22 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use jammy for base jobs https://review.opendev.org/c/openstack/ironic/+/869052 | 08:32 |
| jssfr | good morning & happy new year Ironic! | 08:35 |
| rpittau | great! finally pinning tox<4 in stable branches \o/ | 09:01 |
| opendevreview | Baptiste Jonglez proposed openstack/networking-generic-switch master: Document ngs_trunk_ports and ngs_physical_networks https://review.opendev.org/c/openstack/networking-generic-switch/+/868313 | 10:03 |
| opendevreview | Verification of a change to openstack/sushy-oem-idrac stable/zed failed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/sushy-oem-idrac/+/859264 | 10:52 |
| opendevreview | Verification of a change to openstack/sushy-oem-idrac stable/zed failed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/sushy-oem-idrac/+/859265 | 10:52 |
| opendevreview | Verification of a change to openstack/sushy-oem-idrac master failed: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/sushy-oem-idrac/+/859266 | 10:56 |
| slaweq | rpittau hi, is ironic gate fixed now? If yes, can You maybe check and approve https://review.opendev.org/c/openstack/ironic/+/868191 ? Thx in advance | 11:07 |
| rpittau | slaweq: I approved it, that will also be a good test for the CI :) | 11:10 |
| slaweq | rpittau thx a lot | 11:11 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix selinux context of published image hardlink https://review.opendev.org/c/openstack/ironic/+/868521 | 11:19 |
| kubajj | rpittau: is this problem just a timeout? https://zuul.opendev.org/t/openstack/build/e72fa7a5e2834b1ba072d050e3e52eb4 | 11:30 |
| opendevreview | Merged openstack/networking-generic-switch master: Add ArubaOS-CX switch support https://review.opendev.org/c/openstack/networking-generic-switch/+/868598 | 11:36 |
| opendevreview | Merged openstack/ironic master: Fix "'NoneType' object is not iterable" in RAID https://review.opendev.org/c/openstack/ironic/+/867117 | 11:44 |
| rpittau | kubajj: well it is a timeout but of the bad kind, can't ssh into the instance, port 22 seems closed or the instance is not up | 12:07 |
| rpittau | not sure why we're still using centos8 in metalsmith | 12:08 |
| iurygregory | morning Ironic | 12:18 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith master: Remove not used old job https://review.opendev.org/c/openstack/metalsmith/+/869356 | 12:18 |
| opendevreview | Merged openstack/ironic master: Remove lib/neutron-legacy leftovers https://review.opendev.org/c/openstack/ironic/+/868191 | 12:25 |
| rpittau | openstack-tox-linters is failing in metalsmith because of the "pbr bug", I guess we want to wait for that to be fixed instead of adding py_modules=[] to setup.py ? | 13:46 |
| opendevreview | Riccardo Pittau proposed openstack/metalsmith master: [WIP] Add centos9 based job https://review.opendev.org/c/openstack/metalsmith/+/869374 | 13:59 |
| iurygregory | rpittau, humm probably, do we know when it's going to happen the release? | 14:13 |
| rpittau | I have no idea ¯\_(ツ)_/¯ | 14:14 |
| rpittau | the fix is this one btw https://review.opendev.org/c/openstack/pbr/+/869082 | 14:14 |
| rpittau | not merged yet | 14:15 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix selinux context of published image hardlink https://review.opendev.org/c/openstack/ironic/+/868521 | 14:36 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Fix selinux context of published image hardlink https://review.opendev.org/c/openstack/ironic/+/868521 | 14:37 |
| rpittau | bye everyone, see you on monday o/ | 14:51 |
| opendevreview | Denys Mishchenko proposed openstack/sushy-tools master: Save previous hd boot device when setting Pxe https://review.opendev.org/c/openstack/sushy-tools/+/869379 | 15:05 |
| JasonF | rpittau: if that works around the issue temporarily, we can do it. We just need to make sure we remember to revert it, and that we do not make any releases while the repo is in that state | 15:40 |
| TheJulia | rutro | 16:07 |
| TheJulia | pbr does not look to be happy | 16:08 |
| TheJulia | this is not going to be a quick fix :\ | 16:09 |
| iurygregory | so we should go with the workaround I would say | 16:12 |
| opendevreview | Julia Kreger proposed openstack/ironic-prometheus-exporter master: devstack: fix plugin for local usage https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/869385 | 16:15 |
| TheJulia | well, the issue is more pbr has py27 cruft it looks like | 16:15 |
| TheJulia | and the CI doesn't know about 27 anymore | 16:16 |
| * TheJulia is working to verify locally first | 16:16 | |
| TheJulia | yeah, works *if* i have python2.7 installed | 16:16 |
| TheJulia | oh, 3.7 | 16:22 |
| *** JasonF is now known as JayF | 16:28 | |
| TheJulia | .. okay | 16:44 |
| sschmitt | Back to my left field question about EVPN-VXLAN and baremetal networking from yesterday, it seems like this neutron project is considering what I want | 16:58 |
| sschmitt | https://docs.openstack.org/networking-bagpipe/latest/user/applications.html | 16:58 |
| sschmitt | > networking-l2gw driver leveraging bagpipe-bgp running on a ToR | 16:58 |
| TheJulia | good to know | 17:03 |
| JayF | I sometimes think I could work on openstack for the rest of my life and still not know all the tools in the neutron toolbox :D | 17:03 |
| JayF | I'm glad you found the one for you \o/ | 17:03 |
| opendevreview | Alex Welsh proposed openstack/bifrost master: Use curl to download deploy image if get_url fails https://review.opendev.org/c/openstack/bifrost/+/868387 | 17:04 |
| TheJulia | JayF: do you know if anyone is looking at pbr and it's broken gate? | 17:07 |
| JayF | I have no idea; I was just thinking aobut what IRC channel that'd be in | 17:07 |
| TheJulia | I think it might be opendev tbh | 17:08 |
| TheJulia | clark/jeremy are the maintainers at this point from what I see | 17:08 |
| TheJulia | with latest tox, I can't reproduce the issues | 17:08 |
| fungi | stephenfin has been working on it | 17:08 |
| TheJulia | I mean aside from my machine missing a Python.h file | 17:08 |
| TheJulia | including the fact pep8/py* jobs are dead? | 17:09 |
| fungi | https://review.opendev.org/869190 "Update tox.ini to work with tox 4" | 17:09 |
| fungi | https://review.opendev.org/869082 "Tie recursion calls to Dist object, not module" | 17:09 |
| TheJulia | I've got the first, I'll try applyign the second to see if it, itself, can still work | 17:10 |
| fungi | well, those aren't passing yet either | 17:10 |
| TheJulia | well | 17:10 |
| TheJulia | CI doesn't work for them in general | 17:10 |
| TheJulia | they work locally | 17:10 |
| TheJulia | looks to be they are all trying to end up building wheels for other versions | 17:11 |
| TheJulia | yeah, looks like I get past that point locally | 17:11 |
| fungi | as for "what irc channel" is appropriate for pbr discussion, technically #openstack-oslo since it's an oslo team deliverable, but discussion often spills over into other channels | 17:12 |
| TheJulia | ... could it be the fact the jobs are running on jammy it looks like? | 17:12 |
| JayF | TheJulia: why don't we shift over to #openstack-oslo, try to sync up with stevenfin to make sure we're not duping work | 17:13 |
| TheJulia | already did | 17:14 |
| JayF | of course you have :D | 17:14 |
| * JayF used to being behind :D | 17:14 | |
| TheJulia | well, now that that seems to be on the sorted path | 17:36 |
| TheJulia | I can go back to metrics | 17:36 |
| stevebaker[m] | OK, I've finally got this systemd regression bug parked, time to write some CfPs | 21:56 |
| TheJulia | \o/ | 21:59 |
| JayF | Just helping test a thing | 22:04 |
| JayF | kevinhannon[m]: I can see you in here now, testing? | 22:06 |
| kevinhannon[m] | hey jay! | 22:06 |
| TheJulia | so I'm looking at the prometheus data format, and our dot delimited metric name format... and I'm seriously wondering if we should just shorten the field lenths... then again coding in some structural knowledge into the prometheus exporter allows us to add additional labels to provide clarity | 22:07 |
| JayF | the dot delimited format is common to most metrics stuff I've used | 22:29 |
| JayF | but I kinda feel like prometheus is a whole ecosystem I don't know much about | 22:29 |
| JayF | so it's tough for me to judge if shorter names in that context are good idea or not | 22:29 |
| TheJulia | https://scoutapm.com/blog/prometheus-metrics is a decent primer that I've stumbled upon | 22:29 |
| TheJulia | fwiw | 22:29 |
| TheJulia | well will see I guess, I'm stripping down the strings and applying labels in the code I'm writing | 22:30 |
| TheJulia | so folks can do "give me all the driver stuffs!" | 22:30 |
| JayF | that seems like a good idea | 22:33 |
| JayF | hierarchies can break down into tags | 22:33 |
| JayF | but not vice-versa | 22:33 |
| JayF | so having the hierarchy at the interface step (dotted names) can feed both kinds of metrics | 22:34 |
| TheJulia | yeah, kind of with code under the hood that understands it | 22:34 |
| JayF | btw; topic for my OSS Office Hours (in 18 minutes): learning about nox | 22:43 |
| JayF | youtube.com/jayofdoom | 22:43 |
| JayF | For a net-new python file; do we even put a Copyright line at the top? | 23:10 |
| TheJulia | not really | 23:10 |
| TheJulia | license yes, copyright no | 23:10 |
| TheJulia | if employer wants/expects it, it is okay | 23:10 |
| JayF | ack; I'll leave it out | 23:12 |
| JayF | not that this is likely to be more than a POC | 23:13 |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent master: DNM: Proof of Concept for nox https://review.opendev.org/c/openstack/ironic-python-agent/+/869412 | 23:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!