| opendevreview | Merged openstack/releases master: Release heat RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912395 | 06:09 |
|---|---|---|
| opendevreview | Merged openstack/releases master: Release heat-agents RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912393 | 06:09 |
| opendevreview | Merged openstack/releases master: Release heat-dashboard RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912394 | 06:12 |
| opendevreview | Merged openstack/releases master: Release neutron-fwaas RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912414 | 06:13 |
| opendevreview | Merged openstack/releases master: Release neutron-fwaas-dashboard RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912412 | 06:48 |
| frickler | ok, dealt with rc1 patches as far as possible, did a couple more pings for feedback | 07:05 |
| opendevreview | Merged openstack/releases master: Release neutron-vpnaas-dashboard RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912415 | 07:09 |
| opendevreview | Merged openstack/releases master: Release ovn-bgp-agent RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912425 | 07:09 |
| opendevreview | Merged openstack/releases master: Release tap-as-a-service RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912437 | 07:09 |
| opendevreview | Merged openstack/releases master: Release ovn-octavia-provider RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912426 | 07:09 |
| opendevreview | Merged openstack/releases master: Release manila-ui RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912401 | 07:09 |
| opendevreview | Merged openstack/releases master: Release masakari-dashboard RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912403 | 07:09 |
| opendevreview | Merged openstack/releases master: Release masakari RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912405 | 07:09 |
| opendevreview | Merged openstack/releases master: Release masakari-monitors RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912404 | 07:09 |
| opendevreview | Merged openstack/releases master: Release cyborg RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912389 | 07:09 |
| opendevreview | Merged openstack/releases master: Release blazar-dashboard RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912382 | 07:09 |
| opendevreview | Merged openstack/releases master: Release blazar-nova RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912383 | 07:09 |
| opendevreview | Merged openstack/releases master: Release zaqar RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912449 | 07:09 |
| opendevreview | Merged openstack/releases master: Release zaqar-ui RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912448 | 07:09 |
| tkajinam | a few projects(trove, vitrage and watcher as far as I'm aware of but there can be more) currently have broken CI because these need to adapt to new oslo.db which removed the old interface to enable autocommit. I was hoping to get some working paths for them but learned that I have to consult stephenfin once he comes back from PTO next week. | 07:20 |
| tkajinam | so we may need some activities next week for a few projects to get the required fixes for 2024.1 rc2 (or postpone rc1 until these issues are sorted) | 07:21 |
| tkajinam | just heads up. I guess you see such CI failures in post-release patches. | 07:21 |
| frickler | tkajinam: is that only CI failures or are the services actually broken in the current state? I'd still be leaning towards doing rc1 now anyway | 07:23 |
| tkajinam | frickler, services are actually broken... these services can't write into dbs because sessions do not properly begin/end | 07:24 |
| * tkajinam is leaving for local meetup but will come back online at (probably late) night | 07:26 | |
| opendevreview | Vishal Manchanda proposed openstack/releases master: Release Horizon 23.5.0 as the final tag for 2024.1 https://review.opendev.org/c/openstack/releases/+/912909 | 10:02 |
| elodilles | so then we have the usual churn about "library released a broken change at late point in the cycle" :( nevertheless i think we have merge RC1 releases where possible and hopefully teams will fix the broken services until final RC | 10:27 |
| elodilles | * have to merge | 10:28 |
| frickler | elodilles: when is branching for cycle-with-intermediary supposed to happen? I can't seem to find a reference in the release schedule | 11:08 |
| opendevreview | Merged openstack/releases master: [ironic] Transition EM branches to Unmaintained https://review.opendev.org/c/openstack/releases/+/910398 | 11:10 |
| elodilles | they can branch at their release time (lib freeze, non-client lib freeze a.k.a. milestone-3) but not necessary, so we propose the branching patches in R-4 (1 week after milestone-3) | 11:10 |
| elodilles | (for those that haven't branched already at that time) | 11:11 |
| dtantsur | FYI folks, this looks quite wrong to me https://review.opendev.org/c/openstack/networking-generic-switch/+/912932/1/releasenotes/source/index.rst | 11:20 |
| dtantsur | same with https://review.opendev.org/c/openstack/networking-generic-switch/+/912930 and https://review.opendev.org/c/openstack/networking-generic-switch/+/912931 | 11:21 |
| dtantsur | cc JayF | 11:21 |
| elodilles | dtantsur: that is really quite wrong /o\ | 11:21 |
| dtantsur | ironicclient also affected https://review.opendev.org/c/openstack/python-ironicclient/+/912937 | 11:22 |
| elodilles | it indicates that somehow stable/victoria branch of networking-generic-switch is reopened... :/ | 11:22 |
| dtantsur | yep, I wonder how and why | 11:22 |
| elodilles | same | 11:22 |
| dtantsur | most of our projects are okay, it seems | 11:22 |
| dtantsur | For the (correctly) unmaintained branches, I see a lot of patches to update reno on master, but seemingly no patches to update .gitreview on branches themselves? | 11:29 |
| elodilles | i guess the post-release jobs are still running | 11:30 |
| dtantsur | on the bright side, my review stats will be awesome this month :D | 11:31 |
| elodilles | i expect some more false patches for ironiclient & networking-generic-switch on wallaby and xena... :/ (at least i don't see any reason why those victora patches should appear suddenly as the victoria-eol tags were on place... :/) | 11:32 |
| dtantsur | yeah | 11:32 |
| dtantsur | I wonder if all these reno patches are actually going to merge. Or if they get blocked on other branches no longer existing. | 11:32 |
| elodilles | (wallaby patches: https://review.opendev.org/q/topic:reno-wallaby ) | 11:34 |
| elodilles | dtantsur: to be clear, reno-eom-* patches are valid patches | 11:35 |
| ttx | elodilles: we had a request yesterday to speed up review of https://review.opendev.org/c/openstack/releases/+/911950 | 11:36 |
| dtantsur | yeah, I'm curious how reno behaves in the presence of missing branches | 11:36 |
| dtantsur | the jobs do seem to pass, so maybe my concerns are false | 11:36 |
| elodilles | ttx: +2+W'd | 11:37 |
| elodilles | dtantsur: if these patches weren't merged when the stable/* patches got deleted, then the reno jobs would fail | 11:38 |
| ttx | elodilles: thanks for them! | 11:38 |
| dtantsur | elodilles: so, the branches were not deleted yet? | 11:38 |
| elodilles | dtantsur: nope, the branches are usually deleted in some days/a week after the branching, as we do that by running a separate script | 11:39 |
| elodilles | ttx: np, wanted to do it yesterday, but probably got distracted :/ | 11:40 |
| dtantsur | ah, got it. please make sure we land all these patches before it happens | 11:41 |
| elodilles | please merge them as soon as possible o:) | 11:43 |
| frickler | hmm, https://review.opendev.org/c/openstack/python-ironicclient/+/912966 tries to add wallaby a second time, that looks wrong, how did that happen? | 11:45 |
| dtantsur | elodilles: I've approved all valid ones on Ironic projects | 11:45 |
| frickler | ooh, https://review.opendev.org/c/openstack/releases/+/910398/4/deliverables/victoria/python-ironicclient.yaml | 11:47 |
| frickler | elodilles: so that's why we had marked those branches as closed series :-/ | 11:48 |
| frickler | it seems we need to define two different types of closed. or have better checks against re-opening branches otherwise. (cf. https://review.opendev.org/c/openstack/releases/+/911579) | 11:50 |
| opendevreview | Merged openstack/releases master: Add feature/mpu branch for Swift https://review.opendev.org/c/openstack/releases/+/911950 | 11:50 |
| opendevreview | Merged openstack/releases master: Release manila RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912402 | 11:50 |
| opendevreview | Merged openstack/releases master: Release vitrage RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912445 | 11:50 |
| opendevreview | Vishal Manchanda proposed openstack/releases master: Release Horizon 24.0.0 as the final tag for 2024.1 https://review.opendev.org/c/openstack/releases/+/912909 | 11:51 |
| opendevreview | Merged openstack/releases master: Release vitrage-dashboard RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912444 | 11:52 |
| opendevreview | Merged openstack/releases master: 2024.1 Caracal branch cut for devstack plugins https://review.opendev.org/c/openstack/releases/+/912363 | 11:52 |
| opendevreview | Merged openstack/releases master: Release networking-baremetal 6.3.0 for caracal https://review.opendev.org/c/openstack/releases/+/912089 | 11:54 |
| opendevreview | Merged openstack/releases master: Release OpenStack-Ansible 2023.1 https://review.opendev.org/c/openstack/releases/+/912581 | 12:01 |
| opendevreview | Merged openstack/releases master: Release OpenStack-Ansible Zed https://review.opendev.org/c/openstack/releases/+/912579 | 12:13 |
| opendevreview | Merged openstack/releases master: Release cinder for stable/zed https://review.opendev.org/c/openstack/releases/+/906838 | 12:13 |
| opendevreview | Merged openstack/releases master: Release trove for stable/zed https://review.opendev.org/c/openstack/releases/+/906879 | 12:13 |
| opendevreview | Merged openstack/releases master: [OSA-roles] Transition EM branches to Unmaintained https://review.opendev.org/c/openstack/releases/+/911509 | 12:13 |
| opendevreview | Merged openstack/releases master: Set Xena, Wallaby and Victoria state as Unmaintained https://review.opendev.org/c/openstack/releases/+/911860 | 12:13 |
| elodilles | frickler: i don't think this happened because of we removed victoria from CLOSED_SERIES. rather there is some bug in our script ( https://opendev.org/openstack/project-config/src/branch/master/roles/copy-release-tools-scripts/files/release-tools/make_branch.sh ) | 12:22 |
| elodilles | it seems that the bash variable magic didn't work as intended :/ here we should have 'grep wallaby-eol': https://zuul.opendev.org/t/openstack/build/e657109c5d744524ae395daf51d7f327/log/job-output.txt#17057 | 12:51 |
| elodilles | (this is the corresponding line in the script: https://opendev.org/openstack/project-config/src/branch/master/roles/copy-release-tools-scripts/files/release-tools/make_branch.sh#L60 ) | 12:51 |
| frickler | I'm just looking at the same log, my local test does give the right expansion though. the good news is only two repos were actually affected, n-g-s and pythonclient | 12:59 |
| elodilles | yepp, same here. locally works, and saw only those two (but those two on victoria + wallaby) | 13:02 |
| frickler | hmm, if I test in bash interactively, it works, if I put the expansion into a script, it doesn't. will be away for a bit, continue later | 13:44 |
| frickler | maybe you can run the review abandon script anyway so we can close out those branches later | 13:44 |
| elodilles | frickler: ACK, i'm running it | 13:51 |
| elodilles | frickler: i have to test this, but this should work i think: https://review.opendev.org/c/openstack/project-config/+/913189 | 14:03 |
| opendevreview | Elod Illes proposed openstack/releases master: Release Swift for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912371 | 14:51 |
| frickler | elodilles: being simple instead of fancy, that's a good idea | 14:51 |
| fungi | i raised related concerns when i approved the earlier change. if we need to be doing complex pattern manipulation, better to rewrite in another language where it can be more explicit and testable | 14:57 |
| elodilles | yepp, that makes sense | 14:59 |
| fungi | too much bash magic quickly becomes unreviewable for me, especially in a script where we have no automated tests to confirm correctness | 15:01 |
| fungi | i end up resorting to rereading the bash manpage, which is very unwieldy | 15:02 |
| opendevreview | Gregory Thiemonge proposed openstack/releases master: Release octavia RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912424 | 15:02 |
| frickler | yeah, the nasty part about this one is that it worked fine in interactive testing, but not within a script. but I agree that this might be a bit over the edge for bash. | 15:20 |
| frickler | elodilles: did the abandon script work or is it still running? I did a test run of the eol script and it finds the expected branches but still complains about open reviews. I could also just abandon the bot patches manually if needed | 15:23 |
| elodilles | frickler: victoria part finished, now running against wallaby | 15:26 |
| elodilles | frickler: it's office day to me and both the script is slow and i have also some downstream meeting o:) | 15:29 |
| elodilles | so feel free to run the abandoning script against xena | 15:30 |
| frickler | elodilles: ok, will do that now | 15:32 |
| opendevreview | Takashi Kajinami proposed openstack/releases master: Create new yaql release for Caracal https://review.opendev.org/c/openstack/releases/+/913230 | 15:33 |
| opendevreview | Merged openstack/releases master: Release networking-bagpipe RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912408 | 15:34 |
| opendevreview | Elod Illes proposed openstack/releases master: Release aodh RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912380 | 16:25 |
| fungi | release managers: expedited review requested for https://review.opendev.org/c/openstack/releases/+/913230 since we've got a security note brewing about it | 16:30 |
| frickler | elodilles: ttx: hberaud: ^^ (and how about standardizing on "release-team" for highlights?) | 16:43 |
| elodilles | fungi frickler : +2+W'd | 16:46 |
| elodilles | frickler: "release-team" looks good to me | 16:46 |
| elodilles | added it to my highlight list | 16:48 |
| fungi | thanks! i'll try to remember to use that when i want to bug you all ;) | 16:48 |
| elodilles | :] | 16:49 |
| opendevreview | Merged openstack/releases master: Create new yaql release for Caracal https://review.opendev.org/c/openstack/releases/+/913230 | 16:56 |
| frickler | we should add that to the topic once we all agreed on it | 17:28 |
| ttx | added | 17:32 |
| frickler | list-em-series stopped generating any output after https://review.opendev.org/c/openstack/releases/+/911860, which is correct, but makes the list_eol script a no-op. I'm working on a patch now to move to check eom status instead | 17:51 |
| opendevreview | Merged openstack/releases master: New manila-tempest-plugin release https://review.opendev.org/c/openstack/releases/+/912810 | 18:01 |
| opendevreview | Merged openstack/releases master: Release glance RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912392 | 19:44 |
| opendevreview | Merged openstack/releases master: Release blazar RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912384 | 19:49 |
| opendevreview | Merged openstack/releases master: Release octavia-dashboard RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912423 | 19:51 |
| opendevreview | Merged openstack/releases master: Release octavia RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912424 | 19:51 |
| opendevreview | Brian Haley proposed openstack/releases master: Release neutron RC1 for 2024.1 Caracal https://review.opendev.org/c/openstack/releases/+/912421 | 20:22 |
| opendevreview | Jeremy Stanley proposed openstack/releases master: Publish the 2024.2/Dalmatian key for future use https://review.opendev.org/c/openstack/releases/+/913273 | 21:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!