)]}'
{".zuul.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"724a99b01cfe7cb6ebaf7a357e42ea064bb5fed2","unresolved":true,"context_lines":[{"line_number":61,"context_line":"    parent: telemetry-tempest-base"},{"line_number":62,"context_line":"    override-checkout: stable/2024.2"},{"line_number":63,"context_line":"    vars: \u0026prefix_grouping_vars"},{"line_number":64,"context_line":"      devstack_plugins:"},{"line_number":65,"context_line":"        heat: https://opendev.org/openstack/heat"},{"line_number":66,"context_line":"        ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":67,"context_line":"        aodh: https://opendev.org/openstack/aodh"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"00d305db_2850124a","line":64,"range":{"start_line":64,"start_character":6,"end_line":64,"end_character":22},"updated":"2024-10-23 10:12:32.000000000","message":"zuul do some deep-merge magic when you override vars, so you can omit sections which don\u0027t have actual override. I think in your case all you need is devstack_local_conf .","commit_id":"2dc5eb1f44e2a35eaf65268dd9744425faa43655"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ad93d096240c26a11e5382825db0c29922d3169f","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    parent: telemetry-tempest-base"},{"line_number":62,"context_line":"    override-checkout: stable/2024.2"},{"line_number":63,"context_line":"    vars: \u0026prefix_grouping_vars"},{"line_number":64,"context_line":"      devstack_plugins:"},{"line_number":65,"context_line":"        heat: https://opendev.org/openstack/heat"},{"line_number":66,"context_line":"        ceilometer: https://opendev.org/openstack/ceilometer"},{"line_number":67,"context_line":"        aodh: https://opendev.org/openstack/aodh"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"35030792_344a239d","line":64,"range":{"start_line":64,"start_character":6,"end_line":64,"end_character":22},"in_reply_to":"00d305db_2850124a","updated":"2024-10-23 15:17:49.000000000","message":"Done","commit_id":"2dc5eb1f44e2a35eaf65268dd9744425faa43655"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"724a99b01cfe7cb6ebaf7a357e42ea064bb5fed2","unresolved":true,"context_lines":[{"line_number":178,"context_line":"    vars: *no_prometheus_vars"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"- job:"},{"line_number":181,"context_line":"    name: telemetry-dsvm-integration-ipv6-only"},{"line_number":182,"context_line":"    parent: telemetry-dsvm-integration-ipv6-only-base"},{"line_number":183,"context_line":"    description: |"},{"line_number":184,"context_line":"      Telemetry devstack tempest tests job for IPv6-only deployment"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"12ba895c_0c92be1d","line":181,"range":{"start_line":181,"start_character":10,"end_line":181,"end_character":46},"updated":"2024-10-23 10:12:32.000000000","message":"Please add this before L170","commit_id":"2dc5eb1f44e2a35eaf65268dd9744425faa43655"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ad93d096240c26a11e5382825db0c29922d3169f","unresolved":false,"context_lines":[{"line_number":178,"context_line":"    vars: *no_prometheus_vars"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"- job:"},{"line_number":181,"context_line":"    name: telemetry-dsvm-integration-ipv6-only"},{"line_number":182,"context_line":"    parent: telemetry-dsvm-integration-ipv6-only-base"},{"line_number":183,"context_line":"    description: |"},{"line_number":184,"context_line":"      Telemetry devstack tempest tests job for IPv6-only deployment"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bdb5c82a_b89960b8","line":181,"range":{"start_line":181,"start_character":10,"end_line":181,"end_character":46},"in_reply_to":"12ba895c_0c92be1d","updated":"2024-10-23 15:17:49.000000000","message":"Done","commit_id":"2dc5eb1f44e2a35eaf65268dd9744425faa43655"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"1350456b2d3725d3bca3f8ee3401b5dd4f6cba30","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7ee93a88_a2df837f","updated":"2024-10-16 18:52:02.000000000","message":"Testing the stable branch CI in: https://review.opendev.org/c/openstack/aodh/+/932550","commit_id":"d2e2ba4c6815f12a5bbd84d5921e09aa2188e315"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"55b7a96e18eacb5d407ac1e6d8f50afe0bff0a98","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"84e3badd_fa8b1a6e","updated":"2024-10-17 14:17:25.000000000","message":"Because master of tempest plugin should support all stable branches, we should add the job which runs latest tempest/plugin against stable branch.\nAlso, there are a few other options you have to override to disable sg-core.\n\nSee https://review.opendev.org/c/openstack/telemetry-tempest-plugin/+/932310 and https://review.opendev.org/c/openstack/telemetry-tempest-plugin/+/932615","commit_id":"7b3e72030f85de5e7f825020d86513d2e82116b8"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"9aa12dad52a19addc37e7874f56ee8f0ea9bb11e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b831ac1d_16161c5b","updated":"2024-10-17 08:39:30.000000000","message":"I discovered, that the CI is failing on stable/2023.* because they don\u0027t support prometheus based alarms. I included a job variant for these branches, which disables testing on them. I created a new testing aodh patch on a stable/2024.1 branch. So now the testing is done like this:\n\nTesting that prometheus based autoscaling is disabled on 2023 branches: https://review.opendev.org/c/openstack/aodh/+/932550\nTesting that prefix autoscaling instance grouping is used on 2024 branches: https://review.opendev.org/c/openstack/aodh/+/932590","commit_id":"7b3e72030f85de5e7f825020d86513d2e82116b8"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"9b08d367c1ee1b58aa0b746a50884af61f3ba20a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"54fd5dad_af104b79","in_reply_to":"84e3badd_fa8b1a6e","updated":"2024-10-17 15:50:17.000000000","message":"What I\u0027ve done is actually enough to not execute the prometheus tests and so the CI passes on stable/2023.* as can be seen here: https://review.opendev.org/c/openstack/aodh/+/932550\n\nI see, that we\u0027re doing some similar things and I like your patches. I could rebase this on top of your changes and just have the \"autoscaling_instance_grouping: prefix\" for 2024.* branches. Does that sound like a good idea?","commit_id":"7b3e72030f85de5e7f825020d86513d2e82116b8"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"08a34051b63af9f88a3b0f88158ca36a88830c23","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a275db72_72c0f2d5","updated":"2024-10-23 16:36:43.000000000","message":"Thanks Takashi for implementing your suggestions, I was about to do that. The Zuul quickly looking at the job failure, it seems like something I saw and fixed before. I\u0027ll try to take a deeper look just in case it\u0027s something we should address.","commit_id":"95345f8b26490ed3e8972f0b90b17c819ccca87f"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"e7803ac3b69dc70ce7065c827970e3a2824ed86c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"70750996_f7795f3b","updated":"2024-10-23 16:42:58.000000000","message":"Yep. Based on the aodh api logs. It looks like this: https://review.opendev.org/c/openstack/keystoneauth/+/900530 . I wonder if we should backport that keystoneauth change. WDYT?","commit_id":"95345f8b26490ed3e8972f0b90b17c819ccca87f"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"a6475d4d8401cd58f94f415f2e2d5885f94cc3d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"cf85ea91_c9d78f5a","updated":"2024-11-05 13:30:14.000000000","message":"recheck","commit_id":"95345f8b26490ed3e8972f0b90b17c819ccca87f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ec89331d1790465ef0e6be80c648ede6b10b0c0c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b0401539_a09c8356","updated":"2024-11-13 01:14:47.000000000","message":"@Jamior Could you please propose a follow-up to add the release note describing the change ?","commit_id":"753a8b8b97057e47f0ff3b6a7f6743b1993168be"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"40536b2cd416863c1d3665985b511f9715f5038d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"be728dcb_aa5570a9","in_reply_to":"b0401539_a09c8356","updated":"2024-11-13 21:26:03.000000000","message":"Sure","commit_id":"753a8b8b97057e47f0ff3b6a7f6743b1993168be"}],"telemetry_tempest_plugin/config.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"3def1872fea62fce1571247a5b30ec1424022268","unresolved":true,"context_lines":[{"line_number":103,"context_line":"               help\u003d\"How to group instances for autoscaling testing. \""},{"line_number":104,"context_line":"                    \"\u0027prefix\u0027 relies on the instances having a common string \""},{"line_number":105,"context_line":"                    \"at the start of their name. \u0027metadata\u0027 is a new and \""},{"line_number":106,"context_line":"                    \"prefered way of grouping since 2024.2 relying on \""},{"line_number":107,"context_line":"                    \"metering.server_group instance metadata\")"},{"line_number":108,"context_line":"]"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"telemetry_services_opts \u003d ["}],"source_content_type":"text/x-python","patch_set":4,"id":"6282d179_99042973","line":107,"range":{"start_line":106,"start_character":0,"end_line":107,"end_character":62},"updated":"2024-10-23 10:25:43.000000000","message":"This is not directly related to this change, but I noticed that autoscaling_instance_grouping is used only in prometheus tests. If this is specific to prometheus then we should mention it.","commit_id":"2dc5eb1f44e2a35eaf65268dd9744425faa43655"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ad93d096240c26a11e5382825db0c29922d3169f","unresolved":false,"context_lines":[{"line_number":103,"context_line":"               help\u003d\"How to group instances for autoscaling testing. \""},{"line_number":104,"context_line":"                    \"\u0027prefix\u0027 relies on the instances having a common string \""},{"line_number":105,"context_line":"                    \"at the start of their name. \u0027metadata\u0027 is a new and \""},{"line_number":106,"context_line":"                    \"prefered way of grouping since 2024.2 relying on \""},{"line_number":107,"context_line":"                    \"metering.server_group instance metadata\")"},{"line_number":108,"context_line":"]"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"telemetry_services_opts \u003d ["}],"source_content_type":"text/x-python","patch_set":4,"id":"81800146_9b039213","line":107,"range":{"start_line":106,"start_character":0,"end_line":107,"end_character":62},"in_reply_to":"6282d179_99042973","updated":"2024-10-23 15:17:49.000000000","message":"Done","commit_id":"2dc5eb1f44e2a35eaf65268dd9744425faa43655"}]}