frickler | JayF: 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 |
---|---|---|
frickler | for 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 reno | 06:41 |
frickler | where 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 |
opendevreview | Riccardo Pittau proposed openstack/releases master: Release ironic-inspector 12.0.0 for caracal https://review.opendev.org/c/openstack/releases/+/907217 | 08:54 |
frickler | ttx: hberaud: could one of you look at https://review.opendev.org/c/openstack/releases/+/906626 please? then we should be ready to do some yoga | 09:00 |
hberaud | ack | 09:01 |
hberaud | done | 09:03 |
frickler | ty | 09:04 |
opendevreview | Merged openstack/releases master: Fix validation when moving to unmaintained https://review.opendev.org/c/openstack/releases/+/906626 | 09:17 |
hberaud | elodilles: can we proceed with the new release schedule, or do we want to wait a bit more? | 10:09 |
elodilles | hberaud: let me quickly check whether we got mails regarding it on ML | 10:20 |
hberaud | ack, let you +W | 10:20 |
elodilles | (nevertheless, i'm OK to +W it and as far as i remember ttx also +2'd it already) | 10:20 |
elodilles | hberaud: i see no mails, so please +W it \o/ | 10:21 |
elodilles | hberaud: 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 afternoon | 10:32 |
elodilles | and now i disapper :X | 10:33 |
*** elodilles is now known as elodilles_afk | 10:33 | |
hberaud | done | 10:38 |
hberaud | ack for the yoga patches | 10:38 |
opendevreview | Merged openstack/releases master: [ironic] release ironic-tempest-plugin version 2.8.0 https://review.opendev.org/c/openstack/releases/+/906712 | 10:43 |
opendevreview | Merged openstack/releases master: Release ironic-python-agent 9.9.0 for caracal https://review.opendev.org/c/openstack/releases/+/907218 | 10:45 |
opendevreview | Merged openstack/releases master: Proposed 2024.2 Dalmatian release schedule https://review.opendev.org/c/openstack/releases/+/906050 | 10:49 |
opendevreview | Merged openstack/releases master: python-openstackclient 6.5.0 https://review.opendev.org/c/openstack/releases/+/907325 | 11:15 |
opendevreview | marihan girgis proposed openstack/releases master: Release telemetry-tempest-plugin for caracal https://review.opendev.org/c/openstack/releases/+/907406 | 11:16 |
opendevreview | Merged openstack/releases master: Release ironic 24.0.0 for caracal https://review.opendev.org/c/openstack/releases/+/907216 | 11:16 |
opendevreview | Merged openstack/releases master: [python-manilaclient] New Bobcat stable release https://review.opendev.org/c/openstack/releases/+/906656 | 11:19 |
opendevreview | Merged openstack/releases master: [manila] New 2023.2 stable release https://review.opendev.org/c/openstack/releases/+/906640 | 11:19 |
opendevreview | Merged openstack/releases master: Create new oslo.messaging release for 2023.1 https://review.opendev.org/c/openstack/releases/+/907055 | 11:25 |
opendevreview | Merged openstack/releases master: [python-manilaclient] New 2023.1 stable release https://review.opendev.org/c/openstack/releases/+/906657 | 11:25 |
frickler | we could start with kolla or designate for yoga-eom (or both), those shouldn't affect too many others but still be a good test | 13: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 debugging | 14:03 |
frickler | approved 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 |
frickler | seems that also has some open reviews to abandon, so we get to test elodilles' script | 15:15 |
opendevreview | Merged openstack/releases master: [barbican] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906583 | 15:16 |
*** elodilles_afk is now known as elodilles | 15:21 | |
elodilles | ack, I can run the branch abandoning script against barbican. let's see. | 15:23 |
frickler | hmm, 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 |
elodilles | i think it's OK to have the proper unmainted/<series> branches there, and keep them up-to-date | 15:25 |
elodilles | so if it's working already, then why would we change that? | 15:25 |
frickler | maybe because they're not so much official openstack project branches any more? but I guess it's fine, yes | 15:26 |
elodilles | (the single open barbican stable/yoga patch is now abandoned) | 15:32 |
frickler | there is also at least one for python-barbicanclient. didn't check the other repos yet | 15:34 |
frickler | might be a nice idea to point the script to a deliverable file instead of single repos | 15:35 |
frickler | in particular when looking at "big fish" like neutron | 15:36 |
frickler | I can look at that if you want | 15:37 |
elodilles | well, 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 |
frickler | well but some projects still want to merge changes afaict | 15:38 |
elodilles | yes, in this case, that is not fortunate | 15:39 |
frickler | do 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 myself | 15:39 |
frickler | except maybe sdk which was just a rebase | 15:39 |
elodilles | let me have a quick look at the barbican deliverables whether they have everything in the right place (branch, tag) | 15:40 |
JayF | I 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 | +1 | 15:42 |
elodilles | frickler: 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 |
frickler | o.k., I'll check the list | 15:48 |
frickler | hmm, some failures in https://review.opendev.org/c/openstack/python-barbicanclient/+/907439 seem to happen because the requirements repo branch is missing | 15:49 |
elodilles | and 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 |
frickler | maybe we should approve the patch to create it for reqs, but defer deleting the stable/yoga branch there till the end? | 15:49 |
elodilles | frickler: that's a good idea | 15:50 |
elodilles | frickler: i guess the same will be valid for devstack & grenade as well... | 15:50 |
elodilles | frickler: definitely things need to be hacked to make things work :/ | 15:51 |
elodilles | tonyb 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 |
frickler | well that's up to the unmaintained-core team then I guess ... which iirc contains just you for now? ;) | 15:54 |
frickler | ah, no, you and tonyb indeed | 15:57 |
opendevreview | Merged openstack/releases master: Final Yoga releases of Ironic deliverables https://review.opendev.org/c/openstack/releases/+/907248 | 16:00 |
frickler | elodilles: 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 comparison | 16:02 |
elodilles | that's OK for today then :) i don't want anything to override :) | 16:03 |
opendevreview | Merged openstack/releases master: [Telemetry] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906568 | 16:11 |
frickler | meh, 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 messy | 16:18 |
elodilles | ahh, sushy. yepp, true. :/ | 16:38 |
elodilles | btw, we had a release job failure: https://lists.openstack.org/archives/list/release-job-failures@lists.openstack.org/thread/GEKGQ2EOZHLSUMQYJCB27O6QWJH4UYZQ/ | 16:47 |
elodilles | the propose-update-constraints job failed when it tried to clone the openstack/requirements repository | 16:47 |
elodilles | fungi: can you please re-enqueue this job? this looks like a temporary gitea (?) issue | 16:48 |
fungi | checking | 16:50 |
elodilles | thanks o/ | 16:52 |
fungi | elodilles: 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/bdd92ab904df453fa2dd633e3002ae48 | 16:58 |
fungi | it would be safer for someone to just push the constraints change directly to gerrit, if it's needed | 16:59 |
elodilles | fungi: i see, thanks, then i'll propose that | 19:08 |
fungi | to 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 rerun | 19:11 |
fungi | i'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 longer | 19:12 |
elodilles | yepp, thanks for the explanation, i forget time to time that a single job cannot be re-run from the release pipeline :S | 19:13 |
elodilles | upper constraints bumping is an easy patch anyway, so it's just quicker to propose a patch | 19:14 |
elodilles | fungi: btw, i'll run the EOL and EOM cleanup scripts if infra status allows | 19:16 |
fungi | elodilles: sure, we don't have anything going on that should impact it right now | 19:16 |
elodilles | ack, thanks o/ | 19:17 |
opendevreview | Elod Illes proposed openstack/releases master: WIP: Add helper script to abandon open changes https://review.opendev.org/c/openstack/releases/+/907106 | 19:42 |
opendevreview | Tianqi Xiao proposed openstack/releases master: Update tempest version to latest compatible for yoga, zed & antelope https://review.opendev.org/c/openstack/releases/+/907493 | 20:19 |
opendevreview | Tianqi Xiao proposed openstack/releases master: Update tempest max supporting version for yoga, zed & antelope https://review.opendev.org/c/openstack/releases/+/907493 | 20:24 |
frickler | fyi 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/+/907492 | 20:26 |
opendevreview | Merged openstack/releases master: [manila] New 2023.1 stable release https://review.opendev.org/c/openstack/releases/+/906642 | 20:31 |
opendevreview | Elod Illes proposed openstack/releases master: Add is-eom option to list-deliverables command https://review.opendev.org/c/openstack/releases/+/907501 | 22:18 |
opendevreview | Elod Illes proposed openstack/releases master: DNM: list_eom_stale_branches.sh https://review.opendev.org/c/openstack/releases/+/907502 | 22:23 |
* tonyb is around now. I'll get to my releases reviews in ~60mins | 22:33 | |
elodilles | so, there were two kinds of deletions this time: 1) ussuri EOL branches: https://paste.opendev.org/show/b29Ad43ur8Xuwhfmnl78/ | 22:33 |
tonyb | frickler, elodilles ^ | 22:33 |
elodilles | 2) yoga EOM branches: https://paste.opendev.org/show/bFyVTdFXRPBpZ0bWmzlV/ | 22:33 |
elodilles | tonyb: ack, thanks! | 22:33 |
elodilles | for me it's waaaay EOD, so i'm leaving for the day o:) | 22:34 |
*** tosky_ is now known as tosky | 23:14 | |
opendevreview | Tony Breeds proposed openstack/releases master: [requirements] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906582 | 23:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!