| opendevreview | Scott Tran proposed openstack/sushy master: PUC-491, Contribute Port interface to Sushy https://review.opendev.org/c/openstack/sushy/+/931728 | 03:16 |
|---|---|---|
| rpittau | good morning ironic! o/ | 06:50 |
| iurygregory | good morning ironic | 11:37 |
| iurygregory | oh interesting, so we have NetworkPort and Port in redfish O.o | 11:51 |
| tkajinam | I wonder if anyone knows the current state of molteniron ? https://github.com/openstack/molteniron | 11:53 |
| iurygregory | as far as i remember we didn't do releases (it's been a while) | 11:54 |
| tkajinam | I happened to found that repository as part of mass cleanup work and noticed that its CI is badly broken | 11:55 |
| tkajinam | yeah | 11:55 |
| iurygregory | truth | 11:55 |
| iurygregory | we should probably just drop testing jobs etc on it I would say | 11:55 |
| iurygregory | not even sure if there is still people using it | 11:55 |
| tkajinam | https://review.opendev.org/c/openstack/molteniron/+/927788 | 11:55 |
| tkajinam | I'd leave the repository for now if it is not really active. but its situation looks like a sign for potential retirement | 11:56 |
| iurygregory | yeah | 11:56 |
| masghar | I think its retirement was discussed a while ago, but idk what came of it | 12:35 |
| rpittau | tkajinam: as masghar said, the retirement of molteniron has been discussed some time ago and the repository is currently not receiving updates, probably not worth spending time there | 13:00 |
| TheJulia | I'd just close it up. | 13:47 |
| TheJulia | molteniron | 13:47 |
| TheJulia | It was always scripting for CI, never anything else | 13:48 |
| opendevreview | Merged openstack/ironic-python-agent master: Drop dependency on netifaces https://review.opendev.org/c/openstack/ironic-python-agent/+/931584 | 13:52 |
| opendevreview | Merged openstack/ironic-python-agent master: Get rid of pkg_resources https://review.opendev.org/c/openstack/ironic-python-agent/+/931188 | 13:52 |
| opendevreview | cid proposed openstack/ironic master: Fix double JSON encoding of error message https://review.opendev.org/c/openstack/ironic/+/931795 | 14:06 |
| TheJulia | someone should just do it | 14:09 |
| opendevreview | Scott Tran proposed openstack/sushy master: Contribute Port interface to Sushy https://review.opendev.org/c/openstack/sushy/+/931728 | 14:16 |
| dtantsur | Hey folks, looking for a 2nd +2 https://review.opendev.org/c/openstack/ironic/+/915318 | 14:31 |
| opendevreview | Merged openstack/ironic master: Use oslo.utils implementation to parse server address https://review.opendev.org/c/openstack/ironic/+/931563 | 14:31 |
| iurygregory | dtantsur, looking now | 14:32 |
| TheJulia | done | 14:33 |
| dtantsur | thx! | 14:33 |
| dtantsur | TheJulia: https://review.opendev.org/q/Ic4390d578f564f245d7fb4013f2ba5531aee9ea9 is ready to go, right? | 14:33 |
| * dtantsur makes sure he hasn't missed any conversatons | 14:33 | |
| TheJulia | dtantsur: yeah, it is really straight forward, but that is not the cve :) | 14:35 |
| dtantsur | ah, I see, I only took a brief look at the summaries :D | 14:36 |
| TheJulia | heh | 14:36 |
| dtantsur | okay, so https://review.opendev.org/c/openstack/ironic/+/931277 needs a 2nd +2 as well now | 14:37 |
| TheJulia | https://review.opendev.org/q/Id6185b317aa6e4f4363ee49f77e688701995323a is the cve | 14:38 |
| rpittau | mmm I thought I've added it already, oh well added it now | 14:39 |
| iurygregory | woot crypt got removed in python 3.13.. https://docs.python.org/3/library/crypt.html | 15:01 |
| iurygregory | I just saw the email from zigo on openstack-discuss | 15:02 |
| zigo | :) | 15:02 |
| zigo | What's its use in Ironic btw? | 15:02 |
| zigo | (I had no time to dig into each individual one yet...) | 15:03 |
| iurygregory | password hash if I recall | 15:04 |
| iurygregory | maybe there is more, I would need to check the code to remember =) | 15:04 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: Checksum files before raw conversion https://review.opendev.org/c/openstack/ironic/+/931300 | 15:26 |
| zigo | iurygregory: If you fix it, please send me the patch URL ! | 15:33 |
| opendevreview | Merged openstack/ironic master: Add timeout to SERVICEWAIT https://review.opendev.org/c/openstack/ironic/+/915318 | 15:53 |
| opendevreview | Merged openstack/ironic master: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931277 | 15:53 |
| iurygregory | zigo, will do, looking how we can replace or other libs we can use for it o/ | 15:59 |
| rpittau | iurygregory: should be quick with hashlib in our case, want to give it a try? | 16:03 |
| opendevreview | Stig Telfer proposed openstack/tenks master: Preserve assigned MAC on subsequent invocations https://review.opendev.org/c/openstack/tenks/+/931816 | 16:03 |
| iurygregory | rpittau, yeah I saw hashlib was planning to test it o/ | 16:03 |
| rpittau | sounds good, maybe let's create a bug in launchpad so we can track it | 16:04 |
| iurygregory | yeah, I'm on it o/ | 16:04 |
| rpittau | thanks! | 16:05 |
| opendevreview | Scott Tran proposed openstack/sushy master: Change-Id: Ibfa97060d477d794a7ef8fe2a9675a658a13e1f6 https://review.opendev.org/c/openstack/sushy/+/931818 | 16:09 |
| rpittau | good night! o/ | 16:10 |
| iurygregory | good night! | 16:13 |
| iurygregory | https://bugs.launchpad.net/ironic/+bug/2083955 bug created for ironic and ipa o/ I'm working on it | 16:13 |
| dtantsur | TheJulia: I've approved stable/* versions of your fix. Could you check if any bugfix branches are affected? | 16:28 |
| iurygregory | hashlib doesn't have a function for salt.. wondering if we can just create one or maybe try other lib.. | 17:08 |
| dtantsur | salt is just a large random number, no? | 17:17 |
| * dtantsur might be out of context here | 17:17 | |
| opendevreview | Merged openstack/ironic stable/2024.2: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931278 | 17:30 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: cumulus_nvue: drop unapplied config on failures https://review.opendev.org/c/openstack/networking-generic-switch/+/931829 | 17:36 |
| opendevreview | Merged openstack/ironic stable/2024.1: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931279 | 17:37 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: cumulus_nvue: drop unapplied config on failures https://review.opendev.org/c/openstack/networking-generic-switch/+/931829 | 17:37 |
| opendevreview | Michal Nasiadka proposed openstack/networking-generic-switch master: cumulus_nvue: drop unapplied config on failures https://review.opendev.org/c/openstack/networking-generic-switch/+/931829 | 17:38 |
| opendevreview | Merged openstack/ironic stable/2023.2: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931280 | 17:43 |
| opendevreview | Merged openstack/ironic stable/2023.1: Fix actual size calculation for storage fallback logic https://review.opendev.org/c/openstack/ironic/+/931281 | 17:43 |
| iurygregory | dtantsur, can be letters and symbols if I recall .. | 17:46 |
| dtantsur | well, they all are numbers in this sense :) | 17:46 |
| iurygregory | yeah =) | 17:46 |
| dtantsur | "The passlib package can replace all use cases of this module." reads the page of the deprecated module | 17:47 |
| iurygregory | I only saw something about hashlib, /me looks for passlib ... | 17:47 |
| iurygregory | this would add a new requirement after a quick look at codesearch | 17:49 |
| dtantsur | yeah, I don't see anything immediately related to generating salt | 17:50 |
| dtantsur | iurygregory: what are we doing with crypt now? Something around rescue? | 17:51 |
| iurygregory | dtantsur, correct | 17:52 |
| iurygregory | we have it in utils and we use for the rescue password | 17:52 |
| clarkb | another tool that may do similar things and have a solution you can adopt is cloud-init | 17:52 |
| dtantsur | clarkb: probably won't help: we need to generate the hashed password on the Ironic's side and then apply it at a weird moment on the ramdisk side | 17:54 |
| dtantsur | iurygregory: if we go down the passlib path, https://passlib.readthedocs.io/en/stable/lib/passlib.hash.sha256_crypt.html seems to generate the salt by default | 17:54 |
| iurygregory | oh nice! | 17:55 |
| iurygregory | I will do some testing to see how it goes | 17:56 |
| clarkb | dtantsur: pretty sure cloud-init also sets passwords. But maybe it just shells out to passwd | 17:56 |
| dtantsur | I suspect it does | 18:02 |
| jamesdenton | regarding Swift support w/ virtual media... "While highly scalable, this method does suffer from issues where some vendors BMCs reject URLs with & or ? characters." -- iLo 5. Guilty as charged. | 18:03 |
| iurygregory | oh we saw a few things... | 18:05 |
| iurygregory | but wasn't with swift | 18:05 |
| jamesdenton | hah | 18:05 |
| iurygregory | I do remember some bmcs rejecting the iso because of some characters in the url.. | 18:05 |
| jamesdenton | Well, i spend too much time on it. Works great with DRAC 9. | 18:05 |
| jamesdenton | iLo5 silently failed to even attempt to mount the ISO via Swift. Switched to Conductor HTTP and wouldn't you know, worked the first time. | 18:06 |
| jamesdenton | so, the docs are helpful in that regard :D | 18:06 |
| TheJulia | jamesdenton: I have word it is on their backlog to fix it. We have folks following up every few weeks, we’re still waiting. | 18:09 |
| jamesdenton | glad to hear it! | 18:10 |
| jamesdenton | on a related note... happy to report it's working consistently well, and it the dhcp-less (routed/l3) w/ config-drive is smooth | 18:11 |
| jamesdenton | so, thanks to you all | 18:11 |
| TheJulia | dtantsur: they are, I didn’t have capacity to backport to the bug fix branches last week | 18:11 |
| iurygregory | jamesdenton, in supermicro we saw problems when the image url was https with a port number :D | 18:16 |
| TheJulia | One of the vendors requires a username and password to be submitted too | 18:19 |
| jamesdenton | insert "what year is this meme" | 18:19 |
| TheJulia | Queue table flipping | 18:19 |
| iurygregory | TheJulia, yeah! | 18:19 |
| * TheJulia sips the post meeting margarita | 18:20 | |
| opendevreview | Stig Telfer proposed openstack/tenks master: Preserve assigned MAC on subsequent invocations https://review.opendev.org/c/openstack/tenks/+/931816 | 19:19 |
| opendevreview | Scott Tran proposed openstack/sushy master: Contribute Port interface to Sushy and update chassis check since Dell can have more than one member. https://review.opendev.org/c/openstack/sushy/+/931728 | 21:07 |
| iurygregory | not sure how I feel about shifting things for from network/port.py to network/network_port.py .-. maybe just have both in port.py? .-. | 21:14 |
| iurygregory | this is regarding https://review.opendev.org/c/openstack/sushy/+/931728 | 21:15 |
| duri | hello, does Ironic support UEFI HTTP Booting in alternative to PXE booting ? | 21:17 |
| iurygregory | hi duri o/, we added support for UEFI HTTP Booting for redfish | 21:18 |
| iurygregory | I think it was in Antelope or Bobcat (2023.1 or 2023.2) | 21:18 |
| duri | I am new to openstack, can you share link where I can read about that ? | 21:18 |
| iurygregory | let me try to find here 1min | 21:18 |
| duri | https://docs.openstack.org/ironic/latest/admin/interfaces/boot.html I guess | 21:20 |
| iurygregory | this was the RFE https://bugs.launchpad.net/ironic/+bug/2032380 | 21:20 |
| iurygregory | yeah and the link you just shared =) | 21:20 |
| iurygregory | https://docs.openstack.org/ironic/latest/admin/drivers/redfish.html#redfish-http-s-boot here you can also find some info | 21:21 |
| duri | ty. appreciated | 21:22 |
| iurygregory | yw | 21:24 |
| opendevreview | cid proposed openstack/ironic master: Set all required properies on Gigabyte platforms https://review.opendev.org/c/openstack/ironic/+/931848 | 21:53 |
| opendevreview | Scott Tran proposed openstack/sushy master: Add Port interface and DellSwitchConnection https://review.opendev.org/c/openstack/sushy/+/931728 | 21:57 |
| opendevreview | cid proposed openstack/ironic master: Gracefully handle bad request exception https://review.opendev.org/c/openstack/ironic/+/931849 | 21:59 |
| iurygregory | this is interesting https://review.opendev.org/c/openstack/sushy/+/931728/4/sushy/resources/system/network/switch_connection.py , but we should probably ask to be vendor agnostic ... | 22:10 |
| iurygregory | seems like is an Oem extension for NetworkPorts | 22:10 |
| cardoe | Yeah I told Scott about sushy-oem-idrac | 22:11 |
| cardoe | I'd like to merge sushy-oem-idrac into sushy proper. | 22:11 |
| iurygregory | sounds like a PTG topic or meeting topic I would say | 22:13 |
| cardoe | Already on there. | 22:13 |
| iurygregory | nice! | 22:14 |
| cardoe | Scott's working on my team currently but I haven't had enough cycles to connect with him. But ultimately he's been asked to go through our internal hacks and drag them up to current sushy. | 22:15 |
| iurygregory | Kevin mentioned some people would be contributing to ironic when I was at the OIS Asia, good to see this happening =D | 22:16 |
| cardoe | He should be working with Dell hardware currently. | 22:16 |
| cardoe | So Kevin's the "public cloud" side and I'm the metal-y bits in the DC. Though my team is smaller than Kevin's. | 22:17 |
| iurygregory | got it =) | 22:17 |
| cardoe | So we'll see if I'm crazy but basically from the hardware side I've told everyone redfish >= sushy > * | 22:19 |
| cardoe | If we gotta touch redfish directly we gotta ultimately get it into sushy though. | 22:20 |
| cardoe | So wanting hardware to appear and inspect it out-of-band with enough details to make it go. | 22:24 |
| cardoe | So Scott's got "def dell_switch_connections(self):" but really how that code is done locally is that there's a "def switch_connections(self):" and we attempt the standard LLDP (haven't seen a piece of gear that supports it) and as a fallback we load the OEM module. | 22:26 |
| cardoe | Which is a bit different than how sushy works in general. You have to explicitly load the oem manager. | 22:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!