| *** JasonF is now known as JayF | 00:01 | |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/yoga: [ci] [stable-only] Cinder fixed; make BFV job vote https://review.opendev.org/c/openstack/ironic/+/884221 | 00:05 |
|---|---|---|
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Handle duplicate node inventory entries per node https://review.opendev.org/c/openstack/ironic/+/884608 | 05:38 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Handle duplicate node inventory entries per node https://review.opendev.org/c/openstack/ironic/+/884608 | 05:45 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: Handle duplicate node inventory entries per node https://review.opendev.org/c/openstack/ironic/+/884608 | 06:18 |
| rpittau | good morning ironic! o/ | 06:59 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/zed: Add support auth protocols for iRMC https://review.opendev.org/c/openstack/ironic/+/885231 | 07:44 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/2023.1: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885233 | 08:11 |
| opendevreview | Vanou Ishii proposed openstack/ironic bugfix/21.3: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885234 | 08:25 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/zed: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885235 | 08:41 |
| opendevreview | Vanou Ishii proposed openstack/ironic bugfix/20.2: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885237 | 09:00 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/yoga: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885238 | 09:11 |
| opendevreview | Vanou Ishii proposed openstack/ironic bugfix/19.0: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885239 | 09:18 |
| dtantsur | Team, looking for reviews on the first step towards inspector merger: https://review.opendev.org/c/openstack/ironic/+/875944 | 09:21 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/xena: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885242 | 09:24 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic stable/xena: [Stable-only] Remove Docker from devstack dependencies https://review.opendev.org/c/openstack/ironic/+/885243 | 09:29 |
| opendevreview | Vanou Ishii proposed openstack/ironic bugfix/18.1: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885244 | 09:30 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Remove traces of Docker from devstack dependencies https://review.opendev.org/c/openstack/ironic/+/885245 | 09:31 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/wallaby: [iRMC] Fix parse_driver_info bug enforcing SNMP v3 under FIPS mode https://review.opendev.org/c/openstack/ironic/+/885246 | 09:31 |
| dtantsur | TheJulia: hey, are you planning on backporting https://review.opendev.org/c/openstack/ironic/+/882609 and follows-up? I definitely see the race in stable/zed | 09:47 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/2023.1: [iRMC] Fix IPMI incompatibility handling error https://review.opendev.org/c/openstack/ironic/+/885275 | 10:19 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic master: Firmware Interface https://review.opendev.org/c/openstack/ironic/+/885276 | 11:26 |
| dtantsur | folks, please review https://review.opendev.org/c/openstack/ironic/+/885243/ it's for the Xena CI | 11:31 |
| opendevreview | Vanou Ishii proposed openstack/ironic bugfix/21.3: [iRMC] Fix IPMI incompatibility handling error https://review.opendev.org/c/openstack/ironic/+/885277 | 11:32 |
| iurygregory | good morning Ironic | 11:39 |
| opendevreview | Merged openstack/ironic-ui master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-ui/+/882662 | 11:51 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/zed: [iRMC] Fix IPMI incompatibility handling error https://review.opendev.org/c/openstack/ironic/+/885282 | 12:34 |
| opendevreview | Merged openstack/ironic stable/zed: Handle MissingAttributeError when using OOB inspections to fetch MACs https://review.opendev.org/c/openstack/ironic/+/882523 | 13:13 |
| opendevreview | Verification of a change to openstack/ironic master failed: Remove instance image after deployment https://review.opendev.org/c/openstack/ironic/+/878680 | 13:13 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM Enable OVN https://review.opendev.org/c/openstack/ironic/+/885087 | 13:32 |
| iurygregory | dtantsur, just saw your comment in the DB API and Object patch, maybe we can do like in the BIOS approach on the driver level, I've added a comment in the patch. Tks for the review! | 14:01 |
| dtantsur | I'll need to look up what exactly BIOS does.. | 14:03 |
| dtantsur | iurygregory: I'm not really fond of doing it in Python instead of in a database transaction, but I guess I can live with it | 14:05 |
| iurygregory | at the low level still a DB transaction no? <insert thinking face> | 14:09 |
| iurygregory | or you would expect the driver directly calling the DB? | 14:10 |
| dtantsur | iurygregory: BIOS settings does everything in Python: first fetching, then filtering, then creating/updating/deleting all separately. | 14:10 |
| dtantsur | Thanks to the Ironic-level node lock, it's not racy. | 14:10 |
| iurygregory | yeah | 14:11 |
| iurygregory | I will go with this path and push an update in the patches | 14:13 |
| iurygregory | if we are ok with it | 14:13 |
| TheJulia | good morning | 14:35 |
| iurygregory | good morning TheJulia | 14:36 |
| opendevreview | Merged openstack/ironic master: Prepare [inspector]require_managed_boot to change to True in the future https://review.opendev.org/c/openstack/ironic/+/877470 | 14:37 |
| TheJulia | dtantsur: regarding your question early this morning, I have no plans to do it myself at this time | 14:51 |
| dtantsur | k, got it (and good morning) | 14:51 |
| TheJulia | now, anyone have any good ideas to parse a dict payload printed to str in a api response body from a service? | 14:53 |
| iurygregory | the json lib in python doesn't help? | 14:56 |
| iurygregory | json.loads if I recall | 14:56 |
| iurygregory | the api returns a payload as a single string? instead of a json | 14:58 |
| JayF | json lib will barf on the u'blah' | 14:58 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM Enable OVN https://review.opendev.org/c/openstack/ironic/+/885087 | 14:58 |
| JayF | ast.literal_eval(string), if the dict is trusted, apparently | 14:59 |
| TheJulia | I think ^ will work | 14:59 |
| TheJulia | since it works locally if I do it | 14:59 |
| TheJulia | well, the pattern at least | 14:59 |
| iurygregory | meeting \o/ | 15:00 |
| * TheJulia wonders how much coffee iurygregory has had :) | 15:01 | |
| iurygregory | only 2 | 15:01 |
| JayF | #startmeeting ironic | 15:01 |
| opendevmeet | Meeting started Mon Jun 5 15:01:47 2023 UTC and is due to finish in 60 minutes. The chair is JayF. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:01 |
| JayF | I'm glad you all said something, I'm halfway thru my cup and missed it was 8am | 15:02 |
| JayF | #topic roll call | 15:02 |
| JayF | who all is here today? | 15:02 |
| iurygregory | o/ | 15:02 |
| matfechner | o/ | 15:02 |
| JayF | We don't have a lot of unique items on the agenda. I'm going to move quickly, please say something if you need me to stop for more input | 15:02 |
| JayF | #topic Announcements | 15:02 |
| JayF | #note 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 |
| JayF | No action items from last meeting; skipping agenda item. | 15:03 |
| JayF | #topic Review Ironic CI Status | 15:03 |
| JayF | I don't have a view of the whole thing, but it seems like we've been moving slowly towards more reliablity? | 15:04 |
| iurygregory | stable branches should be better since we merged the cinder changes (xena/ wallaby) probably | 15:05 |
| JayF | we need to make sure we follow up and re-enable any disabled rbac/bfv jobs | 15:05 |
| JayF | I have some changes up for that already | 15:05 |
| iurygregory | +1 | 15:05 |
| rpittau | o/ | 15:05 |
| JayF | Aight, moving on | 15:05 |
| JayF | #topic 2023.2 Workstreams | 15:06 |
| JayF | #link https://etherpad.opendev.org/p/IronicWorkstreams2023.2 | 15:06 |
| JayF | We loosely track status of work this cycle in this etherpad. | 15:06 |
| JayF | Thanks for those who have updated it, it looks like we're making good progress. | 15:06 |
| JayF | Anything else on 2023.2 workstreams? | 15:07 |
| JayF | #topic Open Discussion | 15:07 |
| iurygregory | I've updated the Firmware Interface in the etherpad | 15:07 |
| JayF | There is nothing on the written agenda for open discsusion. Is there anything anyone would like to chat about? | 15:07 |
| JayF | thanks iurygregory | 15:07 |
| JayF | One thing before we close up: next weekend many of us will be in Vancouver for PTG | 15:09 |
| JayF | I propose we cancel the Ironic meeting for next week as a result | 15:10 |
| iurygregory | +1 from me | 15:10 |
| JayF | motion carries :D | 15:10 |
| JayF | #endmeeting | 15:10 |
| opendevmeet | Meeting ended Mon Jun 5 15:10:46 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:10 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-06-05-15.01.html | 15:10 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-06-05-15.01.txt | 15:10 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2023/ironic.2023-06-05-15.01.log.html | 15:10 |
| TheJulia | wow, I blink away to do paperwork and the meeting is over | 15:10 |
| * TheJulia blinks again and hopes it is the end of the day | 15:11 | |
| JayF | TheJulia: I try to move 'em quick if we don't have anything contentious on the agenda | 15:11 |
| iurygregory | I wish I could blink and have code ready :D | 15:11 |
| JayF | don't wanna spend more than a few minutes basically performing ritual lol | 15:11 |
| iurygregory | but I'm not an AI LOL | 15:11 |
| JayF | chatgpt can write really bad code really fast | 15:12 |
| iurygregory | yes | 15:13 |
| JayF | I could write code fast, too, if I imagined a library existed that did all the stuff I needed it to lol | 15:13 |
| JayF | that was always the joke about python back in the day, right, "how to write an imap server in python: import imap; imap.start_server()" | 15:13 |
| TheJulia | I do <3 someone's response on social media recently "yes, everyone wants to focus on the metric of writing code... but the most important skill is more often removing code." | 15:15 |
| iurygregory | nice one! <3 | 15:15 |
| JayF | I'm just stuck on "all the concrete examples I've been shown of chatgpt writing anything beyond boilerplate, the code was nonworking and bad" | 15:17 |
| JayF | it doesn't even write code correctly (yet) | 15:17 |
| JayF | but the #1 reason chatgpt won't take our jobs: you have to be able to describe what you want for an AI to give it to you | 15:18 |
| TheJulia | yeah, someone has to turn the dream into reality and traditionally that is the engineer's role to fuse the knowledge of context in to achieve the goal | 15:20 |
| iurygregory | yup | 15:22 |
| *** dking is now known as Guest2258 | 15:23 | |
| iurygregory | you need clients to clear explain what they want | 15:23 |
| *** Guest2258 is now known as dking | 15:23 | |
| TheJulia | And we all know how that can go | 15:24 |
| iurygregory | yes =) | 15:24 |
| TheJulia | Although, a fun bug I got to dig into on ?Thursday? ... the customer actually sent me all of the information I needed to determine the issue | 15:24 |
| TheJulia | like... they proactively sent it | 15:24 |
| * TheJulia assumes someone, someplace, is thinking it went something like https://www.youtube.com/watch?v=rr6yT7uhTSY | 15:27 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM Enable OVN https://review.opendev.org/c/openstack/ironic/+/885087 | 15:41 |
| dtantsur | Could I get a 2nd +2 please? https://review.opendev.org/c/openstack/ironic/+/885243 | 15:52 |
| JayF | lookin | 15:53 |
| JayF | +2a | 15:53 |
| dtantsur | thx | 15:55 |
| TheJulia | iurygregory: added to things to the forum deck | 15:55 |
| TheJulia | two | 15:55 |
| iurygregory | ack, today I will be adding things to the etherpad | 15:57 |
| TheJulia | I haven't looked at it yet | 15:57 |
| TheJulia | but pre-seeding is always a good idea :) | 15:57 |
| rpittau | good night! o/ | 16:18 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Very basic in-band inspection with a new "agent" interface https://review.opendev.org/c/openstack/ironic/+/877814 | 16:37 |
| NobodyCam | Good morning Ironic Folks... Happy Monday | 16:47 |
| dking | NobodyCam: Happy Monday. | 16:57 |
| opendevreview | Merged openstack/ironic stable/2023.1: Handle MissingAttributeError when using OOB inspections to fetch MACs https://review.opendev.org/c/openstack/ironic/+/882522 | 17:01 |
| kaloyank | Hello everyone | 17:08 |
| JayF | o/ | 17:08 |
| kaloyank | JayF: I'm looking at the OpenConfig addition to the networking-baremetal project. Is the networking-baremetal project planned to supersede in one way or another the networking-generic-switch project? | 17:09 |
| JayF | Going to be honest, you're talking about projects under Ironic I'm the least familiar with. Which also means there aren't any big public plans because nobody has talked about it in context of PTG or similar | 17:09 |
| JayF | so I think no big plans ? | 17:10 |
| JayF | https://review.opendev.org/c/openstack/nova/+/883411 fyi, ironic driver change | 17:10 |
| JayF | fix lgtm and the bug reported around it looks real | 17:10 |
| kaloyank | Understood | 17:11 |
| JayF | those sort of questions are usually way better for the list | 17:13 |
| JayF | because you can get more eyes on it than those of us who are in irc | 17:13 |
| kaloyank | Makes sense to me, was just wondering if it was something I missed on | 17:14 |
| opendevreview | Merged openstack/ironic stable/xena: [Stable-only] Remove Docker from devstack dependencies https://review.opendev.org/c/openstack/ironic/+/885243 | 17:26 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM Enable OVN https://review.opendev.org/c/openstack/ironic/+/885087 | 18:31 |
| TheJulia | JayF: thoughts on just dropping the job w/r/t https://zuul.opendev.org/t/openstack/build/50956e0096b746f79bbfeaf5d648643b | 18:35 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM Enable OVN https://review.opendev.org/c/openstack/ironic/+/885087 | 20:31 |
| TheJulia | I think I'm getting close.... At least I hope I am | 20:32 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM Enable OVN https://review.opendev.org/c/openstack/ironic/+/885087 | 20:56 |
| TheJulia | and hopefully we will have debugigng output in an hour | 20:59 |
| JayF | TheJulia: it's an em branch, do what you need | 21:50 |
| iurygregory | JayF, have you chose the day for the ironic dinner? | 22:20 |
| iurygregory | oh ok, I see you will probably announce tomorrow (just found the etherpad) | 22:21 |
| opendevreview | Julia Kreger proposed openstack/ironic master: DNM Enable OVN https://review.opendev.org/c/openstack/ironic/+/885087 | 23:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!