opendevreview | Iury Gregory Melo Ferreira proposed openstack/sushy-tools master: Update irrelevant-files for sushy-jobs https://review.opendev.org/c/openstack/sushy-tools/+/925069 | 01:22 |
---|---|---|
opendevreview | Merged openstack/sushy-tools master: Add fake system driver docs https://review.opendev.org/c/openstack/sushy-tools/+/924920 | 01:55 |
opendevreview | Jacob Anders proposed openstack/bifrost master: [DNM] test UUID VM definitions https://review.opendev.org/c/openstack/bifrost/+/925074 | 04:28 |
opendevreview | Jacob Anders proposed openstack/bifrost master: [DNM] test UUID VM definitions https://review.opendev.org/c/openstack/bifrost/+/925074 | 05:47 |
rpittau | good morning ironic! o/ | 06:43 |
opendevreview | Jacob Anders proposed openstack/bifrost master: [DNM] test UUID VM definitions https://review.opendev.org/c/openstack/bifrost/+/925074 | 08:12 |
janders | hey rpittau o/ | 08:12 |
rpittau | hey janders :) | 08:12 |
opendevreview | Jacob Anders proposed openstack/bifrost master: [WIP] Use libvirt UUIDs instead of VM names in Redfish system_ids https://review.opendev.org/c/openstack/bifrost/+/925074 | 08:31 |
drannou | Hello ironic | 08:35 |
opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Extend the fake systems driver with external systems status notifier https://review.opendev.org/c/openstack/sushy-tools/+/923111 | 08:46 |
opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Extend the fake systems driver with external systems status notifier https://review.opendev.org/c/openstack/sushy-tools/+/923111 | 08:58 |
opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Extend the fake systems driver with external systems status notifier https://review.opendev.org/c/openstack/sushy-tools/+/923111 | 10:27 |
iurygregory | good morning ironic | 11:09 |
opendevreview | Jonas Schäfer proposed openstack/ironic-python-agent master: agent: make _find_routable_addr dual-stack capable https://review.opendev.org/c/openstack/ironic-python-agent/+/925087 | 11:11 |
jssfr | o/ I just opened https://review.opendev.org/c/openstack/ironic-python-agent/+/925087 by accident because I didn't expect that `git review` does things instead of showing a help page. I haven't checked the tests or lints yet, so sorry for the noise. | 11:13 |
iurygregory | jssfr, no worries =) | 11:15 |
dtantsur | jssfr: you can mark it as not ready by using "Workflow -1" in the UI | 11:27 |
jssfr | let me find out how to log in :) it's been a while :) | 11:30 |
jssfr | do I just post "Workflow -1" in the comments? | 11:31 |
jssfr | I found a "Mark as Work in Progress" button | 11:32 |
jssfr | did that do the right thing? | 11:32 |
dtantsur | jssfr: "Reply" button, then select a Workflow vote -1 | 11:34 |
dtantsur | Or you can mark as WIP (I've never used this feature, it's quite new) | 11:34 |
jssfr | I apparently just did the latter | 11:34 |
jssfr | text search on the page doesn't find a "reply" | 11:35 |
jssfr | ah, "Start review" | 11:35 |
opendevreview | Jonas Schäfer proposed openstack/ironic-python-agent master: agent: make _find_routable_addr dual-stack capable https://review.opendev.org/c/openstack/ironic-python-agent/+/925087 | 11:37 |
jssfr | alright, added tests :) | 11:37 |
opendevreview | Jonas Schäfer proposed openstack/ironic-python-agent master: agent: make _find_routable_addr work with IPv4- and IPv6-only setups https://review.opendev.org/c/openstack/ironic-python-agent/+/925087 | 11:38 |
opendevreview | Mohammed Boukhalfa proposed openstack/sushy-tools master: Extend the fake systems driver with external systems status notifier https://review.opendev.org/c/openstack/sushy-tools/+/923111 | 12:04 |
drannou | Does anyone has some times to check my bug report https://bugs.launchpad.net/ironic/+bug/2072336 ? | 12:15 |
drannou | for me, it's a huge problem, because it's hidden, and at the end it will reach a motherboard limit | 12:16 |
cid | Looking... | 12:25 |
* cid oh, seems I already did. Someone with more experience will need to now. | 12:27 | |
drannou | the patch is very simple, I already apply it on our infra, but I don't know all motherboard integrations, may be some are using UTF-16, but that would be wired in my POV | 12:28 |
dtantsur | drannou: do you just assume utf-8 or what's the fix? | 12:31 |
drannou | dtantsur: yes, only replace utf-16 to utf-8 in the decode call | 12:31 |
drannou | if someone has some running nodes, that had multiple install, to check the EFI boot entries and confirm the bug ? | 12:34 |
dtantsur | I wonder why utf-16 was added. TheJulia was it you? | 12:34 |
dtantsur | maybe we can try both.. | 12:34 |
drannou | https://github.com/openstack/ironic-python-agent/commit/76accfb880474445a5dcb07825889123b3dd0237 | 12:36 |
TheJulia | It is *supposed* to be utf-16, but obviously sometimes jr is not. :( | 12:37 |
dtantsur | okay, so we at least should not just blindly revert it. maybe if we fail with UnicodeError, we can try utf-8 | 12:37 |
TheJulia | S/jr/it/ | 12:37 |
TheJulia | I unfortunately have had customer cases come up with utf-16 encoded regional accent characters | 12:39 |
drannou | Yep not really surprising, but the problem is that it has a huge impact on my hosts: after multiple install, the host refuse to boot automatically, I have to manually select the boot entry in the bios | 12:40 |
TheJulia | That sounds like a vendor firmware bug | 12:47 |
TheJulia | honestly | 12:47 |
TheJulia | plus, we should be removing old entries | 12:48 |
drannou | May be but we should catch this problem to not create a new bug | 12:52 |
TheJulia | I guess | 13:02 |
TheJulia | so about 3 months ago, stevebaker posted https://github.com/openstack/ironic-python-agent/commit/215fecd4470e868e1bac9737417e166a7e10fb64 | 13:04 |
TheJulia | Would be reasonable to backport I guess | 13:04 |
TheJulia | Also, good morning everyone | 13:06 |
rpittau | hey TheJulia :) | 13:11 |
drannou | I saw this commit, but IPA was already doing some clean, so yes it's an improvement, but not fixing the problem :). But I just made a test by removing the ignore errors, and catching the exception, it's working | 13:13 |
TheJulia | Well, ideally it should actually prevent the case your ultimately hitting with excess entries if properly decoded | 13:15 |
TheJulia | the issue, I think is your running the command and the data is unreadable as a result | 13:15 |
TheJulia | ... Which makes me wonder what distribution your using as well | 13:16 |
drannou | on my IPA ? Debian 12 | 13:16 |
TheJulia | Interesting. I wonder if Debian is defaulting with anything which would impact efibootmgr's behavior | 13:33 |
opendevreview | Adam Rozman proposed openstack/ironic-specs master: propose root partition and configdrive encryption https://review.opendev.org/c/openstack/ironic-specs/+/924993 | 13:36 |
TheJulia | braaaains | 13:38 |
opendevreview | Adam Rozman proposed openstack/ironic-specs master: propose root partition and configdrive encryption https://review.opendev.org/c/openstack/ironic-specs/+/924993 | 13:46 |
opendevreview | Merged openstack/ironic master: Log node UUID and not entire node instance to avoid logging its driver_info field https://review.opendev.org/c/openstack/ironic/+/924838 | 14:28 |
rpittau | #startmeeting ironic | 15:00 |
opendevmeet | Meeting started Mon Jul 29 15:00:16 2024 UTC and is due to finish in 60 minutes. The chair is rpittau. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
rpittau | Hello everyone! | 15:00 |
rpittau | Welcome to our weekly meeting! | 15:00 |
rpittau | The meeting agenda can be found here: | 15:00 |
rpittau | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_July_29.2C_2024 | 15:00 |
TheJulia | o/ | 15:01 |
mohammed | o/ | 15:01 |
JayF | O/ | 15:02 |
cardoe | o/ | 15:02 |
rpittau | alright let's start | 15:03 |
masghar | o/ | 15:03 |
rpittau | #topic Announcements/Reminders | 15:03 |
cid | o/ | 15:03 |
rpittau | #info Standing reminder to review patches tagged ironic-week-prio and to hashtag any patches ready for review with ironic-week-prio | 15:03 |
rpittau | #link https://tinyurl.com/ironic-weekly-prio-dash | 15:03 |
iurygregory | o/ | 15:03 |
rpittau | 2-3 patches that looks ready there | 15:04 |
rpittau | #info 2024.2 Dalmatian Release Schedule | 15:05 |
rpittau | #link https://releases.openstack.org/dalmatian/schedule.html | 15:05 |
rpittau | new bugfix branches will be requested this week | 15:05 |
rpittau | we're at R-9, 3 weeks to release for non-client libraries | 15:05 |
rpittau | #info he next OpenInfra PTG which will take place October 21-25, 2024 virtually! Registration is now open! | 15:06 |
rpittau | #link https://ptg.openinfra.dev | 15:06 |
rpittau | the ironic team has been registered! | 15:06 |
rpittau | and JayF prepared a meetpad page | 15:06 |
rpittau | #link https://etherpad.opendev.org/p/ironic-ptg-october-2024 | 15:06 |
TheJulia | ... 21-st through 25th... right when I was planning on being on vacation :( | 15:06 |
TheJulia | le-sigh | 15:06 |
rpittau | put your ides/suggestions/comments there | 15:06 |
rpittau | TheJulia: d'oh | 15:06 |
rpittau | I may miss a couple of days too, probably half of that week | 15:07 |
rpittau | well, we'll see | 15:07 |
rpittau | anything else we want to announce/remind ? | 15:08 |
JayF | I'll note that that ptg schedule is also within a stone's throw of my brother's wedding. I don't have a direct conflict, but I could see enough of us having trouble that we may want to have some type of off-cycle ironic sync | 15:08 |
rpittau | JayF: let's see how many people are available and then we can definitely think about having a separate sync, maybe reducing the official PTG time | 15:09 |
TheJulia | Honestly, I think more high bandwidth/sync communiation is better than once every six months | 15:09 |
TheJulia | so if we have a pre-ptg sync, we could at least get some stuff raised up/identified for wider discussion | 15:10 |
JayF | That's actually a pretty good observation. At a minimum, consider bringing back spuc... We could also consider doing a once-monthly video meeting similar to what the TC does | 15:10 |
TheJulia | which would better enable us to move forward with one person who might have previously been critical for the discussion to take place | 15:10 |
rpittau | sure, although IMHO the PTG helps on planning for the next release, so we should still decide on priorities for the next cycle | 15:10 |
TheJulia | Sanity presevation unconference often did end up being some deep technical discussion | 15:11 |
TheJulia | rpittau: agree completely | 15:11 |
TheJulia | I just don't think we should shoehorn all activities including problem definition into the PTG and expect success to result | 15:11 |
JayF | ++ | 15:12 |
TheJulia | like Adam's whole disk encryption proposal.... that might need some advance discussion before planning or at least base context sharing | 15:12 |
rpittau | I guess we can come up with the priorities during other discussions and then just write them down to have them ready for the next cycle | 15:12 |
JayF | TheJulia: I just wanna talk with him about it b/c it's so dang cool :D | 15:12 |
TheJulia | Well, spreading understanding is distinctly different than defining and deciding on priorities | 15:13 |
TheJulia | JayF: oh my, I haven't even read it yet becuase insanely busy | 15:13 |
JayF | we've often combined what and how in PTG | 15:13 |
* TheJulia resists for now | 15:13 | |
JayF | splitting them is wise | 15:13 |
TheJulia | we don't have to entirely split, just... spread context, and it doesn't have to be formal | 15:14 |
JayF | TheJulia: eh, more that I'm a sicko who appreciates seeing crap I've manually done in a gentoo desktop get mainstream enough that I might to get it for my day job :D | 15:14 |
TheJulia | eh, that is more Freaky, TBH :) | 15:14 |
cardoe | Someone likes self flogging. | 15:14 |
TheJulia | heh | 15:14 |
JayF | not exactly :D | 15:15 |
TheJulia | Okay, back to the topic at hand! | 15:15 |
JayF | It's not that tough anyway, dracut supports it well | 15:15 |
rpittau | let's see how many topics we have for the PTG first | 15:15 |
TheJulia | I do think bringing back the SPUC is worthwhile just from a contributor connection/relationship building standpoint | 15:15 |
JayF | ++ especially with new folks in the community | 15:16 |
cardoe | Can someone define SPUC? (sorry newbie here) | 15:16 |
TheJulia | Sanity Preservation Un-Conference | 15:16 |
masghar | (Also listening) | 15:16 |
TheJulia | A weekly call we did during the pandemic to give people an outlet to vent/connect/relate | 15:16 |
TheJulia | And discuss any topics in high bandwidth | 15:17 |
masghar | Oh that sounds nice | 15:17 |
TheJulia | Keep in mind, when we started it, most of the core contributors were loosing all remaining sanity | 15:17 |
TheJulia | But it really did help | 15:17 |
JayF | We eventually gave up on sanity preservation | 15:18 |
JayF | but friendship is valuable even for those who have no sanity :) | 15:18 |
TheJulia | ++ | 15:18 |
rpittau | I agree | 15:18 |
rpittau | if we bring that back maybe we could do it once per month or something like that | 15:18 |
TheJulia | Anyway, lets get meeting back on track | 15:18 |
rpittau | yep | 15:18 |
rpittau | #topic Review Ironic CI status | 15:19 |
TheJulia | Anything that is not weekly makes it hard to keep it in mind, fwiw. | 15:19 |
rpittau | #info ironic CI was impacted by an issue caused by the removal of simplejson from osc reqs | 15:19 |
JayF | stevebaker[m] and I are doing a stable branch CI audit, I think much of that is in better shape than usual | 15:19 |
JayF | but I need to check in to see how far we've gotten | 15:20 |
rpittau | JayF: I think I reviewed most of the changes already | 15:20 |
rpittau | btw this is the meetpad link | 15:20 |
rpittau | #link https://etherpad.opendev.org/p/ironic-ci-audit-july-2024 | 15:20 |
JayF | yeah I just haven't checked that in a couple days, had other distractions | 15:22 |
rpittau | I'll have another look at the patches during the week | 15:22 |
rpittau | probably we need to clarify that anything that still uses CS8 must be removed or made non-voting | 15:22 |
rpittau | not sure it's worth to upgrade to CS9 unless we still maintain the branch | 15:23 |
TheJulia | Removed, I believe | 15:23 |
TheJulia | since the base images are also gone and the jobs just get ignored as a result | 15:24 |
rpittau | yeah, actually better not waste CI resources | 15:24 |
TheJulia | ++ | 15:24 |
rpittau | anything else CI related ? | 15:24 |
rpittau | alright so we don't have discussion topics, unless someone has something to bring up | 15:25 |
JayF | I think cardoe had something for open discussion | 15:26 |
rpittau | sure thing | 15:26 |
cardoe | Hello all. Got a few asks. | 15:26 |
cardoe | So working on an effort around a sizeable amount of machines living in Ironic (multiple ones that we'll have as regions) | 15:27 |
cardoe | Our deployment is centered around Kubernetes and we're using OpenStack Helm. | 15:27 |
cardoe | We're letting people consume the hardware via Nova and some like jamesdenton consume it via baremetal directly. | 15:27 |
cardoe | Just wanted to share that context. | 15:27 |
TheJulia | Interesting.... | 15:27 |
TheJulia | context++ | 15:28 |
JayF | oh, you're working on my old cluster, or something adjacent to it :D | 15:28 |
cardoe | Happy to give more details anytime as well. | 15:28 |
cardoe | So we've tried to use the redfish inspector via sushy and there's quite a bit of bits lacking. | 15:28 |
JayF | I'm sorry and/or you're welcome as appropriate :) | 15:28 |
chris218 | Hi is the meeting still ongoing? | 15:28 |
TheJulia | chris218: yes, cardoe has the floor | 15:29 |
cardoe | I've been monkey patching some more endpoints into sushy because I know there's a specific targeted profile, but how much appetite is there to extensions there? | 15:29 |
cardoe | e.g. ethernet_interfaces is busted on a bunch of hardware. | 15:29 |
cardoe | https://review.opendev.org/c/openstack/ironic/+/924943 | 15:29 |
TheJulia | cardoe: so there is a lot of history here, so I guess to sort of start off | 15:29 |
cardoe | e.g. Dell R76xx family of hardware with iDRAC 9 7.x.y.z returns only the 2 on board 1GB interfaces and then an empty MAC address and none of the other NIC cards plugged in. | 15:30 |
TheJulia | The out-of-band introspection is minimal information, but I think we would accept patches to obtain/match identify more within reason | 15:30 |
TheJulia | ethernet_interfaces themselves has a long, painful, history | 15:30 |
cardoe | Right. I've switched us to using Ports | 15:30 |
TheJulia | cool cool | 15:30 |
dtantsur | You have a fix though, it just needs to be moved to sushy? | 15:30 |
cardoe | Well the fix for that change request is to just filter out the empty MAC. Which is done in a few places in Ironic code. Probably should be centralized. | 15:31 |
TheJulia | so the other challenge there is depending on the cards, the interface between the BMC and the firmware (i.e. is it dell's version fo the firmware, or chipset OEM's card, or even third party firwmare), your going to get different BMC reporting behavior *as well* | 15:31 |
cardoe | My "full" fix is to stop using ethernet_interfaces and use the ports endpoint in Redfish, but that's a big change to the interop profile. | 15:31 |
TheJulia | filtering the empty mac makes sense | 15:32 |
dtantsur | cardoe: yeah, also we cannot stop using them - we'll definitely find a hardware that only has EthernetInterfaces... | 15:32 |
cardoe | I've got piles of HP and Dell gear only that I'm testing against. So that's gonna be my bias filter. | 15:32 |
dtantsur | (we've been in similar positions several times) | 15:32 |
TheJulia | the underlying problem is.... I'm trying to think of the protocol used for out of band communications between the BMC and the cards, it knows it is a nic, but not enough information/support to pass along the mac address | 15:32 |
TheJulia | so filtering empty makes *tons* of sense | 15:32 |
dtantsur | but we definitely need to fix sushy to rule out empty strings. if you're not planning on it, we should probably Just Do It ourselves | 15:33 |
cardoe | So would there be an appetite for adding a "ports" interface to sushy? Ironic doesn't have to depend on it. | 15:33 |
dtantsur | TheJulia: we already have "is not None" already btw, but that does not capture empty strings | 15:33 |
TheJulia | I would be happy to review such patches | 15:33 |
cardoe | You don't have to do it. You gimme $TIME and I'll cherry-pick them out and push them to gerrit | 15:34 |
dtantsur | cardoe: we're open for any standard-compliant extensions that open new possibilities for Ironic or adjacent projects | 15:34 |
TheJulia | dtantsur: yeah, that was sort of what I was figuring | 15:34 |
cardoe | Yeah so my Ports implementation follows the DMTF spec | 15:34 |
rpittau | that's a great selling point :) | 15:35 |
cardoe | I tried to make it as identical how you guys have done the others. But I'm sure there's some stuff I've missed and don't know. I'm happy to get review and do the lift to get it landed. | 15:35 |
JayF | that sounds like about as easy of a change as we'll ever have to approve from a theoretical standpoint. A standards-compliant implementation of a redfish endpoint mimicing existing style :D | 15:35 |
cardoe | Essentially I'm trying to add as much of what Ironic Inspector did into the Redfish inspector. | 15:35 |
cardoe | To have as much out of band as possible. | 15:36 |
dtantsur | ++ | 15:36 |
cardoe | When I send a human into the DC to fix up a box, I wanna quickly out of band make sure the hardware they swapped is where it's suppose to be. | 15:36 |
cardoe | and before they get back to their desk be able to tell them "no go back and fix it" | 15:37 |
TheJulia | The one thing to keep in mind, you may still want a full OS to boot and look for any devices, because BMC support and reporting replies on firmware on the card behaving on the i2c bus | 15:37 |
TheJulia | ++ | 15:37 |
cardoe | 100%. We're doing out-of-band not using the ironic redfish inspector today. But I'm pushing my team to use sushy (our fork) for all of that. | 15:37 |
cardoe | Then we're flipping it to ironic inspector to do the more complete inspection. | 15:38 |
TheJulia | very cool | 15:38 |
TheJulia | Sounds good | 15:38 |
cardoe | Switching to the "agent" currently. | 15:38 |
cardoe | Just wondering if we can maybe contribute this to the redfish inspector eventually. | 15:38 |
cardoe | Or land our sushy bits into actual upstream. | 15:39 |
JayF | I think the answer is very yes | 15:39 |
cardoe | We did some Dell specific bits in our sushy fork which now wanting us to fork sushy-oem-idrac and get it there. | 15:39 |
cardoe | Okay good. | 15:39 |
JayF | I think we've considered in the past rolling sushy-oem-drac back into sushy | 15:39 |
JayF | might be worth having a larger discussion about that if it makes what you're doing easier, too | 15:39 |
cardoe | The other issue we've got is for new hardware out of the box (or maybe pallet? or shipping container? I dunno never seen the gear in person) | 15:39 |
cardoe | A lot of Dell stuff doesn't PXE boot for example. | 15:40 |
TheJulia | sushy-oem-idrac is largely for the super-specific behavior dell doesn't intend to change/fix/make compliant | 15:40 |
cardoe | We do some Redfish calls to fix it up | 15:40 |
dtantsur | is virtual media an option (instead of pxe)? | 15:40 |
TheJulia | will the dell gear do httpboot out of the box? | 15:40 |
cardoe | Not sure. I need to still test with that. | 15:41 |
cardoe | That was my backport question cause the idrac driver doesn't allow redfish-https | 15:41 |
TheJulia | oh, to send a network boot url out of band? | 15:41 |
cardoe | yeah. It's on a REAL soon sprint for someone to work that. | 15:42 |
cardoe | And confirm | 15:42 |
TheJulia | yeah, that is likely just a bug from my point of view | 15:42 |
cardoe | I assume httpboot you meant redfish-https | 15:42 |
TheJulia | yeah, one of the flavors | 15:42 |
TheJulia | there are two distinct http boot flavors | 15:42 |
cardoe | So we poke the BIOS settings via redfish | 15:43 |
dtantsur | virtual media exists for somewhat longer though | 15:43 |
TheJulia | I think dell gear from the factory ships leaning towards httpboot out of the box, but I've never seen a fresh from the factory box which has not been modified by well meaning humans | 15:43 |
cardoe | yeah virtual media doesn't work on the last dozen? or more racks we've gotten from Dell. | 15:43 |
TheJulia | oh joy | 15:44 |
cardoe | https://paste.opendev.org/show/bpoxYEvFBbqHpsGTy1IF/ https://paste.opendev.org/show/bUZln4wKQ1eINnjpTscw/ | 15:44 |
TheJulia | cardoe: if you can get us details in a bug, we might be able to assist with that | 15:44 |
dtantsur | huh, interesting. we have bugs from time to time, but never seen any issues at scale | 15:44 |
cardoe | Will do. | 15:44 |
cardoe | I mean I didn't try EVERY server in the rack | 15:44 |
TheJulia | I think this one has appeared in very recent firmware | 15:45 |
TheJulia | "Virtual Media is detached or Virtual Media devices are already in use." | 15:45 |
cardoe | But n is definitely n>1 | 15:45 |
cardoe | yes. These boxes all came with iDRAC9 7.00.60.00 | 15:45 |
TheJulia | I ?think? Iury was starting to look a week or two ago | 15:45 |
* iurygregory looks | 15:45 | |
cardoe | Whatever the 6.x.y.z version was didn't do this. | 15:45 |
TheJulia | yeah | 15:46 |
cardoe | So to that effect, I tried to use the bios config of the redfish driver in Ironic. | 15:46 |
iurygregory | oh I saw this error once | 15:46 |
TheJulia | Anything else cardoe? I think chris218 was next up :) | 15:46 |
cardoe | But it doesn't allow for disable_ramdisk=True | 15:46 |
iurygregory | but it was also a networking problem | 15:46 |
dtantsur | cardoe: re disable_ramdisk, janders and I have mid-terms plans to look into that (in the context of servicing) | 15:46 |
iurygregory | ironic couldn't reach the address I had the vmedia | 15:46 |
cardoe | okay I'll work with janders on that. | 15:47 |
cardoe | Perfect. I'll give up the floor. Thank you all. | 15:47 |
rpittau | cardoe: would be nice to continue the discussion, please open a bug and we'll look into it :) | 15:47 |
rpittau | thanks! | 15:47 |
TheJulia | Thanks cardoe! | 15:47 |
rpittau | any other discussion topics? | 15:47 |
cardoe | Just wanna say thank you all for the hard work. I wanna make sure we contribute back and get my folks engaged so thanks for entertaining me and my questions. | 15:47 |
TheJulia | chris218: you were wondering if the meeting was still in progress? Do you have a topic for discussion? | 15:48 |
TheJulia | I guess something got their attention. | 15:50 |
* TheJulia shrugs | 15:50 | |
cid | chris218 is probably away from the keyboard. | 15:51 |
rpittau | let's move on | 15:51 |
rpittau | #topic Bug Deputy Updates | 15:51 |
rpittau | cid: anything to report? | 15:51 |
cid | Nothing really. | 15:51 |
cid | Only a new bug was filed and someone help me with triaging it. | 15:51 |
* cid one | 15:52 | |
rpittau | ok cool | 15:52 |
rpittau | any volunteer for bug deputy for this week ? | 15:52 |
iurygregory | I can | 15:53 |
rpittau | thanks iurygregory :) | 15:53 |
rpittau | #topic RFE review | 15:53 |
rpittau | we have an rfe to discuss apparently https://review.opendev.org/c/openstack/sushy-tools/+/923111 | 15:54 |
JayF | There seems to be a lot of ... undocumented context around the direction of sushy-tools? | 15:54 |
mohammed | a tiny patch on sushy-tools. It's a minimal and generic hook that can be enabled to send status changes to a pluggable component's API endpoint | 15:54 |
JayF | I don't have a strong care about it, other than worrying about if this complexity could be maintained if the folks interested in these advanced features went away | 15:54 |
dtantsur | JayF: not really a lot, the root is very simple: the group of us wants to adapt sushy-tools for scale-testing Ironic | 15:54 |
dtantsur | like, scale scale | 15:55 |
JayF | That sounds like a worthy thing, and exciting, but without the plan laid out for upstream about how it's being done, it does make it difficult to review | 15:55 |
dtantsur | then there is some history on how we came to this specific proposal | 15:55 |
TheJulia | I think the problem is the original discussion didn't spread wide so ongoing context is lost | 15:55 |
dtantsur | JayF: have you seen the updated docs? https://review.opendev.org/c/openstack/sushy-tools/+/923111/14/doc/source/user/dynamic-emulator.rst | 15:56 |
TheJulia | (Hey, this is the sort of thing SPUC would have helped with! *ducks*) | 15:56 |
dtantsur | SPUUUC \o/ | 15:56 |
mohammed | Why do we need it? It extends the fake system, which is currently insufficient for testing end-to-end Ironic deployments. | 15:56 |
masghar | SPUC++ | 15:56 |
JayF | I guess I just wish I could see the whole picture | 15:56 |
mohammed | who will use it? For now, but not limited to, the fake IPA (which mocks some real IPA command executions). This component is in progress on the Metal3 org, and we plan to integrate it with the fake system using this interface. | 15:56 |
JayF | right now it feels like I'm reviewing the "y" of the xy problem on all these changes because I can't see the bigger picture | 15:56 |
chris218 | TheJulia: nah I had a question about implementing custom ironic driver | 15:57 |
JayF | which is mostly fine, I rarely review sushy* stuff anyway, but it seems to need the review attention now | 15:57 |
mohammed | Note: Sushy-tools emulates Redfish-compliant hardware, creating a "fake" environment to mimic the behavior of physical hardware for testing and development purposes. The fake system within sushy-tools adds another layer of simulation, creating fake drivers and components on top of the already emulated hardware environment. Essentially, the fake system is a "fake of the fake" and is intended only for testing purposes. I | 15:57 |
mohammed | just want to use it, and if there is no motivation to merge it into sushy-tools, I can host it elsewhere to continue using it effectively. | 15:57 |
cardoe | I mean I've thought about using something similar to that for some "fast" tests. | 15:58 |
TheJulia | I think it is likely okay | 15:58 |
TheJulia | it being the doing a thing, not having looked at the content yet | 15:58 |
cardoe | But our concern has definitely been to not pull a Crowd Strike and implement our behavior to the fake implementation and then have issues in prod. | 15:58 |
TheJulia | .... too soon | 15:59 |
* TheJulia had to deal with airline flight drama | 15:59 | |
mohammed | i do not think sushy will run in any prod env | 16:00 |
dtantsur | cardoe: sushy-tools is something that should not exist in production hence we're trying to put everything there | 16:00 |
dtantsur | mohammed: please avoid confusion: sushy vs sushy-tools | 16:00 |
dtantsur | sushy is a Python library that Ironic uses | 16:00 |
JayF | mohammed: yeah, mainly my question in the review was in an attempt to have this conversation happen | 16:00 |
rpittau | I guess we need to keep discussing that, maybe one ore more reviews with questions/answers in the patch could help | 16:00 |
JayF | mohammed: the conversation has happened, people are more on the same page, I think you'll get this all landed | 16:01 |
TheJulia | JayF: ++ | 16:01 |
dtantsur | I think the patch looks solid, I just spotted one issue with power state handling | 16:01 |
rpittau | alright then :) | 16:01 |
rpittau | going to close the meeting, we're a couple of minutes out | 16:02 |
rpittau | thanks all! | 16:02 |
rpittau | #endmeeting | 16:02 |
opendevmeet | Meeting ended Mon Jul 29 16:02:14 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:02 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-07-29-15.00.html | 16:02 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-07-29-15.00.txt | 16:02 |
opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-07-29-15.00.log.html | 16:02 |
TheJulia | chris218: feel free to pose your question, the brains are all sort of still engaged | 16:02 |
adam-metal3 | oh so the meetng ends at this time not starts :( | 16:04 |
chris218 | I'm planning to implement driver for intel amt for my homelab and had a few questions, for example I can query amt if secure boot is supported but can't enable/disable it, also can't query whether UEFI/Legacy boot is configured | 16:04 |
chris218 | also I know staging drivers have amt implementation (haven't tested it) but I know console for example isn't implemented with it | 16:04 |
TheJulia | adam-metal3: hi! | 16:04 |
rpittau | good night! o/ | 16:04 |
TheJulia | chris218: might depend on the chips you have with that AMT | 16:04 |
TheJulia | chris218: since some CPUs don't actually have LegacyBios anymore | 16:05 |
cardoe | dtantsur: | 16:05 |
chris218 | yea lenovo doesn't like to enable all the features it's 8th gen thinkcentre | 16:05 |
TheJulia | what model cpus do you have? | 16:05 |
cardoe | dtantsur: https://review.opendev.org/c/openstack/ironic/+/924943 so I'll stick that somewhere more common. There's a spot within the wrapper of sushy in Ironic. | 16:05 |
adam-metal3 | TheJulia: hi! | 16:05 |
dtantsur | cardoe: why not sushy itself? it literally has the check, it just does not go far enough. | 16:06 |
chris218 | thinkstation on the other hand even supports httpboot :( | 16:06 |
chris218 | I have 6 nodes with i5-8600t | 16:06 |
cardoe | Yeah that's what I was gonna say it should drag up into sushy | 16:06 |
TheJulia | coffee lake derived | 16:07 |
TheJulia | Yeah, they should support legacybios | 16:08 |
TheJulia | but yeah, lenovo may have locked that out | 16:08 |
chris218 | yea, it supports it but I can't enable/disable it through amt, should I just omit functions relating to those features in ironic driver? | 16:09 |
cardoe | OH I just remembered the PTG thing I wanted to ask | 16:11 |
cardoe | So some projects have started to adopt using Python Type Whispering^H^H^HHinting and generating OpenAPI from that. | 16:11 |
cardoe | Happy to expand on that. | 16:12 |
cardoe | But you could utilize something like mypy for testing then as well. | 16:12 |
adam-metal3 | I wanted to ask but for some reason I had the meeting time in my calendar shifted one hour, is there anyone who might have read the disk partition encryption proposal, are there any thoughts about it? | 16:15 |
chris218 | Hmm since when does sushy-tools have nova support? | 17:08 |
chris218 | It uses "libvirt:pxe-first" metadata attribute which never was implemented and was used here: https://opendev.org/openstack/openstack-virtual-baremetal/src/branch/stable/2.0/patches/nova/nova-pxe-boot.patch | 17:11 |
TheJulia | ... OVB | 17:12 |
TheJulia | Maybe talk to stevebaker | 17:12 |
TheJulia | chris218: or hjensas, stevebaker did it to support test VMs with nova backing | 17:54 |
chris218 | tbh it's not an issue for me I just found it and thought you guys might want to know | 18:03 |
TheJulia | oh, so I *think* it needs to be used with an ipxe base image which forces loading into it | 18:09 |
chris218 | nah ipxe base image works without it, that metadata parameter does nothing there's no even mention of it in nova source code | 18:25 |
chris218 | you need that 3 line patch linked above to make it work with libvirt but it never got into upstream | 18:26 |
TheJulia | chris218: oh yeah, I know it would work without it :) | 19:06 |
TheJulia | because it is ipxe in a disk | 19:06 |
TheJulia | anyway, stevebaker[m] ^^ | 19:07 |
cid | A little runbook demo video: https://youtu.be/00PJS4SXFYQ. I thought it might help with review. | 19:46 |
opendevreview | cid proposed openstack/ironic master: Use gerunds for task headings https://review.opendev.org/c/openstack/ironic/+/925129 | 19:46 |
cid | o/ | 19:49 |
opendevreview | Doug Goldstein proposed openstack/sushy master: avoid MAC addresses that are the empty string https://review.opendev.org/c/openstack/sushy/+/925130 | 19:50 |
cardoe | How about that dtantsur ^ | 19:50 |
cardoe | Are there some better examples of RAID configuration somewhere. Tried to do something like... {"size_gb": 10000, "number_of_physical_disks": 12, "raid_level": "1+0", "is_root_volume": false, "disk_type": "ssd"} I'm only able to set size_gb to the size of a singular disk. | 19:53 |
JayF | Cid: is that linked in the change? | 19:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!