| sean-k-mooney | gmann: we need to merge the 2 patches before it and a sperate bugfix due to a UC bump that broke the unit tests. we should have them merged in a day or two | 10:37 |
|---|---|---|
| opendevreview | Merged openstack/watcher master: Fix create_continuous_audit_with_wrong_interval test assert https://review.opendev.org/c/openstack/watcher/+/936654 | 12:13 |
| *** haleyb|out is now known as haleyb | 14:03 | |
| opendevreview | Martin Kopec proposed openstack/watcher-tempest-plugin master: Update documentation and add a job to publish it https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/936837 | 14:41 |
| rlandy | sean-k-mooney, wrt comments on https://review.opendev.org/c/openstack/watcher/+/935371/16/.zuul.yaml#294 ... iiuc, the change recommended is just c-vol: false and c-bak: false on the subnode? | 15:53 |
| sean-k-mooney | rlandy: we have 2 options. that or we can reenable cidner on the compute | 15:59 |
| sean-k-mooney | *contoller | 15:59 |
| sean-k-mooney | the curent issue is its dsabeld on the contoler but enabeld on the compute | 15:59 |
| sean-k-mooney | longterm we proably do want cinder | 16:00 |
| rlandy | ok - let me try one run with both disabled | 16:00 |
| sean-k-mooney | so it might be just wroth remvoing the lines that disable it on the contoller but either way works | 16:00 |
| rlandy | then will turn both on | 16:00 |
| sean-k-mooney | +1 | 16:00 |
| sean-k-mooney | while the workload balancing senario can take into account storage pool utilsiation im not sure if the tempest plugin supprots testing that | 16:01 |
| sean-k-mooney | so im not sure how much coverage having cinder awialable actully provides | 16:01 |
| sean-k-mooney | thats another item for use to review when lookign at the existing test coverage i guess | 16:02 |
| rlandy | ok - let's compare the two results | 16:05 |
| rlandy | just commenting out the other tests again and it will run | 16:05 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: WIP: Add Tempest test for Prometheus integration https://review.opendev.org/c/openstack/watcher/+/935371 | 16:08 |
| opendevreview | Merged openstack/watcher-tempest-plugin master: Remove stable/2023.1 job from master gate https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/934267 | 16:39 |
| rlandy | sean-k-mooney, https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_62b/935371/17/check/watcher-prometheus-integration/62b81ee/testr_results.html (disabled on both) | 17:06 |
| rlandy | api, functional and dummy scenario | 17:07 |
| sean-k-mooney | while looking at the results of a diffent job i noticed that he tempest tests have a race condition https://bugs.launchpad.net/watcher-tempest-plugin/+bug/2090854 | 17:07 |
| sean-k-mooney | just and fyi that the watcher-tempest-strategies job is likely unstable becasue of that | 17:08 |
| sean-k-mooney | rlandy: ack thats looking pretty good as a baseline | 17:09 |
| rlandy | test still registers an error on exit | 17:09 |
| rlandy | trying with both enabled so we can compare | 17:09 |
| rlandy | then I can submit with a comment on the open launchpad bug to enabled other scenarios | 17:10 |
| sean-k-mooney | ah i was wondering why that was green but the job failed | 17:10 |
| rlandy | getting | 17:11 |
| sean-k-mooney | oh i see | 17:11 |
| sean-k-mooney | so the test we ran could funciton with ounly the contoller | 17:12 |
| sean-k-mooney | so the fact devstack failed on the compute failed the job but not the tempest run | 17:12 |
| sean-k-mooney | that makes sense | 17:12 |
| sean-k-mooney | so the compute is running aodh and it should not be | 17:12 |
| sean-k-mooney | its failing on tryign to enabel the aodh-api service | 17:13 |
| rlandy | aodh: https://opendev.org/openstack/aodh - enabling plugin | 17:14 |
| sean-k-mooney | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_62b/935371/17/check/watcher-prometheus-integration/62b81ee/compute1/logs/_.localrc_auto.txt | 17:14 |
| sean-k-mooney | so i think we need to overried the devstack plugins enabeld on the subnode | 17:14 |
| rlandy | ok ... | 17:14 |
| sean-k-mooney | at the momept they are inheriting form the contoller so we need to remove sg-core and aodh | 17:14 |
| rlandy | ok - let me see if I can redefine on the subnode | 17:15 |
| sean-k-mooney | i think you just need to copy https://review.opendev.org/c/openstack/watcher/+/935371/17/.zuul.yaml#230 to here? https://review.opendev.org/c/openstack/watcher/+/935371/17/.zuul.yaml#293 and remove the extra values and intent it | 17:16 |
| sean-k-mooney | if we look at th elocal.conf we are not exiplcitly enableing aodh on the subnode os its happening as a sideffect fo the plugin | 17:16 |
| rlandy | so that leaves ceilometer and watcher on the subnode | 17:18 |
| rlandy | let's see what this does | 17:18 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: WIP: Add Tempest test for Prometheus integration https://review.opendev.org/c/openstack/watcher/+/935371 | 17:19 |
| sean-k-mooney | rlandy: actully we shoudl not need the watcher plugin on the subnode it does not have any compoent on the computes | 17:34 |
| sean-k-mooney | so we just need ceilometer for the ceilometer compute agent | 17:34 |
| gmann | sean-k-mooney: ack | 17:36 |
| sean-k-mooney | we can update that next time if it failes but i dont think we are enableing the watcher services on teh compute anyway | 17:36 |
| rlandy | k- will update | 17:42 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: WIP: Add Tempest test for Prometheus integration https://review.opendev.org/c/openstack/watcher/+/935371 | 18:33 |
| opendevreview | Merged openstack/watcher master: [pre-commit] enforce pre-commit checks in ci https://review.opendev.org/c/openstack/watcher/+/934394 | 18:37 |
| opendevreview | Merged openstack/watcher master: Fix incompatiablity between apscheduler and eventlet https://review.opendev.org/c/openstack/watcher/+/933890 | 19:13 |
| opendevreview | Merged openstack/watcher master: Update gate jobs as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/watcher/+/932963 | 19:52 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: WIP: Add Tempest test for Prometheus integration https://review.opendev.org/c/openstack/watcher/+/935371 | 20:20 |
| rlandy | watcher-prometheus-integration success | 20:53 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: WIP: Add Tempest test for Prometheus integration https://review.opendev.org/c/openstack/watcher/+/935371 | 21:39 |
| opendevreview | Ronelle Landy proposed openstack/watcher master: WIP: Add Tempest test for Prometheus integration https://review.opendev.org/c/openstack/watcher/+/935371 | 22:37 |
| rlandy | sean-k-mooney, ok - so both worked ... https://zuul.opendev.org/t/openstack/build/06cde6d3fdea4763b389613fdead806e - including or excluding on both | 23:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!