ykarel | thx elodilles | 04:33 |
---|---|---|
ykarel | ttx, frickler if you can check/clear https://review.opendev.org/c/openstack/releases/+/934476 | 04:34 |
frickler | done | 07:41 |
opendevreview | Merged openstack/releases master: [ironic] Transition 2023.1 Antelope to Unmaintained https://review.opendev.org/c/openstack/releases/+/934476 | 07:51 |
opendevreview | Merged openstack/releases master: [barbican] Transition 2023.1 Antelope to Unmaintained https://review.opendev.org/c/openstack/releases/+/934501 | 08:20 |
frickler | elodilles: 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 cleanups | 08:49 |
elodilles | frickler: ACK, i've added it to my TODO | 09:12 |
elodilles | sean-k-mooney: i still don't have +2 rights on watcher's stable branch :-o i don't even see watcher-stable-maint | 09:13 |
elodilles | frickler: +2+W'd | 09:21 |
opendevreview | Merged openstack/releases master: [kolla] Transition 2023.1 Antelope to Unmaintained https://review.opendev.org/c/openstack/releases/+/934490 | 09:29 |
opendevreview | Dmitriy Rabotyagov proposed openstack/releases master: Release OpenSTack-Ansible Antelope https://review.opendev.org/c/openstack/releases/+/936714 | 10:13 |
frickler | thx, /me is afk for a bit, will run the scripts later | 10:25 |
elodilles | ACK | 10:44 |
elodilles | hmm, 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 |
elodilles | this is the last log line before the timeout: https://zuul.opendev.org/t/openstack/build/2bb14f86aff148ed884da9270baade57/log/job-output.txt#3858 | 11:40 |
elodilles | maybe a re-enqueue can be tried for this | 11:41 |
elodilles | though i haven't checked yet if the tags exist or not | 11:41 |
elodilles | btw, 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 |
elodilles | though 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#3858 | 11:45 |
sean-k-mooney | elodilles: that becasue its called watcher-release-core. maybe they have the gerrit acls set up diffently | 11:58 |
sean-k-mooney | thre are two diffent convetions i was aware of stable banches are either owned by -release-core or -stable-maint | 11:59 |
sean-k-mooney | im 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 projects | 12:00 |
sean-k-mooney | ill look into the acls shortly | 12:00 |
elodilles | sean-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.config | 12:49 |
sean-k-mooney | elodilles: it shoudl not be in watcher-core | 12:49 |
elodilles | ah sorry, you wrote watcher-release-core :X | 12:50 |
sean-k-mooney | yep so that the problem however | 12:50 |
sean-k-mooney | they didnt set up watcher-release-core to be able to merge on stable branches | 12:50 |
sean-k-mooney | ill submit a patche to do that for the relvent watcher repos | 12:50 |
elodilles | sean-k-mooney: yeah, there is no such thing as watcher-release-core for the record. rather watcher-release. | 12:51 |
sean-k-mooney | i 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 group | 12:51 |
sean-k-mooney | oh sorry yes that what i ment | 12:51 |
sean-k-mooney | watcher-release | 12:52 |
elodilles | i'd rather use watcher-stable-maint as i don't even know what watcher-release is meant for | 12:52 |
sean-k-mooney | its ment for stable branch mantaince :) | 12:52 |
sean-k-mooney | as i said there are two partterns in use in openstack | 12:52 |
elodilles | then i'd rather name it watcher-stable-maint o:) | 12:52 |
sean-k-mooney | well we coudl but we will have to rename the existing groups | 12:53 |
elodilles | i see | 12:53 |
sean-k-mooney | so in the past any project not governed by the unified release process had a -release group to allow pushing tags and merging on stable branchs | 12:54 |
sean-k-mooney | "core" project had a -stable-maint team but in things like stackforge and neutron stadium | 12:54 |
sean-k-mooney | -release was more common as far as i was aware | 12:54 |
sean-k-mooney | im not sure how much fo that is still true or relevent | 12:55 |
sean-k-mooney | elodilles: so it looks like watcher-release and watcher-sadhboard-release are currently unused | 12:56 |
elodilles | sean-k-mooney: oh, i did not know this part of the history o:) | 13:02 |
elodilles | good to know | 13:02 |
elodilles | yepp, i haven't seen that group used in watcher.config | 13:02 |
sean-k-mooney | given its unused in our gerrit acls | 13:11 |
sean-k-mooney | woudl you prefer to just change too the -stable-maint nameing then | 13:11 |
sean-k-mooney | before doing that i woudl want ot document that on the maining list | 13:12 |
sean-k-mooney | but i dont nessiarly disagree with that, its just a littme more work then reusign the exisitng groups | 13:12 |
elodilles | sean-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-mooney | ok 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 things | 13:14 |
elodilles | +1 | 13:14 |
frickler | 50 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 repos | 13:29 |
elodilles | frickler: looks awesome, thanks o/ | 13:43 |
opendevreview | Dmitriy Rabotyagov proposed openstack/releases master: [OpenStack-Ansible] Branch for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/936753 | 14:28 |
noonedeadpunk | it would be nice to land that sooner then later ^ | 14:28 |
elodilles | +2'd | 14:46 |
noonedeadpunk | thanks! | 14:47 |
sean-k-mooney | elodilles: sorry got distracted talking to someone else but https://review.opendev.org/c/openstack/project-config/+/936754 should correct the stable branch permissions | 14:49 |
elodilles | sean-k-mooney: +1 thanks o/ | 15:09 |
opendevreview | Carlos Eduardo proposed openstack/releases master: Add manila deadlines to the official schedule https://review.opendev.org/c/openstack/releases/+/935891 | 18:05 |
opendevreview | Merged openstack/releases master: [OpenStack-Ansible] Branch for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/936753 | 18:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!