| dviroel | it seems that our stable branches (2025.1 at least) is staill facing issue with "No module named 'pkg_resources'" in docs job, did we missed somthing? | 12:20 |
|---|---|---|
| opendevreview | Douglas Viroel proposed openstack/watcher stable/2025.2: DNM - Testing CI https://review.opendev.org/c/openstack/watcher/+/978875 | 12:58 |
| opendevreview | Douglas Viroel proposed openstack/watcher stable/2025.1: [Stable-only][CI] Pin setuptools in docs job https://review.opendev.org/c/openstack/watcher/+/978879 | 13:39 |
| dviroel | sean-k-mooney: jgilaber ^ please take a look, to fix docs job that is failing | 14:07 |
| sean-k-mooney | when i have done that in other repos i have used <= 82 rather then a hard pin to 81.0.0 | 14:10 |
| sean-k-mooney | the chagne is needed but it woudl be better to use <= so that if there are bugfix patches in the future they can be included | 14:11 |
| sean-k-mooney | and to not force 81 as the min version | 14:11 |
| sean-k-mooney | dviroel: ^ | 14:11 |
| dviroel | sean-k-mooney: ah ok, work for me | 14:16 |
| sean-k-mooney | to be clear we did explcitly pin in many of the proejct too but i woudl prefer to not force it | 14:18 |
| dviroel | yeah, i see the pin in other projects, but make sense to replace with <= | 14:21 |
| dviroel | updating.. | 14:21 |
| jgilaber | +1 to what sean-k-mooney but why are we seeing this now in 2025.1? wasn't that problem fixed in testtools? | 14:21 |
| opendevreview | Douglas Viroel proposed openstack/watcher stable/2025.1: [Stable-only][CI] Pin setuptools in docs job https://review.opendev.org/c/openstack/watcher/+/978879 | 14:22 |
| dviroel | the current solution is to pin, iiuc, devstack already pinned in stable branches | 14:23 |
| jgilaber | ack, thanks | 14:23 |
| dviroel | but tox docs we install directly from requirements | 14:23 |
| dviroel | oops, my bad, needs to be <82.0.0 | 14:44 |
| opendevreview | Douglas Viroel proposed openstack/watcher stable/2025.1: [Stable-only][CI] Pin setuptools in docs job https://review.opendev.org/c/openstack/watcher/+/978879 | 14:45 |
| sean-k-mooney | ah yes your right | 14:47 |
| sean-k-mooney | +2 | 14:47 |
| dviroel | sean-k-mooney: thanks, since joan is out, and voted in previous job, I think we can W+1 to unblock CI | 15:08 |
| dviroel | s/previous job/previous patch | 15:08 |
| dviroel | done | 15:10 |
| sean-k-mooney | ack ya i considerd doing that but i think it fiar | 15:10 |
| opendevreview | Merged openstack/watcher stable/2025.1: [Stable-only][CI] Pin setuptools in docs job https://review.opendev.org/c/openstack/watcher/+/978879 | 15:16 |
| sean-k-mooney | dviroel: would you mind backporting ^ to all the relevent stable btranches | 15:22 |
| dviroel | sean-k-mooney: ack, let me take a look on 2024.2 | 15:23 |
| sean-k-mooney | did we skip 2025.2 by mistake ro is that already fixed? | 15:24 |
| dviroel | wasn't failing | 15:25 |
| dviroel | https://review.opendev.org/c/openstack/watcher/+/978875 | 15:25 |
| sean-k-mooney | hum ok we might need it in the furue but lest see | 15:25 |
| sean-k-mooney | we can pull in the commit to it if it starts failing | 15:25 |
| dviroel | "Successfully installed pbr-7.0.3 setuptools-82.0.0" | 15:26 |
| dviroel | ack | 15:26 |
| opendevreview | Marcin Wilk proposed openstack/watcher stable/2025.1: Add debug message to report calculated metric for workload_balance https://review.opendev.org/c/openstack/watcher/+/977299 | 15:33 |
| opendevreview | Joan Gilabert proposed openstack/watcher-specs master: Add 2026.2 OpenStackSDK migration spec for remaining services https://review.opendev.org/c/openstack/watcher-specs/+/978897 | 15:56 |
| jgilaber | dviroel, when you have some time, could you take a look at https://review.opendev.org/c/openstack/watcher/+/971219? It cleans up a few nova_helper methods that are not used | 16:07 |
| sean-k-mooney | that i thinkis small enough to inlcude before rc1 | 16:07 |
| sean-k-mooney | but after that i woudl perhaps wait till we have tagged that to merge anythign that is not a bug intoduced in this cycle | 16:08 |
| sean-k-mooney | i.e. a new regression that we want to adress before we release | 16:08 |
| jgilaber | yes, that's why I ask, I think the change should be safe even at this point of the cycle | 16:10 |
| sean-k-mooney | yep | 16:12 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: Add 2026.1 release notes prelude https://review.opendev.org/c/openstack/watcher/+/976015 | 17:08 |
| opendevreview | Merged openstack/watcher master: Add 2026.1 release notes prelude https://review.opendev.org/c/openstack/watcher/+/976015 | 17:39 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a software upgrade | 18:12 | |
| opendevreview | Merged openstack/watcher master: Remove unused methods from NovaHelper https://review.opendev.org/c/openstack/watcher/+/971219 | 21:52 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!