| opendevreview | Takashi Kajinami proposed openstack/releases master: Release oslo.utils 7.3.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926622 | 02:42 |
|---|---|---|
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.messaging 14.9.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926623 | 02:42 |
| tkajinam | damani, fyi... you don't have to put all oslo release patches in the single chain but these can be independent. Actually these SHOULD be independent IMO so that we can merge these independently. For this time we can keep the chain and break it when we need to do | 02:50 |
| tkajinam | I mean, you don't have to updated these patches now. let's do it when we need | 02:50 |
| *** bauzas_ is now known as bauzas | 03:51 | |
| *** bauzas_ is now known as bauzas | 06:34 | |
| *** elodilles_ooo is now known as elodilles | 06:44 | |
| *** bauzas_ is now known as bauzas | 07:34 | |
| damani | tkajinam, ok | 08:26 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.log 6.1.2 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926712 | 08:42 |
| elodilles | frickler: i'm back, so let me know if you need help with the non-client libs release patches (meanwhile damani proposed oslo release patches, and Pranali glance patches, so those can be skipped when generating the dalmatian-final-non-client-libs patches) | 09:48 |
| frickler | elodilles: I must admit that I didn't get to look at this yet, but I hope to do so later today | 09:52 |
| opendevreview | yatin proposed openstack/releases master: Release oslo.serialization 5.4.1 for 2024.1 (Caracal) https://review.opendev.org/c/openstack/releases/+/926724 | 10:03 |
| elodilles | frickler: no problem, feel free to ping me if you don't have time and then i can generate the patches | 10:04 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.upgradecheck 2.4.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926535 | 10:53 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.vmware 4.5.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926543 | 10:53 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.config 9.6.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926544 | 10:53 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.utils 7.3.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926622 | 10:53 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.messaging 14.9.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926623 | 10:53 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.log 6.1.2 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926712 | 10:53 |
| frickler | elodilles: hmm, ./tools/list_library_unreleased_changes.sh only outputs "Finding zed library repositories..." and then nothing for half an hour. that doesn't sound like what is expected to happen? | 12:23 |
| frickler | seems detection of the current series is broken since we wrapped the alphabet | 12:26 |
| frickler | I wonder how this has been used for the last cycles | 12:26 |
| elodilles | hmmm, indeed | 12:28 |
| elodilles | well, you can use it like this: tools/list_library_unreleased_changes.sh dalmatian >/tmp/cwiff.log | 12:29 |
| frickler | yes, hardcoded that now. but the next issue now seems to be that connectivity to vexxhost via IPv6 is broken and python doesn't implement happy eyeballs. will need to workaround | 12:31 |
| frickler | damani: the above check script lists five oslo repos for which there are no releases proposed yet, are you still working on those or should I autogenerate them? | 12:58 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final ceilometermiddleware for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926756 | 13:09 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final heat-translator for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926757 | 13:10 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final ironic-lib for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926758 | 13:11 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final keystoneauth for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926759 | 13:12 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final keystonemiddleware for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926760 | 13:12 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final kuryr for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926761 | 13:13 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final mistral-extra for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926762 | 13:15 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final neutron-lib for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926764 | 13:16 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final openstacksdk for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926765 | 13:18 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final os-brick for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926766 | 13:18 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final os-ken for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926767 | 13:19 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final os-vif for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926768 | 13:19 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final stevedore for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926769 | 13:21 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final sushy for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926770 | 13:22 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release final taskflow for 2024.2 (Dalmatian) https://review.opendev.org/c/openstack/releases/+/926771 | 13:23 |
| frickler | elodilles: ^^ please have a look. some git logs are broken where there were signed commits. also some repos were in the generated list that didn't seem to have changes, the script skipped those | 13:26 |
| frickler | also the docs didn't warn me that I need to provide a template for the commit message. I copied it from some older reviews | 13:27 |
| frickler | and there were tracebacks in get-versions-on-series that I still need to look into | 13:27 |
| elodilles | ACK, thanks frickler o/ i'll look into the patches | 13:50 |
| elodilles | yeah, there's no template, you can write what you want, though i usually do the same, just use a random former patch's text o:) | 13:51 |
| elodilles | about the tracebacks: i don't remember whether i saw any last time :S (we fixed at least a dozen issue when we moved from alphabetical branch names to number based, so yeah, we might still miss some :S) | 13:54 |
| *** jbernard_ is now known as jbernard | 14:02 | |
| frickler | seems the five oslo repos I mentioned didn't actually have any pending changes, either, guess I parsed the output from the script wrongly | 14:18 |
| *** bauzas_ is now known as bauzas | 14:23 | |
| frickler | elodilles: there is a commit message mismatch in https://review.opendev.org/c/openstack/releases/+/926513 and others, do you think I should pull my approval and amend that? or does it not matter enough? 6.0.1 vs. 6.1.0 | 14:36 |
| elodilles | :S | 14:36 |
| frickler | maybe I'll leave the first one but stop approving the others? | 14:37 |
| elodilles | yeah, thanks | 14:37 |
| elodilles | if you see other mismatch then let's ask damani to update | 14:37 |
| elodilles | btw, i see one missing release: oslo.policy needs a release as it has a merged but unreleased patch (even if that is a non-functional change) | 14:38 |
| elodilles | damani: fyi ^^^ could you propose oslo.policy as well? | 14:38 |
| frickler | there are more, I only noticed it for the second patch and then went back double checking. I'll amend myself to speed up things and possibly also do ptl-approved override | 14:39 |
| elodilles | frickler: ACK, thanks o/ | 14:39 |
| frickler | elodilles: ah, I saw that one but wasn't sure whether it actually needs a release just for the pre-commit change | 14:39 |
| opendevreview | Jens Harbott proposed openstack/releases master: Release oslo.context 5.6.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926515 | 14:40 |
| frickler | meh, now I also need to rebase the whole stack. I'll take the opportunity to rebase them all onto master, unstacking things | 14:41 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.18n 6.4.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926516 | 14:41 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.limit 2.5.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926517 | 14:42 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.metrics 0.9.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926518 | 14:43 |
| opendevreview | Merged openstack/releases master: Release oslo.concurrency 6.0.1 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926513 | 14:44 |
| frickler | luckily this only resets zuul checks, but not existing reviews, so I can still immediately approve those that don't need editing | 14:44 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.middleware 6.2.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926519 | 14:44 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.privsep 3.4.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926531 | 14:45 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.reports 3.4.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926533 | 14:46 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.rootwrap 7.3.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926534 | 14:46 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.upgradecheck 2.4.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926535 | 14:47 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.vmware 4.5.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926543 | 14:48 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.config 9.6.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926544 | 14:50 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.utils 7.3.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926622 | 14:51 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.messaging 14.9.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926623 | 14:54 |
| opendevreview | Daniel Bengtsson proposed openstack/releases master: Release oslo.log 6.1.2 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926712 | 14:54 |
| opendevreview | Merged openstack/releases master: Release oslo.context 5.6.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926515 | 14:58 |
| frickler | sometimes I'm wondering whether Sean McGinnis is using a bot to add all those reviewers so fast ;) sadly not around on IRC anymore afaict | 15:02 |
| elodilles | frickler: yepp, he does | 15:05 |
| elodilles | tools/add_reviewers.sh | 15:05 |
| elodilles | you can run it, too :) | 15:06 |
| opendevreview | Merged openstack/releases master: Release oslo.metrics 0.9.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926518 | 15:06 |
| opendevreview | Merged openstack/releases master: Release oslo.middleware 6.2.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926519 | 15:06 |
| opendevreview | Merged openstack/releases master: Release oslo.18n 6.4.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926516 | 15:06 |
| opendevreview | Merged openstack/releases master: Release oslo.upgradecheck 2.4.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926535 | 15:07 |
| opendevreview | Merged openstack/releases master: Release oslo.privsep 3.4.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926531 | 15:09 |
| opendevreview | Merged openstack/releases master: Release oslo.reports 3.4.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926533 | 15:11 |
| opendevreview | Merged openstack/releases master: Release oslo.rootwrap 7.3.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926534 | 15:11 |
| frickler | nice | 15:14 |
| frickler | ah, now that I see the next reqs updates showing up, I remember I wanted to merge my fix. let me ask for some infra reviews | 15:19 |
| opendevreview | Merged openstack/releases master: Release final glance_store for Dalmatian https://review.opendev.org/c/openstack/releases/+/926608 | 15:23 |
| opendevreview | Merged openstack/releases master: Release final python-glanceclient for Dalmatian https://review.opendev.org/c/openstack/releases/+/926607 | 15:27 |
| frickler | elodilles: with https://review.opendev.org/c/openstack/project-config/+/926405 merged let's now only do a single release and then check results before proceeding | 15:59 |
| frickler | approved https://review.opendev.org/c/openstack/releases/+/926543 now | 16:00 |
| opendevreview | Merged openstack/releases master: Release oslo.vmware 4.5.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926543 | 16:12 |
| elodilles | is the u-c job running already? :-o | 16:26 |
| elodilles | hmmm. i don't see the job yet | 16:29 |
| frickler | elodilles: no, tag-release failed due to opendev having switched to noble https://zuul.opendev.org/t/openstack/build/c282702c8e5f47af8cf6f6ec9e5ef8c8 | 16:31 |
| frickler | release team: ^^ please refrain from approving things until this is fixed, clarkb is about to push a patch | 16:31 |
| elodilles | yeah, just wanted to link tag-release job list: https://zuul.opendev.org/t/openstack/builds?job_name=tag-releases&project=openstack/releases | 16:33 |
| clarkb | remote: https://review.opendev.org/c/openstack/project-config/+/926821 Use yaml.safe_load in release jobs | 16:37 |
| elodilles | it seems that flake8 is too old for the existing importlib_metadata there ^^^ we probably have to bump 'hacking' version in project-config's tox.ini as well | 16:59 |
| clarkb | elodilles: yup I just pushed a change for that | 17:03 |
| clarkb | I'll sort out any rule violations then rebase 926821 on it | 17:04 |
| elodilles | clarkb: thanks o/ | 17:07 |
| *** bauzas_ is now known as bauzas | 18:30 | |
| opendevreview | Lajos Katona proposed openstack/releases master: Release oslo.utils 7.3.0 for 2024.2 dalmatian. https://review.opendev.org/c/openstack/releases/+/926622 | 18:38 |
| clarkb | elodilles: the yaml.safe_load fix has landed finally. Sorry that took longer than I expected it to | 19:50 |
| frickler | clarkb: could you re-enqueue the tag-release job? I think elod is offline by now and I'm as good as gone, too | 19:58 |
| frickler | else I'll continue tomorrow | 19:58 |
| Clark[m] | Sorry just popped out for lunch. Maybe fungi can? | 20:03 |
| Clark[m] | Also we have to reenwueue the whole build set is that safe? | 20:03 |
| frickler | this buildset only has that single job https://zuul.opendev.org/t/openstack/buildset/b9ea6cb2ab074da19a164d6faa8ef0c4 | 20:04 |
| frickler | but if you're unsure I can check with elod tomorrow | 20:04 |
| clarkb | reviewing what ran I think it is safe beacuse it never got to the point of pushing a tag | 20:32 |
| clarkb | https://opendev.org/openstack/oslo.vmware/src/tag/4.5.0 is a 404 which helps confirm this | 20:32 |
| fungi | looking | 20:35 |
| clarkb | `zuul-client enqueue-ref --tenant openstack --pipeline release-post --project opendev.org/openstack/releases --newrev 6e2b2154b4e80678885cf01d5940c944380dfe35 --oldrev d6ba5257898d8130fc3fab62a63de97fcd9f1a10 --ref refs/heads/master` is the command I've constructed if we think this is safe does that look right to you? | 20:36 |
| fungi | yeah, the tag-releases job is all it runs, i think? and it's idepotent | 20:37 |
| clarkb | ya I confirmed that buildset only had the one job as frickler stated | 20:37 |
| fungi | so even if it breaks halfway through tagging things it should be safe to rerun | 20:37 |
| clarkb | cool in that case does the command above look correct to you? | 20:37 |
| fungi | sudo zuul-client enqueue-ref --tenant=openstack --pipeline=release-post --project=openstack/releases --ref=refs/heads/master --newrev=6e2b2154b4e80678885cf01d5940c944380dfe35 | 20:39 |
| fungi | is what i would have tried to run | 20:39 |
| fungi | looks like --oldrev isn't needed | 20:40 |
| clarkb | ok I can run your construction. I'm guessing thats a known good command based on historical actions? | 20:40 |
| fungi | on zuul01 i ran `/home/fungi/reenqueue.py https://399637c6f98f105a3f15-e3daadb986270c27e8869be52c5f0413.ssl.cf1.rackcdn.com/6e2b2154b4e80678885cf01d5940c944380dfe35/release-post/tag-releases/c282702/` | 20:40 |
| clarkb | ok executing your command now | 20:41 |
| fungi | these days i can probably redo that script to extract the raw file url from the buildset | 20:41 |
| clarkb | done, I see it in the zuul status | 20:41 |
| clarkb | the job is running now | 20:48 |
| clarkb | https://zuul.opendev.org/t/openstack/stream/c1599b47e6484a099e90d6476649c00d?logfile=console.log | 20:48 |
| clarkb | job succeeded and https://opendev.org/openstack/oslo.vmware/src/tag/4.5.0 exists now | 20:50 |
| clarkb | hash looks good on the tag too | 20:50 |
| clarkb | https://zuul.opendev.org/t/openstack/buildset/813f914fbfd84cd5a3cc464969f39f9c looks like there was a second job though that failed (it didn't run previously because it depends on the tag job I guess) | 22:19 |
| clarkb | wheel build for python-pcre failed. I'm going to guess this is a similar issue to Pillow where there isn't a wheel for python3.12 on the version we're forcing | 22:19 |
| clarkb | ok this is slightly different. python-pcre is absolutely ancient and doesn't seem to be maintained anymore so we were previously relying on wheels built for our wheel mirror to provide these wheels. I was hopign we wouldn't need to build those wheels for noble so I'm going to stick with that and instead add the pcre dev lib to openstack/releases. Looks like whereto depends | 22:23 |
| clarkb | on python-pcre and whereto is a direct dependency of releases. Honestly you can probably just use python regexes... | 22:23 |
| clarkb | oh and the raeson gate/check didn't catch this is releases uses a template for the check/gate docs builds but a special job for publication things. I'll leave that as an exercise to the reader to determine if they want to change anything related to that | 22:28 |
| opendevreview | Clark Boylan proposed openstack/releases master: Add libpcre3-dev to bindep https://review.opendev.org/c/openstack/releases/+/926860 | 22:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!