| opendevreview | Alfredo Moralejo proposed openstack/watcher-tempest-plugin master: Add api test for skip action https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/955775 | 06:25 |
|---|---|---|
| amoralej | dviroel, wrt https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/956306/3i tried to apply it in | 08:20 |
| amoralej | https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/956306/ i tried to apply it in https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/955775/ but it's skipping the new test in master job | 08:20 |
| amoralej | https://zuul.opendev.org/t/openstack/build/d382e11cd50745f3aef79264069af452/ | 08:21 |
| amoralej | although watcher is reporting 'openstack-api-maximum-version': '1.6' | 08:21 |
| amoralej | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_d38/openstack/d382e11cd50745f3aef79264069af452/controller/logs/tempest_log.txt | 08:22 |
| amoralej | 2025-08-04 06:59:21.135 93231 INFO tempest.test [-] <class 'unittest.case.SkipTest'> raised in TestPatchAction.setUpClass. Invoking tearDownClass | 08:22 |
| amoralej | maybe I'm doing something wrong | 09:25 |
| opendevreview | David proposed openstack/watcher master: Disable real metrics on devstack injected data jobs https://review.opendev.org/c/openstack/watcher/+/955281 | 10:42 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: WIP: Update Overload standard deviation doc https://review.opendev.org/c/openstack/watcher/+/954067 | 10:44 |
| dviroel | amoralej: hi, sorry, missed your ping here | 11:47 |
| amoralej | no problem | 11:48 |
| dviroel | amoralej: you test is setting min_midroversion correctly, but you still need to provide the min and max supported by the cloud, which we are adding in another patch | 11:50 |
| amoralej | ok, so, doesn't it | 11:51 |
| dviroel | amoralej: we were discussing it here https://review.opendev.org/c/openstack/watcher/+/956380 | 11:51 |
| amoralej | so, it takes it from config, not from the supported versions according to api response | 11:51 |
| dviroel | correct, following the tempest docs and how other projects are doing to | 11:51 |
| amoralej | so i need to set 1.6 in my patch for the functional job for master | 11:51 |
| dviroel | s/to/too | 11:51 |
| amoralej | ack, got it | 11:52 |
| dviroel | in the end, we will not set the default in the config of tempest.conf, but in the openstack/watcher repo | 11:52 |
| amoralej | i just need to set the variable TEMPEST_WATCHER_MAX_MICROVERSION | 11:52 |
| dviroel | so each stable branch will have its min/max | 11:52 |
| dviroel | we are going to always hardcoded in devstack plugin.sg | 11:53 |
| dviroel | https://review.opendev.org/c/openstack/watcher/+/956380/2/devstack/lib/watcher line 271 | 11:53 |
| dviroel | so everyone that propose a new microversion, will need to update the default hardcoded | 11:54 |
| dviroel | once we cut the stable branch, it will have the latest for that release | 11:54 |
| amoralej | good, so for the stable jobs in the plugin https://github.com/openstack/watcher-tempest-plugin/blob/master/.zuul.yaml#L28 we'll inherit what is in the stable branches of watcher | 11:55 |
| dviroel | yes, since tempest-plugin is branchless, and watcher is not | 11:55 |
| amoralej | got it | 11:55 |
| amoralej | wfm | 11:55 |
| amoralej | thanks for taking care of this dviroel++ | 11:55 |
| dviroel | it is similar from what happens to nova | 11:55 |
| dviroel | but nova/neutron/etc are configures in the devstack repo instead | 11:56 |
| amoralej | for some reason i thought nova and others had max-min version discovery via api calls, i was wrong | 11:56 |
| dviroel | yeah, gmaan was pointing that to us on friday, on how it works today | 11:57 |
| amoralej | so, before merging https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/956306 we will also need to backport the changes in the stable branches that we test | 12:01 |
| dviroel | correct, or it will break the stable branches, the jobs the runs scenario tests | 12:08 |
| dviroel | going to propose the stable patche now | 12:08 |
| opendevreview | Douglas Viroel proposed openstack/watcher stable/2025.1: Configure watcher tempest's microversion in devstack https://review.opendev.org/c/openstack/watcher/+/956465 | 12:18 |
| opendevreview | Douglas Viroel proposed openstack/watcher stable/2025.1: Configure watcher tempest's microversion in devstack https://review.opendev.org/c/openstack/watcher/+/956465 | 12:18 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: DNM: Test commit https://review.opendev.org/c/openstack/watcher/+/956488 | 14:26 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: Update Overload standard deviation doc https://review.opendev.org/c/openstack/watcher/+/954067 | 14:55 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: WIP: Update Overload standard deviation doc https://review.opendev.org/c/openstack/watcher/+/954067 | 15:10 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: Update Overload standard deviation doc https://review.opendev.org/c/openstack/watcher/+/954067 | 15:45 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: Update Overload standard deviation doc https://review.opendev.org/c/openstack/watcher/+/954067 | 20:07 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!