opendevreview | Dan Smith proposed openstack/releases master: Release nova RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928551 | 01:10 |
---|---|---|
dansmith | elodilles: hopefully that's right^ ? | 01:10 |
opendevreview | Riccardo Pittau proposed openstack/releases master: Release Bifrost 19.1.0 for dalmatian https://review.opendev.org/c/openstack/releases/+/929996 | 07:19 |
elodilles | dansmith bauzas gibi : the hash is not the latest anymore, otherwise the RC1 patch LGTM :) | 07:39 |
bauzas | elodilles: lemme look | 07:39 |
bauzas | do we have a merged patch ? | 07:40 |
elodilles | this seems to be the latest: 9c36174539 (HEAD -> master, origin/master, origin/HEAD) Merge "nova-manage: modify image properties in request_spec" | 07:41 |
bauzas | https://review.opendev.org/c/openstack/nova/+/924319 | 07:42 |
bauzas | this is the patch in question | 07:42 |
elodilles | yepp | 07:43 |
elodilles | it's a bugfix | 07:43 |
bauzas | that wasn't tracked as a critical bugfix but this is cheap now to just add it in RC1 | 07:43 |
bauzas | so, either we rebase the RC1 to add it, or we just pin before and then we will backport it | 07:44 |
elodilles | both works for me | 07:44 |
elodilles | do you want me to update the hash? | 07:45 |
bauzas | that's cheap, it would be nice | 07:45 |
bauzas | or maybe me ? | 07:46 |
bauzas | is it okay if you update the hash and then you can +2 it ? | 07:46 |
elodilles | please update then :) | 07:46 |
bauzas | cool, doing it now | 07:46 |
opendevreview | Sylvain Bauza proposed openstack/releases master: Release nova RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928551 | 07:47 |
bauzas | elodilles: done ^ | 07:48 |
bauzas | can you please then reapply your +2 ? | 07:48 |
elodilles | Zuul isn't +1'd yet, but LGTM, so +2'd in advance :] | 07:49 |
* bauzas nods | 07:49 | |
elodilles | release-team: nova folks eagerly wait for a 2nd +2 for their RC1 patch: https://review.opendev.org/c/openstack/releases/+/928551 | 07:56 |
elodilles | when you have time | 07:57 |
ttx | looking | 08:06 |
elodilles | thx ~o~ | 08:06 |
ttx | +2a | 08:06 |
elodilles | \o/ | 08:06 |
opendevreview | Merged openstack/releases master: Release nova RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928551 | 08:11 |
elodilles | now the post-release jobs | 08:15 |
bauzas | \o/ | 08:35 |
zigo | hberaud: In Debian Sid, I'm getting: | 09:26 |
zigo | ImportError: cannot import name 'hubs' from partially initialized module 'eventlet' (most likely due to a circular import) (/usr/lib/python3/dist-packages/eventlet/__init__.py) | 09:26 |
zigo | do you know what's the fix? | 09:26 |
zigo | Oh, shit, my package eventlet in Sid is completely wrong! :) | 09:28 |
*** bauzas_ is now known as bauzas | 10:11 | |
elodilles | rpittau: sorry, could you please double check the following patches? https://review.opendev.org/c/openstack/releases/+/929934 and https://review.opendev.org/c/openstack/releases/+/929934 and https://review.opendev.org/c/openstack/releases/+/929934 | 10:28 |
*** ykarel__ is now known as ykarel | 10:33 | |
opendevreview | Merged openstack/releases master: Release ironic-python-agent 9.14.0 for dalmatian https://review.opendev.org/c/openstack/releases/+/929935 | 10:36 |
opendevreview | Merged openstack/releases master: Release Bifrost 19.1.0 for dalmatian https://review.opendev.org/c/openstack/releases/+/929996 | 10:36 |
ttx | yes this is blocking the stable branch patch for ironic | 11:57 |
rpittau | elodilles: I think you pasted the same link 3 times :) | 12:25 |
rpittau | elodilles: I don't think we need a major bump in https://review.opendev.org/c/openstack/releases/+/929934, that note was added for completeness but it was already like that | 12:29 |
rpittau | the patch does that in an explicit way | 12:30 |
elodilles | rpittau: indeed :D sry. and thanks for the double check, +2+W'd ^^^ | 12:33 |
rpittau | elodilles: thanks! :) | 12:33 |
elodilles | rpittau: this should be the 2nd link: https://review.opendev.org/c/openstack/releases/+/929764 | 12:33 |
elodilles | rpittau: and the 3rd: https://review.opendev.org/c/openstack/releases/+/928305 | 12:34 |
rpittau | elodilles: I will udpate https://review.opendev.org/c/openstack/releases/+/929764 now, thanks | 12:34 |
elodilles | reminder: weekly meeting in less than 30 mins! | 12:34 |
elodilles | rpittau: ACK, thx too | 12:35 |
opendevreview | Riccardo Pittau proposed openstack/releases master: Release ironic-inspector 12.3.0 for dalmatian https://review.opendev.org/c/openstack/releases/+/929764 | 12:36 |
rpittau | I think I got them all :D | 12:36 |
opendevreview | Merged openstack/releases master: Release ironic 26.1.0 for dalmatian https://review.opendev.org/c/openstack/releases/+/929934 | 12:41 |
*** ykarel_ is now known as ykarel | 12:41 | |
opendevreview | Thierry Carrez proposed openstack/releases master: stable/2024.2 branches for Ironic deliverables https://review.opendev.org/c/openstack/releases/+/929409 | 12:46 |
ttx | Alright patch updated with all latest ironic releases | 12:47 |
ttx | Hmm, we still need +2a on https://review.opendev.org/c/openstack/releases/+/929764 | 12:49 |
opendevreview | Riccardo Pittau proposed openstack/releases master: Release ironic-tempest-plugin for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/929526 | 12:50 |
opendevreview | Thierry Carrez proposed openstack/releases master: Release ironic-inspector 12.3.0 for dalmatian https://review.opendev.org/c/openstack/releases/+/929764 | 12:54 |
opendevreview | Thierry Carrez proposed openstack/releases master: stable/2024.2 branches for Ironic deliverables https://review.opendev.org/c/openstack/releases/+/929409 | 12:54 |
ttx | arth | 12:54 |
ttx | my rebase dropped the approvals | 12:55 |
ttx | but at least now the branch cut can be reviewed as it's built on top of the lest required release | 12:55 |
ttx | so please rereview the https://review.opendev.org/c/openstack/releases/+/929764 and https://review.opendev.org/c/openstack/releases/+/929409 | 12:56 |
opendevreview | Elod Illes proposed openstack/releases master: [ironic] Add automatic_lessee to cycle highlights https://review.opendev.org/c/openstack/releases/+/928305 | 12:57 |
elodilles | rpittau: rebased to eliminate merge conflict ^^^ | 12:57 |
rpittau | thanks elodilles +1 :) | 12:59 |
ttx | elodilles: you should be fine just w+1 https://review.opendev.org/c/openstack/releases/+/929764 | 12:59 |
ttx | rpittau: please +1 https://review.opendev.org/c/openstack/releases/+/929409 | 13:00 |
elodilles | done o/ | 13:00 |
elodilles | #startmeeting releaseteam | 13:00 |
opendevmeet | Meeting started Fri Sep 20 13:00:31 2024 UTC and is due to finish in 60 minutes. The chair is elodilles. Information about MeetBot at http://wiki.debian.org/MeetBot. | 13:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 13:00 |
opendevmeet | The meeting name has been set to 'releaseteam' | 13:00 |
elodilles | Ping list: release-team elod armstrong | 13:00 |
elodilles | #link https://etherpad.opendev.org/p/dalmatian-relmgt-tracking | 13:00 |
elodilles | we are @ L397 | 13:01 |
ttx | o/ | 13:01 |
elodilles | ~o~ | 13:01 |
elodilles | let's start! | 13:02 |
elodilles | #topic Review task completion | 13:02 |
elodilles | 1st task was: | 13:02 |
elodilles | ' On Monday, test the release announcement fix (ttx)' | 13:02 |
elodilles | thanks ttx , as we saw, worked well o/ | 13:03 |
ttx | yes that was done | 13:03 |
elodilles | 2nd task: | 13:03 |
elodilles | ' On Monday, finally Approve RC1 releases that did not get a -1 (ttx)' | 13:03 |
elodilles | we got a lot of patches merged | 13:03 |
elodilles | unfortunately there are still some that have broken gates and lacks fixes :( | 13:04 |
elodilles | those are: | 13:04 |
elodilles | kuryr-kubernetes, kuryr-libnetwork, zun, ansible-role-thales-hsm | 13:05 |
ttx | OK let me copy those forward to next week tasks | 13:05 |
elodilles | thanks ttx , | 13:05 |
elodilles | note that we should think about what to do if we won't get updates there | 13:05 |
elodilles | afaik kuryr is about to move to 'inactive' | 13:06 |
ttx | do any of those have someone actively working on them? | 13:06 |
elodilles | (kuryr was taken over by zun team) | 13:06 |
elodilles | ttx: no, that's the problem | 13:06 |
elodilles | i've seen a comment somewhere that we might need to drop them from the release | 13:07 |
elodilles | though we are late in the cycle :( | 13:07 |
fungi | where was the comment? | 13:07 |
ttx | if we don;t have a release we can't release it | 13:07 |
fungi | that certainly seem... tautological yes | 13:07 |
elodilles | fungi: i have to look for it | 13:08 |
elodilles | ttx: the question is whether we release them anyway, with broken gates, | 13:08 |
elodilles | we had to do this in the past once or twice :( | 13:08 |
ttx | ansible-role-thales-hsm is a barbican thing? | 13:08 |
elodilles | yep, barbican | 13:08 |
ttx | I guess we still have some time to get a fix there, can land late | 13:09 |
ttx | Zun/Kuryr is more of a problem | 13:09 |
elodilles | fungi: here is the comment from gmann https://review.opendev.org/c/openstack/releases/+/928531/1#message-347ee36d4b8404e941ddceb2026a998423163f6f | 13:09 |
ttx | Was there a PTL elected there? | 13:09 |
fungi | thanks elodilles | 13:09 |
elodilles | ttx: for zun, yes | 13:10 |
ttx | Still Hongbin? | 13:11 |
elodilles | i think so, but to be honest, i say it because our script added they | 13:12 |
elodilles | as reviewer | 13:12 |
ttx | There was a big holiday in China a few days ago | 13:12 |
elodilles | ah, i see | 13:13 |
elodilles | i can try to drop a mail to hongbin to ask for some help | 13:13 |
ttx | maybe we can get in touch with him via other channels. I'll see what I can do | 13:13 |
elodilles | ttx: ACK, thanks! | 13:13 |
elodilles | anything else to this task? | 13:15 |
ttx | elodilles: you can send Hongbin an email | 13:15 |
elodilles | ttx: ACK, will do then! | 13:15 |
ttx | detailing all the issues and the risk to be dropped from release | 13:15 |
elodilles | #action elod to send a mail to Zun PTL detailing all the issues and the risk to be dropped from release | 13:16 |
ttx | I'll ask Horace our China community manager to ping him on wechat and other solutions to pay attention to his email | 13:16 |
opendevreview | Merged openstack/releases master: Release ironic-inspector 12.3.0 for dalmatian https://review.opendev.org/c/openstack/releases/+/929764 | 13:16 |
elodilles | ttx: sounds like a plan! thanks! | 13:16 |
elodilles | i think that's what we can do about this | 13:17 |
elodilles | so let's move on | 13:17 |
elodilles | 3rd task was: | 13:17 |
elodilles | ' Gracefully release tempest plugins with latest changes available. (elod) | 13:17 |
elodilles | #link https://review.opendev.org/q/topic:dalmatian-tp-latest | 13:17 |
elodilles | i'll double check all the patches after the meeting and i think we should +2+W them as deadline has passed ^^^ | 13:18 |
ttx | wait, what about the ansible HSM thing? Did we ping more directly the Barbican PTL? | 13:19 |
elodilles | either today, or if we don't want Friday-releases, then early Monday | 13:19 |
elodilles | ttx: not yet, at least not me | 13:19 |
elodilles | ttx: but i can do that as well | 13:20 |
ttx | yeah... two separate emails | 13:20 |
ttx | direct, not list | 13:20 |
elodilles | ttx: ACK | 13:20 |
ttx | (in retrospect we should probably have done that a week ago :) ) | 13:21 |
elodilles | ttx: yeah :( | 13:21 |
fungi | he's xek in #openstack-barbican | 13:21 |
elodilles | ACK | 13:21 |
fungi | "idle: 3 days, 22 hours" | 13:21 |
fungi | so maybe not around this week | 13:21 |
ttx | OK once the email is sent we can add a ping there | 13:22 |
ttx | fire on all channels | 13:22 |
elodilles | +1 | 13:22 |
elodilles | can we move on? | 13:26 |
ttx | yes! | 13:26 |
elodilles | ok, sorry :) | 13:26 |
elodilles | so the still open tempest-plugin release patches needs procedural PTL-Approved+1s + reviews + approvals if the patches are still correct | 13:27 |
ttx | fwiw the ansible thing looks easy enough to fix | 13:27 |
elodilles | either today or early Monday | 13:27 |
elodilles | ttx: that's promising then | 13:27 |
ttx | I might do it on Monday | 13:28 |
elodilles | ACK | 13:28 |
elodilles | 4th task was: | 13:28 |
elodilles | ' On the Monday, generate stable branches for all cycle deliverables that are still missing one (ttx)' | 13:28 |
ttx | but would be better if someone who knows what they are doing fixed it | 13:28 |
ttx | yes, so that one resulted in : | 13:28 |
ttx | #link https://review.opendev.org/c/openstack/releases/+/929409 | 13:29 |
ttx | which should be ready to merge about now | 13:29 |
ttx | you can +2a it it will be processed after the inspector release | 13:29 |
elodilles | inspector released already merged :) | 13:30 |
elodilles | s/released/patch/ | 13:30 |
ttx | alright +2a | 13:30 |
elodilles | ~o~ | 13:30 |
elodilles | cool | 13:30 |
elodilles | ' After all the projects enabled in devstack by default have been branched, we can engage with the (elod)' | 13:31 |
elodilles | so | 13:31 |
elodilles | about this task ^^^ | 13:31 |
elodilles | i haven't started it as we were not there yet | 13:31 |
ttx | maybe push it to Monday? | 13:31 |
elodilles | ACK, adding it to next week | 13:32 |
dansmith | elodilles: well, it was the latest when I updated that patch a few minutes after it landed :/ | 13:32 |
elodilles | dansmith: yeah, i saw :) but i was sleeping at that time o:) (sorry, we are on meeting now) | 13:33 |
dansmith | (oh sorry) | 13:33 |
elodilles | 6th task: | 13:34 |
elodilles | ' Ensure that all projects that are publishing release notes have the notes link included in their deliverable file. (elod)' | 13:34 |
elodilles | #link https://review.opendev.org/q/topic:dalmatian-releasenotes | 13:34 |
elodilles | still not all, but quite many reno link added already | 13:34 |
elodilles | and we have a task for release day to do another round | 13:35 |
elodilles | so should be OK (and I'll run another round before that, to land them earlier) | 13:36 |
elodilles | and a semi-task was: | 13:36 |
elodilles | ' Let cycle-with-rc projects iterate on RCs as needed. The final release candidate for each project needs to be prepared at least one week before the final release date. (all / guideline)' | 13:36 |
elodilles | "storlets needs a RC2 to deal with https://bugs.launchpad.net/python-openstackclient/+bug/2080600" | 13:36 |
elodilles | "probably means we need an python-openstackclient release too" | 13:37 |
ttx | maybe that was solved by the pin | 13:37 |
elodilles | has anyone see what happened with storlets? | 13:37 |
elodilles | i don't remember seeing RC2 from storlets | 13:38 |
ttx | tkajinam might know if anything more is needed there | 13:38 |
elodilles | oh, you mean the python-openstackclient<7.0.0 capping "solved" storlets problem as well? | 13:39 |
opendevreview | Merged openstack/releases master: stable/2024.2 branches for Ironic deliverables https://review.opendev.org/c/openstack/releases/+/929409 | 13:39 |
elodilles | i see that one patch merged to storlets stable/2024.2 | 13:40 |
ttx | well if the issue was a 7.x regression I bet it did solve it | 13:41 |
elodilles | but anyway, next week is final rc week, so it is still possible to do an RC2 if needed | 13:41 |
ttx | yes, I'd let it flow | 13:42 |
elodilles | +1 | 13:42 |
elodilles | about python-openstackclient: it is capped to <7.0.0 && 7.1.2 released, just in case... but cap is not removed | 13:43 |
elodilles | i wonder if we might be able to remove the cap next week? | 13:43 |
elodilles | or is that a far-fetched dream of mine? :) | 13:43 |
fungi | it should be possible to push a revert of the cap and then depends-on it with dnm changes in nova and storlets | 13:45 |
ttx | hmm | 13:45 |
ttx | Shouldn't the cap stay on stable/2024.2? | 13:45 |
fungi | not if we want it to reflect the versions on the releases site | 13:46 |
ttx | wouldn't that break Nova again? I mean, we did cap it for a reason | 13:46 |
fungi | since we're saying python-openstackclient 7.1.2 is the 2024.4/dalmatian version | 13:46 |
fungi | i thought 7.1.2 had the (latest) presumed fix for nova's issue | 13:46 |
elodilles | ttx: it might, i haven't looked closely whether nova folks merged a fix at the end. but as fungi says, we have to check it with a DNM patch maybe. | 13:47 |
ttx | i thought we did not have enough trust in current OSC to ship it as a dependency | 13:47 |
elodilles | and of course, if there are objection from any team, then we stay with capping <7.0.0 | 13:47 |
ttx | oh sure | 13:48 |
elodilles | fungi: yes, you are right, 7.1.2 meant to fix the nova issue | 13:48 |
fungi | where i thought we ended up was that the <7 cap was a hopefully temporary measure to unblock nova's tests so they could get rc1 tagged asap | 13:48 |
fungi | and that we could possibly lift the cap again either during the rc period or even after release day | 13:49 |
elodilles | yes, that was my understanding as well | 13:49 |
elodilles | anyway, we can test that next week | 13:49 |
fungi | so that the stable/2024.2 branch is still viable. otherwise how do we, say, backport a security fix there if we don't consider it usable | 13:49 |
elodilles | let's move on as time is flying | 13:49 |
elodilles | fungi: +1 | 13:50 |
elodilles | #topic Assign R-1 week tasks | 13:50 |
elodilles | as i see all tasks taken | 13:50 |
elodilles | thx ttx o/ | 13:50 |
elodilles | #topic Review countdown email for week R-1 | 13:51 |
elodilles | #link https://etherpad.opendev.org/p/relmgmt-weekly-emails | 13:51 |
elodilles | please review ^^^ | 13:51 |
ttx | fwiw Horace is spaeking with Hongbin on wechat right now. I pointed them to the dnm patches, but would be good to sent the more complete email soon after meeting | 13:51 |
elodilles | ttx: will do | 13:52 |
ttx | email looks good after a couple fixes | 13:53 |
elodilles | cool, thanks | 13:54 |
elodilles | #topic Open Discussion | 13:54 |
elodilles | anything to discuss? | 13:54 |
ttx | I started filling the "things to change" section of the doc | 13:54 |
ttx | Re: PTG I booked a run at our usual meeting time | 13:54 |
ttx | booked a room | 13:55 |
elodilles | +1+1+1 | 13:55 |
ttx | that is all from me | 13:55 |
elodilles | and thanks :) | 13:55 |
fungi | an early heads up that i'll probably miss the release team session at the ptg, i've got some more family travel that's come up on short notice which will impact my ptg friday, but i'll catch up from the notes | 13:55 |
ttx | let me know when the Hongbin email is sent so that I let them know to expect it | 13:55 |
elodilles | fungi: ack, thanks for letting us know | 13:56 |
elodilles | ttx: ACK | 13:56 |
ttx | looks like uncap was proposed at https://review.opendev.org/c/openstack/requirements/+/929938 | 13:57 |
elodilles | well, it's a bit confusing as we haven't branched yet with requirements repo, but the uncap needs to be tested there (though stable/2024.2 is still master for requirements) | 13:58 |
elodilles | but sure, we can use that for testing :) | 13:59 |
elodilles | anything else before we run out of time? :) | 13:59 |
fungi | the comments say it's already tested from a nova depends-on | 14:00 |
fungi | i commented suggesting doing the same for storlets | 14:00 |
elodilles | good, even better then \o/ | 14:00 |
elodilles | let's close the meeting, but we can continue the discussion after the meeting | 14:01 |
elodilles | thanks everyone for participating o/ | 14:01 |
elodilles | #endmeeting | 14:01 |
opendevmeet | Meeting ended Fri Sep 20 14:01:15 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:01 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/releaseteam/2024/releaseteam.2024-09-20-13.00.html | 14:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/releaseteam/2024/releaseteam.2024-09-20-13.00.txt | 14:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/releaseteam/2024/releaseteam.2024-09-20-13.00.log.html | 14:01 |
* elodilles start to compose the mail to Hongbin | 14:01 | |
ttx | you can just cc me -- I'll let him know | 14:02 |
ttx | thanks elodilles ! | 14:02 |
tkajinam | sorry I was afk during the meeting | 14:03 |
tkajinam | storlets CI was fixed by https://review.opendev.org/c/openstack/storlets/+/929335 . (now osc was downgraded so we may not need it, though ). Because the change is made only in devstack script I don't think we need rc2 | 14:03 |
ttx | tkajinam: perfect | 14:04 |
elodilles | tkajinam: thanks for the info! | 14:14 |
elodilles | ttx: mail sent | 14:14 |
ttx | Hongbin said he will follow up on Zun and Libnetwork. That said it's the weekend there already so maybe not immediate progress | 15:02 |
elodilles | ACK | 15:24 |
fungi | https://review.opendev.org/923072 is related to the earlier kuryr-kubernetes and kuryr-tempest-plugin discussion | 15:39 |
gmann | those are still not retired yet because of tacker dependencies and tacker team is working on that | 16:35 |
gmann | this one https://review.opendev.org/c/openstack/tacker/+/926504 | 16:36 |
gmann | but it is clear that these deliverables are not maintained and about to be retired | 16:36 |
gmann | IMO, we do not need to release them as such | 16:37 |
opendevreview | Ghanshyam proposed openstack/releases master: Release Tempest 41.0.0 as final tag for 2024.2 https://review.opendev.org/c/openstack/releases/+/930093 | 21:50 |
*** bauzas_ is now known as bauzas | 22:07 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!