)]}'
{".zuul.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f1b7185d9f0b75bf4655278575e8400cb5f1fc8f","unresolved":true,"context_lines":[{"line_number":48,"context_line":"          mysql: false"},{"line_number":49,"context_line":"    vars:"},{"line_number":50,"context_line":"      devstack_localrc:"},{"line_number":51,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-low-rate\""},{"line_number":52,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":53,"context_line":"      devstack_services:"},{"line_number":54,"context_line":"        ceilometer-acompute: false"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"953c2f64_4fca4caa","line":51,"range":{"start_line":51,"start_character":52,"end_line":51,"end_character":55},"updated":"2025-07-31 17:50:40.000000000","message":"we probaly shoudl set this to `ceilometer-high-rate`\n\n\nlow rate i think has a long time before the stats show up or a longer aggreation time i.e. an hour vs 5 mins kind of thing.\n\nthey are using high in the telemetry plugin i htink for that reason","commit_id":"5994bda16a0d03e95af2f29a2cb8433cd1c6ba56"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"33d6d96826d00ca40a9ba122f2823c180172740c","unresolved":true,"context_lines":[{"line_number":48,"context_line":"          mysql: false"},{"line_number":49,"context_line":"    vars:"},{"line_number":50,"context_line":"      devstack_localrc:"},{"line_number":51,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-low-rate\""},{"line_number":52,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":53,"context_line":"      devstack_services:"},{"line_number":54,"context_line":"        ceilometer-acompute: false"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"6474bebf_2cbee6a3","line":51,"range":{"start_line":51,"start_character":52,"end_line":51,"end_character":55},"in_reply_to":"51bcd7b4_b5656d81","updated":"2025-08-01 12:05:44.000000000","message":"well no\nthe test shoudl idealy not depend on the granulatiry at all\n\nit should not be assertign any expclit numbers.\nthe only reaosn the test exist is to make sure teh stragey works witout injectign metrics\n\nwatcher behavior need to be consistent regardles of the arcive policy that is configure on gnocchi\n\nif its not then that a bug in watcher.","commit_id":"5994bda16a0d03e95af2f29a2cb8433cd1c6ba56"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"669e7a026ba29de920cc3f2fcf264b704edbeebb","unresolved":true,"context_lines":[{"line_number":48,"context_line":"          mysql: false"},{"line_number":49,"context_line":"    vars:"},{"line_number":50,"context_line":"      devstack_localrc:"},{"line_number":51,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-low-rate\""},{"line_number":52,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":53,"context_line":"      devstack_services:"},{"line_number":54,"context_line":"        ceilometer-acompute: false"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"80c7d7c1_4facd1b6","line":51,"range":{"start_line":51,"start_character":52,"end_line":51,"end_character":55},"in_reply_to":"6474bebf_2cbee6a3","updated":"2025-08-04 09:04:38.000000000","message":"maybe dependency is not the correct name, I would say the test has to be correlated with the granularity.\n\nGranularity is a parameter only used by gnocchi, but we need to know the archive policy used by ceilometer, and create the tests according to it.\n\nSetting to ceilometer-low-rate, we have to use 300 as granularity, and inject metrics matching that (1 metric each minute during 10 minutes)\n\nIf we change to ceilometer-high-rate, we need to think which granulary we have to use (because 300 fails, I have already tried) and also rethink the timeframe we use on injected metrics.\n\nThat is a huge change and that is why I think it is better to keep using ceilometer-low-rate","commit_id":"5994bda16a0d03e95af2f29a2cb8433cd1c6ba56"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"463c54780fd054070b29729c723ab60551963ef7","unresolved":true,"context_lines":[{"line_number":48,"context_line":"          mysql: false"},{"line_number":49,"context_line":"    vars:"},{"line_number":50,"context_line":"      devstack_localrc:"},{"line_number":51,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-low-rate\""},{"line_number":52,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":53,"context_line":"      devstack_services:"},{"line_number":54,"context_line":"        ceilometer-acompute: false"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"4298985f_a7f68243","line":51,"range":{"start_line":51,"start_character":52,"end_line":51,"end_character":55},"in_reply_to":"80c7d7c1_4facd1b6","updated":"2025-08-05 14:56:57.000000000","message":"ack low is OK but the test was failing in ci with low using the default granularity that is why i suggested trying high.","commit_id":"5994bda16a0d03e95af2f29a2cb8433cd1c6ba56"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"77bc9b7c78b2d1cd96f4e39a9e50235972148a78","unresolved":true,"context_lines":[{"line_number":48,"context_line":"          mysql: false"},{"line_number":49,"context_line":"    vars:"},{"line_number":50,"context_line":"      devstack_localrc:"},{"line_number":51,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-low-rate\""},{"line_number":52,"context_line":"        CEILOMETER_PIPELINE_INTERVAL: 15"},{"line_number":53,"context_line":"      devstack_services:"},{"line_number":54,"context_line":"        ceilometer-acompute: false"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"51bcd7b4_b5656d81","line":51,"range":{"start_line":51,"start_character":52,"end_line":51,"end_character":55},"in_reply_to":"953c2f64_4fca4caa","updated":"2025-08-01 11:24:34.000000000","message":"I think it is a good idea, but I would accomplish on a future change, as it requires changes on the job, but also on the tests because of the  granularity and the number and timeframe of injected metrics\n\nceilometer-high-rate decreases the granularity to 1 second (ceilometer-low-rate is 5 minutes) so changing default to high rate requires a granularity change on all the tests requiring metrics:\n\nAug 01 10:38:44 morenod-devstack-1 watcher-decision-engine[1015]: DEBUG urllib3.connectionpool [None req-c3604f86-87ee-45ce-af00-ba8c91803ed1 None None] http://192.168.50.108:80 \"GET /metric/v1/resource/generic/ebbe5669-f9ce-46c3-84bc-b8cbc872ddb4/metric/cpu/measures?start\u003d2025-08-01T10%3A36%3A44.084974\u0026stop\u003d2025-08-01T10%3A38%3A44.084974\u0026aggregation\u003drate%3Amean\u0026granularity\u003d300\u0026refresh\u003dFalse HTTP/1.1\" 404 170 {{(pid\u003d1015) _make_request /opt/stack/data/venv/lib/python3.12/site-packages/urllib3/connectionpool.py:475}}\n\nAug 01 10:38:44 morenod-devstack-1 watcher-decision-engine[1015]: DEBUG keystoneauth.session [None req-c3604f86-87ee-45ce-af00-ba8c91803ed1 None None] RESP BODY: {\"code\": 404, \"title\": \"Not Found\", \"description\": \"Aggregation method \u0027rate:mean\u0027 at granularity \u0027300.0\u0027 for metric b53e27b8-4c01-4ae1-9b45-42079202f742 does not exist\"} {{(pid\u003d1015) _http_log_response /opt/stack/data/venv/lib/python3.12/site-packages/keystoneauth1/session.py:656}}\n\n\nAlso it would require a change on number of injected metrics and timeframe between them.","commit_id":"5994bda16a0d03e95af2f29a2cb8433cd1c6ba56"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"44c4b2501f0de924508815e60c872132529b2a01","unresolved":true,"context_lines":[{"line_number":278,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":279,"context_line":"            optimize:"},{"line_number":280,"context_line":"              datasource: \"\""},{"line_number":281,"context_line":"              real_workload_period: 300"},{"line_number":282,"context_line":"      # All tests inside watcher_tempest_plugin.tests.scenario with tag \"real_load\""},{"line_number":283,"context_line":"      tempest_test_regex: (^watcher_tempest_plugin.tests.scenario)(.*\\[.*\\breal_load\\b.*\\].*)"},{"line_number":284,"context_line":"      tempest_exclude_regex: \"\""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"205de710_79afab64","line":281,"updated":"2025-08-05 15:02:04.000000000","message":"i suspect this was actully what was missing in v14","commit_id":"0435200fb118fc232a61ea0aa86f0c034ecb4960"}],"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"44c4b2501f0de924508815e60c872132529b2a01","unresolved":true,"context_lines":[{"line_number":13,"context_line":"We already did this on watcher-operator disabling ceilometer agent and"},{"line_number":14,"context_line":"node_exported on [1] so now we have to do it on devstack installations,"},{"line_number":15,"context_line":"disabling meminfo on node_exporter for host metrics (cpu is already"},{"line_number":16,"context_line":"disabled) and sg-core for instance metrics"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"[1] https://github.com/openstack-k8s-operators/watcher-operator/pull/196"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":17,"id":"94099991_47b95a71","line":16,"updated":"2025-08-05 15:02:04.000000000","message":"downstream ci decision wont always be correct upstream so we shoudl not assume just because we did this change in watcher operator that its valid for upstream.\n\nin this case it is but watcher operator supports fewer configuration and feature then watcher does.\n\nthe fact we already made this change downstream does not generally add any weight to doing it upstream and i would prefer not to include references to the watcher-operator repo in commit messages.","commit_id":"0435200fb118fc232a61ea0aa86f0c034ecb4960"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"f6b12583a63563266805de1dca4b3d6d0dc70df6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"caff3af6_b140f1e4","updated":"2025-07-17 12:19:41.000000000","message":"recheck","commit_id":"f922a37948d3b11c85349bece9fe82760aa5b946"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"4c17886381d5e4371b199c0206f55799b93963ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3a17c71b_07e43153","updated":"2025-07-21 09:19:44.000000000","message":"check experimental","commit_id":"ccad37f3492fdce2e420a71422625b8a54df7dad"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"eb93899f3178c4b947583c3843fdb5b541657a0f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d4e49bd9_9c17c274","updated":"2025-07-18 13:57:28.000000000","message":"check experimental","commit_id":"ccad37f3492fdce2e420a71422625b8a54df7dad"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"97c2f22eb522303db7ed879e418c6304f658f851","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"fb1f77ad_08a33a0f","updated":"2025-07-21 11:04:55.000000000","message":"check experimental","commit_id":"ccad37f3492fdce2e420a71422625b8a54df7dad"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"a1b243e24cc15899f6b8982a21c507ff7dd8f5c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"437437fc_5d5c3b93","updated":"2025-07-18 13:44:05.000000000","message":"recheck","commit_id":"ccad37f3492fdce2e420a71422625b8a54df7dad"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"1ea5dabe6f904c5ce4d794b506df814e203faf5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5a438a0c_265084ad","updated":"2025-07-21 11:04:48.000000000","message":"recheck","commit_id":"ccad37f3492fdce2e420a71422625b8a54df7dad"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"8f3d241ab08504956ae11ca01e0e11243376cbde","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"5850a2fd_c7985778","updated":"2025-07-22 09:06:20.000000000","message":"This patch is not disabling real metrics in gnocchi, only in prometheus, so the tests still are still failing in the gnocchi job","commit_id":"1cbf6216043b99feabf3ba1b78a8a95459485eac"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"6aef9ed7272fc6f586ecf27155e1631085e55995","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"3d957a8d_63d2e73c","updated":"2025-07-22 06:35:22.000000000","message":"check experimental","commit_id":"1cbf6216043b99feabf3ba1b78a8a95459485eac"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"5dc5c50eae5b29978b394ae075c1f758280e56db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f9c3cbf0_5f14a0ad","updated":"2025-07-22 06:38:38.000000000","message":"we should test this with the recently disabled tests. iiuc, this may fix one of them, could we do it with a patch on top of this?","commit_id":"1cbf6216043b99feabf3ba1b78a8a95459485eac"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f1b7185d9f0b75bf4655278575e8400cb5f1fc8f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"93644171_5ddae586","in_reply_to":"5850a2fd_c7985778","updated":"2025-07-31 17:50:40.000000000","message":"ah good spot.\n\ni was not aware we ran the real data test in gnocci\nthey were intended ot test the promethues datasouce originally\n\ntesting the data collection adn reporting into the datasouce is technially out os scope of the watcher tempest plugin and shoudl be done via the templetmety tempst plugin so we coudl just skip those test for gnoicci.","commit_id":"1cbf6216043b99feabf3ba1b78a8a95459485eac"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"77bc9b7c78b2d1cd96f4e39a9e50235972148a78","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cd22b27a_bf5c119e","in_reply_to":"93644171_5ddae586","updated":"2025-08-01 11:24:34.000000000","message":"Comment from alfredo was related to the fact that I was changing values on the incrrect job parent, but it is outdated.\n\nThe changes are now on watcher-tempest-multinode job, which is the parent of the gnocchi jobs\n\nthere is not intention at this moment of adding real data jobs for gnocchi, they are only configured on the job installed with probmetheus.\n\n\nA valid execution of all the jobs (gnocchi and prometheus) enabling again workload balance jobs can be found here:\n\nhttps://zuul.opendev.org/t/openstack/buildset/d36b76590ce0484e8d906fadf3282ecf (https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/953853)","commit_id":"1cbf6216043b99feabf3ba1b78a8a95459485eac"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"8922cff88f8773d7871d55fa553f613c12f68f1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"a8dd3f8f_4ee04712","updated":"2025-07-29 11:58:57.000000000","message":"check experimental","commit_id":"7121bfcad8f82d3e6ec89cc2197ddc842091ef83"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"aa8b3f3c30359100cada0336fff46fe5bded5e67","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"2db49e90_33087bf9","updated":"2025-07-29 11:58:47.000000000","message":"recheck","commit_id":"7121bfcad8f82d3e6ec89cc2197ddc842091ef83"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"d8a445f74c0a110c5eb8a0df1256e6c2215d2da3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"5c723331_761b41db","updated":"2025-07-23 09:37:20.000000000","message":"recheck","commit_id":"7121bfcad8f82d3e6ec89cc2197ddc842091ef83"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"3fd18b7b393892c1a2da57b60391692ca6f9eadd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"286ad3e0_cfbbde23","updated":"2025-07-31 14:21:11.000000000","message":"check experimental","commit_id":"5994bda16a0d03e95af2f29a2cb8433cd1c6ba56"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"6ccc224eab9466680458154a50118f3ca3ed8122","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"54e41639_0a75988e","updated":"2025-07-31 14:21:03.000000000","message":"recheck","commit_id":"5994bda16a0d03e95af2f29a2cb8433cd1c6ba56"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"37e7d28e1efc3cab171035cace746dd4dbe8cf43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"1b0b8fb1_af9da704","updated":"2025-08-04 13:52:26.000000000","message":"check experimental","commit_id":"0435200fb118fc232a61ea0aa86f0c034ecb4960"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"a281f7c4928066bb65a997158aa62724523038b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"5a7c0806_afc2207a","updated":"2025-08-05 07:32:08.000000000","message":"check experimental","commit_id":"0435200fb118fc232a61ea0aa86f0c034ecb4960"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"91f40f9011281d450bba6937506df96ef0176aee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"d1baa873_f95812e8","updated":"2025-08-06 14:51:15.000000000","message":"i would still prefer not to reference watcher operator prs or changes in the commit message but that not enough to block this.\n\nif you have to respin this patch please remove them","commit_id":"0435200fb118fc232a61ea0aa86f0c034ecb4960"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"0b639bc32dc330ca13a00f3ee1e037f36fededc6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"15b1bed6_159831bb","updated":"2025-08-05 14:58:12.000000000","message":"it looks like the latest version may not be passing so ill try and re review this again tommrow or later this week","commit_id":"0435200fb118fc232a61ea0aa86f0c034ecb4960"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"fa1232e67c0e6117bc2648805cd27b1e8ae60b5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"957dc75e_42268670","updated":"2025-08-11 16:44:52.000000000","message":"looks good, services are enabled when they should, and CI is happy with the change, thanks!","commit_id":"0435200fb118fc232a61ea0aa86f0c034ecb4960"}]}