Friday, 2024-09-20

opendevreviewDan Smith proposed openstack/releases master: Release nova RC1 for 2024.2 Dalmatian  https://review.opendev.org/c/openstack/releases/+/92855101:10
dansmithelodilles: hopefully that's right^ ?01:10
opendevreviewRiccardo Pittau proposed openstack/releases master: Release Bifrost 19.1.0 for dalmatian  https://review.opendev.org/c/openstack/releases/+/92999607:19
elodillesdansmith bauzas gibi : the hash is not the latest anymore, otherwise the RC1 patch LGTM :)07:39
bauzaselodilles: lemme look07:39
bauzasdo we have a merged patch ?07:40
elodillesthis seems to be the latest: 9c36174539 (HEAD -> master, origin/master, origin/HEAD) Merge "nova-manage: modify image properties in request_spec"07:41
bauzashttps://review.opendev.org/c/openstack/nova/+/92431907:42
bauzasthis is the patch in question07:42
elodillesyepp07:43
elodillesit's a bugfix07:43
bauzasthat wasn't tracked as a critical bugfix but this is cheap now to just add it in RC107:43
bauzasso, either we rebase the RC1 to add it, or we just pin before and then we will backport it07:44
elodillesboth works for me07:44
elodillesdo you want me to update the hash?07:45
bauzasthat's cheap, it would be nice07:45
bauzasor maybe me ?07:46
bauzasis it okay if you update the hash and then you can +2 it ?07:46
elodillesplease update then :)07:46
bauzascool, doing it now07:46
opendevreviewSylvain Bauza proposed openstack/releases master: Release nova RC1 for 2024.2 Dalmatian  https://review.opendev.org/c/openstack/releases/+/92855107:47
bauzaselodilles: done ^07:48
bauzascan you please then reapply your +2 ?07:48
elodillesZuul isn't +1'd yet, but LGTM, so +2'd in advance :]07:49
* bauzas nods07:49
elodillesrelease-team: nova folks eagerly wait for a 2nd +2 for their RC1 patch: https://review.opendev.org/c/openstack/releases/+/92855107:56
elodilleswhen you have time07:57
ttxlooking08:06
elodillesthx ~o~08:06
ttx+2a08:06
elodilles\o/08:06
opendevreviewMerged openstack/releases master: Release nova RC1 for 2024.2 Dalmatian  https://review.opendev.org/c/openstack/releases/+/92855108:11
elodillesnow the post-release jobs08:15
bauzas\o/08:35
zigohberaud: In Debian Sid, I'm getting:09:26
zigoImportError: 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
zigodo you know what's the fix?09:26
zigoOh, shit, my package eventlet in Sid is completely wrong! :)09:28
*** bauzas_ is now known as bauzas10:11
elodillesrpittau: 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/+/92993410:28
*** ykarel__ is now known as ykarel10:33
opendevreviewMerged openstack/releases master: Release  ironic-python-agent 9.14.0 for dalmatian  https://review.opendev.org/c/openstack/releases/+/92993510:36
opendevreviewMerged openstack/releases master: Release Bifrost 19.1.0 for dalmatian  https://review.opendev.org/c/openstack/releases/+/92999610:36
ttxyes this is blocking the stable branch patch for ironic11:57
rpittauelodilles: I think you pasted the same link 3 times :)12:25
rpittauelodilles: 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 that12:29
rpittauthe patch does that in an explicit way12:30
elodillesrpittau: indeed :D sry. and thanks for the double check, +2+W'd ^^^12:33
rpittauelodilles: thanks! :)12:33
elodillesrpittau: this should be the 2nd link: https://review.opendev.org/c/openstack/releases/+/92976412:33
elodillesrpittau: and the 3rd: https://review.opendev.org/c/openstack/releases/+/92830512:34
rpittauelodilles: I will udpate https://review.opendev.org/c/openstack/releases/+/929764 now, thanks12:34
elodillesreminder: weekly meeting in less than 30 mins!12:34
elodillesrpittau: ACK, thx too12:35
opendevreviewRiccardo Pittau proposed openstack/releases master: Release ironic-inspector 12.3.0 for dalmatian  https://review.opendev.org/c/openstack/releases/+/92976412:36
rpittauI think I got them all :D12:36
opendevreviewMerged openstack/releases master: Release ironic 26.1.0 for dalmatian  https://review.opendev.org/c/openstack/releases/+/92993412:41
*** ykarel_ is now known as ykarel12:41
opendevreviewThierry Carrez proposed openstack/releases master: stable/2024.2 branches for Ironic deliverables  https://review.opendev.org/c/openstack/releases/+/92940912:46
ttxAlright patch updated with all latest ironic releases12:47
ttxHmm, we still need +2a on https://review.opendev.org/c/openstack/releases/+/92976412:49
opendevreviewRiccardo Pittau proposed openstack/releases master: Release ironic-tempest-plugin for 2024.2 Dalmatian  https://review.opendev.org/c/openstack/releases/+/92952612:50
opendevreviewThierry Carrez proposed openstack/releases master: Release ironic-inspector 12.3.0 for dalmatian  https://review.opendev.org/c/openstack/releases/+/92976412:54
opendevreviewThierry Carrez proposed openstack/releases master: stable/2024.2 branches for Ironic deliverables  https://review.opendev.org/c/openstack/releases/+/92940912:54
ttxarth12:54
ttxmy rebase dropped the approvals12:55
ttxbut at least now the branch cut can be reviewed as it's built on top of the lest required release12:55
ttxso please rereview the https://review.opendev.org/c/openstack/releases/+/929764 and https://review.opendev.org/c/openstack/releases/+/92940912:56
opendevreviewElod Illes proposed openstack/releases master: [ironic] Add automatic_lessee to cycle highlights  https://review.opendev.org/c/openstack/releases/+/92830512:57
elodillesrpittau: rebased to eliminate merge conflict ^^^12:57
rpittauthanks elodilles +1 :)12:59
ttxelodilles: you should be fine just w+1 https://review.opendev.org/c/openstack/releases/+/92976412:59
ttxrpittau: please +1 https://review.opendev.org/c/openstack/releases/+/92940913:00
elodillesdone o/13:00
elodilles#startmeeting releaseteam13:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.13:00
opendevmeetThe meeting name has been set to 'releaseteam'13:00
elodillesPing list: release-team elod armstrong 13:00
elodilles#link https://etherpad.opendev.org/p/dalmatian-relmgt-tracking13:00
elodilleswe are @ L39713:01
ttxo/13:01
elodilles~o~13:01
elodilleslet's start!13:02
elodilles#topic     Review task completion13:02
elodilles1st task was:13:02
elodilles'    On Monday, test the release announcement fix (ttx)'13:02
elodillesthanks ttx , as we saw, worked well o/13:03
ttxyes that was done13:03
elodilles2nd task:13:03
elodilles'    On Monday, finally Approve RC1 releases that did not get a -1 (ttx)'13:03
elodilleswe got a lot of patches merged13:03
elodillesunfortunately there are still some that have broken gates and lacks fixes :(13:04
elodillesthose are:13:04
elodilleskuryr-kubernetes, kuryr-libnetwork, zun, ansible-role-thales-hsm13:05
ttxOK let me copy those forward to next week tasks13:05
elodillesthanks ttx , 13:05
elodillesnote that we should think about what to do if we won't get updates there13:05
elodillesafaik kuryr is about to move to 'inactive'13:06
ttxdo any of those have someone actively working on them?13:06
elodilles(kuryr was taken over by zun team)13:06
elodillesttx: no, that's the problem13:06
elodillesi've seen a comment somewhere that we might need to drop them from the release13:07
elodillesthough we are late in the cycle :(13:07
fungiwhere was the comment?13:07
ttxif we don;t have a release we can't release it13:07
fungithat certainly seem... tautological yes13:07
elodillesfungi: i have to look for it13:08
elodillesttx: the question is whether we release them anyway, with broken gates,13:08
elodilleswe had to do this in the past once or twice :(13:08
ttxansible-role-thales-hsm is a barbican thing?13:08
elodillesyep, barbican13:08
ttxI guess we still have some time to get a fix there, can land late13:09
ttxZun/Kuryr is more of a problem13:09
elodillesfungi: here is the comment from gmann https://review.opendev.org/c/openstack/releases/+/928531/1#message-347ee36d4b8404e941ddceb2026a998423163f6f13:09
ttxWas there a PTL elected there?13:09
fungithanks elodilles13:09
elodillesttx: for zun, yes13:10
ttxStill Hongbin?13:11
elodillesi think so, but to be honest, i say it because our script added they13:12
elodillesas reviewer13:12
ttxThere was a big holiday in China a few days ago13:12
elodillesah, i see13:13
elodillesi can try to drop a mail to hongbin to ask for some help13:13
ttxmaybe we can get in touch with him via other channels. I'll see what I can do13:13
elodillesttx: ACK, thanks!13:13
elodillesanything else to this task?13:15
ttxelodilles: you can send Hongbin an email13:15
elodillesttx: ACK, will do then!13:15
ttxdetailing all the issues and the risk to be dropped from release13:15
elodilles#action elod to send a mail to Zun PTL detailing all the issues and the risk to be dropped from release13:16
ttxI'll ask Horace our China community manager to ping him on wechat and other solutions to pay attention to his email13:16
opendevreviewMerged openstack/releases master: Release ironic-inspector 12.3.0 for dalmatian  https://review.opendev.org/c/openstack/releases/+/92976413:16
elodillesttx: sounds like a plan! thanks!13:16
elodillesi think that's what we can do about this13:17
elodillesso let's move on13:17
elodilles3rd 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-latest13:17
elodillesi'll double check all the patches after the meeting and i think we should +2+W them as deadline has passed ^^^13:18
ttxwait, what about the ansible HSM thing? Did we ping more directly the Barbican PTL?13:19
elodilleseither today, or if we don't want Friday-releases, then early Monday13:19
elodillesttx: not yet, at least not me13:19
elodillesttx: but i can do that as well13:20
ttxyeah... two separate emails13:20
ttxdirect, not list13:20
elodillesttx: ACK13:20
ttx(in retrospect we should probably have done that a week ago :) )13:21
elodillesttx: yeah :(13:21
fungihe's xek in #openstack-barbican13:21
elodillesACK13:21
fungi"idle: 3 days, 22 hours"13:21
fungiso maybe not around this week13:21
ttxOK once the email is sent we can add a ping there13:22
ttxfire on all channels13:22
elodilles+113:22
elodillescan we move on?13:26
ttxyes!13:26
elodillesok, sorry :)13:26
elodillesso the still open tempest-plugin release patches needs procedural PTL-Approved+1s + reviews + approvals if the patches are still correct13:27
ttxfwiw the ansible thing looks easy enough to fix13:27
elodilleseither today or early Monday13:27
elodillesttx: that's promising then13:27
ttxI might do it on Monday 13:28
elodillesACK13:28
elodilles4th task was:13:28
elodilles'    On the Monday, generate stable branches for all cycle deliverables that are still missing one (ttx)'13:28
ttxbut would be better if someone who knows what they are doing fixed it13:28
ttxyes, so that one resulted in :13:28
ttx#link https://review.opendev.org/c/openstack/releases/+/92940913:29
ttxwhich should be ready to merge about now13:29
ttxyou can +2a it it will be processed after the inspector release13:29
elodillesinspector released already merged :)13:30
elodilless/released/patch/13:30
ttxalright +2a13:30
elodilles~o~13:30
elodillescool13:30
elodilles'    After all the projects enabled in devstack by default have been branched, we can engage with the (elod)'13:31
elodillesso13:31
elodillesabout this task ^^^13:31
elodillesi haven't started it as we were not there yet13:31
ttxmaybe push it to Monday?13:31
elodillesACK, adding it to next week13:32
dansmithelodilles: well, it was the latest when I updated that patch a few minutes after it landed :/13:32
elodillesdansmith: yeah, i saw :) but i was sleeping at that time o:) (sorry, we are on meeting now)13:33
dansmith(oh sorry)13:33
elodilles6th 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-releasenotes13:34
elodillesstill not all, but quite many reno link added already13:34
elodillesand we have a task for release day to do another round13:35
elodillesso should be OK (and I'll run another round before that, to land them earlier)13:36
elodillesand 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
ttxmaybe that was solved by the pin13:37
elodilleshas anyone see what happened with storlets?13:37
elodillesi don't remember seeing RC2 from storlets13:38
ttxtkajinam might know if anything more is needed there13:38
elodillesoh, you mean the python-openstackclient<7.0.0 capping "solved" storlets problem as well?13:39
opendevreviewMerged openstack/releases master: stable/2024.2 branches for Ironic deliverables  https://review.opendev.org/c/openstack/releases/+/92940913:39
elodillesi see that one patch merged to storlets stable/2024.213:40
ttxwell if the issue was a 7.x regression I bet it did solve it13:41
elodillesbut anyway, next week is final rc week, so it is still possible to do an RC2 if needed13:41
ttxyes, I'd let it flow13:42
elodilles+113:42
elodillesabout python-openstackclient: it is capped to <7.0.0 && 7.1.2 released, just in case... but cap is not removed13:43
elodillesi wonder if we might be able to remove the cap next week?13:43
elodillesor is that a far-fetched dream of mine? :)13:43
fungiit should be possible to push a revert of the cap and then depends-on it with dnm changes in nova and storlets13:45
ttxhmm13:45
ttxShouldn't the cap stay on stable/2024.2?13:45
funginot if we want it to reflect the versions on the releases site13:46
ttxwouldn't that break Nova again? I mean, we did cap it for a reason13:46
fungisince we're saying python-openstackclient 7.1.2 is the 2024.4/dalmatian version13:46
fungii thought 7.1.2 had the (latest) presumed fix for nova's issue13:46
elodillesttx: 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
ttxi thought we did not have enough trust in current OSC to ship it as a dependency13:47
elodillesand of course, if there are objection from any team, then we stay with capping <7.0.013:47
ttxoh sure13:48
elodillesfungi: yes, you are right, 7.1.2 meant to fix the nova issue13:48
fungiwhere 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 asap13:48
fungiand that we could possibly lift the cap again either during the rc period or even after release day13:49
elodillesyes, that was my understanding as well13:49
elodillesanyway, we can test that next week13:49
fungiso 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 usable13:49
elodilleslet's move on as time is flying13:49
elodillesfungi: +113:50
elodilles#topic     Assign R-1 week tasks13:50
elodillesas i see all tasks taken13:50
elodillesthx ttx o/13:50
elodilles#topic     Review countdown email for week R-113:51
elodilles#link https://etherpad.opendev.org/p/relmgmt-weekly-emails13:51
elodillesplease review ^^^13:51
ttxfwiw 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 meeting13:51
elodillesttx: will do13:52
ttxemail looks good after a couple fixes13:53
elodillescool, thanks13:54
elodilles#topic     Open Discussion13:54
elodillesanything to discuss?13:54
ttxI started filling the "things to change" section of the doc13:54
ttxRe: PTG I booked a run at our usual meeting time13:54
ttxbooked a room13:55
elodilles+1+1+113:55
ttxthat is all from me13:55
elodillesand thanks :)13:55
fungian 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 notes13:55
ttxlet me know when the Hongbin email is sent so that I let them know to expect it13:55
elodillesfungi: ack, thanks for letting us know13:56
elodillesttx: ACK13:56
ttxlooks like uncap was proposed at https://review.opendev.org/c/openstack/requirements/+/92993813:57
elodilleswell, 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
elodillesbut sure, we can use that for testing :)13:59
elodillesanything else before we run out of time? :)13:59
fungithe comments say it's already tested from a nova depends-on14:00
fungii commented suggesting doing the same for storlets14:00
elodillesgood, even better then \o/14:00
elodilleslet's close the meeting, but we can continue the discussion after the meeting14:01
elodillesthanks everyone for participating o/14:01
elodilles#endmeeting14:01
opendevmeetMeeting ended Fri Sep 20 14:01:15 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:01
opendevmeetMinutes:        https://meetings.opendev.org/meetings/releaseteam/2024/releaseteam.2024-09-20-13.00.html14:01
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/releaseteam/2024/releaseteam.2024-09-20-13.00.txt14:01
opendevmeetLog:            https://meetings.opendev.org/meetings/releaseteam/2024/releaseteam.2024-09-20-13.00.log.html14:01
* elodilles start to compose the mail to Hongbin14:01
ttxyou can just cc me -- I'll let him know14:02
ttxthanks elodilles !14:02
tkajinamsorry I was afk during the meeting14:03
tkajinamstorlets 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 rc214:03
ttxtkajinam: perfect14:04
elodillestkajinam: thanks for the info!14:14
elodillesttx: mail sent14:14
ttxHongbin said he will follow up on Zun and Libnetwork. That said it's the weekend there already so maybe not immediate progress15:02
elodillesACK15:24
fungihttps://review.opendev.org/923072 is related to the earlier kuryr-kubernetes and kuryr-tempest-plugin discussion15:39
gmannthose are still not retired yet because of tacker dependencies and tacker team is working on that16:35
gmannthis one https://review.opendev.org/c/openstack/tacker/+/92650416:36
gmannbut it is clear that these deliverables are not maintained and about to be retired 16:36
gmannIMO, we do not need to release them as such16:37
opendevreviewGhanshyam proposed openstack/releases master: Release Tempest 41.0.0 as final tag for 2024.2  https://review.opendev.org/c/openstack/releases/+/93009321:50
*** bauzas_ is now known as bauzas22:07

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!