Thursday, 2024-02-01

fricklerJayF: elodilles: working my way through the review dashboard I found https://review.opendev.org/c/openstack/releases/+/897054 , did you just loose track of this or can it be abandoned?06:27
fricklerfor https://review.opendev.org/c/openstack/releases/+/906640 the list-changes job says "WARNING: Did not find revision '17.2.0' in list of versions with release notes ['17.0.0-56'], skipping reno06:41
fricklerwhere 17.0.0-56 is the current reno version on master instead of 2023.2, cf. https://docs.openstack.org/releasenotes/manila/unreleased.html. is this a known issue happening while master hasn't had a new release or is this something for further investigation?06:43
opendevreviewRiccardo Pittau proposed openstack/releases master: Release ironic-inspector 12.0.0 for caracal  https://review.opendev.org/c/openstack/releases/+/90721708:54
fricklerttx: hberaud: could one of you look at https://review.opendev.org/c/openstack/releases/+/906626 please? then we should be ready to do some yoga09:00
hberaudack09:01
hberauddone09:03
fricklerty09:04
opendevreviewMerged openstack/releases master: Fix validation when moving to unmaintained  https://review.opendev.org/c/openstack/releases/+/90662609:17
hberaudelodilles: can we proceed with the new release schedule, or do we want to wait a bit more?10:09
elodilleshberaud: let me quickly check whether we got mails regarding it on ML10:20
hberaudack, let you +W10:20
elodilles(nevertheless, i'm OK to +W it and as far as i remember ttx also +2'd it already)10:20
elodilleshberaud: i see no mails, so please +W it \o/10:21
elodilleshberaud: i'll have downstream meeting and other things today, but for a trial we can start merging some yoga-eom patches (not too many at the same time) and I'll clean those yoga branches some time at late afternoon10:32
elodillesand now i disapper :X10:33
*** elodilles is now known as elodilles_afk10:33
hberauddone10:38
hberaudack for the yoga patches10:38
opendevreviewMerged openstack/releases master: [ironic] release ironic-tempest-plugin version 2.8.0  https://review.opendev.org/c/openstack/releases/+/90671210:43
opendevreviewMerged openstack/releases master: Release ironic-python-agent 9.9.0 for caracal  https://review.opendev.org/c/openstack/releases/+/90721810:45
opendevreviewMerged openstack/releases master: Proposed 2024.2 Dalmatian release schedule  https://review.opendev.org/c/openstack/releases/+/90605010:49
opendevreviewMerged openstack/releases master: python-openstackclient 6.5.0  https://review.opendev.org/c/openstack/releases/+/90732511:15
opendevreviewmarihan girgis proposed openstack/releases master: Release telemetry-tempest-plugin for caracal  https://review.opendev.org/c/openstack/releases/+/90740611:16
opendevreviewMerged openstack/releases master: Release ironic 24.0.0 for caracal  https://review.opendev.org/c/openstack/releases/+/90721611:16
opendevreviewMerged openstack/releases master: [python-manilaclient] New Bobcat stable release  https://review.opendev.org/c/openstack/releases/+/90665611:19
opendevreviewMerged openstack/releases master: [manila] New 2023.2 stable release  https://review.opendev.org/c/openstack/releases/+/90664011:19
opendevreviewMerged openstack/releases master: Create new oslo.messaging release for 2023.1  https://review.opendev.org/c/openstack/releases/+/90705511:25
opendevreviewMerged openstack/releases master: [python-manilaclient] New 2023.1 stable release  https://review.opendev.org/c/openstack/releases/+/90665711:25
fricklerwe could start with kolla or designate for yoga-eom (or both), those shouldn't affect too many others but still be a good test13:51
frickler"reno -q semver-next" is failing on cliff for stable/yoga https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_09a/906576/3/check/releases-tox-list-changes/09a313a/tox/list-changes/list-changes-results.log /me goes debugging14:03
fricklerapproved barbican now, that was one I didn't touch so far https://review.opendev.org/c/openstack/releases/+/906583 seems > 50% of the patches still need PTL feedback though :-(15:00
fricklerseems that also has some open reviews to abandon, so we get to test elodilles' script15:15
opendevreviewMerged openstack/releases master: [barbican] Transition Yoga to Unmaintained  https://review.opendev.org/c/openstack/releases/+/90658315:16
*** elodilles_afk is now known as elodilles15:21
elodillesack, I can run the branch abandoning script against barbican. let's see.15:23
fricklerhmm, do we want to keep mirroring the unmaintained branches to github? seems the current setup does this. likely better discuss with the TC?15:23
elodillesi think it's OK to have the proper unmainted/<series> branches there, and keep them up-to-date15:25
elodillesso if it's working already, then why would we change that?15:25
fricklermaybe because they're not so much official openstack project branches any more? but I guess it's fine, yes15:26
elodilles(the single open barbican stable/yoga patch is now abandoned)15:32
fricklerthere is also at least one for python-barbicanclient. didn't check the other repos yet15:34
fricklermight be a nice idea to point the script to a deliverable file instead of single repos15:35
fricklerin particular when looking at "big fish" like neutron15:36
fricklerI can look at that if you want15:37
elodilleswell, the easiest is to do it as it's usage help says (in that way run it against all the deliverables of a branch)15:37
fricklerwell but some projects still want to merge changes afaict15:38
elodillesyes, in this case, that is not fortunate15:39
fricklerdo you want me to approve some more patches or is this enough for a first test? I don't want to +3 those patches that I touched myself15:39
fricklerexcept maybe sdk which was just a rebase15:39
elodilleslet me have a quick look at the barbican deliverables whether they have everything in the right place (branch, tag)15:40
JayFI would personally think not mirroring unmaintained/ branches to github would likely confuse downstreams. It hadn't occured to me we'd consider not mirroring them.15:42
JayF(To state that with less negatives: Yes, I think we should mirror them)15:42
elodilles+115:42
elodillesfrickler: barbican yoga-eom tags and unmaintained/yoga branches are there, so I guess we can +W some other patches that are ready for EOM'ing :)15:47
fricklero.k., I'll check the list15:48
fricklerhmm, some failures in https://review.opendev.org/c/openstack/python-barbicanclient/+/907439 seem to happen because the requirements repo branch is missing15:49
elodillesand when those patches merged & tags + unmaintained branches are created i can run the delete branch script (might be that i'll commute home before i start doing that)15:49
fricklermaybe we should approve the patch to create it for reqs, but defer deleting the stable/yoga branch there till the end?15:49
elodillesfrickler: that's a good idea15:50
elodillesfrickler: i guess the same will be valid for devstack & grenade as well... 15:50
elodillesfrickler: definitely things need to be hacked to make things work :/15:51
elodillestonyb already signaled that the upper constraints redirections won't work as soon as we delete the requirments repo yoga branch... and i think that will be among the smallest issues we'll face...15:53
fricklerwell that's up to the unmaintained-core team then I guess ... which iirc contains just you for now? ;)15:54
fricklerah, no, you and tonyb indeed15:57
opendevreviewMerged openstack/releases master: Final Yoga releases of Ironic deliverables  https://review.opendev.org/c/openstack/releases/+/90724816:00
fricklerelodilles: would you want to override tonyb's -1 on https://review.opendev.org/c/openstack/releases/+/906582? otherwise I'd think we should rather wait for him to be online. I only approved telemetry now to have a second project for test comparison16:02
elodillesthat's OK for today then :) i don't want anything to override :)16:03
opendevreviewMerged openstack/releases master: [Telemetry] Transition Yoga to Unmaintained  https://review.opendev.org/c/openstack/releases/+/90656816:11
fricklermeh, we just made another yoga release which should be added into stable/yoga reqs before we move on I guess. let's hope CI still passes on that, else it will get even more messy16:18
elodillesahh, sushy. yepp, true. :/16:38
elodillesbtw, we had a release job failure: https://lists.openstack.org/archives/list/release-job-failures@lists.openstack.org/thread/GEKGQ2EOZHLSUMQYJCB27O6QWJH4UYZQ/16:47
elodillesthe propose-update-constraints job failed when it tried to clone the openstack/requirements repository16:47
elodillesfungi: can you please re-enqueue this job? this looks like a temporary gitea (?) issue16:48
fungichecking16:50
elodillesthanks o/16:52
fungielodilles: i don't think we should. the failed build was part of this buildset, and we don't want to rerun the successful jobs (particularly release-openstack-python): https://zuul.opendev.org/t/openstack/buildset/bdd92ab904df453fa2dd633e3002ae4816:58
fungiit would be safer for someone to just push the constraints change directly to gerrit, if it's needed16:59
elodillesfungi: i see, thanks, then i'll propose that19:08
fungito be clear, i think what would happen if i tried to reenqueue that tag is the release-openstack-python job would fail (because pypi will refuse to allow reuploading the same sdist/wheel), and then the previously failed constraints job would get skipped and not rerun19:11
fungii'd have to look through the order of operations in release-openstack-python but rerunning it might replace the original sdist and wheel on our tarballs site before trying the pypi upload, which would be even worse because then the checksums won't be the same and the signatures we publish for them won't validate the files on pypi for that version any longer19:12
elodillesyepp, thanks for the explanation, i forget time to time that a single job cannot be re-run from the release pipeline :S19:13
elodillesupper constraints bumping is an easy patch anyway, so it's just quicker to propose a patch19:14
elodillesfungi: btw, i'll run the EOL and EOM cleanup scripts if infra status allows19:16
fungielodilles: sure, we don't have anything going on that should impact it right now19:16
elodillesack, thanks o/19:17
opendevreviewElod Illes proposed openstack/releases master: WIP: Add helper script to abandon open changes  https://review.opendev.org/c/openstack/releases/+/90710619:42
opendevreviewTianqi Xiao proposed openstack/releases master: Update tempest version to latest compatible for yoga, zed & antelope  https://review.opendev.org/c/openstack/releases/+/90749320:19
opendevreviewTianqi Xiao proposed openstack/releases master: Update tempest max supporting version for yoga, zed & antelope  https://review.opendev.org/c/openstack/releases/+/90749320:24
fricklerfyi looking at the log for the above failure led me to find an error in the script, not directly related though https://review.opendev.org/c/openstack/project-config/+/90749220:26
opendevreviewMerged openstack/releases master: [manila] New 2023.1 stable release  https://review.opendev.org/c/openstack/releases/+/90664220:31
opendevreviewElod Illes proposed openstack/releases master: Add is-eom option to list-deliverables command  https://review.opendev.org/c/openstack/releases/+/90750122:18
opendevreviewElod Illes proposed openstack/releases master: DNM: list_eom_stale_branches.sh  https://review.opendev.org/c/openstack/releases/+/90750222:23
* tonyb is around now. I'll get to my releases reviews in ~60mins22:33
elodillesso, there were two kinds of deletions this time: 1) ussuri EOL branches: https://paste.opendev.org/show/b29Ad43ur8Xuwhfmnl78/22:33
tonybfrickler, elodilles ^22:33
elodilles2) yoga EOM branches: https://paste.opendev.org/show/bFyVTdFXRPBpZ0bWmzlV/22:33
elodillestonyb: ack, thanks!22:33
elodillesfor me it's waaaay EOD, so i'm leaving for the day o:)22:34
*** tosky_ is now known as tosky23:14
opendevreviewTony Breeds proposed openstack/releases master: [requirements] Transition Yoga to Unmaintained  https://review.opendev.org/c/openstack/releases/+/90658223:33

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