| *** bauzas_ is now known as bauzas | 09:30 | |
| opendevreview | Gregory Thiemonge proposed openstack/releases master: Release octavia RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928553 | 12:33 |
|---|---|---|
| opendevreview | Merged openstack/releases master: 2024.2 Dalmatian branch cut for devstack plugins https://review.opendev.org/c/openstack/releases/+/928504 | 12:58 |
| opendevreview | Merged openstack/releases master: Release barbican RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928514 | 13:03 |
| opendevreview | Merged openstack/releases master: Release ansible-role-atos-hsm RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928510 | 13:03 |
| opendevreview | Merged openstack/releases master: Release ansible-role-lunasa-hsm RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928511 | 13:06 |
| opendevreview | Merged openstack/releases master: Release venus RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928567 | 13:06 |
| opendevreview | Merged openstack/releases master: Release octavia-dashboard RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928552 | 13:06 |
| opendevreview | Merged openstack/releases master: Release zaqar-ui RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928572 | 13:06 |
| opendevreview | Merged openstack/releases master: Release zaqar RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928573 | 13:06 |
| opendevreview | Merged openstack/releases master: Nova 2024.2 Dalmatian cycle highlights https://review.opendev.org/c/openstack/releases/+/928777 | 13:06 |
| opendevreview | Merged openstack/releases master: [Telemetry] Create 2024.2 branch for client and non-client libs https://review.opendev.org/c/openstack/releases/+/928423 | 13:07 |
| opendevreview | Merged openstack/releases master: [adjutant] Create 2024.2 branch for client and non-client libs https://review.opendev.org/c/openstack/releases/+/928426 | 13:07 |
| opendevreview | Merged openstack/releases master: [watcher] Create 2024.2 branch for client and non-client libs https://review.opendev.org/c/openstack/releases/+/928424 | 13:07 |
| opendevreview | Merged openstack/releases master: Release Swift for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928505 | 13:08 |
| opendevreview | Merged openstack/releases master: Release networking-bagpipe RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928542 | 13:08 |
| opendevreview | Merged openstack/releases master: Release networking-sfc RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928544 | 13:11 |
| opendevreview | Merged openstack/releases master: Release neutron-vpnaas-dashboard RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928548 | 13:11 |
| opendevreview | Merged openstack/releases master: Release neutron-dynamic-routing RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928545 | 13:11 |
| opendevreview | Merged openstack/releases master: Release neutron-fwaas RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928547 | 13:14 |
| opendevreview | Merged openstack/releases master: Release tap-as-a-service RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928563 | 13:14 |
| opendevreview | Merged openstack/releases master: Release ovn-bgp-agent RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928554 | 13:14 |
| opendevreview | Merged openstack/releases master: Add Neutron cycle highlights (Dalmatian release) https://review.opendev.org/c/openstack/releases/+/928289 | 13:21 |
| opendevreview | Merged openstack/releases master: Release neutron-fwaas-dashboard RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928546 | 13:21 |
| opendevreview | Merged openstack/releases master: Release neutron-vpnaas RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928549 | 13:22 |
| opendevreview | Merged openstack/releases master: Release networking-bgpvpn RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928543 | 13:22 |
| opendevreview | Merged openstack/releases master: Release glance RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928526 | 13:22 |
| opendevreview | Merged openstack/releases master: Tag monthly kolla stable releases https://review.opendev.org/c/openstack/releases/+/928908 | 13:22 |
| opendevreview | Merged openstack/releases master: Release barbican for stable/2023.1 https://review.opendev.org/c/openstack/releases/+/925334 | 13:22 |
| frickler | this is weird, why did zuul reset the ptl-approved flag and cause a gate failure? https://review.opendev.org/c/openstack/releases/+/928566 | 13:37 |
| frickler | clarkb: fungi: ^^ any idea before I start digging zuul logs? | 13:46 |
| opendevreview | Merged openstack/releases master: Release octavia RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928553 | 13:55 |
| ttx | woah, yeah that's weird | 14:02 |
| ttx | We also got a failed release-announce job for swift at https://zuul.opendev.org/t/openstack/build/f6fa299e49b14cc6ba5bccc148ca2d22 | 14:03 |
| opendevreview | Merged openstack/releases master: Release keystone RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928530 | 14:04 |
| ttx | (not even sure why the release announcement would need to build pyEClib) | 14:04 |
| ttx | Re: venus-dashboard I'll reapprove it | 14:06 |
| opendevreview | Merged openstack/releases master: Release ovn-octavia-provider RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928555 | 14:07 |
| opendevreview | Merged openstack/releases master: Release storlets RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928560 | 14:10 |
| opendevreview | Merged openstack/releases master: Release heat RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928529 | 14:13 |
| opendevreview | Merged openstack/releases master: Release heat-agents RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928527 | 14:15 |
| opendevreview | Merged openstack/releases master: Release heat-dashboard RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928528 | 14:17 |
| frickler | the swift failure looks like another broken-on-noble issue, maybe timburke can check this | 14:18 |
| ttx | yeah, probably missing liberasurecode-dev | 14:18 |
| ttx | although.. the bindep has it | 14:19 |
| opendevreview | Pierre Riteau proposed openstack/releases master: Release blazar-dashboard RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928515 | 14:19 |
| ttx | https://opendev.org/openstack/swift/src/branch/master/bindep.txt | 14:20 |
| frickler | ttx: yeah, the issue seems to be that that job installs bindep from the releases repo, not from the deliverable https://zuul.opendev.org/t/openstack/build/f6fa299e49b14cc6ba5bccc148ca2d22/console#1/0/14/ubuntu-noble | 14:21 |
| elodilles | so we should pin announce-release job to ubuntu-jammy as well, right? | 14:22 |
| frickler | I wonder why this doesn't fail more often then. likely pampered over by our wheel cache | 14:22 |
| elodilles | probably it depends how actively bindep.txt is updated for a project | 14:24 |
| ttx | I actually question why the release-announce job needs to install the project requirements | 14:25 |
| elodilles | or some projects don't have so detailed bindep list in their bindep.txt :) | 14:25 |
| frickler | elodilles: well as ttx mentioned, bindep for swift is fine, but the job doesn't use it | 14:25 |
| frickler | I'd rather see us fix that bug instead of doing more distro pinning | 14:26 |
| elodilles | yeah we discussed this in the past if i remember well :) | 14:26 |
| ttx | Is it just to be able to run python -c 'import importlib.metadata; print(importlib.metadata.metadata('''barbican''')['''Name'''])' | 14:27 |
| ttx | in order to compute the full name in the "barbican 16.0.1: OpenStack Secure Key Management" line | 14:27 |
| elodilles | there are some comments from former relmgt members that it was just a safety net as 'python setup.py --name' etc might want to build the package | 14:27 |
| ttx | Reading announce.sh, it appears we are running "python -m pip install ." only to be able to import the main module to grab project_name and description | 14:35 |
| ttx | and then project_name is overwritten , so it's just to grab description | 14:35 |
| ttx | https://opendev.org/openstack/releases/src/branch/master/tools/announce.sh#L177 | 14:36 |
| opendevreview | Merged openstack/releases master: Release venus-dashboard RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928566 | 14:36 |
| elodilles | yepp | 14:39 |
| ttx | If my read is correct, then it's a lot of added complexity (and computing power) spent to compute a description that nobody reads | 14:43 |
| clarkb | frickler: resetting votes is done in pipeline configs | 14:48 |
| clarkb | frickler: its likely the pipeline that checks ptl approval resets things? | 14:48 |
| frickler | clarkb: but why? it also only happened on that single patch, not on the 20 others that hot approved and merged in parallel | 14:56 |
| clarkb | https://opendev.org/openstack/project-config/src/branch/master/zuul.d/pipelines.yaml#L284-L287 | 15:01 |
| clarkb | the job failed but you dont get a comment | 15:02 |
| clarkb | maybe remove comment: false so it is more obvious? | 15:02 |
| opendevreview | Tatiana Ovchinnikova proposed openstack/releases master: Release Horizon 25.1.0 for Dalmatian https://review.opendev.org/c/openstack/releases/+/928847 | 15:03 |
| opendevreview | Thierry Carrez proposed openstack/releases master: Remove description to simplify release-announce https://review.opendev.org/c/openstack/releases/+/929126 | 15:03 |
| frickler | clarkb: the question is actually why it failed in that case when it shouldn't have. seems it was a post_failure with no logs saved https://zuul.opendev.org/t/openstack/build/fd72e4a05a724e38b15a5469807d6a70 so I guess I'll need to check zuul logs after all | 15:04 |
| ttx | It's a bit late in the release schedule to make that change ^ but I figured I should throw it out there before I forget | 15:04 |
| ttx | Not sure how to best test it... | 15:05 |
| frickler | ttx: elodilles: I'd be fine with just merging that and watching the next announcements. the code change lgtm | 15:17 |
| frickler | also if one of you could check https://review.opendev.org/c/openstack/releases/+/928862 that'd be nice | 15:18 |
| frickler | clarkb: ok, so it was log upload failing, not much we can do, just bad timing for it to happen after the +W | 15:27 |
| elodilles | frickler: i've +2+W'd the ironic release patch | 15:31 |
| frickler | thx | 15:35 |
| frickler | should horizon also create a stable branch in https://review.opendev.org/c/openstack/releases/+/928847 or should that happen later? | 15:36 |
| opendevreview | Merged openstack/releases master: [ironic] Releases for OSSA-2024-003 https://review.opendev.org/c/openstack/releases/+/928862 | 15:46 |
| opendevreview | Jon Bernard proposed openstack/releases master: Add 2024.2 (Dalmatian) cycle highlights for cinder https://review.opendev.org/c/openstack/releases/+/929154 | 16:28 |
| frickler | ttx: why were you hesitating on https://review.opendev.org/c/openstack/releases/+/927030 ? IMO it should be fine to publish the new key well before it is being put into service | 16:39 |
| opendevreview | Carlos Eduardo proposed openstack/releases master: Release manila RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928536 | 18:21 |
| *** bauzas_ is now known as bauzas | 19:48 | |
| opendevreview | Brian Haley proposed openstack/releases master: Release neutron RC1 for 2024.2 Dalmatian https://review.opendev.org/c/openstack/releases/+/928550 | 23:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!