Friday, 2024-11-15

opendevreviewRodolfo Alonso proposed openstack/releases master: New os-ken release 2.11.1  https://review.opendev.org/c/openstack/releases/+/93526507:47
fricklerJayF: I can do the eol tags for you, no need to go through the project-config hassle unless you really want to. I'd also argue that we likely still want to merge that patch anyway, if only for documentation purposes, since we do have the same in place for ocata09:00
fricklerrelease-team: please review https://review.opendev.org/c/openstack/releases/+/935064 so that we can proceed with the eom->eol transitions. I'll then update 935041 according to the feedback and make it a stack starting at the latest branch09:08
opendevreviewBartosz Bezak proposed openstack/releases master: kayobe: 2024.2 rc1  https://review.opendev.org/c/openstack/releases/+/93527109:55
fricklerelodilles: fyi this would be my next dataset to base eols on, note that is does include nova, so you may want to actually looks into this https://zuul.opendev.org/t/openstack/config-errors?branch=unmaintained%2Fvictoria&severity=error&skip=010:09
elodillesfrickler: ACK, noted. i tend to check the zuul config errors and fixed a lot already, but i always try to start from the most recent branch, hence i did not get to victoria in many cases (sometimes stable branches are also affected and even master). the problem is that time to time new things were deleted, but i'm on it, beside my other duties10:30
fricklerelodilles: well it was you who requested that I start at victoria ;)10:40
elodillesindeed :)10:44
elodillesas that is the process o:) (no release should be skipped)10:44
fricklerthe process also says all branches with broken CI shall be closed, but you seem to object to that part of the process10:46
elodillesi don't recall, is there a grace period or should we delete them instantly?10:50
fricklerthe policy doesn't mention any grace period. I would consider the time since the branch transition to eom until the next eom event to be sufficient for that10:50
elodillesi still think that it could be useful for some people to keep old branches open, even if we see only minimal activities (and less and less on the oldest branches). i volunteered to do some maintenance on these old branches, and although i get help time to time, there's always new things to fix. if the community insists on deleting these old branches then i have to accept that10:57
elodillesand note, that i didn't want to delete instantly any CI jobs that are failing, rather try to find why it is failing. it's relatively easy to disable any failing CI job to make the gate pass. but that is not my intention at all.10:59
fricklerto me, the question really is what is the benefit that you get out of keeping these branches open that couldn't also be fulfilled by say a company owned fork on codeberg or even github if needed. is it really worth the added efforts on the sides of the release team, the opendev team and others? especially if you only really care for a single branch?12:17
elodillesfrickler: CI + the possibility for everyone to do some maintenance upstream, where other can consume bug fixes. it's not only for victoria, but for other branches as well. maybe some vendor / company / developer see some value in it and put some effort in it, if not now, then later. if we don't even give the possibility, then it could never happen. i know, maybe this approach is too idealistic13:17
elodillesnoonedeadpunk: hi, sorry for the late ping: we have a task for this week to propose branch cuts for trailing projects where there are no stable/2024.2 yet. should i push the patches for that or do you plan to do that?13:34
elodillesnoonedeadpunk: i'm talking about OSA + OSA-roles o:)13:35
noonedeadpunkelodilles: I'll do that, but we're still finilizing some things13:45
noonedeadpunkso probably it won't be this week13:45
opendevreviewMerged openstack/releases master: Release python-zaqarclient for Epoxy-1 milestone  https://review.opendev.org/c/openstack/releases/+/93463113:47
opendevreviewMerged openstack/releases master: kayobe: 2024.2 rc1  https://review.opendev.org/c/openstack/releases/+/93527113:47
elodillesnoonedeadpunk: sure, no problem, thanks in advance o/13:48
fricklerelodilles: yes, I see the idea, but I don't think we can keep everything open forever, just in case someone comes along years later. I think waiting for one cycle after the eom tag was made must be enough13:49
fricklerelodilles: what I won't object to is keeping branches open that actually have had non-bot-generated patches proposed or merged https://review.opendev.org/q/branch:unmaintained/victoria+-owner:infra-root@openstack.org13:49
fricklerthat's 65 patches spread over 30 repos13:51
elodillesfrickler: yeah, i don't expect and want to keep it open forever, but one cycle after EOM is too short in my opinion. when on summits on forums the question was on the agenda 'what versions do you have on production' there were insanely old versions mentioned every time.13:52
elodillesof course the best is if everyone upgrades all the time to the latest release. but that's not happening all the time in reality13:54
fricklerI have two questions regarding this (not necessarily to you, but more in general): a) how many of these old deployments would even be in a position to and willing to deploy bugfixes? so would they really profit from the unmaintained branches?13:58
fricklerand second: won't that number only increase even more, if we decrease the pressure to do timely upgrades?13:58
elodillesgood point :)14:00
elodillesbut let's have a meeting14:00
elodilles#startmeeting releaseteam14:00
opendevmeetMeeting started Fri Nov 15 14:00:56 2024 UTC and is due to finish in 60 minutes.  The chair is elodilles. 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
ttxo/14:01
frickler\o14:01
elodillesPing list: release-team14:01
elodilles#link https://etherpad.opendev.org/p/epoxy-relmgt-tracking14:01
elodilles(my copy paste buffer is dying...)14:01
elodilleso/14:02
elodilleswe are at line 8414:02
elodilleslet's start!14:03
elodilles#topic Review task completion14:03
elodilles1st task was:14:03
elodilles'Ensure that all trailing projects have been branched for the previous series. (elod)'14:03
elodillesi saw 3 repos from 2 teams that needed stable/2024.2 branching:14:04
elodilleskayobe: https://review.opendev.org/c/openstack/releases/+/935271 just merged14:04
elodillesso this is done ^^^ \o/14:04
elodillesand14:05
elodillesOSA + OSA-roles14:05
fricklerseems noonedeadpunk is onto that14:05
elodillesi've pinged noonedeadpunk and he promised to prepare the patches 14:05
elodillesin the coming weeks14:05
elodilleswe still have 3 weeks until Dalmatian cycle-trailing release deadline: https://releases.openstack.org/epoxy/schedule.html#e-cycle-trail14:06
noonedeadpunkI'm aware of deadlines and will push patches once we will be ready for them14:07
noonedeadpunkwe had very quiet cycle and were hoping to get release early14:07
noonedeadpunkbut plenty of ideas raised right after ptg 14:07
elodillesnoonedeadpunk: thanks for the heads up, sounds like a plan!14:08
elodillesthat's also good to hear :)14:08
noonedeadpunk(and CI failing on us heavily recently)14:08
elodilles:-o14:08
elodilleshmm, that's not good to hear :/14:08
elodillesanyway... fingers crossed, and thanks again for the info.14:09
elodilles2nd task was:14:09
elodilles'Propose autoreleases for cycle-with-intermediary libraries which did not release since the previous release. (elod)'14:09
elodilleshttps://review.opendev.org/q/topic:epoxy-milestone-114:10
elodillesas you can see, about third of them have merged,14:10
ttxshould we force or just ignore the other ones?14:10
elodillessince we are at milestone-1 i guess we can ignore14:11
elodillesbut i'll ping damani to review the patches maybe14:11
elodillesas most of the remaining patches are oslo deliverables14:11
elodillesif that is OK for you14:12
ttxok... maybe we can abandon them next week14:12
ttxgive a last chance to everyone14:12
ttxand yes oslo ping sounds like a good idea14:12
elodillesttx: ACK, i'll do the pinging and abandon the ones that do not get responses in some days14:13
elodilleswe have a task for that anyways o:)14:13
opendevreviewMerged openstack/releases master: New os-ken release 2.11.1  https://review.opendev.org/c/openstack/releases/+/93526514:13
elodillesi mean, to keep track of the 'milestone-1 exceptions'14:13
elodilles3rd task then:14:14
elodilles'To catch if there are acl issues in newly created repositories, run tools/aclissues.py (ttx)'14:14
ttxdone that -- no issue reported14:14
elodillescool \o/14:14
elodillesand that was all the tasks14:15
elodilles#topic Assign R-19 and R-16 week tasks14:15
elodillesthough as i see we have not much to assign,14:15
elodillesi've added 'all' to 'milestone-1 exceptions'14:16
ttxTook the chairing14:16
elodillesttx: +1, thanks!14:16
elodilles#topic Review weekly countdown email14:16
elodilles#link https://etherpad.opendev.org/p/relmgmt-weekly-emails14:17
elodillesplease review ^^^14:17
fricklerlgtm14:18
ttxlgtm, added final release date14:18
elodillesgood, thanks!14:18
elodilleswill send if later today14:18
elodilless/if/it14:18
elodilles#topic Open Discussion14:19
elodilles(frickler) Fix for eom->eol transition14:19
elodilleshttps://review.opendev.org/c/openstack/releases/+/93506414:19
frickleras discussed yesterday, the validation currently fails since the e.g. stable/zed branch is gone14:20
fricklerif someone feels inclined to amend the code to actually check the unmaintained/* branch, that would be nice, too14:20
fricklerbut the above would just skip the check for -eol tags14:21
frickler(the check whether tag-eol is actually on a proper branch)14:21
ttxyes I think that works14:22
elodillessounds + looks good to me with a quick glance, i'll review it thorougly after the meeting14:22
elodillesanything else to add here?14:23
ttx(nothing from me)14:25
elodillesOK then let's move on14:25
elodilles'(frickler) Directly EOL EOM branches for retired projects?'14:25
elodilles    (solum) https://review.opendev.org/c/openstack/releases/+/93447514:26
elodilles    (ec2-api) https://review.opendev.org/c/openstack/releases/+/93446914:26
elodilles    (kuryr) https://review.opendev.org/c/openstack/releases/+/93447714:26
elodilles    (winstackers) https://review.opendev.org/c/openstack/releases/+/93448314:26
elodilles    (senlin) https://review.opendev.org/c/openstack/releases/+/93450814:26
elodilles    (sahara) https://review.opendev.org/c/openstack/releases/+/93449614:26
elodilles    (murano) https://review.opendev.org/c/openstack/releases/+/93450414:26
fricklerI must admit I didn't check yet how we handled these last cycle, but I think I remember we did some direct eols there, too?14:26
elodillesi don't have strong feelings here14:26
elodillesin general, i'm OK to EOL them14:27
ttxI think it makes sense. If ether is no team to continue maintaining them, it's even more true for old branches14:27
elodillesbut have to check whether older branches are still open (unmaintained/*)14:27
ttxMaybe just a quick email thread to explain and give anyone opportunity to object14:27
fungitechnically, unmaintained branches are supposed to be eol'd if nobody volunteers to be their caretaker14:28
fungii had previously assumed, based on the original discussions at the forum and subsequently in drafting of the tc resolutions, that most of those branches would go eol after one cycle of being unmaintained14:29
fungisince usually nobody volunteers to keep them in working condition14:29
fricklerfungi: well the problem with the latter is that nothing is happening by default14:29
fricklerso we still need to actually implement that part of the policy14:29
fungithe plan was that they be eol'd by default14:29
fricklerfor some value of "we"14:30
elodillesACK, then I will drop a mail to ML and state that we are about to EOL solum, [..], murano in a week if noone steps up.14:30
fungibut yeah, maybe that part of the process was never clearly defined, so it's not actionable as-is14:30
fricklerwell it is clearly defined in the policy. it is just that the implementation differs14:31
elodilles#action elod to drop a mail to ML and state that we are about to EOL solum, [..], murano in a week if noone steps up14:31
elodillesis that OK for you? ^^^14:31
fungiregardless, yes, if the project is retired then newer branches aren't being maintained which means the unmaintained branches can't reasonably continue as-is either14:31
ttxworksforme14:31
fricklerelodilles: I'm not convinced that it is worth the effort, but I won't stop you, either14:31
fricklerif someone did step up, they'd also have to revive the main branch, wouldn't they?14:33
elodillesyeah, since i haven't seen any activity on them (not a surprise, they are retired) so i don't expect either any response, but we will see14:33
elodillesfrickler: that's a good question. probably yes14:33
elodillessome bug fix backport + CI fixes could have been done without master revival... but that's it14:34
elodillesalrighty then, this was the last topic14:35
frickleranyway, maybe some miracle happens and someone shows up, we can discuss further steps then14:35
elodilles:)14:35
elodillesindeed14:35
elodillesanything else to bring up before we close the meeting?14:35
elodilles#info next meeting is at 13th December14:36
elodillesjust to note that we will skip some meetings in the coming weeks ^^^14:37
ttxenjoy the break!14:37
elodillesyepp-yepp :)14:37
elodillesokay, if nothing else, then thanks everyone for joining o/14:38
elodilles#endmeeting14:38
opendevmeetMeeting ended Fri Nov 15 14:38:16 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:38
opendevmeetMinutes:        https://meetings.opendev.org/meetings/releaseteam/2024/releaseteam.2024-11-15-14.00.html14:38
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/releaseteam/2024/releaseteam.2024-11-15-14.00.txt14:38
opendevmeetLog:            https://meetings.opendev.org/meetings/releaseteam/2024/releaseteam.2024-11-15-14.00.log.html14:38
frickler\o14:38
elodillesDamani_: hi, since you are here, could you please review these oslo releases? https://review.opendev.org/q/topic:epoxy-milestone-1+is:open+reviewer:self14:42
elodillesmail sent to ML about the retired projects: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/J34NQ7WQWM2RZ5Y6XD6I4WB6MHENLN7V/16:27
elodillesi'll wait a week and prepare the patches depending on the replies (or lack of them)16:28
opendevreviewDr. Jens Harbott proposed openstack/releases master: Transition zed-eom branches to EOL  https://review.opendev.org/c/openstack/releases/+/93504116:38
opendevreviewDr. Jens Harbott proposed openstack/releases master: Transition victoria-eom branches to EOL  https://review.opendev.org/c/openstack/releases/+/93537316:38
opendevreviewDr. Jens Harbott proposed openstack/releases master: Transition wallaby-eom branches to EOL  https://review.opendev.org/c/openstack/releases/+/93537416:38
opendevreviewDr. Jens Harbott proposed openstack/releases master: Transition xena-eom branches to EOL  https://review.opendev.org/c/openstack/releases/+/93537516:38
opendevreviewDr. Jens Harbott proposed openstack/releases master: Transition yoga-eom branches to EOL  https://review.opendev.org/c/openstack/releases/+/93537616:38
opendevreviewMerged openstack/releases master: [cinder] Transition 2023.1 Antelope to Unmaintained  https://review.opendev.org/c/openstack/releases/+/93448916:58
opendevreviewDr. Jens Harbott proposed openstack/releases master: Transition zed-eom branches to EOL  https://review.opendev.org/c/openstack/releases/+/93504117:46
elodillesjust a heads up that i'll run the eom branch clean up script (and keep requirements + devstack branches) in a minute17:57
elodillesClark[m]: fungi: sorry, i see that some operation is ongoing on #opendev is this a wrong timing to do some stable/2023.1 branch clean-up, because i was about to run the script ^^^18:00
sean-k-mooneyfrickler: probaly best to chat here, i dont know if you saw may watcher unmainteind branch email to the list or my commnet on https://review.opendev.org/c/openstack/releases/+/93504118:00
sean-k-mooneyi see htat the older branhc transtion patches have been created18:00
fungielodilles: should be fine, what's going on currently is unrelated to gerrit18:01
sean-k-mooneyi was going to ask if it made sense to create a seperate patch for watcher and related patches18:01
clarkbelodilles: the operation is pruning the intermediate container registry used by ci jobs. Agreed it shouldn't affect branch cleanups in gerrit18:01
fungijust doing some cleanup of data in swift18:01
sean-k-mooneyrelated repos to do them all in one commit or not18:01
clarkbthe last time we tried to prune this is made some jobs sad beacuse there were bugs. We think they have been fixed and don't expect problems this time but either way gerrit itself should be unaffected18:01
elodillesfungi: clarkb: ACK, thanks, then i'll run the script18:02
sean-k-mooneyfrickler: the current transtion patch move watcher branche to eol but does not do the same for watcher-dashboard and python-watcherclient which i think should all be move to eol togehter18:04
elodillessean-k-mooney: thanks for your mail and review on the patch, i guess frickler doesn't insist on a separate patch, though i also wanted to / plan to create separate patch for the retired projects as i don't want everything to be EOL'd in the above patches... (even if that means i have to fix / reduce CI jobs for some of the projects)18:06
elodillessean-k-mooney: and yepp, if watcher is about to be EOL'd then watcher-dashboard and python-watcherclient EOL'ing makes sense at the same time18:07
sean-k-mooneywell im asking more so we are not workign across purposes18:07
sean-k-mooneyi.e. i dont want to submit a configlcit patch for watcher* if we prefer to go with the combinded ones18:08
sean-k-mooneybut i was going to do that before i saw frickler's patch18:09
elodillesfrickler did it to spare some release review effort, though i'm happier with some smaller patches (though not a separate patch for every repo) than a huuuuge almost infinite patch :S18:10
sean-k-mooneymy plan was to wait a week or so for any respone on the mailing list and create  a patch late next week to eol them if no one objected. bu ti would like to cordinate that with the wider release team then just go my own way so whatever works for folks in general18:10
elodillesi like your plan and thanks for doing it o:)18:11
sean-k-mooneythe other thing i need to do carfully is cordiante acess to the stable (and master) brach core teams18:12
elodillesbut yes, let's wait for frickler's response18:12
sean-k-mooneybaisally when we are doign this i woudl expect we may need to tweak grenade or simialr?18:12
sean-k-mooneyi.e. to not break the 2023.1 unmaineded barnches18:13
elodillesis watcher used in any project's grenade job?18:13
sean-k-mooneyother then its own i dont think so18:14
sean-k-mooneyand of late it hasd not had really any mantaince on stable let alone unmainteed/*18:14
elodillesthen i guess it shouldn't be a problem18:14
sean-k-mooneyso i dont expect it to break other proejcts18:14
sean-k-mooneybut i just dont want to creat a buch fo zuul error ectra18:15
elodillesthen i think it won't cause any issue18:15
sean-k-mooneyso if we need to pin to a tag or soemthing i just want to make sure people have access to do that18:15
elodillesand unfortunately time to time we have new zuul config errors when things are deleted (devstack-gate, nodesets, py38, etc-etc), so that's kind of normal operation :D18:16
sean-k-mooneyoh interesting. the github mirror sync does not delete the stabel branch when it moves to unmaintained18:17
elodillessean-k-mooney: ping me anytime as i'm in stable-maint group + openstack-unmaintained-core as well, so most things i could cover18:17
sean-k-mooneythat does seam to work fine on opendev.org18:17
elodillessean-k-mooney: if i remember well, we delete the branch in gerrit, and it get synced later on18:18
sean-k-mooneyelodilles: there is one patch that woudl be nice tomerge on stable if you have time https://review.opendev.org/c/openstack/watcher/+/93499618:19
elodillesand note that i'm running the cleanup script (deleting stable/2023.1 branches) right now, so there are still some left-over stable/2023.1 branches18:19
sean-k-mooneyah18:19
opendevreviewStephen Finucane proposed openstack/releases master: oslo.context 5.7.0  https://review.opendev.org/c/openstack/releases/+/93539318:20
sean-k-mooneyif its not synced/fixed in a week or so ill let you know but ill assume it will resolve itself 18:20
elodillessean-k-mooney: will look into the patch shortly18:20
sean-k-mooneyno rush, it was just pointed out that the stable/2023.2 branch was failing so i cherry picked the 2024.1 patch that fixed it but was not backported18:22
elodillesand the result of the cleanup for now: https://paste.opendev.org/show/bixe2g2g9S6pQ83WBIvx/18:39
fricklersean-k-mooney: if you want to eol watcher separately, please comment in the patch or amend it yourself. as I wrote in a comment there already, this thing is mostly autogenerated as a first easy step for cleanup, more stuff will follow when checking project+CI status individually18:39
fricklerelodilles: nice18:40
sean-k-mooneyfrickler: well im more lookign for guidance18:40
sean-k-mooneyfrickler: as i said i am going ot wait a week for respocnes on the maining list18:41
sean-k-mooneyso i can either update the patch, create my own or create one just for watcher-dashbaord adn watcher client18:41
sean-k-mooneyi just dont want to create more work for you18:41
fricklerthen the third option would sound easiest I'd say. I personally would think having waited half a year for progress is long enough, but a week more or not doesn't really matter then, too18:43
sean-k-mooneysure ill go for option 3 then and just put it at the end of your existing patches18:44
elodillessean-k-mooney: the watcher backport looks good to me, though watcher is one of the projects where somehow stable-maint is not in the <project>-stable-maint group so i cannot +2 it :/18:51
elodillesrelease-team: cinderlib release-job failure (publish-openstack-releasenotes-python3 job) can be ignored i think, as cinderlib is retired and i guess the emptied repo causing the issue18:57

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