| rpittau | good morning ironic! o/ | 08:07 |
|---|---|---|
| rpittau | mmm I think I lost all the scroll logs from the weekend :/ | 08:51 |
| *** tosky_ is now known as tosky | 11:36 | |
| *** nfedorov_ is now known as jingvar | 12:08 | |
| opendevreview | Merged openstack/ironic stable/2023.2: ci: pin CI to dnsmasq 2.85 https://review.opendev.org/c/openstack/ironic/+/910669 | 12:09 |
| iurygregory | good morning Ironic | 13:12 |
| TheJulia | rpittau: it was a quiet weekend | 13:36 |
| TheJulia | Also, good morning everyone! | 13:36 |
| masghar | Good morning! | 13:57 |
| dtantsur | Does anyone understand the new unmaintained/ thing? I'm working on the gophercloud CI breakage and would like to understand if supporting Yoga- is even possible at this pont. | 14:20 |
| dtantsur | TheJulia, JayF maybe ^^ | 14:20 |
| TheJulia | understand no, and I have no idea why yoga got unmaintained when folks are still maintaining things like wallaby | 14:20 |
| dtantsur | yeah, that's another brain-twisting aspect | 14:21 |
| dtantsur | should I expect these to be broken similarly in the near future? | 14:21 |
| TheJulia | I think it is a good thing to drop unused things, but the stable policy needs to be revised if it is okay to drop intermediate branches | 14:21 |
| dtantsur | yeah.. | 14:21 |
| JayF | We're going backwards in the rollout. Things back all the way to W will be retired or moved to unmaintained | 14:21 |
| JayF | I sent an email about the changes in releases repository about this to the mailing list Friday | 14:22 |
| dtantsur | a written reference is nice, thank you | 14:22 |
| dtantsur | I now need to find where the ML archives are nowadays.. | 14:23 |
| * dtantsur hates the decision to rename branches btw | 14:24 | |
| TheJulia | yeah, having been burned by that already, I'm super frustarated | 14:24 |
| opendevreview | Merged openstack/ironic master: Don't import sushy conditionally, it's a requirement https://review.opendev.org/c/openstack/ironic/+/910529 | 14:33 |
| dtantsur | JayF: do I get it right that only Yoga will be left unmaintainted by default, while the older branches will be EOL-ed unless the team opts out? | 14:35 |
| dtantsur | sorry, I somehow missed this whole topic | 14:35 |
| JayF | Yes, but someone already opted in to maintain stuff back to victoria. And please do look at the copious amounts of documentation we produced including beforehand so that people could provide feedback before we made the decision | 14:35 |
| dtantsur | I'm looking at https://docs.openstack.org/project-team-guide/stable-branches.html#unmaintained, just wanted to make sure I don't play broken phone with gophercloud people | 14:36 |
| TheJulia | I guess there is a plus, the move to unmaintained at least dialed back testing substantailly | 14:55 |
| * dtantsur nods | 14:55 | |
| dtantsur | TheJulia: do you have any ideas what to do with https://github.com/gophercloud/gophercloud/actions/runs/8142059704/job/22250749967?pr=2962#step:5:12786 ? | 14:56 |
| dtantsur | Make the removal optional? | 14:56 |
| TheJulia | I guess we could | 14:58 |
| TheJulia | challenge is, we're going to have to stomp on it no matter what to have working CI | 14:58 |
| TheJulia | building/running 2.85 runs a slight risk if there are other dependencies, dunno | 14:58 |
| TheJulia | seems super weird that there is that requirement | 14:58 |
| JayF | dtantsur: we also have details about the transition in the resolution itself. Right now I'm trying to quickly get in front of the physical computer to run the meeting though so I can find it after if you want | 14:59 |
| dtantsur | Probably just included in github images.. | 14:59 |
| JayF | #startmeeting ironic | 15:01 |
| opendevmeet | Meeting started Mon Mar 4 15:01:44 2024 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 |
| rpittau | o/ | 15:01 |
| dtantsur | o/ | 15:01 |
| JayF | Good morning, welcome to the Ironic meeting. | 15:01 |
| masghar | o/ | 15:01 |
| JayF | #topic Announcements / Reminder | 15:02 |
| JayF | #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:02 |
| JayF | #info Project Teams Gathering (PTG) will be held from Monday, April 8 to Friday, April 12 2024 | 15:02 |
| JayF | We have a dedicated topic about this later | 15:02 |
| JayF | #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:02 |
| JayF | #topic Caracal Release Schedule | 15:02 |
| JayF | milestone-3 has passed, Ironic should be landing our final things for the 2024.1 release | 15:02 |
| JayF | Is there anything of specific note that needs reviews/attention? | 15:02 |
| JayF | #topic Review Ironic CI Status | 15:04 |
| JayF | Should we start by listing the things that *weren't broken* in the gate over the last week? | 15:04 |
| JayF | #link https://bugs.launchpad.net/dnsmasq/+bug/2026757 | 15:04 |
| TheJulia | Toasty, tl;dr is our CI can't run with newer dnsmasqs due to what appears to be a memory corruption issue | 15:04 |
| JayF | reared it's head again, Julia figured out to back it out to 0.85 | 15:04 |
| TheJulia | 2.85 | 15:05 |
| JayF | Unless someone else already is; I'm going to make a point to ensure upstraem is aware of this bug | 15:05 |
| JayF | TheJulia: I swear, everytime I write that I put a different number to the left of the decimal :\ | 15:05 |
| TheJulia | JayF: I've been chatting with some upstream folks, we're still trying to figure out *what* exactly is going on | 15:05 |
| JayF | ack; thank you sounds good I'll leave it alone then | 15:05 |
| JayF | We also added a new multinode job, non-voting, which uses two shards | 15:06 |
| JayF | Anything else on gate? | 15:06 |
| JayF | #topic vPTG Planning | 15:07 |
| JayF | #info current schedule is 4 hours tuesday and wednesday (13-17 UTC), 3 hours on thursday (13-16 UTC) | 15:07 |
| JayF | #undo | 15:07 |
| opendevmeet | Removing item from minutes: #info current schedule is 4 hours tuesday and wednesday (13-17 UTC), 3 hours on thursday (13-16 UTC) | 15:07 |
| JayF | #info current schedule is 4 hours tuesday and wednesday (13-17 UTC), 3 hours on thursday (13-16 UTC) in Mitaka room at https://ptg.opendev.org | 15:08 |
| JayF | #info topics etherpad https://etherpad.opendev.org/p/ironic-ptg-april-2024 | 15:08 |
| JayF | Please take time in the next week-ish to look over ptg topics | 15:08 |
| JayF | add/comment/etc | 15:08 |
| JayF | rpittau: anything to add? | 15:08 |
| JayF | #topic Bug Deputy | 15:09 |
| rpittau | nothing for the moment | 15:09 |
| rpittau | once we finalize the list of topics we just need to split them on the scheduled days | 15:09 |
| JayF | ack, sounds good | 15:09 |
| JayF | TheJulia: you were bug deputy anything interesting other than the dmsmasq bug from heck :P | 15:10 |
| dtantsur | I was :) | 15:10 |
| TheJulia | JayF: I think it was dmitry last week | 15:10 |
| JayF | I did an excellent job of indicating that on the meeting agenda lol | 15:10 |
| dtantsur | I've merged one of the contributions to the dashboard and added a list of bugs older than 1 year | 15:10 |
| dtantsur | On top of that, I've cleaned up some invalid or already fixed bugs from the list of untriaged and triaged in progress | 15:11 |
| dtantsur | Unfortunately, I haven't captured any numbers to boast, so you'll have to trust me on this :) | 15:11 |
| JayF | We're starting to make a really noticible dent | 15:11 |
| JayF | in our bug backlog being a group of actually actionable things that are relevant | 15:11 |
| JayF | thanks for helping make that happen | 15:11 |
| TheJulia | we're down to 174 open bugs, we were at 200 just a month and a half ago, so progress \o/ | 15:11 |
| JayF | No RFEs to review, passing that agenda item | 15:11 |
| JayF | Oh wow, another 26 gone poof | 15:12 |
| JayF | nice work! | 15:12 |
| dtantsur | I cannot promise to be the deputy again, so we need another volunteer | 15:12 |
| JayF | #success Ironic project down to 174 open bugs after having what seemeed like an infinite number this time last cycle | 15:12 |
| opendevstatus | JayF: Added success to Success page (https://wiki.openstack.org/wiki/Successes) | 15:12 |
| JayF | #action JayF is bug deputy | 15:12 |
| JayF | I know a guy :D | 15:12 |
| rpittau | I'll do it next week then :) | 15:12 |
| JayF | You have a PTG to plan | 15:13 |
| JayF | speaking of | 15:13 |
| JayF | #topic Open Discussion | 15:13 |
| JayF | #info rpittau has been elected by consensus the PTL for the Dalmation cycle | 15:13 |
| JayF | Congratulations rpittau, and thanks to the community for trusting me with the spot for the last three cycles. | 15:13 |
| rpittau | thanks! :) | 15:13 |
| JayF | Anything else for Open Discussion? | 15:14 |
| dtantsur | rpittau++ | 15:14 |
| JayF | When does the new PTL actually take over? | 15:15 |
| rpittau | I think it's end of March | 15:15 |
| JayF | 3/20 | 15:15 |
| JayF | I think I'll call it there | 15:16 |
| JayF | o/ | 15:16 |
| JayF | #endmeeting | 15:16 |
| opendevmeet | Meeting ended Mon Mar 4 15:16:30 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:16 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-03-04-15.01.html | 15:16 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-03-04-15.01.txt | 15:16 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2024/ironic.2024-03-04-15.01.log.html | 15:16 |
| JayF | I'm going to be working on cycle highlights today | 15:20 |
| JayF | if there's anything you feel strongly should be included, please let me know | 15:20 |
| dtantsur | New inspection, firmware upgrades (did they come this cycle?) | 15:20 |
| TheJulia | httpboot all the things! | 15:21 |
| dtantsur | ++ | 15:22 |
| JayF | https://releases.openstack.org/bobcat/highlights.html#ironic api, but not driver, so I'll mention it | 15:27 |
| JayF | (re firmware) | 15:27 |
| opendevreview | Julia Kreger proposed openstack/ironic stable/2023.1: stable-only: pin proliantutils/scciclient to prevent break https://review.opendev.org/c/openstack/ironic/+/910528 | 15:47 |
| TheJulia | so yeah, definite memory corruption issue, | 16:20 |
| dtantsur | \o/ | 16:20 |
| TheJulia | I'm retrying with ipv6 turned off in the dnsmasq build just to see if that helps identify the where | 16:23 |
| TheJulia | since a lot of the code is patterned, "do the thing for normal cases, then do it for v6 as a separate thing" | 16:23 |
| dtantsur | that's why we cannot have nice things.. | 16:24 |
| TheJulia | the pit of sadness exists to torture ironic developers | 16:25 |
| dtantsur | ERROR ironic.api.method [None req-a96d3053-ff8d-43ff-b9f1-564645eaca92 admin admin] Server-side error: ""baremetal:conductor:get": "(role:reader and system_scope:all) or (role:service and system_scope:all) or rule:service_role" requires a scope of ['system'], request was made with project scope." | 16:33 |
| dtantsur | A lot of sadness in the gophercloud CI too. TheJulia do you know why we return HTTP 500 on ^^^ though? | 16:33 |
| TheJulia | ugh, the scope only restriction needs to be removed | 16:33 |
| dtantsur | Looks like a case for 403? | 16:33 |
| dtantsur | Is it something to be fixed in the gophercloud CI or in Ironic (or both)? | 16:34 |
| TheJulia | likely ironic | 16:34 |
| rpittau | need to drop, see you tomorrow, good night! o/ | 16:34 |
| TheJulia | o/ | 16:34 |
| dtantsur | On the positive side, removing dnsmasq in the job definition helped | 16:35 |
| TheJulia | what do you mean removing dnsmasq? | 16:35 |
| dtantsur | https://github.com/gophercloud/gophercloud/commit/888aa7be4c474e58a9477e43a7ae631b29f0e5a7 | 16:36 |
| dtantsur | do you have an idea for the ironic fix for baremetal:conductor:get? | 16:37 |
| TheJulia | yeah, its a policy fix most likely | 16:37 |
| TheJulia | oh! | 16:37 |
| TheJulia | OH! | 16:37 |
| TheJulia | conductor get | 16:37 |
| TheJulia | so it is still going to fail for the user unless they have service role rights under the rbac model | 16:37 |
| dtantsur | "source openrc admin admin" | 16:38 |
| TheJulia | I'll post a change to ironic in a minute for that side | 16:38 |
| TheJulia | that is a project scoped admin | 16:38 |
| TheJulia | not a system | 16:38 |
| * dtantsur is going to cry | 16:38 | |
| dtantsur | Is it an easy to use replacement or will I break something else? | 16:38 |
| TheJulia | TC decided to kick system scope to the fucking curb | 16:38 |
| dtantsur | \o/ | 16:39 |
| dtantsur | It seems that I'll become collateral damage | 16:39 |
| TheJulia | of course, after we shipped support | 16:39 |
| TheJulia | testing locally | 16:41 |
| TheJulia | to do the list, change the command to OS_CLOUD=devstack-system-admin openstack put rest of command here | 16:42 |
| dtantsur | Now the same but in Go please :D | 16:42 |
| JayF | or `--os-cloud=devstack-system-admin` as an option to the command | 16:42 |
| dtantsur | It's Gophercloud acceptance tests. No OSC involved. | 16:42 |
| JayF | that in devstack just changes your account login | 16:42 |
| dtantsur | It does `source openrc admin admin` | 16:43 |
| TheJulia | ... they are using env vars? | 16:43 |
| JayF | to be system admin scope | 16:43 |
| JayF | egad | 16:43 |
| TheJulia | try unsetting PROJECT_NAME and TENANT_NAME parameters | 16:43 |
| TheJulia | and try setting OS_SYSTEM_SCOPE=all | 16:43 |
| * TheJulia fixes unit tests | 16:44 | |
| dtantsur | TheJulia: something like this https://github.com/gophercloud/gophercloud/pull/2964? | 16:48 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix artifical rbac policy constraint that resulted in 500s https://review.opendev.org/c/openstack/ironic/+/910969 | 16:49 |
| TheJulia | dtantsur: it has to be set to "all" | 16:49 |
| TheJulia | not an empty string | 16:49 |
| dtantsur | TheJulia: line 16? | 16:50 |
| dtantsur | I don't want to set it for versions that work as they are | 16:50 |
| dtantsur | (yes, the matrix syntax is weird in github) | 16:50 |
| TheJulia | okay, didn't grok line 16 | 16:50 |
| TheJulia | yeah, that should work | 16:50 |
| opendevreview | cid proposed openstack/ironic master: Fix data length exceeding limit error https://review.opendev.org/c/openstack/ironic/+/910973 | 17:57 |
| opendevreview | Julia Kreger proposed openstack/ironic master: docs: troubleshooting addition for sync period https://review.opendev.org/c/openstack/ironic/+/908342 | 21:07 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: stable-only: pin proliantutils/scciclient to prevent break https://review.opendev.org/c/openstack/ironic/+/910528 | 21:14 |
| iurygregory | CI seems a bit unhappy in stable | 21:58 |
| TheJulia | that feels like an understatement | 22:09 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: stable-only: pin proliantutils/scciclient to prevent break https://review.opendev.org/c/openstack/ironic/+/910528 | 22:18 |
| iurygregory | yeah | 22:36 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!