opendevreview | Pranali Deore proposed openstack/releases master: Release glance-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876609 | 06:45 |
---|---|---|
*** amoralej|off is now known as amoralej | 07:13 | |
opendevreview | Merged openstack/releases master: Release zun-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876623 | 09:39 |
opendevreview | Merged openstack/releases master: Release freezer-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876608 | 09:39 |
opendevreview | Merged openstack/releases master: Release ec2api-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876607 | 09:55 |
opendevreview | Merged openstack/releases master: Release vitrage-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876622 | 09:55 |
opendevreview | Merged openstack/releases master: Release manila-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876613 | 09:55 |
opendevreview | Merged openstack/releases master: Release telemetry-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876619 | 09:56 |
opendevreview | Merged openstack/releases master: Release heat-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876610 | 10:00 |
opendevreview | Merged openstack/releases master: Release neutron-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876616 | 10:02 |
opendevreview | Merged openstack/releases master: Release trove-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876620 | 10:02 |
opendevreview | Merged openstack/releases master: Release senlin-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876618 | 10:02 |
opendevreview | Merged openstack/releases master: Release glance-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876609 | 10:02 |
hberaud | elodilles, ttx: Well I think we have an issue with tools/add_release_note_links.sh because no antelope deliverable files are modified once I ran the command. | 10:03 |
hberaud | https://releases.openstack.org/reference/process.html#r-2-week | 10:03 |
hberaud | I think it is related to the branch naming | 10:04 |
hberaud | I'll try to see if I can fix it | 10:04 |
hberaud | it check over links likes this one https://docs.openstack.org/releasenotes/taskflow/antelope.html | 10:05 |
hberaud | while https://docs.openstack.org/releasenotes/taskflow/2023.1.html exists | 10:06 |
hberaud | this script expect the series's name to be passed as an args of the CLI, however if I pass `tools/add_release_note_links.sh 2023.1` everything is ignored | 10:08 |
elodilles | looking | 10:08 |
elodilles | hberaud: yepp, because the script searches here: deliverables/$SERIES/*.yaml | 10:09 |
elodilles | so we have to have the $series name and relese-id as well | 10:09 |
hberaud | exact | 10:10 |
hberaud | we have to pass the both | 10:10 |
elodilles | or get release id based on series name | 10:11 |
elodilles | though the 1st option is really simple | 10:12 |
hberaud | fixed | 10:18 |
hberaud | yeah this script is really simple, I'm not sure it worth the case to modify it to load python module and hence retrieves the series's id rather than simply pass it manually | 10:20 |
elodilles | yes, that is not a big issue to pass both if needed | 10:23 |
opendevreview | Hervé Beraud proposed openstack/releases master: allow to series's ID to generate correct releasenotes links https://review.opendev.org/c/openstack/releases/+/876951 | 10:27 |
hberaud | the fix ^ | 10:27 |
opendevreview | Hervé Beraud proposed openstack/releases master: allow to pass the series's ID to generate correct releasenotes links https://review.opendev.org/c/openstack/releases/+/876951 | 10:27 |
opendevreview | Hervé Beraud proposed openstack/releases master: add releasenotes links for antelope 2023.1 https://review.opendev.org/c/openstack/releases/+/876952 | 10:30 |
opendevreview | Hervé Beraud proposed openstack/releases master: allow to pass the series's ID to generate correct releasenotes links https://review.opendev.org/c/openstack/releases/+/876951 | 10:32 |
elodilles | added a comment. i was thinking about adding $SERIES as default name for series_id, but that is unnecessary as we will need both in the future, so why bother with it.... | 10:35 |
opendevreview | Hervé Beraud proposed openstack/releases master: allow to pass the series's ID to generate correct releasenotes links https://review.opendev.org/c/openstack/releases/+/876951 | 10:36 |
hberaud | yeah that more or less what I was thinking with this latest patch set ^ | 10:37 |
hberaud | elodilles: I'd argue that we don't need a default case with the else condition | 10:37 |
hberaud | as all the next series should be in the form <series> <series_id> | 10:38 |
hberaud | so IMO the patch set 2 is correct | 10:38 |
hberaud | sorry the patch set 3 | 10:39 |
hberaud | (is correct) | 10:39 |
opendevreview | Hervé Beraud proposed openstack/releases master: allow to pass the series's ID to generate correct releasenotes links https://review.opendev.org/c/openstack/releases/+/876951 | 10:40 |
opendevreview | Hervé Beraud proposed openstack/releases master: allow to pass the series's ID to generate correct releasenotes links https://review.opendev.org/c/openstack/releases/+/876951 | 10:41 |
opendevreview | Elod Illes proposed openstack/releases master: Allow to pass release ID to generate correct releasenotes links https://review.opendev.org/c/openstack/releases/+/876951 | 10:54 |
elodilles | hberaud: updated a bit ^^^ hope it's not a problem o:) this is closer to PS3, with minor edits | 10:55 |
hberaud | lGTM | 10:56 |
hberaud | np | 10:57 |
opendevreview | Elod Illes proposed openstack/releases master: Allow to pass release ID to generate correct releasenotes links https://review.opendev.org/c/openstack/releases/+/876951 | 10:59 |
elodilles | actually, modified a bit more ^^^ to stick to the naming we use so far (RELEASE_ID) :S | 10:59 |
hberaud | ok np | 11:04 |
opendevreview | Sylvain Bauza proposed openstack/releases master: Update Nova release liaisons https://review.opendev.org/c/openstack/releases/+/876758 | 11:04 |
opendevreview | Elod Illes proposed openstack/releases master: Add missing release note links for old releases https://review.opendev.org/c/openstack/releases/+/876962 | 11:40 |
opendevreview | Elod Illes proposed openstack/releases master: Add missing release note links for Zed https://review.opendev.org/c/openstack/releases/+/876963 | 11:42 |
opendevreview | Iury Gregory Melo Ferreira proposed openstack/releases master: Release ironic-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876611 | 12:27 |
iurygregory | elodilles, ^ this should be ready to go | 12:27 |
iurygregory | I've also checked the release for inspector, ironic, ipa | 12:36 |
*** amoralej is now known as amoralej|lunch | 13:15 | |
opendevreview | Merged openstack/releases master: Release cyborg-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876605 | 13:33 |
opendevreview | Merged openstack/releases master: Release ironic-inspector 11.4.0 and branch stable/2023.1 https://review.opendev.org/c/openstack/releases/+/876569 | 13:33 |
opendevreview | Merged openstack/releases master: release IPA 9.4.0 and branch stable/2023.1 https://review.opendev.org/c/openstack/releases/+/876572 | 13:42 |
opendevreview | Merged openstack/releases master: Release bifrost 16.1.0 and branch stable/2023.1 https://review.opendev.org/c/openstack/releases/+/876567 | 13:42 |
opendevreview | Merged openstack/releases master: Update Nova release liaisons https://review.opendev.org/c/openstack/releases/+/876758 | 13:55 |
*** amoralej|lunch is now known as amoralej | 14:12 | |
opendevreview | Merged openstack/releases master: Release ironic-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876611 | 14:22 |
elodilles | iurygregory: thanks, approved ^^^ | 14:26 |
iurygregory | yw | 14:26 |
*** sfinucan is now known as stephenfin | 14:44 | |
opendevreview | Merged openstack/releases master: Release venus-tempest-plugin for 2023.1 Antelope https://review.opendev.org/c/openstack/releases/+/876621 | 14:47 |
bauzas | elodilles: hberaud: any reason why *not* including nova's relnotes https://review.opendev.org/c/openstack/releases/+/876952 ? | 14:55 |
elodilles | bauzas: there is: nova reno patch was not merge at the time when the patch was generated :) | 14:56 |
hberaud | bauzas: hm maybe because the script that generated this patch ignored nova | 14:56 |
bauzas | oh nevermind, that's because when you uploaded it, the nova patch wasn't merged yet :) | 14:56 |
bauzas | yeah that | 14:56 |
elodilles | we'll need to run the script later, when this 'reno links' patch has merged | 14:58 |
elodilles | i mean: we need to propose another patch when the 1st one has merged | 14:58 |
bauzas | ok, then let's switch my vote to +1 with a comment saying that I'm eagerly waiting for a FUP | 14:59 |
bauzas | elodilles: wfy ? | 14:59 |
elodilles | bauzas: perfectly wfm :) | 15:00 |
bauzas | done | 15:01 |
elodilles | thx o/ | 15:01 |
opendevreview | Elod Illes proposed openstack/releases master: Add Axel as release liaison https://review.opendev.org/c/openstack/releases/+/876990 | 15:44 |
*** amoralej is now known as amoralej|off | 16:04 | |
hberaud | elodilles, ttx I don't think we will get response from swift https://review.opendev.org/c/openstack/releases/+/876579 | 16:14 |
elodilles | is Tim active in swift's IRC channel? | 16:16 |
elodilles | is he still the PTL? :S | 16:16 |
hberaud | good questions | 16:17 |
hberaud | let me try to ping him on irc | 16:18 |
elodilles | I don't see swift here: https://governance.openstack.org/election/#bobcat-ptl-candidates | 16:18 |
elodilles | so probably he did not candidate himself as Swift PTL for the next cycle | 16:19 |
elodilles | at least i see some merged patches in swift, so it's not inactive | 16:20 |
elodilles | hberaud: ack. if we don't get an answer from him then tomorrow we can force merge swift release patch | 16:21 |
hberaud | I found him on IRC, I left him a message | 16:21 |
elodilles | hberaud: ack, see my prev line ^^^ | 16:21 |
hberaud | ack | 16:22 |
hberaud | I'll check if the used sha is the last one | 16:22 |
opendevreview | Hervé Beraud proposed openstack/releases master: Release swift 2.32.0 and cut branch stable/2023.1 https://review.opendev.org/c/openstack/releases/+/876579 | 16:23 |
hberaud | updated | 16:23 |
hberaud | and yeah he doesn't seems to be PTL anymore | 16:24 |
elodilles | did the swift team changed to DPTL model? | 16:32 |
elodilles | anyway, I've reviewed the patch, LGTM, so +1'd it for now. thanks for the update hberaud | 16:32 |
opendevreview | Merged openstack/releases master: release ironic 21.4.0 and branch stable/2023.1 https://review.opendev.org/c/openstack/releases/+/876573 | 16:37 |
opendevreview | Merged openstack/releases master: Release python-openstackclient 6.2.0 and create branch stable/2023.1 https://review.opendev.org/c/openstack/releases/+/876575 | 16:37 |
hberaud | ok thanks | 16:48 |
hberaud | np | 16:48 |
ttx | elodilles: hberaud Tim is still active, he just requested space for PTG | 16:52 |
hberaud | ack | 16:53 |
ttx | but I suspect he does not pay daily attention | 16:53 |
hberaud | ttx, elodilles he just replied to me on IRC, he will have a look | 17:00 |
elodilles | ack, sounds good, thx | 17:06 |
opendevreview | Merged openstack/releases master: Allow to pass release ID to generate correct releasenotes links https://review.opendev.org/c/openstack/releases/+/876951 | 17:21 |
opendevreview | Gregory Thiemonge proposed openstack/releases master: Release Octavia RC2 for Antelope https://review.opendev.org/c/openstack/releases/+/877008 | 17:24 |
-opendevstatus- NOTICE: Yesterday's change to Gerrit configs to use submit-requirements had a boolean logic bug. This has now been corrected and any changes that did not merge as a result can be rechecked. We have reenqueued the changes we identified as being affected. | 17:24 | |
opendevreview | Merged openstack/releases master: Release virtualbmc 3.0.1 https://review.opendev.org/c/openstack/releases/+/876399 | 17:33 |
gthiemonge | Hi Folks, I proposed a patch for an RC2 for Octavia but it fails in the CI, it seems to search for a stable/antelope branch: | 20:00 |
gthiemonge | DEBUG: did not find remotes/origin/stable/antelope in branches containing 4486d2416f600df15df1b5c98a676f898e2b1d42: ['remotes/origin/stable/2023.1'] DEBUG: cwd = /home/zuul/src/opendev.org/openstack/releases/.tox/validate/tmp/releases-dljfyhng/openstack/octavia | 20:00 |
gthiemonge | https://zuul.opendev.org/t/openstack/build/f64cdb0dabe3437d82e6e2757d595ce3 | 20:00 |
opendevreview | Tim Burke proposed openstack/releases master: Release swift 2.32.0 and cut branch stable/2023.1 https://review.opendev.org/c/openstack/releases/+/876579 | 21:34 |
* johnsom Yeah, not sure how this didn't fail before.... | 22:29 | |
johnsom | https://github.com/openstack/releases/blob/master/openstack_releases/deliverable.py#L440 | 22:29 |
johnsom | https://zuul.opendev.org/t/openstack/build/f64cdb0dabe3437d82e6e2757d595ce3/log/job-output.txt#5728 | 22:31 |
johnsom | The deliverable series is based of the file path, so pulls in antelope as the deliverable series | 22:33 |
johnsom | So this call is broken: https://github.com/openstack/releases/blob/master/openstack_releases/cmds/validate.py#L1412 | 22:34 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!