)]}'
{".zuul.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e412882c0e1c505e3290027769d59d8a756b6ac1","unresolved":true,"context_lines":[{"line_number":48,"context_line":"        test-config:"},{"line_number":49,"context_line":"          $WATCHER_CONFIG:"},{"line_number":50,"context_line":"            watcher_strategies.vm_workload_consolidation:"},{"line_number":51,"context_line":"              datasource: ceilometer"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"- job:"},{"line_number":54,"context_line":"    name: watcher-tempest-workload_balancing"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"ff8ee06d_f7d50d57","side":"PARENT","line":51,"updated":"2024-12-16 07:45:24.000000000","message":"in time we may bring some of this back with the new prometheus data source and devstack with sg-core but for now makes sense to remove if these are all using ceilo datasource","commit_id":"811a704f809652af660b2ffd3a4c76e4196f930f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"0ce2fb6e286f09cc8d8ae890ef61d7d321954961","unresolved":true,"context_lines":[{"line_number":48,"context_line":"        test-config:"},{"line_number":49,"context_line":"          $WATCHER_CONFIG:"},{"line_number":50,"context_line":"            watcher_strategies.vm_workload_consolidation:"},{"line_number":51,"context_line":"              datasource: ceilometer"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"- job:"},{"line_number":54,"context_line":"    name: watcher-tempest-workload_balancing"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3d49a754_28378159","side":"PARENT","line":51,"in_reply_to":"7115601f_0de2d0b7","updated":"2024-12-19 02:01:55.000000000","message":"We can add it back (of cause with modification when we add a meaningful test)\n\nOne minor note I\u0027d add here is that the watcher_strategies.vm_workload_consolidation section does not exist in watcher.conf really (which clearly indicates this job is just useless)","commit_id":"811a704f809652af660b2ffd3a4c76e4196f930f"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a9f74adc251d603ef43e7171a186917d5e9cab81","unresolved":true,"context_lines":[{"line_number":48,"context_line":"        test-config:"},{"line_number":49,"context_line":"          $WATCHER_CONFIG:"},{"line_number":50,"context_line":"            watcher_strategies.vm_workload_consolidation:"},{"line_number":51,"context_line":"              datasource: ceilometer"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"- job:"},{"line_number":54,"context_line":"    name: watcher-tempest-workload_balancing"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7115601f_0de2d0b7","side":"PARENT","line":51,"in_reply_to":"ff8ee06d_f7d50d57","updated":"2024-12-17 15:17:52.000000000","message":"Since ceilomer and gnocchi datasource have the same metric list, we could just replace ceilometer  with gnocchi in all. But TBH, we are not using these jobs anywhere, so I am in favor on removing them, and continue with jobs that tests more than one strategy.","commit_id":"811a704f809652af660b2ffd3a4c76e4196f930f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"0a114fe2c3441be9667734a3c1aab2b77517d3f4","unresolved":true,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- job:"},{"line_number":43,"context_line":"    name: watcher-tempest-vm_workload_consolidation"},{"line_number":44,"context_line":"    parent: watcher-tempest-multinode"},{"line_number":45,"context_line":"    vars:"},{"line_number":46,"context_line":"      tempest_test_regex: watcher_tempest_plugin.tests.scenario.test_execute_vm_workload_consolidation"},{"line_number":47,"context_line":"      devstack_local_conf:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"05309f32_cfb2a62f","side":"PARENT","line":44,"updated":"2024-12-17 06:45:26.000000000","message":"watcher-tempest-multinode job definition contains reference of ceilometer.\nhttps://opendev.org/openstack/watcher/src/branch/master/.zuul.yaml#L143 and https://opendev.org/openstack/watcher/src/branch/master/.zuul.yaml#L114 Do we want to drop those also?","commit_id":"811a704f809652af660b2ffd3a4c76e4196f930f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"26d19e8aee4bb938e31f06607e1fe042ee45f0da","unresolved":true,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"- job:"},{"line_number":43,"context_line":"    name: watcher-tempest-vm_workload_consolidation"},{"line_number":44,"context_line":"    parent: watcher-tempest-multinode"},{"line_number":45,"context_line":"    vars:"},{"line_number":46,"context_line":"      tempest_test_regex: watcher_tempest_plugin.tests.scenario.test_execute_vm_workload_consolidation"},{"line_number":47,"context_line":"      devstack_local_conf:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"8977a770_2201e967","side":"PARENT","line":44,"in_reply_to":"05309f32_cfb2a62f","updated":"2024-12-17 09:05:08.000000000","message":"IIUC we need telemetry services to use gnocchi datasources in this job so the reference should be kept. These We don\u0027t have to install telemetry services in subnodes.","commit_id":"811a704f809652af660b2ffd3a4c76e4196f930f"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"030d5adabde8ed8fc35e56b9180d69f16e60e245","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9b683f82_53466aad","updated":"2024-12-16 06:21:20.000000000","message":"https://992db71bf452589ff860-e0419d2da2313928ed85c22410bb6c0a.ssl.cf2.rackcdn.com/937625/4/check/watcher-tempest-functional/d07f902/job-output.txt\n```\nFailed to fetch https://mirror-int.ord.rax.opendev.org/ubuntu/pool/main/h/haproxy/haproxy_2.8.5-1ubuntu3.1_amd64.deb  404  Not Found [IP: 10.209.96.36 443]\n2024-12-12 14:57:44.232496 | controller | E: Failed to fetch https://mirror-int.ord.rax.opendev.org/ubuntu/pool/main/p/postgresql-16/libpq5_16.4-0ubuntu0.24.04.2_amd64.deb  404  Not Found [IP: 10.209.96.36 443]\n2024-12-12 14:57:44.232526 | controller | E: Failed to fetch https://mirror-int.ord.rax.opendev.org/ubuntu/pool/main/p/postgresql-16/libpq-dev_16.4-0ubuntu0.24.04.2_amd64.deb  404  Not Found [IP: 10.209.96.36 443]\n2024-12-12 14:57:44.232541 | controller | E: Failed to fetch https://mirror-int.ord.rax.opendev.org/ubuntu/pool/main/p/postgresql-16/postgresql-client-16_16.4-0ubuntu0.24.04.2_amd64.deb  404  Not Found [IP: 10.209.96.36 443]\n2024-12-12 14:57:44.232554 | controller | E: Failed to fetch https://mirror-int.ord.rax.opendev.org/ubuntu/pool/universe/p/postgresql-16/postgresql-server-dev-16_16.4-0ubuntu0.24.04.2_amd64.deb  404  Not Found [IP: 10.209.96.36 443]\n2024-12-12 14:57:44.232567 | controller | E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?\n```\nMirror issue\n\nrecheck","commit_id":"d45902fcd43aafbc8285779d2259d5ad8ee77f3e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e412882c0e1c505e3290027769d59d8a756b6ac1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"25284d63_e21c30b7","updated":"2024-12-16 07:45:24.000000000","message":"thanks for working on that Takashi. from a quick review i don\u0027t think there will be any conflict with the new prometheus data source https://review.opendev.org/c/openstack/watcher/+/934423. no vote as it needs update for the pep8 which is hilighted inline","commit_id":"d45902fcd43aafbc8285779d2259d5ad8ee77f3e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"45b85ea92cebe68ecd500afcfc12a137f1c2af0a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4caee75f_175d04d2","updated":"2025-01-27 07:27:56.000000000","message":"bump @smooney@redhat.com i think this can go now too? you added +2 10 Jan so last friday 24th was 2 weeks already","commit_id":"da23fdc621bfd3211da4946ac5bdc017fb867c5a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"7990e12547ebfa5218de1b7d8f53451810148e2a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c326823b_ed3d8f14","updated":"2024-12-17 06:41:54.000000000","message":"https://opendev.org/openstack/watcher-dashboard/src/branch/master/watcher_dashboard/templates/infra_optim/_performance_chart_box.html#L5 contains reference of ceilometer stats, I think we also need a patch in watcher_dashboard to drop ceilometer stats from there.","commit_id":"da23fdc621bfd3211da4946ac5bdc017fb867c5a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f7956b6c5ece6715f442a45b261c7affdc3871f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3f91261f_fc80a97f","updated":"2025-01-29 12:49:39.000000000","message":"per conversation on irc,\nthis has +1s form both doug and marios who i am proposing adding as core after the next watcher meeting.\nadditionally this has been available for review for over a month and is removing functionality that no longer works in any supported release\n\nbased on all of the above reasons im upgrading to +2w","commit_id":"da23fdc621bfd3211da4946ac5bdc017fb867c5a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"46ecc16e526e9dcc9a7e034ddc91fd7981fadda8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"14964a66_e7da33fc","updated":"2025-01-08 07:22:47.000000000","message":"recheck \n\nstrategies sometimes hits a transient issue (I believe related to the bug Sean has been chasing down with patched/unpatched apscheduler vs eventlet eg https://review.opendev.org/c/openstack/watcher/+/933890)","commit_id":"da23fdc621bfd3211da4946ac5bdc017fb867c5a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"baff0c5d2b903330df0ed422659aaab141dc0712","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"82a0934e_3cecfd23","updated":"2025-01-09 22:54:03.000000000","message":"recheck the job failure looks unrelated to me","commit_id":"da23fdc621bfd3211da4946ac5bdc017fb867c5a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a9f74adc251d603ef43e7171a186917d5e9cab81","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"848ea8a3_f766364c","updated":"2024-12-17 15:17:52.000000000","message":"still reviewing it, trying to see if we are missing things...","commit_id":"da23fdc621bfd3211da4946ac5bdc017fb867c5a"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"10bcb1c6b3f35aee058e17ce4706d18c5c4aa9cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"361efff4_7521c92d","updated":"2025-01-20 15:18:44.000000000","message":"updating to +1: CI is happy with the removal.\nThanks Takashi.","commit_id":"da23fdc621bfd3211da4946ac5bdc017fb867c5a"}],"releasenotes/notes/remove-ceilometer-datasource-8d9ab7d64d61e405.yaml":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a9f74adc251d603ef43e7171a186917d5e9cab81","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Ceilometer datasource has been completely removed. The datasource requires"},{"line_number":5,"context_line":"    ceilometer API which was already removed from Ceilometer. Use the other"},{"line_number":6,"context_line":"    datasources such as Gnocchi."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"52ad6b01_2f2bb2a0","line":6,"range":{"start_line":4,"start_character":0,"end_line":6,"end_character":32},"updated":"2024-12-17 15:17:52.000000000","message":"Maybe clarify to users that ceilometer can still be used with gnocchi as backend?","commit_id":"da23fdc621bfd3211da4946ac5bdc017fb867c5a"}],"watcher/tests/decision_engine/planner/test_workload_stabilization_planner.py":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e412882c0e1c505e3290027769d59d8a756b6ac1","unresolved":true,"context_lines":[{"line_number":27,"context_line":"from watcher.tests.db import base"},{"line_number":28,"context_line":"from watcher.tests.db import utils as db_utils"},{"line_number":29,"context_line":"from watcher.tests.decision_engine.model import gnocchi_metrics as fake"},{"line_number":30,"context_line":"from watcher.tests.decision_engine.model import faker_cluster_state"},{"line_number":31,"context_line":"from watcher.tests.objects import utils as obj_utils"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"dd3d3646_5197034e","line":30,"in_reply_to":"58392e1c_f3fea491","updated":"2024-12-16 07:45:24.000000000","message":"\u003e pep8: H306: imports not in alphabetical order (watcher.tests.decision_engine.model.gnocchi_metrics, watcher.tests.decision_engine.model.faker_cluster_state)\n\nPlease fix.","commit_id":"d45902fcd43aafbc8285779d2259d5ad8ee77f3e"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"28d9fc13688cf1f376387f6be720496d69b0d7de","unresolved":false,"context_lines":[{"line_number":27,"context_line":"from watcher.tests.db import base"},{"line_number":28,"context_line":"from watcher.tests.db import utils as db_utils"},{"line_number":29,"context_line":"from watcher.tests.decision_engine.model import gnocchi_metrics as fake"},{"line_number":30,"context_line":"from watcher.tests.decision_engine.model import faker_cluster_state"},{"line_number":31,"context_line":"from watcher.tests.objects import utils as obj_utils"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"b8dce10b_cc31fc7b","line":30,"in_reply_to":"dd3d3646_5197034e","updated":"2024-12-16 13:08:28.000000000","message":"Done","commit_id":"d45902fcd43aafbc8285779d2259d5ad8ee77f3e"}]}