Wednesday, 2025-05-28

opendevreviewMerged openstack/watcher-tempest-plugin master: Add metrics for RAM to gnocchi and prometheus injected data  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94977307:18
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Add tests for workload_balance with injected data  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/94972207:38
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Create 2 instances per host on test_execute_workload_stabilization_strategy to avoid creating empty actionplans  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95096907:52
amoralejdviroel, wrt https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/950389 , I also proposed updating version of nova in the watcher jobs in https://review.opendev.org/c/openstack/watcher/+/950387 but i understand from the comments there to implement it in the plugin itself09:17
amoralejand honestly, I liked the idea of decoupling the implementation of that from the jobs configuration in the different deployment tools that may be configuring tempest09:17
amoralejalthough, I'm fine with updating the min_version to something more recent and in the scope of the current supportability of the plugin09:17
sean-k-mooneyamoralej: the way microversion are ment to work is you are ment to set it on every rest call to the version required by the request10:38
sean-k-mooneythe min and max version are more for deteminign if a cloud can suppprot the test then contoleing what version tehy use10:38
sean-k-mooneythis si somehting that the watcher project did incorrectly in both watcher and the tempest plugin10:39
sean-k-mooneywe shoudl not have https://docs.openstack.org/watcher/latest/configuration/watcher.html#nova_client.api_version as a config option at all. its more reasoable to have https://github.com/openstack/tempest/blob/master/tempest/config.py#L380-L39710:42
sean-k-mooneybut they are not for selecting what microverison to use they are for selecting which test to run10:43
opendevreviewRonelle Landy proposed openstack/watcher master: Update Workload Balance strategy documentation  https://review.opendev.org/c/openstack/watcher/+/95102511:56
dviroelamoralej: ack, continuing our discussion in the tempest patch12:02
opendevreviewDouglas Viroel proposed openstack/watcher-specs master: Adds spec for extend compute model attributes  https://review.opendev.org/c/openstack/watcher-specs/+/94728212:37
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: I have observe that we have a high ratio of failures on this test because the action plan do not contain any "migrate" action.  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95096913:06
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Update parameters on test_execute_workload_stabilization_strategy  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95096913:18
rlandyreminder: watcher meeting tomorrow - pls add topics to: https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L5413:52
rlandythank you13:52
amoralejdviroel, sean-k-mooney thanks for the clarification on the review, now i understand it better.14:12
amoralejBut i still have doubts. My understanding is that the parameter https://github.com/openstack/watcher/blob/59757249bbbbb45bead28bfb2899242aea97dfc1/.zuul.yaml#L60 and/or https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/950389 will not affect at all to the version of the nova api version that watcher uses14:13
amoralejonly the version that the tempest plugin uses when calling directly to nova to create instances14:14
amoralejso, i see two different topics14:14
amoralejtempest may use 2.74 when creating an instance, while watcher keep using 2.56 when doing api calls to nova14:14
amoralejalthough, it may be a good practice to be consistent14:15
amoralejso, getting rid of the https://docs.openstack.org/watcher/latest/configuration/watcher.html#nova_client.api_version and replacing it by a better approach as unrelated to changing the api version that the tempest plugin itself uses14:16
sean-k-mooneyamoralej: yes they are two diffent topic. related but different14:28
sean-k-mooneyamoralej: the tempest plugign shoudl ideally explcitly use 2.57 to specify the host when creating the vm 14:29
amoralej2.74, right?14:29
sean-k-mooneyand the test shoudl be annotated with a min nova microvsion fo 2.57 unless you implemnt a fallback for the test to the older way to specify the host14:29
sean-k-mooneyam well which ever the correct one is fro --host14:30
amoralejyeah, got it14:31
amoraleji hope, that was i was doing with my patch14:31
amoralejat the nova client creation time inside each test14:31
amoralejnot per api call but per client creation14:31
amoralejwrt fallback for older way, i'd say it's not worthy, tbh14:32
amoralejI took the idea about implementating fixin the api version from https://docs.openstack.org/tempest/16.1.0/microversion_testing.html14:35
amoralejmaybe it's wrong way to do it14:35
dviroelamoralej: right, when you define the tempest min compute version, it only affects the resources created by tempest, but watcher still keep using its minimal. I missed that detail, that we don't change the minimal in watcher side14:55
dviroelwe will still testing watcher on 2.56 in this way, but it was confuding for me when I looked at the first time14:56
dviroelnot sure if makes sense to create the workloads with 2.74 while watcher is always using 2.5614:57
amoralejyes, i was missing we set specific minimal version of nova api in watcher :)14:58
amoralejactually, in the "real world" out of openstack, the version used to create workloads it's independent of the version that watcher uses14:59
amoralejs/out of openstack/out of tempest tests/14:59
dviroeli think that sean was mentioning that is still possible to select the host in 2.56, using the AZ14:59
dviroelamoralej: ack, older clients should be able to retrieve info in the same way15:00
amoraleji can investigate the AZ option, if you think that's a better approach15:02
opendevreviewMerged openstack/grian-ui master: Add basic docs and runserver command  https://review.opendev.org/c/openstack/grian-ui/+/95047215:29
opendevreviewRonelle Landy proposed openstack/watcher master: Update Host Maintenance strategy documentation  https://review.opendev.org/c/openstack/watcher/+/95100216:02
opendevreviewDavid proposed openstack/watcher-tempest-plugin master: Update parameters on test_execute_workload_stabilization_strategy  https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/95096916:03
opendevreviewJoan Gilabert proposed openstack/watcher master: Handle missing dst_pool parameter in zone_migration  https://review.opendev.org/c/openstack/watcher/+/95014916:18
opendevreviewRonelle Landy proposed openstack/watcher master: Update Host Maintenance strategy documentation  https://review.opendev.org/c/openstack/watcher/+/95100218:59
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org is temporarily unreachable due to an ongoing issue in the hosting provider where it resides22:08

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!