| frickler | jbernard: rosmaita: commented in the review, what about https://review.opendev.org/c/openstack/releases/+/918347 ? I think we would need that tag made first? | 06:19 |
|---|---|---|
| *** elodilles_ooo is now known as elodilles | 06:32 | |
| noonedeadpunk | frickler: if around - would be awesome to land that really asap... https://review.opendev.org/c/openstack/releases/+/920944/1 | 07:50 |
| frickler | noonedeadpunk: yes, I saw that already, will have to take a look at the shas, since I don't think the automation actually verifies them, but it is on my list | 07:55 |
| noonedeadpunk | aha, ok, I thought it should... | 07:55 |
| noonedeadpunk | I can add to my "to do" list to add such check fwiw | 07:57 |
| noonedeadpunk | as we do have such verification of shas in osa CI | 07:57 |
| frickler | noonedeadpunk: I found one outdated commit for openstack/ansible-role-systemd_networkd, not sure it is relevant, but I didn't find a warning about this in the test logs, so certainly room for improvement there | 08:14 |
| noonedeadpunk | frickler: well, that's reno for unmaintained/zed so meh | 08:16 |
| noonedeadpunk | If you;'re talking about that https://opendev.org/openstack/ansible-role-systemd_networkd/commit/8a4a59afd916bfd7667723f24359b0ac50d1c694 | 08:16 |
| noonedeadpunk | and the sha before that is 19a276e6195399ea117e72afeba050a41db106d1 | 08:16 |
| noonedeadpunk | which seems valid | 08:17 |
| noonedeadpunk | so in sake of progress I'd prefer to ignore that | 08:17 |
| frickler | noonedeadpunk: yes, fine with ignoring that in that case, same for os_blazar, I'm almost done checking the others | 08:18 |
| frickler | noonedeadpunk: hmm, cloudkitty seems to have had a different update, can you check that? | 08:19 |
| frickler | also os_glance | 08:20 |
| noonedeadpunk | cloudkitty is kinda fine - it's marked for backport anyway: https://review.opendev.org/c/openstack/openstack-ansible-os_cloudkitty/+/920432 | 08:21 |
| noonedeadpunk | so we can cherry-pick after branching | 08:21 |
| jrosser | we dont stop merging things just because there is a rc do we | 08:21 |
| jrosser | so i think this is al pretty much expected | 08:21 |
| jrosser | *all | 08:21 |
| noonedeadpunk | yeah, it's kinda fine as long as SHAs are valid and not messed up in terms of wrong sha for the repo | 08:22 |
| noonedeadpunk | for glance it's fine to be left on master, as that's more of a CI change | 08:23 |
| frickler | well other projects actually do some kind of freeze around branching time, but I guess that's not a hard requirement. so let's go on, then | 08:28 |
| noonedeadpunk | we usually take approach of backporting important things and marking them with backport-cadidate label | 08:30 |
| noonedeadpunk | that we can track later on trhough gerrit dashboard we have | 08:31 |
| opendevreview | Merged openstack/releases master: [OpenStack-Ansible Roles] Branch repos for 2024.1 (Caracal) https://review.opendev.org/c/openstack/releases/+/920944 | 08:41 |
| opendevreview | marihan girgis proposed openstack/releases master: Tag a new patch release for the telemetry-tempest-plugin https://review.opendev.org/c/openstack/releases/+/921275 | 10:58 |
| opendevreview | marihan girgis proposed openstack/releases master: Tag a new patch release for the telemetry-tempest-plugin https://review.opendev.org/c/openstack/releases/+/921275 | 11:04 |
| opendevreview | marihan girgis proposed openstack/releases master: Tag a new patch release for the telemetry-tempest-plugin https://review.opendev.org/c/openstack/releases/+/921275 | 11:05 |
| frickler | gouthamr: did you intend to update https://review.opendev.org/c/openstack/releases/+/920145 or should I submit some changes? | 11:12 |
| opendevreview | Elod Illes proposed openstack/releases master: nova: Release 2024.1 Caracal 29.0.2 https://review.opendev.org/c/openstack/releases/+/921287 | 13:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!