Friday, 2024-11-29

ykarelthx elodilles 04:33
ykarelttx, frickler if you can check/clear https://review.opendev.org/c/openstack/releases/+/93447604:34
fricklerdone07:41
opendevreviewMerged openstack/releases master: [ironic] Transition 2023.1 Antelope to Unmaintained  https://review.opendev.org/c/openstack/releases/+/93447607:51
opendevreviewMerged openstack/releases master: [barbican] Transition 2023.1 Antelope to Unmaintained  https://review.opendev.org/c/openstack/releases/+/93450108:20
fricklerelodilles: ttx: maybe you can also check https://review.opendev.org/c/openstack/releases/+/934490 when you're around, I would wait for that before doing the next round of branch cleanups08:49
elodillesfrickler: ACK, i've added it to my TODO09:12
elodillessean-k-mooney: i still don't have +2 rights on watcher's stable branch :-o i don't even see watcher-stable-maint09:13
elodillesfrickler: +2+W'd09:21
opendevreviewMerged openstack/releases master: [kolla] Transition 2023.1 Antelope to Unmaintained  https://review.opendev.org/c/openstack/releases/+/93449009:29
opendevreviewDmitriy Rabotyagov proposed openstack/releases master: Release OpenSTack-Ansible Antelope  https://review.opendev.org/c/openstack/releases/+/93671410:13
fricklerthx, /me is afk for a bit, will run the scripts later10:25
elodillesACK10:44
elodilleshmm, we have release job failure, which seems to be important: it is a tag-releases job failure. the job got timed out: https://lists.openstack.org/archives/list/release-job-failures@lists.openstack.org/thread/AQC6HPZF646VB5TEC7IUD6TZZPKJ7C74/11:39
elodillesthis is the last log line before the timeout: https://zuul.opendev.org/t/openstack/build/2bb14f86aff148ed884da9270baade57/log/job-output.txt#385811:40
elodillesmaybe a re-enqueue can be tried for this11:41
elodillesthough i haven't checked yet if the tags exist or not11:41
elodillesbtw, there was a couple of other release job failures for kuryr-kubernetes, specifically their reno publish job failed, but those can be ignored in my opinion, as that is due to the repo is retired and was emptied (hence the job does not find 'releasenotes')11:43
elodillesthough 1 another error is also somewhat valid: it looks like ansible-collections-openstack-release job needs some investigation: https://zuul.opendev.org/t/openstack/build/2bb14f86aff148ed884da9270baade57/log/job-output.txt#385811:45
sean-k-mooneyelodilles: that becasue its called watcher-release-core. maybe they have the gerrit acls set up diffently11:58
sean-k-mooneythre are two diffent convetions i was aware of stable banches are either owned by -release-core or -stable-maint11:59
sean-k-mooneyim more familar with the former owning it because that was more common for neutron related porjects but i belive the -stable-maint naming it more common with other projects12:00
sean-k-mooneyill look into the acls shortly12:00
elodillessean-k-mooney: i don't see stable-maint in 'watcher-core' ( https://review.opendev.org/admin/groups/77cd139bd57514e98e5e694a82f7e261b4748a8a,members ) but i think the best would be to add [access "refs/heads/stable/*"] section is watcher's config: https://opendev.org/openstack/project-config/src/branch/master/gerrit/acls/openstack/watcher.config12:49
sean-k-mooneyelodilles: it shoudl not be in watcher-core12:49
elodillesah sorry, you wrote watcher-release-core :X12:50
sean-k-mooney yep so that the problem however12:50
sean-k-mooneythey didnt set up watcher-release-core to be able to merge on stable branches12:50
sean-k-mooneyill submit a patche to do that for the relvent watcher repos12:50
elodillessean-k-mooney: yeah, there is no such thing as watcher-release-core for the record. rather watcher-release.12:51
sean-k-mooneyi have made watcher-core part of watcher-release-core but i want to be able to add other stabel cores eventually without addign directly to the main core group12:51
sean-k-mooneyoh sorry yes that what i ment12:51
sean-k-mooneywatcher-release12:52
elodillesi'd rather use watcher-stable-maint as i don't even know what watcher-release is meant for12:52
sean-k-mooneyits ment for stable branch mantaince :)12:52
sean-k-mooneyas i said there are two partterns in use in openstack12:52
elodillesthen i'd rather name it watcher-stable-maint o:)12:52
sean-k-mooneywell we coudl but we will have to rename the existing groups12:53
elodillesi see12:53
sean-k-mooneyso in the past any project not governed by the unified release process had a -release group to allow pushing tags and merging on stable branchs12:54
sean-k-mooney"core" project had a -stable-maint team but in things like stackforge and neutron stadium12:54
sean-k-mooney-release was more common as far as i was aware12:54
sean-k-mooneyim not sure how much fo that is still true or relevent12:55
sean-k-mooneyelodilles: so it looks like watcher-release and watcher-sadhboard-release are currently unused12:56
elodillessean-k-mooney: oh, i did not know this part of the history o:)13:02
elodillesgood to know13:02
elodillesyepp, i haven't seen that group used in watcher.config13:02
sean-k-mooneygiven its unused in our gerrit acls13:11
sean-k-mooneywoudl you prefer to just change too the -stable-maint nameing then13:11
sean-k-mooneybefore doing that i woudl want ot document that on the maining list13:12
sean-k-mooneybut i dont nessiarly disagree with that, its just a littme more work then reusign the exisitng groups 13:12
elodillessean-k-mooney: well, i liked that idea better, but given the history, i'm ok to keeping it watcher-release. and as you say it requires less churn, so let's stick to watcher-release then o:)13:13
sean-k-mooneyok let me grab a cup of tea and ill push up a patch for the acl update when i get back. we can alwasy update it again in the future if we watnt to rename things13:14
elodilles+113:14
frickler50 more branches deleted https://paste.opendev.org/show/brvi5eR5LyYbwbGhfHGr/ the abandon script didn't find some of the reviews, needed to do it manually for two repos13:29
elodillesfrickler: looks awesome, thanks o/13:43
opendevreviewDmitriy Rabotyagov proposed openstack/releases master: [OpenStack-Ansible] Branch for 2024.2 (Dalmatian)  https://review.opendev.org/c/openstack/releases/+/93675314:28
noonedeadpunkit would be nice to land that sooner then later ^14:28
elodilles+2'd14:46
noonedeadpunkthanks!14:47
sean-k-mooneyelodilles: sorry got distracted talking to someone else but https://review.opendev.org/c/openstack/project-config/+/936754 should correct the stable branch permissions14:49
elodillessean-k-mooney: +1 thanks o/15:09
opendevreviewCarlos Eduardo proposed openstack/releases master: Add manila deadlines to the official schedule  https://review.opendev.org/c/openstack/releases/+/93589118:05
opendevreviewMerged openstack/releases master: [OpenStack-Ansible] Branch for 2024.2 (Dalmatian)  https://review.opendev.org/c/openstack/releases/+/93675318:19

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