| amoralej_ | Hi, should we do a review of the open https://review.opendev.org/q/project:openstack/watcher+branch:stable/2025.1 before https://review.opendev.org/c/openstack/releases/+/1010956 ? | 07:35 |
|---|---|---|
| jgilaber | that is a great question amoralej_ I was looking at that too | 08:51 |
| jgilaber | it seems we have 35 patches open to that branch | 08:51 |
| jgilaber | I'm leaning towards only merging https://review.opendev.org/c/openstack/watcher/+/996337 which might be helpful for CI and then moving to unmaintained | 08:52 |
| amoralej_ | merging that one | 09:25 |
| amoralej_ | we need to -1 https://review.opendev.org/c/openstack/releases/+/1010956 if we want to get backports before the new branchy and tag | 09:28 |
| jgilaber | done | 09:28 |
| opendevreview | Merged openstack/watcher stable/2025.1: Pin Jammy Tempest job inputs https://review.opendev.org/c/openstack/watcher/+/996337 | 10:32 |
| dviroel | it seems that https://review.opendev.org/c/openstack/watcher/+/951531 would be a starting point for merging the backports | 11:52 |
| jgilaber | yes, what we need to decide if merging the long list of patches there is worth it or not | 12:27 |
| opendevreview | Merged openstack/watcher stable/2025.1: Add test for missing destination in zone migration https://review.opendev.org/c/openstack/watcher/+/951531 | 13:04 |
| dviroel | there are a lots of small and self contained fixes that I think worth merging. | 13:18 |
| dviroel | since CI is green and we still have some time, we can get a few in | 13:18 |
| jgilaber | ack, I already -1 the unamaintained patch,we have a couple of weeks to make progress | 13:19 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add Prometheus emulator and workload_stabilization functional tests https://review.opendev.org/c/openstack/watcher/+/993543 | 13:58 |
| opendevreview | Alfredo Moralejo proposed openstack/watcher master: Add Cinder emulator and zone_migration functional tests https://review.opendev.org/c/openstack/watcher/+/993634 | 13:58 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Remove compute max_microversion set in ci https://review.opendev.org/c/openstack/watcher/+/1011300 | 14:14 |
| dviroel | amoralej_: or jgilaber: as discussed last week in PTG, we agreed to merge this: https://review.opendev.org/c/openstack/watcher/+/1006771 - PTAL when possible. Thanks | 14:15 |
| jgilaber | sure,will review shortly | 14:16 |
| amoralej_ | merging, i was just waiting for PTG to +2 | 14:16 |
| jgilaber | that works for me too :) | 14:17 |
| opendevreview | Merged openstack/watcher stable/2025.1: Handle missing dst_node parameter in zone_migration https://review.opendev.org/c/openstack/watcher/+/951535 | 14:21 |
| opendevreview | Merged openstack/watcher master: Deprecate the Grafana datasource https://review.opendev.org/c/openstack/watcher/+/1006771 | 16:04 |
| opendevreview | Merged openstack/watcher stable/2025.1: Set keystone_client default interface to public https://review.opendev.org/c/openstack/watcher/+/950294 | 16:45 |
| opendevreview | Merged openstack/watcher stable/2025.1: Check logs in some cinder and nova helper tests https://review.opendev.org/c/openstack/watcher/+/960715 | 21:11 |
| opendevreview | Douglas Viroel proposed openstack/watcher-specs master: Add spec for zone migration availability zone drain https://review.opendev.org/c/openstack/watcher-specs/+/1007853 | 21:37 |
| opendevreview | Merged openstack/watcher stable/2025.1: Fix incorrect logging format https://review.opendev.org/c/openstack/watcher/+/960716 | 22:41 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!