| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Fix SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/856336 | 01:01 |
|---|---|---|
| opendevreview | Merged openstack/ironic master: Cleanup submitted SNMP driver code for additional PDUs https://review.opendev.org/c/openstack/ironic/+/840986 | 03:12 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/train: Fix deprecated SQL syntax on test-setup.sh https://review.opendev.org/c/openstack/ironic/+/857004 | 07:59 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/train: Fix deprecated SQL syntax in test-setup.sh https://review.opendev.org/c/openstack/ironic/+/857004 | 08:11 |
| opendevreview | Mark Goddard proposed openstack/tenks master: Follow ups to venv https://review.opendev.org/c/openstack/tenks/+/857008 | 08:43 |
| opendevreview | SONG SHUKUN proposed openstack/ironic master: [WIP] Add support auth protocols for iRMC https://review.opendev.org/c/openstack/ironic/+/857035 | 09:40 |
| iurygregory | good morning Ironic | 10:56 |
| TheJulia | good morning | 12:59 |
| dtantsur | morning iurygregory, TheJulia | 13:06 |
| dtantsur | TheJulia: back home safe? | 13:06 |
| smachi | Hi Julia, would you please review https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/848709 | 13:06 |
| TheJulia | yeah, safe and sound. Didn't *actually* need to evacuate. information got out of hand on Thursday evening | 13:06 |
| TheJulia | Rain from the hurricane helped a lot, but we're going to drive back Friday I think | 13:07 |
| dtantsur | great | 13:08 |
| TheJulia | so yeah, destroying my sleep cycle for a few days was not what I had planned | 13:10 |
| TheJulia | On friday, we got to our house around 3 AM. I unloaded the corgi and the feline overlords... woke up at 8 AM, went and got breakfast, fell back asleep at 10 AM until the mid-afternoon :( | 13:11 |
| dtantsur | :( | 13:15 |
| TheJulia | i did have quiet time to bang my head against sqlalchemy last night | 13:16 |
| TheJulia | well, yesterday afternoon | 13:18 |
| iurygregory | happy to hear you are safe TheJulia =) good morning | 13:59 |
| arne_wiebalck | Good morning, Ironic! | 13:59 |
| arne_wiebalck | iurygregory: I added https://review.opendev.org/c/openstack/ironic-python-agent/+/852999 to the weekly prio list (not sure anyone from our team will be available for the weekly meeting later today). | 14:00 |
| iurygregory | arne_wiebalck, sure! tks for adding | 14:00 |
| TheJulia | okay, two more sqlalchemy issues figured out | 14:06 |
| TheJulia | (... and truth be told, it was bugs in our code) | 14:06 |
| iurygregory | so we can probably say yay | 14:12 |
| TheJulia | 2 more to go and we should be able to remove the autocommit setting | 14:13 |
| opendevreview | Vanou Ishii proposed openstack/ironic stable/ussuri: Fix CI failure on Redfish Virtual Media on Ussuri https://review.opendev.org/c/openstack/ironic/+/857194 | 14:32 |
| iurygregory | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Sep 12 15:00:06 2022 UTC and is due to finish in 60 minutes. The chair is iurygregory. 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 |
| TheJulia | o/ | 15:00 |
| iurygregory | Hello ironicers, welcome to our weekly meeting o/ | 15:00 |
| matfechner | o/ | 15:00 |
| rloo | o/ | 15:00 |
| JayF | o/ | 15:00 |
| ajya | o/ | 15:00 |
| iurygregory | The agenda for today's meeting can be found in the wiki | 15:00 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:00 |
| iurygregory | #topic Announcements / Reminder | 15:01 |
| iurygregory | #info PTL and TC elections starting today till Sep 19. | 15:02 |
| iurygregory | to receive the email you had to opt-in for the civs pool | 15:02 |
| iurygregory | there was an email last week about it | 15:03 |
| iurygregory | #info Check PTG Topics | 15:03 |
| iurygregory | last week I've sent an email with the doodle so we can see a time slot to check the topics and try to group for our PTG | 15:04 |
| iurygregory | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-September/030406.html | 15:04 |
| iurygregory | we only have two people voting, so I will give time till wed this week | 15:04 |
| iurygregory | #link https://doodle.com/meeting/participate/id/dBL8ZZXa | 15:04 |
| iurygregory | this is the link for the doodle | 15:04 |
| iurygregory | #info Antelope PTG etherpad | 15:05 |
| iurygregory | #link https://etherpad.opendev.org/p/ironic-antelope-ptg | 15:05 |
| iurygregory | feel free to add topics for the PTG \o/ | 15:06 |
| dtantsur | o/ | 15:06 |
| iurygregory | and don't forget to register | 15:06 |
| iurygregory | #link https://openinfra-ptg.eventbrite.com/ | 15:06 |
| iurygregory | Does anyone have something that would like to add ? | 15:06 |
| rloo | (so exciting to have PTL elections for ironic...) | 15:07 |
| iurygregory | yeah :D | 15:07 |
| hjensas | o/ | 15:07 |
| iurygregory | ok, moving on | 15:08 |
| iurygregory | #topic Review subteam status reports | 15:08 |
| iurygregory | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:09 |
| iurygregory | starting around L91 | 15:10 |
| TheJulia | we ought to track sqlalchemy | 15:10 |
| TheJulia | or at least, start | 15:10 |
| TheJulia | It is going to take some work overall for 2.0 compatability | 15:10 |
| TheJulia | *but* we should be able to backport the autocommit fix | 15:11 |
| iurygregory | makes sense to me | 15:11 |
| iurygregory | happy to review the patches | 15:11 |
| TheJulia | I'll start a list sometime this week, but first a section. | 15:13 |
| iurygregory | tks TheJulia | 15:16 |
| iurygregory | #topic Deciding on priorities for the coming week | 15:18 |
| iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:18 |
| iurygregory | Does anyone have patches that would like to add to the list? last week I was focused on releases so this week I will review a bit more since the list started to grow | 15:19 |
| iurygregory | JayF, tks for working on the backports o/ | 15:20 |
| JayF | Np | 15:20 |
| TheJulia | I'll add the review priority flag to this sqlalchemy autocommit revert fixes patch once I've got it ready, but it might take a couple days. :\ | 15:22 |
| iurygregory | TheJulia, ack | 15:22 |
| iurygregory | #topic Baremetal SIG | 15:23 |
| iurygregory | so, I was talking with arne_wiebalck last week, we need to decide on the future of the SIG meetings (Arne probably can't attend upstream meetings because of some conflicts) | 15:25 |
| iurygregory | any thoughts on it? | 15:26 |
| iurygregory | normally we need to reach out to folks to see if they would have time to present something | 15:26 |
| TheJulia | my plate is overflowing at the moment | 15:26 |
| TheJulia | so my ability to organize is limited at the moment | 15:26 |
| JayF | I honestly wonder how sustainable it is to do monthly talks for the BM SIG | 15:26 |
| TheJulia | it works really well as a forum to get people talking from different orgs outside of the developer context | 15:27 |
| TheJulia | but someone needs to organize, and maybe we don't need presentations | 15:27 |
| TheJulia | but at that point, we're almost re-imagining some of the interaction style | 15:28 |
| iurygregory | hummm | 15:28 |
| JayF | Yeah; that's kinda what I'm getting at. The current model of "a talk then discussion about the talk" is not really sustainable | 15:28 |
| arne_wiebalck | so far only presentations achieved that people are joining and talking | 15:28 |
| JayF | Why don't we punt this to the mailing list? A longer form discussion might be valuable | 15:29 |
| arne_wiebalck | but it is not that people come forward by themselves | 15:29 |
| JayF | I can also ask the devrel professionals here at GR-OSS if they have ideas as to how to keep something like this going | 15:29 |
| iurygregory | JayF, that would be much appreciated I would say | 15:30 |
| JayF | ack; I'll take an action item to mail out to the list about the SIG and maybe find some ideas from our devrel folks | 15:30 |
| TheJulia | ++ punt to ml | 15:32 |
| iurygregory | ok, good \o/ tks JayF | 15:32 |
| iurygregory | we don't have any RFE, moving to discussion | 15:32 |
| iurygregory | #topic Open discussion | 15:33 |
| iurygregory | Does anyone have something that would like to discuss? | 15:33 |
| JayF | Just wanted to let folks know, I'll be starting up doing some bug triage this week. | 15:33 |
| JayF | Please help me out if I ask strange questions in here :D | 15:33 |
| JayF | and if I am feeling up to it; I might even live-stream some of it on twitch -- I'll post here + on my twitter if/when I do the streaming | 15:34 |
| iurygregory | oh wow | 15:34 |
| iurygregory | live-stream bug triage O.o | 15:34 |
| JayF | Performative open source, for people who want to distract from their boring job with my boring job ;) | 15:35 |
| iurygregory | lol | 15:35 |
| iurygregory | so if we don't think our job is boring we can't join? | 15:35 |
| JayF | I'm just making jokes. I actually think we do lots of cool stuff and it's way better than doing it behind closed doors :D | 15:36 |
| iurygregory | <insert sad emoji here> | 15:36 |
| iurygregory | yeah I know :D | 15:36 |
| rloo | I was thinking Tom Sawyer... | 15:36 |
| TheJulia | JayF: sounds like a great idea | 15:36 |
| * iurygregory has no idea who Tom Sawyer is... | 15:37 | |
| rloo | https://akitajet.com/wiki/Tom_Sawyer_Paints_the_Fence | 15:37 |
| rloo | bug triage is fun. no, you can't help out cuz... ha ha | 15:37 |
| iurygregory | tks for sharing rloo =) | 15:38 |
| dtantsur | :D | 15:38 |
| * dtantsur wishes someone could triage this customer bug for him... | 15:39 | |
| iurygregory | I can take a look later today =) | 15:39 |
| iurygregory | so I think that's all for today | 15:40 |
| iurygregory | tks everyone! | 15:40 |
| JayF | \o | 15:40 |
| iurygregory | #endmeeting | 15:40 |
| opendevmeet | Meeting ended Mon Sep 12 15:40:56 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:40 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-09-12-15.00.html | 15:40 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-09-12-15.00.txt | 15:40 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-09-12-15.00.log.html | 15:40 |
| ajya | iurygregory: re tempest+bifrost, there was a patch mentioned at https://etherpad.opendev.org/p/IronicWhiteBoard#L378 Was it ever pushed? Still wondering what would be necessary to get it working. | 15:42 |
| opendevreview | Merged openstack/ironic master: Enables event subscription methods for ilo and ilo5 hardware types https://review.opendev.org/c/openstack/ironic/+/853690 | 15:49 |
| TheJulia | one unit test \o/ | 15:56 |
| opendevreview | Merged openstack/sushy stable/train: Do not send compression as acceptable encoding https://review.opendev.org/c/openstack/sushy/+/855555 | 16:14 |
| dtantsur | see you tomorrow folks o/ | 16:28 |
| TheJulia | would anyone object if I break tags code internally to ironic? | 17:14 |
| TheJulia | since it was never finished/exposed to users? | 17:14 |
| opendevreview | Verification of a change to openstack/tenks master failed: Follow ups to venv https://review.opendev.org/c/openstack/tenks/+/857008 | 17:32 |
| iurygregory | ajya, wasn't . I probably have in some env, I can look if I hasn't rebuild my lab machines | 18:10 |
| iurygregory | TheJulia, considering what you said that it was never finished/exposed I'm fine with it | 18:11 |
| * TheJulia table flips over joins | 18:11 | |
| TheJulia | pulling out tags *would* make underlying ops faster | 18:14 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/train: Make test-setup.sh compatible with mysql8 https://review.opendev.org/c/openstack/ironic/+/802460 | 18:39 |
| JayF | https://review.opendev.org/c/openstack/ironic/+/802460 trivial fix (backport) for https://storyboard.openstack.org/#!/story/2010287 which | 18:41 |
| JayF | clearly someone is pining for as they tried to backport the fix themselves | 18:41 |
| * JayF is tagging this and any other patches he'll mention here with ironic-week-prio | 18:57 | |
| opendevreview | Jay Faulkner proposed openstack/ironic-python-agent bugfix/8.3: Fix software raid output poisoning https://review.opendev.org/c/openstack/ironic-python-agent/+/857064 | 18:58 |
| JayF | other than `ironic-triaged`, and `rfe`/`rfe-approved` what other tags do we typically use in storyboard? is it doc'd anywhere? | 19:02 |
| TheJulia | low-hanging-fruit, and outreachy if memory serves | 19:06 |
| TheJulia | they are freeform, and I believe in the contributor guide | 19:06 |
| JayF | ack | 19:06 |
| JayF | Curious if folks have an opinion on this: https://storyboard.openstack.org/#!/story/2009865 tl;dr: vmedia boot on supermicro requires sushy =>4, we currently only limit to =>3.10.0 --- my thought is, we should bump this once zed is cut | 19:36 |
| JayF | right? | 19:36 |
| TheJulia | Yeah, I think so | 19:40 |
| JayF | I've gotta be honest, a large % of the "not triaged bugs" (that I have gotten to so far) in our repo are usually ones created by contributor to match their patch, but it's just never marked triage | 19:59 |
| JayF | It's kinda unclear, with no status of "CONFIRMED" or "ACCEPTED" or similar, what triaged means other than "yep this is valid" | 20:00 |
| JayF | so I wonder if we should make it clear (mailing list? IRC meeting?) to folks to just mark their bugs tag:ironic-triaged if they are a contributor and intend on patching it | 20:00 |
| TheJulia | there is definitely some of that | 20:33 |
| TheJulia | I think a PTG topic is..bug tracking | 20:33 |
| TheJulia | becuase storyboard has not evolved | 20:33 |
| JayF | I mean, I won't put it that gently: spending hours in it today, storyboard, in it's current form, is inferior to bugzilla and (blah) launchpad | 20:54 |
| TheJulia | it was really meant for a differnet class of problem | 20:55 |
| TheJulia | and to get off launchpad | 20:55 |
| TheJulia | and bugzilla... just yeah. | 20:55 |
| TheJulia | le sigh | 20:55 |
| JayF | I've been using BZ more as I've contributed to gentoo more | 20:56 |
| JayF | it's good but has very old school vibes | 20:56 |
| janders | good morning Ironic o/ | 21:15 |
| JayF | o/ | 21:17 |
| janders | JayF TheJulia huh it's an interesting one https://storyboard.openstack.org/#!/story/2009865 - I fixed this through upstream for a particular downstream (internal) customer request, bumped sushy version used in metal3 but this one fell through the cracks so to speak - I will raise this with the team to see if maybe we can improve something in the | 21:17 |
| janders | process so fixes propagate better in upstream, too | 21:17 |
| janders | (my first impression was: how come this is still broken?!?) | 21:18 |
| JayF | In this case, the only real "fix" is raising the requirement which really can't happen unless it does right at the release border | 21:18 |
| JayF | I've got a note (I added a future-port section to my backport etherpad lol) to do it right after Zed is cut | 21:18 |
| janders | agreed but we bumped sushy in metal3 ages ago - IIUC while this can't happen right-now for ironic, it could have been triggered at the same time as the metal3 bump and it would have taken effect by now, right? | 21:19 |
| janders | or is this more of a "missing backport" kind of a problem? | 21:19 |
| JayF | So with OpenStack, we just try not to bump requirements late in the cycle because it's unneccessarily stressful on folks packaging downstream | 21:20 |
| JayF | I can't speak to timing of the metal3 change; I just don't know it, but either way, it's not falling thru the cracks again | 21:20 |
| janders | the patch in question merged 10 months ago | 21:22 |
| janders | what is the best way to get the mapping of releases (wallaby, xena, etc) to sushy "numeric" versions (3.1, 4.0, etc)? | 21:23 |
| janders | turns out I did backport it to two releases before master at that time | 21:23 |
| janders | so it's been in the four most recent releases now (W, X, Y, Z) | 21:27 |
| JayF | I'm ... surprised that was eligible for backport | 21:31 |
| JayF | I assumed it wasn't given the content of it | 21:31 |
| JayF | but it'd make sense if we are more flexible around that for sushy | 21:31 |
| JayF | I do not know that name:number mapping | 21:31 |
| JayF | I have to keep it all written down explicitly for the ironic stable/ and bugfix/ branches; the matrix is so large I can't keep it in my head | 21:32 |
| janders | right! I will have a chat with the folks later today in the upstream-downstream context and see what they think... thank you JayF | 21:36 |
| opendevreview | Jacob Anders proposed openstack/sushy master: [WIP] Improve resiliency of eTag handling https://review.opendev.org/c/openstack/sushy/+/856123 | 21:52 |
| iurygregory | JayF, TheJulia re-bumping sushy to >=4 in Zed, I know we are close to the end of the cycle but we did the final release for zed like two weeks ago so the requirements are frozen, we can probably update in ironic the requirements.txt | 22:47 |
| JayF | Aha, so zed is already tagged? | 22:53 |
| JayF | (does that mean new backports have to go to zed first?) | 22:53 |
| JayF | or do you mean, sushy final release for zed | 22:53 |
| iurygregory | JayF, for clients and non-clients libs yes | 22:59 |
| JayF | ack | 23:00 |
| JayF | is it not too late to bump it for ironic? | 23:00 |
| JayF | I guess we're the only consumer which makes it a tad simpler | 23:00 |
| JayF | Do we need to hit requirements first? | 23:00 |
| iurygregory | let me check this | 23:00 |
| iurygregory | uc is pinned for 4.3.0 | 23:01 |
| iurygregory | so we can totally update requirements in ironic for >= 4.0 for Zed since we will do a release | 23:02 |
| TheJulia | sorry, accidently closed the window | 23:03 |
| TheJulia | so, we can do it this cycle if we want, it is not too late | 23:03 |
| TheJulia | yeah | 23:03 |
| * TheJulia tosses things at sqlalchemy | 23:03 | |
| JayF | I'm on it. | 23:04 |
| TheJulia | ack ack | 23:04 |
| JayF | Do we wanna make it >4 or =>4.3? | 23:04 |
| TheJulia | still fighting sqlalchemy | 23:04 |
| JayF | Why would we want someone running Ironic with older sushy? | 23:04 |
| TheJulia | I don't htink we really would at this point | 23:04 |
| TheJulia | we're just bad at bumping constraints | 23:05 |
| TheJulia | like.. with these sqlalchemy changes, the minimum will need to go to 1.4 | 23:05 |
| TheJulia | ... that is likely implied already | 23:05 |
| TheJulia | but yeah | 23:05 |
| iurygregory | normally we try to ask 3rd party CI to bump their requirements (since they have features etc) | 23:06 |
| TheJulia | proliantutils :\ | 23:06 |
| opendevreview | Merged openstack/ironic stable/train: Make test-setup.sh compatible with mysql8 https://review.opendev.org/c/openstack/ironic/+/802460 | 23:06 |
| TheJulia | I've heard it is broken against sushy 4 | 23:06 |
| iurygregory | I don't think we bump sushy that much just because normally we are able to backport things | 23:06 |
| iurygregory | TheJulia, I think it was just some missing things in unit testing... | 23:07 |
| JayF | Well, one way to find out, eh? | 23:07 |
| iurygregory | push the patch | 23:08 |
| iurygregory | let's wait for CI | 23:08 |
| * JayF is on it | 23:08 | |
| JayF | yeah, and check 3rd party CI | 23:08 |
| JayF | all of them but that one HP job for the HP CI had been passing | 23:08 |
| iurygregory | <evil laugh> | 23:08 |
| TheJulia | true, and it can always just be fixed after the fact. just pain() at times | 23:09 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Zed Ironic requires Sushy >4 https://review.opendev.org/c/openstack/ironic/+/857238 | 23:09 |
| JayF | I'll follow up on it tomorrow | 23:10 |
| JayF | have a good day folks o/ | 23:10 |
| TheJulia | ++ | 23:10 |
| TheJulia | o/ | 23:10 |
| iurygregory | JayF, bye o/ | 23:10 |
| iurygregory | going to grab some dinner, bbl o/ | 23:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!