)]}'
{".zuul.yaml":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"2c20663d849861ce2fa999a05f01bc7bb40aa668","unresolved":true,"context_lines":[{"line_number":33,"context_line":"      # We will need to figure out how we refactor them to be stable in the CI."},{"line_number":34,"context_line":"      tempest_exclude_regex: (^telemetry_tempest_plugin\\.scenario\\.test_telemetry_integration\\.)"},{"line_number":35,"context_line":"      devstack_services:"},{"line_number":36,"context_line":"        tempest: true"},{"line_number":37,"context_line":"      devstack_localrc:"},{"line_number":38,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/telemetry-tempest-plugin /opt/stack/heat-tempest-plugin\"\u0027"},{"line_number":39,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-high-rate\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"59d028c1_1abc01eb","line":36,"updated":"2025-03-25 07:54:42.000000000","message":"devstack-plugin-prometheus also enables node_exporter service by default. https://opendev.org/openstack/devstack-plugin-prometheus/src/branch/master/devstack/settings#L4\n\nhttps://7ae9da1647ae1519b362-ee4c5d809145a8a3246cc4d26d65fbe0.ssl.cf1.rackcdn.com/945108/2/check/telemetry-dsvm-integration/45b2308/controller/logs/screen-node-exporter.txt\n\nUnder devstack_services, we can add node_exporter: false to disable it.","commit_id":"61cbe7f2b64017111fb0f6378194136a730186aa"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1301968713c322e4eed22a9553e179d701a77691","unresolved":false,"context_lines":[{"line_number":33,"context_line":"      # We will need to figure out how we refactor them to be stable in the CI."},{"line_number":34,"context_line":"      tempest_exclude_regex: (^telemetry_tempest_plugin\\.scenario\\.test_telemetry_integration\\.)"},{"line_number":35,"context_line":"      devstack_services:"},{"line_number":36,"context_line":"        tempest: true"},{"line_number":37,"context_line":"      devstack_localrc:"},{"line_number":38,"context_line":"        TEMPEST_PLUGINS: \u0027\"/opt/stack/telemetry-tempest-plugin /opt/stack/heat-tempest-plugin\"\u0027"},{"line_number":39,"context_line":"        GNOCCHI_ARCHIVE_POLICY_TEMPEST: \"ceilometer-high-rate\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"4b8e619d_3ea5aef0","line":36,"in_reply_to":"59d028c1_1abc01eb","updated":"2025-04-02 12:42:49.000000000","message":"Done","commit_id":"61cbe7f2b64017111fb0f6378194136a730186aa"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"2c20663d849861ce2fa999a05f01bc7bb40aa668","unresolved":true,"context_lines":[{"line_number":42,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":43,"context_line":"        AODH_DEPLOY: uwsgi"},{"line_number":44,"context_line":"        PROMETHEUS_CONF_DIR: /opt/stack/"},{"line_number":45,"context_line":"        PROMETHEUS_CONFIG_FILE: /opt/stack/prometheus.yml"},{"line_number":46,"context_line":"      devstack_local_conf:"},{"line_number":47,"context_line":"        test-config:"},{"line_number":48,"context_line":"          $TEMPEST_CONFIG:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"2a4ee31e_3721bb95","line":45,"updated":"2025-03-25 07:54:42.000000000","message":"We can also add following lines to collect prometheus config file. It will be helpful for debugging.\n```\n      zuul_copy_output:\n        /etc/prometheus/prometheus.yml: logs\n```","commit_id":"61cbe7f2b64017111fb0f6378194136a730186aa"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1301968713c322e4eed22a9553e179d701a77691","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        CEILOMETER_ALARM_THRESHOLD: 6000000000"},{"line_number":43,"context_line":"        AODH_DEPLOY: uwsgi"},{"line_number":44,"context_line":"        PROMETHEUS_CONF_DIR: /opt/stack/"},{"line_number":45,"context_line":"        PROMETHEUS_CONFIG_FILE: /opt/stack/prometheus.yml"},{"line_number":46,"context_line":"      devstack_local_conf:"},{"line_number":47,"context_line":"        test-config:"},{"line_number":48,"context_line":"          $TEMPEST_CONFIG:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ff05ceae_3cd5764b","line":45,"in_reply_to":"2a4ee31e_3721bb95","updated":"2025-04-02 12:42:49.000000000","message":"Done","commit_id":"61cbe7f2b64017111fb0f6378194136a730186aa"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1301968713c322e4eed22a9553e179d701a77691","unresolved":true,"context_lines":[{"line_number":27,"context_line":"        heat: https://opendev.org/openstack/heat"},{"line_number":28,"context_line":"        ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":29,"context_line":"        aodh: https://opendev.org/openstack/aodh"},{"line_number":30,"context_line":"        sg-core: https://github.com/paramite/sg-core # for test purposes"},{"line_number":31,"context_line":"        devstack-plugin-prometheus: https://opendev.org/openstack/devstack-plugin-prometheus"},{"line_number":32,"context_line":"      # NOTE(jokke): The following will disable the gabbi based integration tests for now."},{"line_number":33,"context_line":"      # We will need to figure out how we refactor them to be stable in the CI."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"ec5b30b7_da7bd7b3","line":30,"updated":"2025-04-02 12:42:49.000000000","message":"Can we drop this one and switch back to openstack-k8s-operators/sg-core repo?","commit_id":"0e832bc320abb4c30839046854d34d4a62e2d292"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"6b17e33e2392e021c8381619ae925ffef04055eb","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        heat: https://opendev.org/openstack/heat"},{"line_number":28,"context_line":"        ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":29,"context_line":"        aodh: https://opendev.org/openstack/aodh"},{"line_number":30,"context_line":"        sg-core: https://github.com/paramite/sg-core # for test purposes"},{"line_number":31,"context_line":"        devstack-plugin-prometheus: https://opendev.org/openstack/devstack-plugin-prometheus"},{"line_number":32,"context_line":"      # NOTE(jokke): The following will disable the gabbi based integration tests for now."},{"line_number":33,"context_line":"      # We will need to figure out how we refactor them to be stable in the CI."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"8ecd2b81_a9c6d6bd","line":30,"in_reply_to":"93625d0a_2250f9b4","updated":"2025-04-03 12:44:09.000000000","message":"Done","commit_id":"0e832bc320abb4c30839046854d34d4a62e2d292"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"a0bea2fb2fad5b2df334aeedaf943744b38066a3","unresolved":true,"context_lines":[{"line_number":27,"context_line":"        heat: https://opendev.org/openstack/heat"},{"line_number":28,"context_line":"        ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":29,"context_line":"        aodh: https://opendev.org/openstack/aodh"},{"line_number":30,"context_line":"        sg-core: https://github.com/paramite/sg-core # for test purposes"},{"line_number":31,"context_line":"        devstack-plugin-prometheus: https://opendev.org/openstack/devstack-plugin-prometheus"},{"line_number":32,"context_line":"      # NOTE(jokke): The following will disable the gabbi based integration tests for now."},{"line_number":33,"context_line":"      # We will need to figure out how we refactor them to be stable in the CI."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"93625d0a_2250f9b4","line":30,"in_reply_to":"ec5b30b7_da7bd7b3","updated":"2025-04-03 11:40:04.000000000","message":"Sure, will do. I wanted to make sure correct code is being taken into account since I had a bit trouble finding an issue.","commit_id":"0e832bc320abb4c30839046854d34d4a62e2d292"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"44aebeb0679b0dc96e6f80cf2220ca92559481e2","unresolved":true,"context_lines":[{"line_number":59,"context_line":"      tox_envlist: all"},{"line_number":60,"context_line":"      zuul_copy_output:"},{"line_number":61,"context_line":"        /etc/prometheus/prometheus.yml: logs"},{"line_number":62,"context_line":"        /etc/openstack/prometheus.yaml: logs"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"- job:"},{"line_number":65,"context_line":"    name: telemetry-dsvm-integration"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"8381721d_8bd2f440","line":62,"updated":"2025-04-02 12:43:22.000000000","message":"we can also drop `/etc/openstack/prometheus.yaml: logs` it will create confusion.","commit_id":"0e832bc320abb4c30839046854d34d4a62e2d292"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"6b17e33e2392e021c8381619ae925ffef04055eb","unresolved":false,"context_lines":[{"line_number":59,"context_line":"      tox_envlist: all"},{"line_number":60,"context_line":"      zuul_copy_output:"},{"line_number":61,"context_line":"        /etc/prometheus/prometheus.yml: logs"},{"line_number":62,"context_line":"        /etc/openstack/prometheus.yaml: logs"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"- job:"},{"line_number":65,"context_line":"    name: telemetry-dsvm-integration"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"07c585d5_705b1fed","line":62,"in_reply_to":"80c2feb9_092b140a","updated":"2025-04-03 12:44:09.000000000","message":"Done","commit_id":"0e832bc320abb4c30839046854d34d4a62e2d292"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"a0bea2fb2fad5b2df334aeedaf943744b38066a3","unresolved":true,"context_lines":[{"line_number":59,"context_line":"      tox_envlist: all"},{"line_number":60,"context_line":"      zuul_copy_output:"},{"line_number":61,"context_line":"        /etc/prometheus/prometheus.yml: logs"},{"line_number":62,"context_line":"        /etc/openstack/prometheus.yaml: logs"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"- job:"},{"line_number":65,"context_line":"    name: telemetry-dsvm-integration"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"80c2feb9_092b140a","line":62,"in_reply_to":"8381721d_8bd2f440","updated":"2025-04-03 11:40:04.000000000","message":"Well we can\u0027t. This is important configuration for observabilityclient and we need to make sure it is created correctly.","commit_id":"0e832bc320abb4c30839046854d34d4a62e2d292"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"2c20663d849861ce2fa999a05f01bc7bb40aa668","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dd1230ae_27200b46","updated":"2025-03-25 07:54:42.000000000","message":"Thank you for adding  me! Overall looks good, Left few comments.","commit_id":"61cbe7f2b64017111fb0f6378194136a730186aa"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"d152d9b4ea4d1fcacc7009aaab4fb4ab815072d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"747dd263_2d5108dc","updated":"2025-03-25 11:03:40.000000000","message":"recheck","commit_id":"61cbe7f2b64017111fb0f6378194136a730186aa"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"5b4a65373b4d337d6ad929baa8a28ad6276fb9b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"010f730a_012ef5b7","updated":"2025-03-25 12:56:39.000000000","message":"recheck","commit_id":"0a433d2a6d670ed39b3f50003f7fcd5d85aab589"},{"author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"change_message_id":"b545690689e0a317fdbf400a74e41e0b17786343","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d2ae6c16_13874f74","updated":"2025-03-26 16:40:18.000000000","message":"We need to fix the testing here. This fails due to setuptools requirements conflicts\n \n      pkg_resources.VersionConflict: (setuptools 59.6.0 (/usr/lib/python3/dist-packages), Requirement.parse(\u0027setuptools\u003e\u003d61\u0027))","commit_id":"ac6127db456ea5632de73b1ff6923b4599f1397a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1301968713c322e4eed22a9553e179d701a77691","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"1b154788_71512a6b","updated":"2025-04-02 12:42:49.000000000","message":"Overall it looks good.\nhttps://ed106298e21a2f3ff245-78c6b016591aee5bdb634ced0dc80e36.ssl.cf2.rackcdn.com/openstack/42c2cc10ed5a4e7eb5d61875b45e91b9/controller/logs/screen-prometheus.txt\n```\nApr 02 11:42:33.071809 np0040353303 prometheus[117783]: time\u003d2025-04-02T11:42:33.071Z level\u003dINFO source\u003dmain.go:1482 msg\u003d\"Completed loading of configuration file\" db_storage\u003d2.064µs remote_storage\u003d2.515µs web_handler\u003d1.209µs query_engine\u003d1.774µs scrape\u003d627.144µs scrape_sd\u003d89.438µs notify\u003d2.551µs notify_sd\u003d1.953µs rules\u003d3.883µs tracing\u003d13.561µs filename\u003d/etc/prometheus/prometheus.yml totalDuration\u003d4.283468ms\n```\nand \nhttps://ed106298e21a2f3ff245-78c6b016591aee5bdb634ced0dc80e36.ssl.cf2.rackcdn.com/openstack/42c2cc10ed5a4e7eb5d61875b45e91b9/controller/logs/prometheus_yml.txt\n```\nglobal:\n  scrape_interval:     15s # By default, scrape targets every 15 seconds.\n\nscrape_configs:\n  - job_name: \u0027sg-core\u0027\n\n    static_configs:\n      - targets: [\u0027localhost:3000\u0027]\n```","commit_id":"0e832bc320abb4c30839046854d34d4a62e2d292"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"b61368c3e5548141f5e348e6421e39c4ea27a62e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"b654ecfb_32175dbf","updated":"2025-04-02 11:06:24.000000000","message":"recheck","commit_id":"0e832bc320abb4c30839046854d34d4a62e2d292"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"97c422249a269631d9a441b28453624a14640e87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"c293d66d_960370b4","updated":"2025-04-03 12:57:44.000000000","message":"Looking at this a bit more. I\u0027m not sure what you mean by the TODOs? Those are variant of jobs, which will be run when a change on a particular stable branch is proposed IINM. So we may need to define variants for newer stable branches and remove the ones that aren\u0027t supported anymore, but I don\u0027t know what \"switching to a newer version means in this case\"","commit_id":"25b070f7acd3eaa6bd929b57716cf7d9f57f48bf"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"6b17e33e2392e021c8381619ae925ffef04055eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"1d2260f3_16b69f97","updated":"2025-04-03 12:44:09.000000000","message":"Thanks @mmagr@redhat.com. :-)","commit_id":"25b070f7acd3eaa6bd929b57716cf7d9f57f48bf"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"5918259f5c80e72d60374dc3cbd02a1c2c01dc3e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"b7894567_e8238a17","updated":"2025-04-03 19:56:18.000000000","message":"recheck","commit_id":"25b070f7acd3eaa6bd929b57716cf7d9f57f48bf"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"5918259f5c80e72d60374dc3cbd02a1c2c01dc3e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"d94c5cef_c12cdcb2","in_reply_to":"c293d66d_960370b4","updated":"2025-04-03 19:56:18.000000000","message":"Creating jobs for supported versions and removing unsupported version was what I meant. But that\u0027s good point. Will. be more specific next time","commit_id":"25b070f7acd3eaa6bd929b57716cf7d9f57f48bf"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"d2f7a3de6a9a91adbc6f45e2d09e9b6dcd4af86c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"b4cb9731_4bd7762f","updated":"2025-04-04 08:14:53.000000000","message":"Let\u0027s get this merged as it seems to block telemetry jobs on all other patches.","commit_id":"6833c236ee1b13536d6a6eb9f20c00a52de595a6"},{"author":{"_account_id":5241,"name":"Martin Magr","email":"mmagr@redhat.com","username":"mmagr"},"change_message_id":"ff2f48af29b8e07add4ce2c7a7f3a48bc45e60ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"1425031e_02d3c695","updated":"2025-04-07 12:54:26.000000000","message":"We gonna need W+1 from somebody to merge this.","commit_id":"6833c236ee1b13536d6a6eb9f20c00a52de595a6"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"a9bf28ac63e4d3ac47b8d8a341654fd11afc2486","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"73906cef_70cfb954","updated":"2025-04-04 06:56:31.000000000","message":"recheck","commit_id":"6833c236ee1b13536d6a6eb9f20c00a52de595a6"}]}