| vanou | Hello Ironic | 01:15 |
|---|---|---|
| vanou | I noticed JayF's comment on colleague's patch (https://review.opendev.org/c/openstack/ironic/+/850553) that said there is backport order in Ironic. I didn't know that. Is there any doc on this? And what the purpose of bugfix/xxx branch? | 01:18 |
| arne_wiebalck | Good morning, Ironic! | 06:33 |
| arne_wiebalck | vanou: this follows OpenStack's stable branch policy documented here: https://docs.openstack.org/project-team-guide/stable-branches.html : ensuring the order avoids update surprises where a patch is missing in a more recent release. The bugfix/xxx branches are explained here https://docs.openstack.org/ironic/latest/contributor/releasing.html#bugfix-branches and basically provide in-cycle releases. I am not sure if | 06:41 |
| arne_wiebalck | the order as outlined in JayF's comment is documented, but maybe that is sth we could add. | 06:41 |
| vanou | arne_wiebalck: Thanks! I overlooked that.. Yes, it's helpfull if there is a guide for backporting order against bugfix & stable branches. | 06:49 |
| opendevreview | SONG SHUKUN proposed openstack/ironic bugfix/20.2: [iRMC] Add SNMPv3 authentication functionality https://review.opendev.org/c/openstack/ironic/+/859216 | 07:17 |
| rpittau | good morning ironic! o/ | 07:42 |
| rpittau | vanou: for a complete list of the current supported bugfix branches you can also check the ironic meetpad around L71 https://etherpad.opendev.org/p/IronicWhiteBoard | 07:44 |
| rpittau | errr etherpad | 07:44 |
| rpittau | that remind me I have to update some docs | 07:45 |
| vanou | rpittau: good morning. Thanks! I'll check it. Regarding https://review.opendev.org/c/openstack/ironic/+/858896 , unit test fails because that bugfix branch doesn't merge SNMPv3 backport patch (https://review.opendev.org/c/openstack/ironic/+/845347). I'll point out what code should be change soon. | 08:28 |
| rpittau | vanou: that means there's probably one backport missing | 08:31 |
| rpittau | and that reminds me one more thing | 08:32 |
| rpittau | 790165 | 08:32 |
| rpittau | wrong window :/ | 08:32 |
| vanou | Ah... I appreciate you for complicated work :) | 08:36 |
| vanou | What can I help? | 08:40 |
| rpittau | vanou: you got it already, we're missing the backport of https://review.opendev.org/c/openstack/ironic/+/845347 to bugfix/19.0 | 09:03 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Update release versions for zed https://review.opendev.org/c/openstack/ironic/+/859228 | 09:08 |
| vanou | rpittau: OK | 09:09 |
| vanou | got it | 09:09 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/20.2: Pin bugfix 20.2 ci jobs to zed https://review.opendev.org/c/openstack/ironic/+/859250 | 09:26 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/20.2: Pin bugfix 20.2 ci jobs to zed https://review.opendev.org/c/openstack/ironic/+/859250 | 09:28 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/20.2: Pin bugfix 20.2 ci jobs to zed https://review.opendev.org/c/openstack/ironic/+/859250 | 09:31 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/21.0: Pin bugfix 21.0 ci jobs to zed https://review.opendev.org/c/openstack/ironic/+/859251 | 09:37 |
| opendevreview | Riccardo Pittau proposed openstack/ironic bugfix/21.0: Pin bugfix 21.0 ci jobs to zed https://review.opendev.org/c/openstack/ironic/+/859251 | 09:40 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Pin bugfix 8.6 ci jobs to zed https://review.opendev.org/c/openstack/ironic-python-agent/+/859253 | 09:53 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Pin bugfix 8.6 ci jobs to zed https://review.opendev.org/c/openstack/ironic-python-agent/+/859253 | 09:54 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Pin bugfix 8.6 ci jobs to zed https://review.opendev.org/c/openstack/ironic-python-agent/+/859253 | 10:00 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/9.0: Pin bgufix 9.0 ci jobs to zed https://review.opendev.org/c/openstack/ironic-python-agent/+/859255 | 10:05 |
| opendevreview | Aija Jauntēva proposed openstack/sushy-oem-idrac stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/sushy-oem-idrac/+/859264 | 11:01 |
| opendevreview | Aija Jauntēva proposed openstack/sushy-oem-idrac stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/sushy-oem-idrac/+/859265 | 11:01 |
| opendevreview | Aija Jauntēva proposed openstack/sushy-oem-idrac master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/sushy-oem-idrac/+/859266 | 11:03 |
| iurygregory | good morning Ironic | 11:28 |
| dtantsur | rpittau: not sure which patch of the chain https://review.opendev.org/c/openstack/bifrost/+/848828/ to -1, but in one of them 1) a release note has to be present, 2) a documentation update is required, 3) one bionic job must be left, at least as non-voting (but ideally voting) | 11:35 |
| dtantsur | to elaborate on #3: otherwise upgrades are not possible (Zed only bionic, A only jammy) | 11:36 |
| dtantsur | maybe actually merge 3 changes into one? they don't seem any invasive | 11:44 |
| janders | TheJulia w/r/t PTG / active steps - apologies, been off Friday - let me touch base with Moshe and report back | 12:03 |
| rpittau | dtantsur: I will merge all into one change at this point, CI seems passing, and local tests too, so can add reno and doc change at the same time | 12:24 |
| dtantsur | ++ | 12:29 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Move bifrost tests to Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/848828 | 12:51 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Move bifrost tests to Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/848828 | 12:55 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Move bifrost tests to Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/848828 | 12:58 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Move bifrost tests to Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/848828 | 13:01 |
| dtantsur | rpittau, iurygregory, maybe it's worth documenting to avoid the confusion in the future, but please keep in mind: if we support $DISTRO $VERSION in $BRANCH, we cannot drop it in $BRANCH+1 in favour of $VERSION+1. Worst case, we need to backport support for $VERSION+1, but ideally we need to keep support for $VERSION | 13:13 |
| dtantsur | (remember how we did that for CS 8->9) | 13:13 |
| dtantsur | reason: upgrades. asking operator to upgrade the distro AND bifrost in a lockstep is very unfortunate, and we should generally avoid it. | 13:14 |
| dtantsur | I also don't think we have a strong reason to drop focal now, do we? | 13:14 |
| iurygregory | dtantsur, gotcha | 13:15 |
| iurygregory | we should keep focal for now I would say | 13:15 |
| iurygregory | we used to keep 2 LTS versions *I think* in bifrost | 13:16 |
| dtantsur | I think so too, yeah. we kept bionic a bit longer | 13:16 |
| dtantsur | Has anyone ever wondered how virtualenvs work when the default Python's version change (i.e. on upgrades)? | 13:16 |
| iurygregory | nope .-. | 13:17 |
| TheJulia | the binaries reference the venv python and the venv python is a copy of the python it was created with | 13:22 |
| opendevreview | Merged openstack/ironic-inspector master: Imported Translations from Zanata https://review.opendev.org/c/openstack/ironic-inspector/+/859173 | 13:24 |
| TheJulia | Also, good morning | 13:27 |
| rpittau | good morning TheJulia :) | 13:29 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Move bifrost tests to Ubuntu Jammy https://review.opendev.org/c/openstack/bifrost/+/848828 | 13:29 |
| opendevreview | Riccardo Pittau proposed openstack/bifrost master: Upgrade from zed https://review.opendev.org/c/openstack/bifrost/+/859075 | 13:30 |
| dtantsur | morning TheJulia. yeah, I wonder if it's going to recognize the installed site-packages | 13:31 |
| TheJulia | I don't even think it would look | 13:31 |
| TheJulia | well | 13:31 |
| TheJulia | it might look where it expects them, but new ones wouldn't be found I think and if there is compiled bytecode then the seed won't mach | 13:32 |
| dtantsur | hmm, so bifrost is kinda broken on an OS upgrade | 13:32 |
| rpittau | mmmmm I'm not sure I would do an OS upgrade just like that knowing I have bifrost on virtualenv, or any other virtualenvs, probably a reinstall | 13:35 |
| rpittau | I actually think I never done that in my operator days, because of python binary change | 13:35 |
| rpittau | there is a way to update the python binary in a venv though | 13:35 |
| TheJulia | well, i thought os upgrades leave the old python alone | 13:35 |
| TheJulia | that is if the venv is dependent on system site packages | 13:36 |
| rpittau | using the python 3 venv module you can use the --upgrade option | 13:36 |
| dtantsur | never heard about it. I wonder if it's something we should start doing. | 13:40 |
| * TheJulia tries to wake up | 13:54 | |
| * TheJulia needs a very big cup of coffee this morning | 13:55 | |
| JayF | rpittau: python3 venv --upgrade worked-ish, but wiped out local changes in a venv for a (different) python thing I run locally | 14:23 |
| JayF | rpittau: I would not go that route if you can help it :| | 14:23 |
| rpittau | JayF: what you mean by "local changes" ? | 14:23 |
| JayF | So I run homeassistant, and I had a couple other things in the dir with the venv (not the best design for running things, obviously). I backed up the dir, did the python3 -m venv --upgrade ... and literally the end result looked like if I had rm venv_dir; python3 -m venv venv_dir | 14:24 |
| JayF | All data in venv_dir/ went kaput and was replaced with a fresh venv | 14:25 |
| JayF | (I had backed up, because I've been doing this long enough to expect things to break; but I had to take a different approach to upgrading the venv) | 14:25 |
| rpittau | JayF: thanks, I don't remember such behavior, I'll do some tests | 14:27 |
| JayF | rpittau: as always; possible I did something wrong or something crazy about the env. I don't always ... exercise sane system admin practices on homelab stuff | 14:28 |
| rpittau | :) | 14:28 |
| dtantsur | hmm so it may purge site-packages.. damn | 14:28 |
| * dtantsur is pondering how to conductor an experiment | 14:29 | |
| TheJulia | I *think* what jayf has indicated in the pattern, is literally what the code does now that I think of it | 14:38 |
| * TheJulia pulled it up a year or three ago | 14:38 | |
| rpittau | bye everyone, see you next week o/ | 14:54 |
| JayF | o/ | 14:56 |
| JayF | iurygregory: let me run the .startmeeting to see if it works for me | 14:59 |
| iurygregory | JayF, ack | 14:59 |
| JayF | iurygregory: .startmeeting ironic # right ? | 14:59 |
| dtantsur | it works for everyone, I think | 14:59 |
| dtantsur | I think it starts with # | 14:59 |
| dtantsur | not with . | 14:59 |
| JayF | that's what Iury suggested in DM, but I know it wasn't always configured that way | 14:59 |
| iurygregory | dtantsur, yeah | 14:59 |
| JayF | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Sep 26 15:00:02 2022 UTC and is due to finish in 60 minutes. The chair is JayF. 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 |
| dtantsur | yay o/ | 15:00 |
| iurygregory | yay =) | 15:00 |
| erbarr | o/ | 15:00 |
| JayF | Hey all, meeting time. Who is here? | 15:00 |
| JayF | o/ | 15:00 |
| iurygregory | o/ | 15:00 |
| ajya | o/ | 15:00 |
| matfechner | o/ | 15:00 |
| JayF | #chair iurygregory | 15:01 |
| opendevmeet | Current chairs: JayF iurygregory | 15:01 |
| iurygregory | you can find the agenda for our meeting in the wiki | 15:01 |
| iurygregory | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| iurygregory | #topic Announcements / Reminder | 15:01 |
| rloo | o/ | 15:02 |
| iurygregory | #info We have released Zed for our projects, congratulations everyone! The official OpenStack Zed release will be next week! | 15:02 |
| iurygregory | #info JayF will be our PTL during the Antelope cycle, congratulations! | 15:02 |
| JayF | It's the 18th release of OpenStack with Ironic included in some way. Next year is our 10th anniversary as a project :D | 15:02 |
| dtantsur | \o/ | 15:03 |
| rloo | wheeee. congrats and congrats and yay!!! | 15:03 |
| ajya | congrats JayF | 15:03 |
| ajya | and thanks iurygregory | 15:03 |
| iurygregory | :D | 15:03 |
| JayF | Thanks everyone o/ | 15:04 |
| iurygregory | #info PTG topics discussion from last week | 15:05 |
| iurygregory | last week we have a meeting to schedule time slots for our next PTG and also to check how much time we would like to spend in each topic | 15:05 |
| iurygregory | some information can be found in the etherpad for the ptg | 15:05 |
| iurygregory | #link https://etherpad.opendev.org/p/ironic-antelope-ptg | 15:05 |
| iurygregory | this week is last call for topics for the PTG, so we can book rooms for discussion =) | 15:06 |
| JayF | Related to that, we have a co-session with Nova that I'll be scheduling with them this week. | 15:06 |
| iurygregory | so, if you haven't added your topic to the etherpad, please do | 15:06 |
| JayF | Mainly focusing on trying to find a path forward on Nova<>Ironic driver improvements | 15:06 |
| JayF | If you want to be included in that scheduling; please reach out to me | 15:07 |
| JayF | Once the time is chosen it'll obviously be open to the public; but I certainly don't want to be the only Ironic person there :D | 15:07 |
| JayF | #info JayF is scheduling a Nova<>Ironic driver PTG session with Nova, if you wish to be included in scheduling discussions reach out to him. | 15:08 |
| iurygregory | #info hjensas is now part of the metalsmith-core \o/ since there was no objections | 15:10 |
| iurygregory | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-September/030524.html | 15:10 |
| JayF | congrats hjensas | 15:10 |
| rloo | congrats hjensas! | 15:11 |
| hjensas | o/ thanks | 15:11 |
| iurygregory | does anyone have something we should add to Announcements / Reminder ? | 15:11 |
| dtantsur | nice! | 15:11 |
| * TheJulia blinks and realizes the time | 15:12 | |
| TheJulia | Related to, seems like the foundation will be doing a summary episode for the Zed release next week for Open Infra live | 15:14 |
| TheJulia | They are presently seeking projects which would be interested in providing updates as part of it. See the discussion mailing list from Allison Price. | 15:14 |
| JayF | I will ensure Ironic is represented unless someone else wants to volunteer. | 15:15 |
| iurygregory | JayF, go ahead =) | 15:15 |
| JayF | ack, adding to my list | 15:16 |
| iurygregory | ok, moving on o/ | 15:17 |
| iurygregory | no action items from last week skipping | 15:17 |
| iurygregory | review subteam status, I'm wondering if makes sense atm.. we have the Zed release for our projects, thoughts? | 15:18 |
| TheJulia | I did put in an update late last week w/r/t sqlalchemy | 15:19 |
| JayF | ack; thanks for your work on that Julia | 15:20 |
| iurygregory | tks TheJulia | 15:20 |
| TheJulia | the tl;dr being inspector never got a delineated declaritive reader/writer setup as most projects went towards. A first logical step is to pickup that patch, which I have already started to move forward, and use that as a foundation to begin to clean things up. It will make the merge into ironic cleaner since it should put them into the same style... eventually. | 15:21 |
| iurygregory | #topic Deciding on priorities for the coming week | 15:21 |
| iurygregory | I think we can start adding the migration as priority for Antelope right? | 15:21 |
| iurygregory | we can start reviewing things to see how CI will react in the long run | 15:22 |
| TheJulia | migration to sqlalchemy 2? | 15:22 |
| TheJulia | or inspector? | 15:22 |
| iurygregory | migration to sqlalchemy in general | 15:23 |
| iurygregory | you have ironic patch up, so I think we can start looking at it (ie, adding to the prio-list) | 15:24 |
| JayF | I've been looking at them already; not sure they are passing CI and ready for general consumption but for a change of that size we should have folks looking at it. | 15:26 |
| iurygregory | Does anyone have patches we should add to the list to review? | 15:27 |
| TheJulia | JayF: I rebased them on Friday, but have not looked since. | 15:28 |
| JayF | I have some stable items that I'll add a tag to once I get back around to them. I generally will add the prio to them as long as they are clean, useful backports :) | 15:28 |
| TheJulia | fwiw, I think merging them will result in a major number bump since we will be moving up the constraint on sqlalchemy to 1.4.0 | 15:28 |
| TheJulia | not that we should let that drive our decision making process, mentioning it more as an FYI | 15:29 |
| iurygregory | tks for the fyi =) | 15:31 |
| iurygregory | ok, moving on | 15:31 |
| iurygregory | #topic Baremetal SIG | 15:31 |
| iurygregory | #link https://etherpad.opendev.org/p/bare-metal-sig | 15:31 |
| * arne_wiebalck found the recording of the last meeting ... | 15:32 | |
| arne_wiebalck | so expect the video soon | 15:32 |
| arne_wiebalck | (somehow I missed the upload) | 15:32 |
| JayF | arne_wiebalck: oh, good stuff! Do you mind sharing the source with me, G-Research may want to post it on their channel too (that was the G-R presentation, right?) | 15:32 |
| arne_wiebalck | yes | 15:33 |
| iurygregory | we have a sig session at the PTG \o/ | 15:33 |
| arne_wiebalck | JayF: the raw video you mean? | 15:33 |
| JayF | arne_wiebalck: ack, if possible, the source video | 15:33 |
| arne_wiebalck | JayF: sure, will send you a link | 15:33 |
| iurygregory | arne_wiebalck probably wasn't aware but now he is =P | 15:33 |
| JayF | arne_wiebalck: good stuff, thanks you! | 15:33 |
| arne_wiebalck | iurygregory: yeah, I was not :-D | 15:33 |
| arne_wiebalck | apart from this the only other thing is to discuss how to move fwd with the sig in general ... maybe at the PTG then :) | 15:34 |
| iurygregory | "operator-hour-baremetal-sig" Wed 19 at 13 UTC | 15:34 |
| arne_wiebalck | nice | 15:34 |
| arne_wiebalck | so this is not the place for a general discussion | 15:34 |
| arne_wiebalck | unless noone shows up :-D | 15:35 |
| JayF | I think we can spend that hour talking about whatever the operators who show up wanna talk about :) | 15:35 |
| arne_wiebalck | heh | 15:35 |
| JayF | I'm fairly sure that number will be nonzero, because I believe some operational folks from here at G-Research are going to try and attend. | 15:35 |
| arne_wiebalck | awesome, ok | 15:36 |
| arne_wiebalck | that is all from me for the SIG | 15:36 |
| iurygregory | tks arne_wiebalck | 15:36 |
| iurygregory | skipping RFE review topic | 15:36 |
| iurygregory | #topic Open discussion | 15:36 |
| iurygregory | Do we have something we would like to discuss? | 15:37 |
| rloo | what do you think of those crickets? | 15:39 |
| iurygregory | I think it's time to end the meeting :D | 15:39 |
| TheJulia | w/r/t the operator discussion, if there is a meeting invite sent out, I can forward it to a few | 15:39 |
| TheJulia | I suspect the crickets are plotting to take over the world... | 15:39 |
| arne_wiebalck | for this, it would be good to have some pre-designed thingy to retweet, in fact | 15:40 |
| arne_wiebalck | with a catchy slogan, a nice picture, ... :) | 15:40 |
| iurygregory | both invitation for email + tweet I would say :D | 15:40 |
| iurygregory | maybe something we need to decide and do it :D | 15:41 |
| TheJulia | ++ | 15:41 |
| iurygregory | #topic Who is going to run the next meeting? | 15:43 |
| TheJulia | I appear to have a conflict next week | 15:44 |
| iurygregory | TheJulia, ack, no worries | 15:45 |
| iurygregory | since I'm only hearing crickets, I will run the meeting next Monday o/ | 15:45 |
| iurygregory | thanks everyone! | 15:45 |
| iurygregory | #endmeeting | 15:46 |
| opendevmeet | Meeting ended Mon Sep 26 15:46:05 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:46 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-09-26-15.00.html | 15:46 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-09-26-15.00.txt | 15:46 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2022/ironic.2022-09-26-15.00.log.html | 15:46 |
| JayF | I'll be here next week and can run the meeting :) | 15:46 |
| JayF | I already relocated to my double-booked 8:45am meeting once I heard the chirping of crickets lol | 15:47 |
| JayF | or iurygregory can :) I don't ever care as long as it gets done | 15:47 |
| TheJulia | And lots of coffee appears | 15:47 |
| iurygregory | JayF, ack =) | 15:47 |
| dtantsur | oh, JFYI I won't be around for the next 2 meetings | 15:49 |
| TheJulia | vacation? | 15:49 |
| * TheJulia needs to take a vacation | 15:49 | |
| dtantsur | one Monday holiday, then Monday-Tuesday a small vacation, yes | 15:50 |
| dtantsur | time to run to the woods! | 15:51 |
| TheJulia | post awesome photos from the woods! | 15:51 |
| dtantsur | will do! the place is quite nice, fingers crossed for the weather :) | 15:52 |
| dtantsur | https://en.wikipedia.org/wiki/Winterberg | 15:52 |
| TheJulia | wow, known for witch trials... | 15:54 |
| TheJulia | 8| | 15:54 |
| dtantsur | quite a bit of history everywhere here :D | 15:54 |
| JayF | I was about to say "Wow, looks like Leavenworth". Then I realized Leavenworth is a vacation destination here in WA because it's a *German village* (turned tourist trap) | 15:54 |
| JayF | "Hey, your original looks like my copy" LOL | 15:55 |
| JayF | I wanna go to Germany sometime :) | 15:55 |
| TheJulia | Germany is awesome | 15:55 |
| dtantsur | I'd be so happy to see you both here :) | 15:55 |
| JayF | My wife has a lot of family there. Which makes it more complex to book, b/c we both generally don't travel together (dogsitting is expensive) | 15:55 |
| dtantsur | true | 15:56 |
| JayF | (and we'll probably have to get my MIL a ticket too lol) | 15:56 |
| JayF | <on topic> If there are any specific-to-zed-release highlights you want me to call out in the openinfra live, let me know, otherwise going to just start reading release notes </on topic> | 15:57 |
| TheJulia | They elected a scientist to be their leader, it just makes me feel all warm and fuzzy. | 15:58 |
| TheJulia | JayF: hmmm... I woudn't just read the release notes, but there are some things which would be good to highlight | 16:02 |
| JayF | I mean like, I'm using rel notes to pick out highlights | 16:03 |
| JayF | not that I'll recite them as my presi :D | 16:03 |
| TheJulia | oh, no, I think that is totally cool | 16:03 |
| * TheJulia would have done the same | 16:03 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: Phase 1 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/856336 | 20:39 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Phase 2 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/857516 | 20:39 |
| TheJulia | JayF: the reason they didn't pass, is another patch raised the warnings to be fatal to operation | 20:40 |
| JayF | oh, heh | 20:48 |
| JayF | TheJulia: these are ready to merge (1 and 2) AFAYK, right? | 20:49 |
| * JayF going to give it a through review this afternoon | 20:49 | |
| opendevreview | Julia Kreger proposed openstack/ironic master: Phase 3 - SQLAlchemy 2.0 Compatability https://review.opendev.org/c/openstack/ironic/+/857932 | 20:56 |
| TheJulia | I think they are, 3rd *should* be good but just found an issue with the test in CI since walk_version is not happy locally | 20:58 |
| JayF | sounds good, I'll make sure they get a review before I leave today | 21:02 |
| TheJulia | ack | 21:04 |
| TheJulia | https://review.opendev.org/q/If69ce1cf2789d9d60fb8e544596cf7d29eab514d <-- I added ironic-week-prio to these so we can get them some review traction | 21:17 |
| opendevreview | Merged openstack/ironic master: Update release versions for zed https://review.opendev.org/c/openstack/ironic/+/859228 | 21:21 |
| JayF | stable patch that could use a review -> https://review.opendev.org/c/openstack/ironic/+/858903 | 21:52 |
| JayF | Hey, thanks to whoever at HPE updated the 3rd party CI job to only contain expected-to-pass jobs \o/ | 22:05 |
| JayF | that's really, really nice | 22:05 |
| TheJulia | \o/ | 22:06 |
| TheJulia | I'll guess Nisha or stendulker :) | 22:07 |
| JayF | TheJulia: https://zuul.opendev.org/t/openstack/build/980b5bbf04f149daa8f6bb5d9ffae41f this failure on that phase 2 patch makes me go hrm | 22:15 |
| JayF | TheJulia: given this is passing other places, I wonder if there's something race-y in there? | 22:16 |
| TheJulia | oh joy | 22:16 |
| TheJulia | I've seen that once but was hoping it was a pure fluke | 22:16 |
| TheJulia | It is likely rooted in the first patch as that one works on locking | 22:17 |
| TheJulia | I'll look tomorrow morning | 22:17 |
| JayF | yeah I'm propogating -1 back | 22:17 |
| JayF | if you want extra eyes lmk | 22:17 |
| opendevreview | Merged openstack/ironic stable/ussuri: Fix install of virtualbmc on ussuri (again) https://review.opendev.org/c/openstack/ironic/+/859189 | 22:30 |
| opendevreview | Merged openstack/ironic bugfix/21.0: Pin bugfix 21.0 ci jobs to zed https://review.opendev.org/c/openstack/ironic/+/859251 | 22:30 |
| opendevreview | Jay Faulkner proposed openstack/ironic stable/wallaby: Do not reboot into nowhere after BIOS settings with fast-track https://review.opendev.org/c/openstack/ironic/+/859234 | 23:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!