Friday, 2021-09-10

*** amoralej|off is now known as amoralej06:54
*** udesale is now known as udesale|holiday06:58
opendevreviewMartin Kopec proposed openstack/releases master: Release devstack-tools 1.2.1  https://review.opendev.org/c/openstack/releases/+/80808708:22
opendevreviewIury Gregory Melo Ferreira proposed openstack/releases master: Cycle With Intermediary Unreleased Ironic  https://review.opendev.org/c/openstack/releases/+/80818609:11
opendevreviewIury Gregory Melo Ferreira proposed openstack/releases master: Release sushy-tools 0.18.0  https://review.opendev.org/c/openstack/releases/+/80818809:19
iurygregoryhberaud, hey fyi we are trying to get a few patches in the ironic-python-agent-builder (CI is a bit broken due to Debian) we are trying to fix things to request the release next week09:21
hberaudiurygregory: ack thanks for the heads up09:21
iurygregorynp =)09:21
hberaudiurygregory: [FYI] here are a couple of related info: next week will propose releases for deliverables that are "cycle-with-intermediary" and "type:other" (https://releases.openstack.org/reference/process.html#r-3-week-rc1-deadline), that's the case of ironic-python-agent-builder, so that's not too late to merge patches. Also we should notice that IPAB didn't yet received release for09:28
hberaudXena09:28
hberaudhttps://opendev.org/openstack/releases/src/branch/master/deliverables/xena/ironic-python-agent-builder.yaml09:29
iurygregoryack o/09:29
iurygregoryhberaud, I'm wondering why https://review.opendev.org/c/openstack/releases/+/808186 complains about ironic ui with 4.4.009:35
iurygregoryin wallaby we have 4.3.0..09:35
* hberaud check09:35
hberaudiurygregory: it complain because these deliverables are RC deliverables and didn't provided RC1 first/yet09:37
*** elodilles_pto is now known as elodilles09:37
hberaudIn other words you couldn't create final release before providing RC09:37
hberauds/couldn't/can't/09:38
iurygregoryoh09:39
iurygregoryso how I handle that ?09:39
hberaudiurygregory: Next week is the RC1 deadline, so we will generate RC patches https://releases.openstack.org/reference/process.html#r-3-week-rc1-deadline09:39
iurygregoryneed to creat as rc instead of feature?09:39
hberaudand those deliverables will be included09:39
hberaudWell, usually RC start with a major version each cycle09:40
iurygregoryoh wow .-.09:41
hberaudand the command to create RC is the same than the command you/we use to create standard release, you just need to select "rc"09:41
hberaud(instead of bugfix, feature, etc...)09:41
iurygregoryI see09:41
hberaudlemme give you an example09:41
hberaudiurygregory: https://releases.openstack.org/reference/using.html#using-new-release-command09:42
iurygregoryinstead of tox -e venv -- new-release xena ironic-ui feature would be tox -e venv -- new-release xena ironic-ui rc09:42
hberaudyes exact09:42
iurygregoryI would say a major bump would be a bit too much for projects like ironic-ui ipe where we don't see a lot of changes09:44
opendevreviewMartin Kopec proposed openstack/releases master: Release os-testr 2.0.1  https://review.opendev.org/c/openstack/releases/+/80809409:46
opendevreviewMartin Kopec proposed openstack/releases master: Release bashate 2.1.0  https://review.opendev.org/c/openstack/releases/+/80808609:47
hberaudiurygregory: unfortunatelly I don't think that we could bypass this rule at this point https://docs.openstack.org/project-team-guide/release-management.html#services-horizon-plugins-and-other-deliverables Maybe we should think about changing the release model or the deliverable type of these deliverables for Y09:47
iurygregoryhberaud, ack09:49
iurygregoryI will push with rc in the patch =)09:49
hberaudiurygregory: thanks09:49
iurygregoryhberaud, np =)09:50
hberaudindeed you don't have a lot of significant changes, so, yes, I think it's time to start thinking about updating the deliverables type/models for those deliverables09:50
hberaudiurygregory: ^ 09:50
hberaudiurygregory: do you will remain release liaison for Y?09:51
hberaud(for ironic)09:51
iurygregoryhberaud, yes and I will be the PTL also09:51
hberaudCool, then I think you need to put this topic on your todo list for Y :)09:52
hberaudiurygregory: Congrats for your PTL role!09:52
iurygregoryhberaud, tks! I will add the topic and I will discuss with the community =)09:52
hberaudExcellent! Do not hesitate to ping us if needed09:53
iurygregorysure!09:54
opendevreviewIury Gregory Melo Ferreira proposed openstack/releases master: RC release for ironic-ui and IPE  https://review.opendev.org/c/openstack/releases/+/80818609:57
hberaudelodilles: o/ is it you who deleted the PTL approved badge on this patch? https://review.opendev.org/c/openstack/releases/+/80761510:35
elodilleshberaud: nope. I'm not even able to do that, am I? :-o10:41
opendevreviewMerged openstack/releases master: [manila] Add Xena cycle highlights  https://review.opendev.org/c/openstack/releases/+/80739110:42
elodillessometimes, in rare cases the PTL-approved flag behaves strangly. Usually it just isn't triggered. :S10:43
opendevreviewMerged openstack/releases master: Release devstack-tools 1.2.1  https://review.opendev.org/c/openstack/releases/+/80808710:55
hberaudelodilles: I don't know if we can remove this tag manually, I don't think is it possible.11:17
hberaudapprently zuul removed this badge in this review11:17
hberaudThe weird thing is that it was added yesterday when the PTL/liaison validated this patch11:18
hberaudHere it was triggered but it was removed in a second time without specific reason11:19
elodilles(I've checked meanwhile and for PTL-approved flag gerrit says "You don't have permission to edit this label.")11:27
hberaudthis is what I was thinking thank for your confirmation11:28
elodilleshberaud: according to this: https://zuul.opendev.org/t/openstack/build/2dbae464f15b48f0869917e1cb330796/console11:30
elodilleshberaud: "monasca team does not exist in liaisons file"11:31
hberaudthose are juste warning, isn't?11:31
elodillesyes, that's true11:31
elodilleshmmm11:31
hberaudif you look at the history of comments this patches received the badge 4 minutes later than this job11:32
hberaudSo I would argue that he worked as expected11:33
hberaudah sorry I'm bad11:34
hberaudWrong date conversion on my side11:34
elodilleshberaud: this is the key to our mistery https://review.opendev.org/c/openstack/governance/+/80785811:37
elodilles:)11:37
hberaudOh yeah! good catch elodilles 11:38
elodillesit was merged yesterday afternoon11:38
elodillesso my comment triggered the check and based on the new data (no PTL) the flag was cleared11:39
hberaudThat's a bit annoying... I didn't see this before xena11:39
hberaudYes11:39
elodillesI guess you haven't met a case where a project was left without a PTL11:40
hberaudSo for the current opened patches we should do diligent reviews on this point and consider this side effect too (ttx, smcginnis FYI ^)11:40
elodillesI don't think we will have that many similar issue11:41
hberauduntil we approve the other patches no... :)11:42
elodillesthis was a 'fortunate' case, where the PTL (for Xena) is still active and voted, but due to the update of the PTL list (when it was merged) his vote does not trigger anymore the PTL-approved flag11:45
elodillesis monasca changing to leaderless model (like Oslo)?11:47
hberaudI don't know11:49
hberaudapparently nope11:50
hberaudelse it will be set to "distributed"11:50
elodillesoh, i see.11:51
hberaud(I think)11:51
hberaud#notsure11:51
*** amoralej is now known as amoralej|lunch12:13
*** amoralej|lunch is now known as amoralej13:14
ttxnote that I'll multiplex during our meeting so if I'm slow to answer that would be it13:28
hberaudack13:58
hberaud#startmeeting releaseteam14:00
opendevmeetMeeting started Fri Sep 10 14:00:08 2021 UTC and is due to finish in 60 minutes.  The chair is hberaud. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'releaseteam'14:00
hberaudPing list: elodilles armstrong14:00
hberaud#link https://etherpad.opendev.org/p/xena-relmgt-tracking14:00
hberaudWe're way down on line 324 now. 14:00
hberaudWill just wait a couple minutes for folks.14:00
ttxo/14:01
elodilleso/14:01
hberaud#topic Review task completion14:03
hberaudSo the 2 main things was14:03
hberaud1) Propose stable/$series branch creation => https://review.opendev.org/q/topic:%22xena-stable-branches%22+(status:open%20OR%20status:merged)14:03
hberaudI think we can trigger the merge for validated and unvalidated patches, and leave only the patch with -114:04
* hberaud starting the +W session14:04
hberaudelodilles, ttx: feel free to approve those with only 1 validation from us14:05
ttxok14:07
hberaudgibi: please can you review https://review.opendev.org/c/openstack/releases/+/80768514:09
hberaudttx, elodilles let's wait a bit for nova14:10
elodilleshberaud: sure14:11
gibihberaud: I will check quickly14:13
hberaudgibi: thank you14:13
hberaudSo I +W all the patches with 2 review from us and I +2'ed those without response.14:14
hberaudWell, we can switch to the second task14:14
gibihberaud: done, thanks14:14
diablo_rojo_phoneo/14:14
hberauddiablo_rojo_phone: o/14:15
hberaudgibi: excellent thanks14:15
hberaudso only cyborg remain waiting for an RFE http://lists.openstack.org/pipermail/openstack-discuss/2021-September/024802.html, if we don't get response from prometheanfire early Monday I'll cut the branch here too and I'll propose to the team to backport their fixes14:17
hberaudSo, back to the second task14:18
hberaudit was "List cycle-with-intermediary deliverables that have not been refreshed in the last 2 months"14:18
hberaudand apparently "This did not catch anything that was not already mentioned in http://lists.openstack.org/pipermail/openstack-discuss/2021-August/024534.html"14:18
hberaudthanks ttx and armstrong14:18
ttxnp14:19
hberaud#topic Assign R-3 tasks14:19
prometheanfirehberaud: reqs breanches last, generally, so if we are last to branch we could do so now14:19
hberaudyep I know14:19
opendevreviewMerged openstack/releases master: New releases of the Neutron stable versions  https://review.opendev.org/c/openstack/releases/+/80801214:19
opendevreviewMerged openstack/releases master: Release os-testr 2.0.1  https://review.opendev.org/c/openstack/releases/+/80809414:20
opendevreviewMerged openstack/releases master: Release sushy-tools 0.18.0  https://review.opendev.org/c/openstack/releases/+/80818814:20
hberaudprometheanfire: the question here is about branching cyborg14:20
hberaudThey wait for a RFE and the current proposed branching don't take account of that14:20
prometheanfireah, email wasn't tagged for reqs :P14:20
prometheanfireI don't think the cyborg review requires reqs signoff14:21
prometheanfirewell, release does I guess14:21
prometheanfireI think it's fine, I'll email14:21
hberaudthanks14:21
hberaudI'll update the patch after the meeting14:22
hberaudwith the new version and the cut based on that14:22
hberaudSo, concerning the task assignment I took all the items14:23
opendevreviewMerged openstack/releases master: [oslo] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80758814:23
opendevreviewMerged openstack/releases master: [monasca] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80761514:23
opendevreviewMerged openstack/releases master: [cloudkitty] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80762214:23
opendevreviewMerged openstack/releases master: [heat ] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80761214:23
opendevreviewMerged openstack/releases master: [octavia] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80762814:23
opendevreviewMerged openstack/releases master: [octavia] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80762014:23
opendevreviewMerged openstack/releases master: [manila] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80762714:23
opendevreviewMerged openstack/releases master: [OpenStackSDK] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80760114:23
hberaud#topic Review countdown email contents14:24
hberaud#link https://etherpad.opendev.org/p/relmgmt-weekly-emails14:24
hberaudlet me know if it LGTY14:25
opendevreviewMerged openstack/releases master: [ironic ] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80760614:26
opendevreviewMerged openstack/releases master: [magnum] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80762614:26
opendevreviewMerged openstack/releases master: [glance ] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80760814:26
opendevreviewMerged openstack/releases master: [neutron] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80761614:26
opendevreviewMerged openstack/releases master: [Telemetry] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80759914:27
opendevreviewMerged openstack/releases master: [trove] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80765214:27
opendevreviewMerged openstack/releases master: [watcher] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80765414:27
opendevreviewMerged openstack/releases master: [zun] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80765514:27
opendevreviewMerged openstack/releases master: [winstackers] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80762114:27
elodilleshberaud: shouldn't be the mail from the process from R-3?14:27
hberaudnope14:27
hberaudWe are in R-414:27
hberaudthe rc deadline is next week14:27
elodillessorry, looked at the wrong week o:)14:28
hberaud(RC1)14:28
ttxchecking really fast14:29
opendevreviewHervé Beraud proposed openstack/releases master: [cyborg] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80762414:30
ttxhberaud: lgtm14:30
hberaudthanks14:30
elodilleslgtm too14:30
hberaudthanks14:30
hberaudsent14:31
hberaud#topic Is it a good idea to pass https://review.opendev.org/c/openstack/project-config/+/801376 pre-release?14:32
hberaudI don't think this is a good idea14:32
hberaudI also left a comment on the review 14:32
ttxok then14:32
ttxlets freeze it14:32
ttxI fear it will double processing time14:32
hberaud+114:32
hberaudyes not the right time14:33
hberaud#topic Create stable/stein for devstack-plugin-ceph14:33
hberaudFor more context please have a look to https://review.opendev.org/c/openstack/releases/+/80774514:33
hberaudthe QA team asked for a branch creation stein, I manually created it as they wanted to backport a couple of changes from train too14:34
hberaudthis is project a tagless so I don't think created it manually is a big deal14:35
hberaudThey merged a patch to prepare the newest branch14:36
hberaudI'll update the sha in this review to see if the CI is more happy with that14:36
elodillesit served the purpose if I'm not mistaken, so I guess it is OK :)14:36
hberaudIt could be worth to merge this patch for the sake of a proper history14:37
elodillesyes, that's true.14:38
elodillesthe details are in the comments of the patch so at least we have something there14:38
hberaudyes14:39
opendevreviewHervé Beraud proposed openstack/releases master: devstack-plugin-ceph: create also a stein branch  https://review.opendev.org/c/openstack/releases/+/80774514:39
hberaudupdate done! ^14:39
hberaud#topic Do we want to participate to Y PTG - deadline for sign up is today14:39
hberaudelodilles: I remember that we already discussed a bit about this but I just want to confirm that we don't want a Y PTG, ttx, elodilles please confirm14:40
hberaudI need to send our response to Ashlee today14:41
elodillessomeone wrote that a 1 hr long one could be held14:41
hberaudI think it's Thierry14:41
hberaudelodilles: You'll own the PTG so what do you think about that?14:42
fungiit might be an opportunity to cover the release metadata api implementation zigo was talking about this week14:42
hberaudindeed14:42
elodillesmaybe we can have one and if not so much topic rises then we can just have a short one14:42
hberaudSold14:43
ttxyep, I said one hour14:43
hberaudThen I'll reply to Ashlee to keep us a place14:43
elodillesthanks hberaud o/14:43
elodillesshould we have an etherpad for the PTG?14:44
hberaudyes14:45
hberaudnormally it will be created manually14:45
hberaudusually we copy past our things to changes into the that etherpad too14:46
elodillesok, will do that next week14:46
hberaudExcellent thanks14:46
hberaudI'll cc you on the email to allow you to follow this topic14:47
elodilleshberaud: cool, thanks!14:47
hberauddone14:48
hberaud#topic Newton and Ocata branch deletion for storlets14:48
elodillesI was planning to run the clean up script during the week but missed it so far :)14:49
hberaudelodilles: yesterday tkajinam asked us about branch deletion for N and O on storelets14:49
elodillesI'll do it now14:49
hberaudtkajinam: FYI ^14:49
hberaudthanks14:49
hberaud#topic Open Discussion14:50
hberaudAnything else that you want to discuss today?14:50
elodillesnothing from me right now14:51
hberaudWhen the PTG etherpad will be available I'll add the release metadata API implementation on it14:53
elodilleshberaud: ++14:53
* hberaud forgot that he already added it to our "things to changes"14:54
opendevreviewMerged openstack/releases master: [vitrage] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80765314:54
opendevreviewMerged openstack/releases master: [tacker] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80765114:54
hberaudOK, thanks everyone. Let's wrap up.14:54
hberaud#endmeeting14:54
opendevmeetMeeting ended Fri Sep 10 14:54:21 2021 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:54
opendevmeetMinutes:        https://meetings.opendev.org/meetings/releaseteam/2021/releaseteam.2021-09-10-14.00.html14:54
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/releaseteam/2021/releaseteam.2021-09-10-14.00.txt14:54
opendevmeetLog:            https://meetings.opendev.org/meetings/releaseteam/2021/releaseteam.2021-09-10-14.00.log.html14:54
opendevreviewMerged openstack/releases master: [kuryr] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80761914:54
opendevreviewMerged openstack/releases master: [designate] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80762514:54
opendevreviewMerged openstack/releases master: [swift] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80765014:54
opendevreviewMerged openstack/releases master: [keystone] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80761814:54
opendevreviewMerged openstack/releases master: [mistral] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80761414:54
opendevreviewMerged openstack/releases master: [senlin] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80762914:54
opendevreviewMerged openstack/releases master: [nova] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80768514:54
elodillesthanks hberaud o/14:54
elodilleshberaud: I could help with the task "On the Monday, generate release requests for all deliverables that have do not have a suitable candidate yet."14:55
hberaudas you want14:56
elodilleshberaud: of course if you are not insisting on doing it by yourself o:)14:56
elodillesthough I might need some minor pointers14:56
hberaudI'll let you handle the cwi part https://releases.openstack.org/reference/process.html#r-3-week-rc1-deadline14:57
elodillesi guess it can be done with one of our tools14:57
hberaudelodilles: I'll do the other one14:58
elodilleshberaud: OK, cool, thanks!14:58
hberauddo not hesitate to ping me if something isn't clear14:58
elodilleshberaud: I'll do it Monday afternoon and let you know if anything is not clear :)14:58
hberaudelodilles: Thanks for your help 14:58
hberaudexcellent!14:58
elodilles:)14:59
armstrongHello hberaud15:24
hberaudarmstrong: o/15:25
armstrongI missed the meeting today15:26
hberaudnp15:26
hberaudI seen you put your name on next weeks items15:26
hberaudthank you15:26
armstrongYou’re welcome15:29
hberaudarmstrong: Patches of your series are almost all merged https://review.opendev.org/q/topic:%22xena-stable-branches%22+(status:open%20OR%20status:merged)15:30
armstrongNice!15:30
opendevreviewMerged openstack/releases master: [blazar] Create for all client and non-client lib xena branch  https://review.opendev.org/c/openstack/releases/+/80759615:46
*** amoralej is now known as amoralej|off15:58
*** marios is now known as marios|out16:14
elodillestkajinam: hi, fyi, storlet's *-eol branches are deleted now (along with other eol'd branches; https://paste.opendev.org/show/809239/ )19:39
opendevreviewGage Hugo proposed openstack/releases master: [openstack-helm] Add Xena cycle highlights  https://review.opendev.org/c/openstack/releases/+/80849420:44
opendevreviewGage Hugo proposed openstack/releases master: [openstack-helm] Add Xena cycle highlights  https://review.opendev.org/c/openstack/releases/+/80849420:47
opendevreviewGage Hugo proposed openstack/releases master: [openstack-helm] Add Xena cycle highlights  https://review.opendev.org/c/openstack/releases/+/80849420:59
opendevreviewGage Hugo proposed openstack/releases master: [openstack-helm] Add Xena cycle highlights  https://review.opendev.org/c/openstack/releases/+/80849421:19

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