| *** amoralej|off is now known as amoralej | 06:12 | |
| opendevreview | Dmitriy Rabotyagov proposed openstack/releases master: Release 2023.1 RC2 for OpenStack-Ansible https://review.opendev.org/c/openstack/releases/+/884984 | 08:45 |
|---|---|---|
| opendevreview | Dmitriy Rabotyagov proposed openstack/releases master: Stable release of OpenStack-Ansible for 2023.1 https://review.opendev.org/c/openstack/releases/+/884985 | 08:46 |
| noonedeadpunk | Would be awesome to land this until deadline ^ :) | 08:50 |
| mnasiadka | noonedeadpunk: seems last minute releases in deployment projects are not only Kolla thing ;) | 08:51 |
| noonedeadpunk | nah, for sure it's not :( each release we plan to do that early and that never happens | 08:52 |
| mnasiadka | with that dependencies hell it's not easy ;-) | 08:54 |
| noonedeadpunk | Yeah, and then there's some OS that making release somewhere in April, or OSSA, or smth else... And PTG where there're some "low-hanging-fruits" are raised that could be covered "easily" in the upcoming relse... | 08:57 |
| noonedeadpunk | ugh | 08:57 |
| opendevreview | Rodolfo Alonso proposed openstack/releases master: [neutron] New Yoga releases for Neutron projects https://review.opendev.org/c/openstack/releases/+/885030 | 10:45 |
| opendevreview | Rodolfo Alonso proposed openstack/releases master: [neutron] New Zed releases for Neutron projects https://review.opendev.org/c/openstack/releases/+/885031 | 10:56 |
| opendevreview | Rodolfo Alonso proposed openstack/releases master: [neutron] New 2023.1 (Antelope) releases for Neutron projects https://review.opendev.org/c/openstack/releases/+/885033 | 11:18 |
| *** amoralej is now known as amoralej|lunch | 12:09 | |
| opendevreview | Merged openstack/releases master: New stable releases for celiometer yoga and zed https://review.opendev.org/c/openstack/releases/+/883843 | 12:12 |
| opendevreview | Merged openstack/releases master: Set Xena status to Extended Maintenance https://review.opendev.org/c/openstack/releases/+/881254 | 12:12 |
| opendevreview | Rodolfo Alonso proposed openstack/releases master: [neutron] New Zed releases for Neutron projects https://review.opendev.org/c/openstack/releases/+/885031 | 12:50 |
| opendevreview | Rodolfo Alonso proposed openstack/releases master: [neutron] New Yoga releases for Neutron projects https://review.opendev.org/c/openstack/releases/+/885030 | 12:51 |
| *** amoralej|lunch is now known as amoralej | 12:59 | |
| elodilles | hberaud ttx : note that i have +W'd the tools/announce.sh fix ( https://review.opendev.org/c/openstack/releases/+/879273 ) so please take an extra look for the announce mailing list that the announce mails arrive in the needed format :) | 13:02 |
| hberaud | ack | 13:02 |
| opendevreview | Mark Goddard proposed openstack/releases master: kolla/kolla-ansible/kayobe: 2023.1 rc2 https://review.opendev.org/c/openstack/releases/+/884937 | 13:26 |
| opendevreview | Merged openstack/releases master: Release 2023.1 RC2 for OpenStack-Ansible https://review.opendev.org/c/openstack/releases/+/884984 | 14:40 |
| opendevreview | Merged openstack/releases master: Release ironic-inspector 11.5 with bugfix branch https://review.opendev.org/c/openstack/releases/+/884900 | 14:44 |
| opendevreview | Merged openstack/releases master: Fix tools/announce by using importlib.metadata instead of setuptools https://review.opendev.org/c/openstack/releases/+/879273 | 14:46 |
| opendevreview | Merged openstack/releases master: Release ironic-python-agent 9.5.0 with bugfix branch https://review.opendev.org/c/openstack/releases/+/884901 | 14:50 |
| opendevreview | Merged openstack/releases master: release oslo.messaging 14.3.1 https://review.opendev.org/c/openstack/releases/+/884841 | 14:55 |
| opendevreview | Merged openstack/releases master: release oslo's zed deliverables https://review.opendev.org/c/openstack/releases/+/884847 | 15:11 |
| opendevreview | Elod Illes proposed openstack/releases master: Fix package description in announce mail https://review.opendev.org/c/openstack/releases/+/885066 | 15:27 |
| elodilles | hberaud ttx : ^^^ the announce mail is a bit garbled, but see my fix for it | 15:28 |
| elodilles | (an example mail: https://lists.openstack.org/pipermail/release-announce/2023-June/014404.html ) | 15:29 |
| hberaud | Summary LGTM | 15:30 |
| hberaud | thanks elodilles | 15:30 |
| elodilles | np :) | 15:30 |
| mnasiadka | elodilles: can you +w https://review.opendev.org/c/openstack/releases/+/884937? it has 2x+2 but no +w ;-) | 15:47 |
| ttx | mnasiadka: done | 15:57 |
| opendevreview | Merged openstack/releases master: kolla/kolla-ansible/kayobe: 2023.1 rc2 https://review.opendev.org/c/openstack/releases/+/884937 | 16:12 |
| *** amoralej is now known as amoralej|off | 16:23 | |
| JayF | I have a weird problem. A few months ago I retired several bugfix branches with these commands: https://etherpad.opendev.org/p/IronicBugfixBranchCleanupRFR -- now, many of those branches have been re-created | 17:21 |
| JayF | I need to EOL more brnaches, but I'd like to solve whatever caused the branches to get recreated first, so I can do one cleanup for it all | 17:21 |
| JayF | note the presence of both https://opendev.org/openstack/ironic/src/branch/bugfix/18.0 and https://opendev.org/openstack/ironic/src/tag/bugfix-18.0-eol for instance | 17:22 |
| frickler | JayF: seems at least here there is an open patch https://review.opendev.org/c/openstack/ironic/+/881942 , it may be possible that the release bot also recreated the branch? | 19:12 |
| frickler | and then the branches are still listed here https://opendev.org/openstack/releases/src/branch/master/deliverables/xena/ironic.yaml#L38-L44 | 19:16 |
| JayF | so do the branches need to be removed from the yaml file upon retirement? | 19:16 |
| JayF | if so that's not really ideal, but none of this is ideal :/ | 19:16 |
| frickler | and that file was updates with https://review.opendev.org/c/openstack/releases/+/870589/3 | 19:17 |
| frickler | I think they should be removed, yes, but maybe wait for confirmation from "real" release people | 19:18 |
| JayF | so we need to either: 1) patch release tooling to recognize an extra key there (like retired:true or active:false) to avoid recreating it or 2) just remove the branches from the yaml file | 19:18 |
| JayF | (1) would be preferable so we don't erase history | 19:18 |
| frickler | which history exactly do you want to keep, the branch/tag mapping? maybe keep it just commented out, then? | 19:21 |
| JayF | the aggressive linting might not like that; but that'd be a good option too | 19:22 |
| elodilles | JayF: I had a fix (probably) for the same issue: https://review.opendev.org/c/openstack/project-config/+/881731 | 19:25 |
| elodilles | JayF: do you see recreated branch anywhere after this patch has merged? | 19:25 |
| elodilles | JayF: it merged May 8th | 19:26 |
| JayF | that looks like a good fix, but I don't have logs (or haven't unearthed them, if I have access to them) for when they were recreated | 19:26 |
| JayF | but I'm game to re-execute the branch deletions and monitor if they come back | 19:26 |
| elodilles | it's also a good sign of a branch recreation when you see 'create-<branch>' topic patches with 'Update .gitreview for <branch>' title | 19:27 |
| elodilles | but yes, anyway, please delete the branches again then and let us know if any of them gets recreated | 19:31 |
| JayF | yeah, when frickler linked that I realized it was a good way to get there | 19:31 |
| frickler | also remember to abandon those new open patches | 19:32 |
| frickler | the one I quoted above is from May 3rd, so just before the fix merged | 19:32 |
| JayF | 10-4 I'll follow up on this after some meetings this afternoon | 19:33 |
| elodilles | frickler: abandoning those patches is needed yes, but branches cannot be deleted until patches are open to that branch anyway... | 19:34 |
| elodilles | btw, I quickly tried to look for 'Update .gitreview ...' patches and i don't see other re-opened ones | 19:35 |
| elodilles | (s/until/as long as/) | 19:36 |
| opendevreview | Michal Nasiadka proposed openstack/releases master: kolla/kolla-ansible/kayobe: final Antelope release https://review.opendev.org/c/openstack/releases/+/885093 | 19:38 |
| mnasiadka | rc2 merged, now final and we're done... | 19:39 |
| elodilles | thanks mnasiadka :) | 19:39 |
| elodilles | i'll review as soon as the zuul jobs finished | 19:39 |
| mnasiadka | thanks :) now I can pack and prepare for 9 hours jet lag ;-) | 19:41 |
| elodilles | have a nice trip & safe travel | 19:47 |
| mnasiadka | zuul results came in and green - thanks :) | 19:49 |
| elodilles | +2'd | 19:50 |
| *** JayF is now known as Guest1962 | 21:37 | |
| *** JasonF is now known as JayF | 21:37 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!