| *** dtantsur_ is now known as dtantsur | 00:08 | |
| opendevreview | Takashi Kajinami proposed openstack/ironic-inspector master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic-inspector/+/915773 | 01:13 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/ironic master: db: Replace use of Engine.execute() method https://review.opendev.org/c/openstack/ironic/+/915789 | 02:48 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-inspector master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic-inspector/+/915773 | 02:52 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic/+/915792 | 02:56 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: db: Replace use of Engine.execute() method https://review.opendev.org/c/openstack/ironic/+/915789 | 05:11 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic/+/915792 | 05:11 |
| opendevreview | Takashi Kajinami proposed openstack/ironic-inspector master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic-inspector/+/915773 | 06:01 |
| rpittau | good morning ironic! o/ | 06:51 |
| opendevreview | Adam Rozman proposed openstack/ironic-python-agent master: add quiet cleanup option https://review.opendev.org/c/openstack/ironic-python-agent/+/915825 | 06:54 |
| opendevreview | Jacob Anders proposed openstack/ironic master: Add states.SERVICING and SERVICEWAIT to _FASTTRACK_HEARTBEAT_ALLOWED https://review.opendev.org/c/openstack/ironic/+/915389 | 08:12 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: db: Replace use of Engine.execute() method https://review.opendev.org/c/openstack/ironic/+/915789 | 08:27 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic/+/915792 | 08:27 |
| opendevreview | Merged openstack/ironic master: Fix spelling reported by codespell https://review.opendev.org/c/openstack/ironic/+/915457 | 10:01 |
| opendevreview | Merged openstack/ironic master: Make codespell voting https://review.opendev.org/c/openstack/ironic/+/915459 | 10:04 |
| opendevreview | Merged openstack/ironic stable/2024.1: Handle servicing failures in the Redfish BIOS interface https://review.opendev.org/c/openstack/ironic/+/915530 | 10:31 |
| iurygregory | good morning Ironic o/ | 10:33 |
| dtantsur | Hey folks, please review two more servicing fixes https://review.opendev.org/c/openstack/ironic/+/915725 and https://review.opendev.org/c/openstack/ironic/+/915321 | 11:08 |
| iurygregory | ack, I will add to my list for reviews | 12:43 |
| opendevreview | Adam Rozman proposed openstack/ironic-python-agent master: add mixed matching of root device hints https://review.opendev.org/c/openstack/ironic-python-agent/+/915858 | 12:55 |
| TheJulia | good morning | 12:55 |
| JayF | o/ | 14:59 |
| rpittau | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Apr 15 15:00:10 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 |
| JayF | o/ | 15:00 |
| rpittau | hello everyone and welcome to our weekly meeting! | 15:00 |
| masghar | o/ | 15:00 |
| rpittau | The meeting agenda can be found here: | 15:00 |
| rpittau | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:00 |
| rpittau | wrong copy-paste of course | 15:00 |
| rpittau | here | 15:00 |
| rpittau | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_April_15th_2024 | 15:01 |
| TheJulia | o/ | 15:01 |
| rpittau | let's wait 2 minutes for people to join | 15:02 |
| dtantsur | o/ | 15:02 |
| TheJulia | The week after the PTG is often light in attendance | 15:02 |
| rpittau | that's true | 15:03 |
| dtantsur | Ironic overdose | 15:03 |
| rpittau | there's not a lot to discuss today, let's start | 15:03 |
| rpittau | #topic Announcements / Reminders | 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: https://tinyurl.com/ironic-weekly-prio-dash | 15:03 |
| rpittau | #info Project Teams Gathering (PTG) 2024.2 topics and discussions https://etherpad.opendev.org/p/ironic-ptg-april-2024 | 15:03 |
| rpittau | I left the link of the PTG for this week in case someone whatns to review the topics ^ | 15:04 |
| rpittau | #info Ironic Meetup/BareMetal SIG June 5, OpenInfra Days June 6 @ CERN. Signup at https://indico.cern.ch/event/1378171/ and https://indico.cern.ch/event/1376907/ | 15:04 |
| rpittau | anything else to announce/remind ? | 15:04 |
| rpittau | oh wait, btw I'm working on the work items for 2024.2, I'll ahve something up tomorrow | 15:05 |
| dtantsur | I haven't followed up on my items yet - we're in a regular pre-freeze panic here | 15:05 |
| rpittau | dtantsur: I feel the cold :) | 15:06 |
| dtantsur | Unfinished tasks, broken CI, customer escalations, you know the business | 15:06 |
| rpittau | I do know very well | 15:06 |
| rpittau | I'd like to write a summary of the PTG too, hopefully before wednesday, let's see where we are in the next days | 15:07 |
| rpittau | anyway, moving on | 15:07 |
| rpittau | #topic Review Ironic CI status | 15:07 |
| iurygregory | o/ | 15:07 |
| * iurygregory a bit late | 15:07 | |
| rpittau | #info ironic-status upgrade check is broken with SQLALchemy 2.0 https://bugs.launchpad.net/ironic/+bug/2061345 | 15:07 |
| rpittau | you probably saw the thread in openstack-discuss about ^ | 15:07 |
| rpittau | we have people working on that, so it's kind of under control | 15:08 |
| TheJulia | I have not, but good to know | 15:08 |
| TheJulia | \o/ | 15:08 |
| rpittau | :) | 15:08 |
| rpittau | anything else on CI ? | 15:08 |
| JayF | I just landed the related patch for that bug jfyi | 15:09 |
| rpittau | oh great, thanks JayF | 15:09 |
| rpittau | onwards! | 15:09 |
| rpittau | I have one topic that was discussed already, but just to clarify if that's anything else we missed, and it's | 15:10 |
| rpittau | #topic rootwrap to privsec migration | 15:10 |
| rpittau | #info preparation work has been done in Ironic and IPA to remove any support for privilege escalation | 15:10 |
| * dtantsur prepares a blowtorch | 15:10 | |
| rpittau | #info now we need to remove run_as_root from ironic-lib | 15:10 |
| rpittau | lol | 15:10 |
| rpittau | yeah | 15:10 |
| dtantsur | I think inspector has rootwrap, but since it's deprecated... | 15:11 |
| rpittau | do we bother ? | 15:11 |
| dtantsur | exactly :) | 15:11 |
| rpittau | meh | 15:11 |
| rpittau | alrighty, anything else we want to discuss today? | 15:12 |
| JayF | I mean, doesn't hurt ot file a low-hanging-fruit bug about it | 15:12 |
| rpittau | JayF: sounds good to me | 15:12 |
| dtantsur | Won't hurt, but it's probably not quite low-hanging | 15:13 |
| dtantsur | Given the contributions rate to inspector, I'd not be overly positive | 15:13 |
| rpittau | it probably needs more than some details | 15:13 |
| JayF | My main thought is wanting it documented that we know it needs doing, and why we meh at it :) | 15:13 |
| dtantsur | or even: if we suddenly have a new contributor looking for work to do, we have so many more useful items | 15:13 |
| rpittau | true ^ | 15:13 |
| JayF | so if someone is later doing a "wipe out rootwrap everywhere" we have something more than 3 lines in a meeting | 15:13 |
| JayF | moreso than looking for a piece of work someone can pick up and do (unless literally we're one of last consumers) | 15:14 |
| rpittau | we can have a very low priority ticket for tracking | 15:14 |
| JayF | ++ | 15:15 |
| dtantsur | ++ | 15:16 |
| rpittau | ok, I can create one | 15:16 |
| rpittau | after the meeting | 15:16 |
| rpittau | moving on? | 15:17 |
| rpittau | #topic bug deputy updates | 15:17 |
| rpittau | we didn't have bug deputy last week because of the PTG, any volunteer for this week? | 15:17 |
| rpittau | I could do it, just not sure how much time I'll have considering our downstream situation | 15:19 |
| iurygregory | I can in about 3 weeks =) | 15:19 |
| rpittau | iurygregory: exactly :D | 15:19 |
| iurygregory | yeah =) | 15:19 |
| dtantsur | Same, of course | 15:19 |
| JayF | "community" bug deputy for a couple of weeks | 15:19 |
| JayF | as we all re-combobulate? lol | 15:19 |
| rpittau | ok, no problem, I'll keep an eye on the bugs this week | 15:20 |
| rpittau | #topic RFE review | 15:21 |
| rpittau | no new RFEs if I'm not mistaken | 15:21 |
| rpittau | #topic Open Discussion | 15:21 |
| rpittau | last chance for an out of agenda discussion here :) | 15:21 |
| TheJulia | I've got nothing right now | 15:22 |
| rpittau | seems we're good | 15:22 |
| rpittau | thanks everyone! | 15:22 |
| rpittau | #endmeeting | 15:22 |
| opendevmeet | Meeting ended Mon Apr 15 15:22:37 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:22 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-04-15-15.00.html | 15:22 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-04-15-15.00.txt | 15:22 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-04-15-15.00.log.html | 15:22 |
| JayF | In about 90 mins, cid and I are pairing on ARM CI | 15:23 |
| JayF | I know some other folks said they wanted in (rpittau?) | 15:23 |
| rpittau | JayF: yes please! | 15:24 |
| JayF | IDR who else, but ping me and I'll add you o/ | 15:24 |
| rpittau | I think it was dtantsur ^ | 15:25 |
| dtantsur | PSA: gertty still does not work with SQLAlchemy 2.0. If you see quick crashes, it's the reason. | 15:25 |
| dtantsur | JayF: yes, it was me | 15:25 |
| dtantsur | Also, I'm quite seriously considering buying a Raspberry PI to put Bifrost on it :D | 15:25 |
| JayF | I have a suggestion that's better than rpi, if you have a couple extra bucks? | 15:26 |
| dtantsur | Depends on the amount of extra bucks :) | 15:26 |
| JayF | rpis are a little difficult to get ahold of, but if you don't need something quite that small, you can get passively cooled old x86 laptop chipsets in a machine from alixpress | 15:26 |
| rpittau | dtantsur: I think you will need a model with some RAM, at least rpi4 with 8GB | 15:26 |
| dtantsur | x86 is something I very explicitly do not want | 15:27 |
| JayF | ahha, okay | 15:27 |
| JayF | I got a $40 chromebook for all my arm toying :D | 15:27 |
| dtantsur | I have no shortage of x86 hardware accessible to me | 15:27 |
| JayF | but I doubt that will run bifrost lol | 15:27 |
| dtantsur | :D | 15:27 |
| dtantsur | yeah, I have some hopes for the 8G model | 15:27 |
| rpittau | I had the same thought btw, I'm going to buy one as I need one new for other.... reasons, so I can probably also give it a try | 15:28 |
| dtantsur | they seem to have them in stock here nowadays | 15:28 |
| JayF | I'm curious what the goal is | 15:28 |
| rpittau | yes, they're wasy to find in EU | 15:28 |
| JayF | like, why arm hardware over a vM? | 15:28 |
| dtantsur | You mean, over an ARM VM? VM is cold and remote, I like things that can touched :) | 15:29 |
| dtantsur | Also, nested virt on ARM... is it even a good idea? | 15:29 |
| JayF | who's going to find out, if not us | 15:30 |
| JayF | and "nested virt" can be disabled in favor of pure qemu | 15:30 |
| JayF | which is how I suspect CI on x86 hosts with ARM vms being provisioned will be shaped to begin | 15:30 |
| dtantsur | yeah, I mean: any VM inside VM, with hardware assistance or not | 15:32 |
| TheJulia | ... I think the newest PIs have the arm chips which support nested virt | 15:33 |
| TheJulia | *think* | 15:33 |
| dtantsur | meanwhile, someone with a Huawei machine has found me. Let's see what it has in terms of Redfishing. | 15:33 |
| TheJulia | row row row | 15:33 |
| dtantsur | Network boot seems to work but they need virtual media. | 15:33 |
| TheJulia | In context of redfish, what is ironic in relation to it? Is it a fisher, or a boat, or ..... *brain explodes* | 15:34 |
| dtantsur | Of course it has a standard compatibility problem, why wouldn't it | 15:34 |
| TheJulia | dtantsur: As foretold in the sacred scrolls. | 15:35 |
| dtantsur | \o/ | 15:35 |
| JayF | TheJulia: as it was written in the scrolls: one fish two fish; red fish ironic fish | 15:36 |
| JayF | Dpm | 15:36 |
| JayF | **Go talk to Mr Geisel if you don't like it ;) | 15:36 |
| TheJulia | JayF: just stop before we get to cylons | 15:36 |
| dtantsur | Huawei has a non-standard link from System to Manager (Redfish uses Links/ManagedBy, they have Links/Manager). | 15:38 |
| dtantsur | It seems to have a virtual media resource even, we just cannot reach it... | 15:38 |
| opendevreview | Verification of a change to openstack/ironic-inspector master failed: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic-inspector/+/915773 | 15:50 |
| opendevreview | Jay Faulkner proposed openstack/ironic-inspector master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic-inspector/+/915773 | 15:54 |
| JayF | fyi updated commit msg and re-landed ^ | 15:55 |
| opendevreview | Verification of a change to openstack/ironic master failed: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic/+/915792 | 15:59 |
| opendevreview | Merged openstack/ironic master: db: Replace use of Engine.execute() method https://review.opendev.org/c/openstack/ironic/+/915789 | 16:11 |
| opendevreview | Alexon Ferreira de Oliveira proposed openstack/ironic master: Changed to the draft-04 schema because other schemas in the same code utilize it https://review.opendev.org/c/openstack/ironic/+/915909 | 16:31 |
| opendevreview | Alexon Ferreira de Oliveira proposed openstack/ironic master: Changed to the draft-04 schema because other schemas in the same code utilize it https://review.opendev.org/c/openstack/ironic/+/915909 | 16:32 |
| opendevreview | Alexon Ferreira de Oliveira proposed openstack/ironic master: Changed to the draft-04 schema because other schemas in the same code utilize it https://review.opendev.org/c/openstack/ironic/+/915909 | 16:33 |
| opendevreview | Merged openstack/networking-generic-switch master: Add python3.10 support in testing runtime https://review.opendev.org/c/openstack/networking-generic-switch/+/902549 | 16:39 |
| opendevreview | Verification of a change to openstack/bifrost stable/zed failed: ironic: Perform online data migrations with localhost DB https://review.opendev.org/c/openstack/bifrost/+/901298 | 16:53 |
| damex | hi, does iconic support provision with a use of vpro/amt ? i haven't found full list of compatible features/platforms so far but i see ' but Ironic also supports vendor-specific plugins which may implement additional functionality.' that's from https://wiki.openstack.org/wiki/Ironic | 17:04 |
| dtantsur | JayF: is the call still on? sorry, lost track of the time | 17:07 |
| JayF | jump on in if you want | 17:07 |
| JayF | I think I added you to inv? | 17:07 |
| dtantsur | trying.. | 17:07 |
| JayF | https://meet.google.com/dep-ruau-ghj?authuser=0. | 17:07 |
| dtantsur | JayF: I'm there, nobody is letting me in :) | 17:08 |
| JayF | what goog acct email | 17:08 |
| JayF | I will try adding that to the invite | 17:08 |
| JayF | moderator for this gmeet is not on the call | 17:09 |
| JayF | we're using the MLH one b/c zoom doesn't like cid's internet | 17:09 |
| dtantsur | JayF: use dtantsur@redhat.com then | 17:09 |
| opendevreview | Merged openstack/metalsmith master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/metalsmith/+/911874 | 17:10 |
| JayF | done, hopefully it works :/ | 17:11 |
| TheJulia | damex: so ages ago we had a out of tree driver for AMT... like version 3 through 6 (or maybe it was 6-9) but intel kept changing the design and basically kept breaking the interface without any backwards support so there was no critical mass behind it. The question I would try to ask is "is there redfish support" in the hardware, since Redfish is the standard most vendors actively support | 17:12 |
| damex | TheJulia: is support unique per amt version? i have fleets of cheap'ish hardware within 10 inch racks of lenovo tiny machines for homelab and other reasons. i thought about using iconic for provisioning them. https://openwsman.github.io/ this one supports modern intel vpro/amt hardware and one can control machine state/change settings and etc. i would expect any proper server platform would just be controlled with ipmitool | 17:17 |
| opendevreview | Merged openstack/ironic stable/2023.2: Marking metalsmith legacy job as non-voting https://review.opendev.org/c/openstack/ironic/+/915728 | 17:18 |
| opendevreview | Merged openstack/ironic-inspector master: Update regex to detect closed branch https://review.opendev.org/c/openstack/ironic-inspector/+/912735 | 17:18 |
| TheJulia | damex: unfortunately AMT has a ton of breaking and unsupportable differences under a marketing banner of "AMT". It wouldn't be terribly difficult to wire up a driver or maybe update what is posted out there (https://opendev.org/x/ironic-staging-drivers/src/branch/master/ironic_staging_drivers/amt) but I remember we looked at it as a community and the breaking to DIY it without some sort of intermediate library were just a | 17:21 |
| TheJulia | barrier for entry for us | 17:21 |
| TheJulia | Proper server platform wise, most hardware vendors are kicking IPMI entirely to the curb in favor of Redfish at this point | 17:22 |
| opendevreview | Verification of a change to openstack/bifrost master failed: CI: Modify upgrade jobs to upgrade from 2024.1 (Caracal) https://review.opendev.org/c/openstack/bifrost/+/913679 | 17:23 |
| damex | at the last job we had lots of server purchases (lots of random vendors) and in the recent years they all came with redfish. they all also supported ipmitool reasonably well. only think one couldn't really do over ipmitool is to update bits and pieces of their firmware (insecure thing mostly because of redfish). its not like old supermicro servers that have all their functionality implemented with ipmitool :) | 17:25 |
| TheJulia | well, even supermicro got to a point where they decided to stop trying to keep ipmi working perfectly forever :) | 17:27 |
| TheJulia | ipmitool had no official mechanism for virtual media as well. Well, there is the in theory mailbox stuff, but that is not in the client and had something like a 28 character limit | 17:27 |
| damex | what is the future of https://opendev.org/x/ironic-staging-drivers/src/branch/master/ironic_staging_drivers/amt ? is it gonna be thrown under the bus? | 17:28 |
| damex | and what is the requirements for such thing to be successfully used with 'ironic'? just implement control of power states? | 17:29 |
| TheJulia | There really is no future of that driver at that point, and there hasn't been for many years. Last time I saw hardware it would "work" with for sale was... 6 or 7 years ago. | 17:29 |
| TheJulia | minimum requirement is power state and boot device. You can do boot mode separately, but modern ironic also defaults to everything is UEFI | 17:30 |
| damex | i took a look at the repo and think that my current m920q/m920x/p330 might work. have like dozens of them :) | 17:30 |
| damex | TheJulia: i don't think there is an implementation for boot device control >_> | 17:31 |
| damex | in amt | 17:31 |
| TheJulia | damex: https://opendev.org/x/ironic-staging-drivers/src/branch/master/ironic_staging_drivers/amt/management.py#L79 <-- That seems a bit overly complex in a private method | 17:33 |
| opendevreview | Alexon Ferreira de Oliveira proposed openstack/ironic master: Remove deprecation warning by setting schema https://review.opendev.org/c/openstack/ironic/+/915909 | 17:34 |
| TheJulia | I do remember a discussion or two, from maybe 8 or 9 years ago where there were challenges with boot mode, I just don't remember the details at htis point | 17:34 |
| opendevreview | Alexon Ferreira de Oliveira proposed openstack/ironic master: Remove deprecation warning by setting schema https://review.opendev.org/c/openstack/ironic/+/915909 | 17:36 |
| damex | TheJulia: main problem there is that you don't just set boot order. it gets set in specific mode that is related to boot mode and then there is other things to worry about. like retail mode/provision mode/whatever else they call ti mode and then there is boot order within each one of them. maybe one can just set whole thing at a time like a config or something >_> | 17:36 |
| TheJulia | oh, well, yeah | 17:36 |
| damex | efibootmgr have better time controlling that thing :) | 17:37 |
| TheJulia | If you create your own, or hack your own updated driver, boot mode is also highly encouraged. | 17:37 |
| TheJulia | actually, that is a good point | 17:37 |
| TheJulia | lenovo hardware is notorious for not liking multiple changes | 17:37 |
| TheJulia | otherwise it considers the nvram inalidated and rolls back to a known good copy | 17:38 |
| TheJulia | invalidated | 17:38 |
| TheJulia | https://review.opendev.org/q/Ic6584479a47146577052d17fa3f697eef64ac73c is the most recent fix we made to ironic | 17:38 |
| TheJulia | we've had a few others over the years of similar headache | 17:39 |
| opendevreview | Merged openstack/ironic-tempest-plugin master: Leverage the remote microversion without requiring human config https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/914270 | 17:43 |
| damex | i usually make pxeboot as a default on machines like that and then just never touch boot order ever again. just make sure from monitoring/whatever that it is set and can boot to pxe. then grub that it receives over pxe decies if one boots from whatever is installed on local media or provision is needed (decided before machine boots over pxe. just reboot and get new settings). | 17:43 |
| damex | TheJulia: can i do something like that with ironic? | 17:44 |
| opendevreview | Verification of a change to openstack/metalsmith stable/2024.1 failed: Update .gitreview for stable/2024.1 https://review.opendev.org/c/openstack/metalsmith/+/911872 | 17:46 |
| opendevreview | Verification of a change to openstack/metalsmith stable/2024.1 failed: Update TOX_CONSTRAINTS_FILE for stable/2024.1 https://review.opendev.org/c/openstack/metalsmith/+/911873 | 17:46 |
| TheJulia | damex: you can, you might want to look at the "ramdisk" deploy_interface. | 17:46 |
| damex | TheJulia: thanks, i will check it out :) | 17:46 |
| TheJulia | damex: just be mindful, shim, which is the signed UEFI loader, auto-injects itself so if you end up in a disk image uefi booting case, it might reset your boot order | 17:47 |
| damex | TheJulia: i don't think it will do something on its own but modification of boot order will definitely be attempted at the end of provision. i think one can mount/remount efivarfs readonly to protect against that :) | 17:52 |
| TheJulia | shim does it directly before an OS even fires up, unfortunately | 17:52 |
| TheJulia | but ipxe doesn't do any of that | 17:53 |
| opendevreview | Alexon Ferreira de Oliveira proposed openstack/ironic master: Remove deprecation warning by setting schema https://review.opendev.org/c/openstack/ironic/+/915909 | 17:54 |
| opendevreview | Merged openstack/ironic-inspector master: Remove SQLAlchemy tips jobs https://review.opendev.org/c/openstack/ironic-inspector/+/915773 | 18:07 |
| opendevreview | Merged openstack/networking-baremetal stable/2023.1: don't force amqp_auto_delete for quorum queues https://review.opendev.org/c/openstack/networking-baremetal/+/911612 | 18:21 |
| opendevreview | Merged openstack/networking-baremetal stable/2023.2: don't force amqp_auto_delete for quorum queues https://review.opendev.org/c/openstack/networking-baremetal/+/911611 | 18:25 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a restart in order to apply a patch update | 18:36 | |
| samcat116 | Probably a can of worms question: but as a heavy user of the SNMP driver, would there be appetite for a redfish-pdu driver? It seems like a lot of newer PDUs claim to support that and I imagine that would be more reliable for power control. | 18:47 |
| JayF | The idea of "should we support PDUs that expose redfish" is very yes | 18:48 |
| JayF | the idea of it being a separate driver... I'm less convinced of? | 18:48 |
| iurygregory | agree ^ | 18:49 |
| iurygregory | unless there is something that will really require a new driver for us to support the use case I would say | 18:49 |
| TheJulia | we would likely need something to denote "there is no system here", so maybe for redfish it could be acceptable to have no Management Interface loaded | 18:50 |
| TheJulia | so you end up with Power only | 18:50 |
| TheJulia | its a weird delineation | 18:50 |
| TheJulia | but might make sense as a separate sort of meta thing too | 18:50 |
| samcat116 | there's a separate pdu standard it seems | 18:51 |
| samcat116 | https://www.dmtf.org/sites/default/files/standards/documents/DSP2056_1.0.0.pdf | 18:51 |
| samcat116 | But yeah I dont think it would make sense on the existing redfish driver, likely a power only redfish-pdu driver or something | 18:52 |
| TheJulia | ... that would be a distinctly different thing then | 18:52 |
| TheJulia | so I sort of see where samcat116 is going | 18:53 |
| TheJulia | samcat116: that being said, likely makes sense to wire that into the sushy library *as* well and kind of follow the same pattern | 18:58 |
| JayF | why can't it be a powerfish | 19:02 |
| JayF | or a purplefish | 19:02 |
| JayF | something other than "redfish, except not" :| | 19:02 |
| TheJulia | We look at redfish as a system oriented thing | 19:02 |
| TheJulia | the reality is more gray, as if we're standing between the shadow and the light (or cat as human and keyboard) | 19:03 |
| TheJulia | ((or as londo would say, ducks) | 19:03 |
| TheJulia | ) | 19:03 |
| samcat116 | Redfish for air conditioning! https://www.dmtf.org/sites/default/files/standards/documents/DSP2064_1.0.0.pdf | 19:07 |
| TheJulia | OHHHHHHH AHHHHHHH | 19:08 |
| TheJulia | .... I feel like this may need to be implemented for my motorhome! | 19:08 |
| samcat116 | "You wouldn't PXE boot a space heater" | 19:09 |
| TheJulia | This is where my wife would chime in "Challenge Accepted" | 19:10 |
| TheJulia | But, I am not my wife. | 19:10 |
| iurygregory | WOOT | 19:45 |
| iurygregory | for air conditioning?! | 19:45 |
| iurygregory | OMG I want :D | 19:45 |
| JayF | with sushy-tools and the pyscript integration for homeassistant, anything can be redfish managed if you write some glue code ;) | 19:47 |
| TheJulia | "Sorry wifey, we decided to re-deploy the thermostat.... and the networking blipped while it was booting. Turns out their boot loader..." | 20:20 |
| JayF | Heh. The ultimate two-sided sword of home automation: "Why do you do that useless stuff anyway?" vs "Hey, $thing you setup stopped working" and there's not much in-between :P | 20:21 |
| TheJulia | "hold on, let me short circuit the wires together so it runs" | 20:25 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/913793 should be trivial to approve, already has +2 from me and a +1 from Takashi | 22:13 |
| opendevreview | Merged openstack/ironic master: Remove deprecation warning by setting schema https://review.opendev.org/c/openstack/ironic/+/915909 | 22:43 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: update redfish docs to detail swift url issues https://review.opendev.org/c/openstack/ironic/+/915931 | 22:53 |
| opendevreview | Julia Kreger proposed openstack/ironic master: redfish: change default virtual media storage to local storage https://review.opendev.org/c/openstack/ironic/+/915932 | 22:53 |
| opendevreview | cid proposed openstack/ironic master: Load machine type from command-line https://review.opendev.org/c/openstack/ironic/+/915934 | 23:29 |
| opendevreview | cid proposed openstack/ironic master: Load machine type from command-line https://review.opendev.org/c/openstack/ironic/+/915934 | 23:31 |
| opendevreview | Merged openstack/ironic master: Add note regarding metal3 ci job in CI config for stable runs https://review.opendev.org/c/openstack/ironic/+/910536 | 23:37 |
| opendevreview | Merged openstack/ironic master: Fix servicing clean-up https://review.opendev.org/c/openstack/ironic/+/915321 | 23:37 |
| opendevreview | Merged openstack/ironic master: Fix the confusion around service_reboot/servicing_reboot https://review.opendev.org/c/openstack/ironic/+/915725 | 23:38 |
| opendevreview | Merged openstack/ironic master: Use the new microversion-parse library to parse the version headers https://review.opendev.org/c/openstack/ironic/+/913793 | 23:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!