| opendevreview | Merged openstack/releases master: Remove data/liaisons.yaml file https://review.opendev.org/c/openstack/releases/+/989164 | 12:52 |
|---|---|---|
| opendevreview | Dmitriy Rabotyagov proposed openstack/releases master: [OSA] Cut stable/2026.1 branch for roles https://review.opendev.org/c/openstack/releases/+/989662 | 13:36 |
| noonedeadpunk | elodilles: we've merged 98% of things we wanted and I have updated ^ | 13:36 |
| noonedeadpunk | we need to branch roles before we can tag/release an integrated repo though | 13:36 |
| noonedeadpunk | as we' | 13:36 |
| noonedeadpunk | as we | 13:36 |
| noonedeadpunk | crap | 13:36 |
| noonedeadpunk | as we are placing SHAs of repos after branching in there | 13:37 |
| noonedeadpunk | so ensure they have proper gitreview | 13:37 |
| opendevreview | Jeremy Stanley proposed openstack/releases master: Fix typo of liaison in list-changes script https://review.opendev.org/c/openstack/releases/+/991648 | 13:55 |
| fungi | noonedeadpunk: that ^ should fix the releases-tox-list-changes job failure on showing up on your osa release request | 13:56 |
| noonedeadpunk | thanks | 13:56 |
| noonedeadpunk | will rebase | 13:56 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/releases master: [OSA] Cut stable/2026.1 branch for roles https://review.opendev.org/c/openstack/releases/+/989662 | 13:56 |
| fungi | elodilles: frickler: ^ easy fix to expedite as it'll be blocking all our reviews until we do | 13:57 |
| frickler | thx, approved | 14:34 |
| opendevreview | Merged openstack/releases master: Fix typo of liaison in list-changes script https://review.opendev.org/c/openstack/releases/+/991648 | 14:48 |
| opendevreview | Riccardo Pittau proposed openstack/releases master: Relase ipa 11.6.0 for hibiscus https://review.opendev.org/c/openstack/releases/+/991677 | 15:04 |
| noonedeadpunk | hm | 16:58 |
| fungi | hm? | 17:33 |
| fungi | oh, looks like list-changes has yet another regression that was being masked by the first | 17:34 |
| elodilles | noonedeadpunk: openstack/openstack-ansible-ops hash is not the latest, is that intentional? | 19:27 |
| fungi | just looking over the list-changes script, i'm not sure whether the liaison object isn't exposing the right level in the governance data structure, or if the problem is that some teams simply don't have any listed liasons and it's coming back as nonetype | 19:32 |
| elodilles | fungi: it seems correct to me. where do you see the error? https://zuul.opendev.org/t/openstack/build/373f78bb402444838fbbbbcc7b075041/log/tox/list-changes/list-changes-results.log#19 | 19:33 |
| fungi | elodilles: you have to cherry-pick in a change like 991677 and then you'll get the same error as https://zuul.opendev.org/t/openstack/build/b632d086673a404488182ba5dbf3ad2b | 19:35 |
| fungi | i'm able to reproduce it locally that way | 19:35 |
| elodilles | hmmm. interesting. | 19:47 |
| fungi | that error was hiding behind the simply typo i fixed earlier, but will require some more debugging, i've just barely had time to reproduce it locally so far | 19:51 |
| opendevreview | Elod Illes proposed openstack/releases master: Fix liaisons listing in list_changes command https://review.opendev.org/c/openstack/releases/+/991735 | 20:15 |
| elodilles | fungi: this worked for me locally ^^^ | 20:15 |
| elodilles | though i also just quickly tested it as it is already EOD to me and i'm also out of office just forgot to change my nick | 20:17 |
| elodilles | o:) | 20:17 |
| elodilles | which i'm doing now | 20:17 |
| *** elodilles is now known as elodilles_OoO | 20:17 | |
| fungi | awesome, i expected the problem was basically that, the liaison lists are a level deeper in the data structure, and also may not exist | 20:17 |
| fungi | thanks elodilles! | 20:17 |
| elodilles_OoO | no problem, thanks for catching it o/ | 20:17 |
| fungi | i'll recheck 991677 once your fix merges and see if there's anything else we've missed | 20:17 |
| elodilles_OoO | thanks o/ | 20:19 |
| * elodilles_OoO disappears now for the day | 20:19 | |
| fungi | yep, see you at the meeting tomorrow | 20:20 |
| opendevreview | Merged openstack/releases master: Fix liaisons listing in list_changes command https://review.opendev.org/c/openstack/releases/+/991735 | 21:10 |
| fungi | looks like it worked finally! | 21:39 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!