)]}'
{".zuul.yaml":[{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"3672cb0b74e6aca9980566eb234fec86c85c1ffb","unresolved":true,"context_lines":[{"line_number":52,"context_line":"        test-config:"},{"line_number":53,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":54,"context_line":"            service_available:"},{"line_number":55,"context_line":"              sg-core: True"},{"line_number":56,"context_line":"            telemetry_services:"},{"line_number":57,"context_line":"              metric_backends: gnocchi,prometheus"},{"line_number":58,"context_line":"            telemetry:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d15d6acb_b0f12dfa","line":55,"updated":"2024-02-20 18:07:00.000000000","message":"IINM, this should be sg_core","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"3aa1af45397ce89f8e7f1666961df5b21dcae61b","unresolved":true,"context_lines":[{"line_number":52,"context_line":"        test-config:"},{"line_number":53,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":54,"context_line":"            service_available:"},{"line_number":55,"context_line":"              sg-core: True"},{"line_number":56,"context_line":"            telemetry_services:"},{"line_number":57,"context_line":"              metric_backends: gnocchi,prometheus"},{"line_number":58,"context_line":"            telemetry:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d37fc6f8_ec9e9459","line":55,"in_reply_to":"d15d6acb_b0f12dfa","updated":"2024-02-21 06:00:53.000000000","message":"Nice find. I fixed this in https://review.opendev.org/c/openstack/telemetry-tempest-plugin/+/908904 but I didn\u0027t pull this change when creating this. I\u0027ll rebase this.","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"197541cf85d8b9742bef46f7fe3b5516e5187699","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        test-config:"},{"line_number":53,"context_line":"          $TEMPEST_CONFIG:"},{"line_number":54,"context_line":"            service_available:"},{"line_number":55,"context_line":"              sg-core: True"},{"line_number":56,"context_line":"            telemetry_services:"},{"line_number":57,"context_line":"              metric_backends: gnocchi,prometheus"},{"line_number":58,"context_line":"            telemetry:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"14d76805_2c123ea5","line":55,"in_reply_to":"d37fc6f8_ec9e9459","updated":"2024-02-21 14:29:53.000000000","message":"Done","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"197541cf85d8b9742bef46f7fe3b5516e5187699","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"82f60fb5_480c5089","updated":"2024-02-21 14:29:53.000000000","message":"The telemetry-operator-tempest CI job hit some issues.","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"6ae1db28f940df9c78d7b7189eb41af3ca8eb4d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4d0f806c_569cd19a","in_reply_to":"82f60fb5_480c5089","updated":"2024-02-21 14:30:27.000000000","message":"I\u0027m working on resolving them, and we\u0027ll hopefully have a test run shortly","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"0df295dddff82539d845f36c8f912fb923071248","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1dd40a16_3d9f75c7","updated":"2024-02-21 17:08:13.000000000","message":"These tests run on openstack-k8s-operators:\nhttps://review.rdoproject.org/zuul/buildset/3a8c11d5761c46db8be781fe7adae568","commit_id":"1c74e7831f57caf6911179469f874341d5a94b7a"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"86e6cf0b2061dcb9a3347d9dea15c7d15460c4cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d2420059_d776ea0f","updated":"2024-02-22 15:11:52.000000000","message":"recheck - status 500 on create stack in autoscaling test.","commit_id":"6b29023d343eb368afd7c75e5f6ebd0ef34787ee"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"e7f2929f9381b086f6fa414e343d623961c050cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f78e76a6_bc160f2e","updated":"2024-03-06 19:04:05.000000000","message":"check rdo experimental","commit_id":"95b65c2d5056837deae2417b539d2efe83eaadab"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"8562daf1a0676b2c7ff23a7dc313237811e51ab3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"223a8abc_ed75e66c","updated":"2024-03-08 10:43:47.000000000","message":"recheck now that gate is unblocked.","commit_id":"95b65c2d5056837deae2417b539d2efe83eaadab"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"6d24d0799bc0273b46567fb7b767067c6c6711d8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"52d788fb_5410180d","in_reply_to":"f78e76a6_bc160f2e","updated":"2024-03-07 14:45:58.000000000","message":"I figured out why this didn\u0027t work. I\u0027ll have to do some more updates.\n\nRDO has a mirror of telemetry-tempest-plugin. I added the jobs to the mirror, not to the opendev/openstack/telemetry-tempest-plugin","commit_id":"95b65c2d5056837deae2417b539d2efe83eaadab"}],"telemetry_tempest_plugin/config.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"204f8bf0358a9943383fba65a836dffa722536c6","unresolved":true,"context_lines":[{"line_number":83,"context_line":"    cfg.StrOpt(\u0027sg_core_service_url\u0027,"},{"line_number":84,"context_line":"               default\u003d\"127.0.0.1:3000\","},{"line_number":85,"context_line":"               help\u003d\"URL to sg-core prometheus endpoint\"),"},{"line_number":86,"context_line":"    cfg.IntOpt(\u0027ceilometer_pipeline_interval\u0027,"},{"line_number":87,"context_line":"               default\u003d300,"},{"line_number":88,"context_line":"               help\u003d\"Polling interval configured for ceilometer. This can \""},{"line_number":89,"context_line":"                    \"be used in test cases to wait for metrics to appear.\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"49da429d_31c4ff6c","line":86,"range":{"start_line":86,"start_character":16,"end_line":86,"end_character":44},"updated":"2024-02-20 08:12:50.000000000","message":"I think *polling*_interval is better than pipeline_interval, because it\u0027s not clear how interval is related to pipeline (while its relation to polling is quite clear)","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"409d714920fa2bdd0ac92525e32815525b24ac2b","unresolved":true,"context_lines":[{"line_number":83,"context_line":"    cfg.StrOpt(\u0027sg_core_service_url\u0027,"},{"line_number":84,"context_line":"               default\u003d\"127.0.0.1:3000\","},{"line_number":85,"context_line":"               help\u003d\"URL to sg-core prometheus endpoint\"),"},{"line_number":86,"context_line":"    cfg.IntOpt(\u0027ceilometer_pipeline_interval\u0027,"},{"line_number":87,"context_line":"               default\u003d300,"},{"line_number":88,"context_line":"               help\u003d\"Polling interval configured for ceilometer. This can \""},{"line_number":89,"context_line":"                    \"be used in test cases to wait for metrics to appear.\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"476b7d22_fbce2a13","line":86,"range":{"start_line":86,"start_character":16,"end_line":86,"end_character":44},"in_reply_to":"30970b00_a30ad9db","updated":"2024-02-21 07:34:43.000000000","message":"IMO devstack is used by developers while this plugin may be used by operators.\n\nAlthough having different names might be consuming for developers, I still think that we should make the naming of the plugin option more friendly for operators and based on this point I\u0027d say polling is still better.","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"53a058c305377ed4699fb8bb258b1a0f25870422","unresolved":true,"context_lines":[{"line_number":83,"context_line":"    cfg.StrOpt(\u0027sg_core_service_url\u0027,"},{"line_number":84,"context_line":"               default\u003d\"127.0.0.1:3000\","},{"line_number":85,"context_line":"               help\u003d\"URL to sg-core prometheus endpoint\"),"},{"line_number":86,"context_line":"    cfg.IntOpt(\u0027ceilometer_pipeline_interval\u0027,"},{"line_number":87,"context_line":"               default\u003d300,"},{"line_number":88,"context_line":"               help\u003d\"Polling interval configured for ceilometer. This can \""},{"line_number":89,"context_line":"                    \"be used in test cases to wait for metrics to appear.\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"a8d4b53f_f08494b0","line":86,"range":{"start_line":86,"start_character":16,"end_line":86,"end_character":44},"in_reply_to":"476b7d22_fbce2a13","updated":"2024-02-22 13:56:17.000000000","message":"I changed it to polling","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"2ba4a07e435404cb1ff77933f11cd110c390763e","unresolved":true,"context_lines":[{"line_number":83,"context_line":"    cfg.StrOpt(\u0027sg_core_service_url\u0027,"},{"line_number":84,"context_line":"               default\u003d\"127.0.0.1:3000\","},{"line_number":85,"context_line":"               help\u003d\"URL to sg-core prometheus endpoint\"),"},{"line_number":86,"context_line":"    cfg.IntOpt(\u0027ceilometer_pipeline_interval\u0027,"},{"line_number":87,"context_line":"               default\u003d300,"},{"line_number":88,"context_line":"               help\u003d\"Polling interval configured for ceilometer. This can \""},{"line_number":89,"context_line":"                    \"be used in test cases to wait for metrics to appear.\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"30970b00_a30ad9db","line":86,"range":{"start_line":86,"start_character":16,"end_line":86,"end_character":44},"in_reply_to":"49da429d_31c4ff6c","updated":"2024-02-20 08:42:32.000000000","message":"\"ceilometer_polling_interval\" was my initial implementation. Then I noticed the ceilometer devstack plugin uses \"CEILOMETER_PIPELINE_INTERVAL\". The goal is for this value to be the same as: https://opendev.org/openstack/telemetry-tempest-plugin/src/commit/9621ee95718cc3be977d57b5ed8d00c0c46553fe/.zuul.yaml#L48\n\nWhile I personally like \"ceilometer_polling_interval\" a little better too, naming the value here differently from the ceilometer devstack plugin would probably be more confusing.","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"3672cb0b74e6aca9980566eb234fec86c85c1ffb","unresolved":true,"context_lines":[{"line_number":92,"context_line":"telemetry_services_opts \u003d ["},{"line_number":93,"context_line":"    cfg.ListOpt(\u0027metric_backends\u0027,"},{"line_number":94,"context_line":"                default\u003d[],"},{"line_number":95,"context_line":"                item_type\u003dtypes.String(choices\u003d[\u0027gnocchi\u0027, \u0027prometheus\u0027]),"},{"line_number":96,"context_line":"                help\u003d\"Backend store used to store metrics\"),"},{"line_number":97,"context_line":"    cfg.StrOpt(\u0027alarm_backend\u0027,"},{"line_number":98,"context_line":"               default\u003d\u0027mysql\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"18ed47d0_c950682f","line":95,"range":{"start_line":95,"start_character":14,"end_line":95,"end_character":74},"updated":"2024-02-20 18:07:00.000000000","message":"I\u0027m starting to think I should set this in the telemetry-operator job [1].\nIs this necessary? I don\u0027t think there are particular tests that rely on prometheus as the backend that we\u0027re currently skipping by not having this set.\n\n[1] https://github.com/openstack-k8s-operators/telemetry-operator/pull/284","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"3aa1af45397ce89f8e7f1666961df5b21dcae61b","unresolved":true,"context_lines":[{"line_number":92,"context_line":"telemetry_services_opts \u003d ["},{"line_number":93,"context_line":"    cfg.ListOpt(\u0027metric_backends\u0027,"},{"line_number":94,"context_line":"                default\u003d[],"},{"line_number":95,"context_line":"                item_type\u003dtypes.String(choices\u003d[\u0027gnocchi\u0027, \u0027prometheus\u0027]),"},{"line_number":96,"context_line":"                help\u003d\"Backend store used to store metrics\"),"},{"line_number":97,"context_line":"    cfg.StrOpt(\u0027alarm_backend\u0027,"},{"line_number":98,"context_line":"               default\u003d\u0027mysql\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5b76e939_9c4501f7","line":95,"range":{"start_line":95,"start_character":14,"end_line":95,"end_character":74},"in_reply_to":"18ed47d0_c950682f","updated":"2024-02-21 06:00:53.000000000","message":"There aren\u0027t any which require prometheus yet. I assume we should require the prometheus backend once we add the autoscaling tests.","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"},{"author":{"_account_id":13177,"name":"Emma Foley","email":"efoley@redhat.com","username":"emma-l-foley"},"change_message_id":"197541cf85d8b9742bef46f7fe3b5516e5187699","unresolved":true,"context_lines":[{"line_number":92,"context_line":"telemetry_services_opts \u003d ["},{"line_number":93,"context_line":"    cfg.ListOpt(\u0027metric_backends\u0027,"},{"line_number":94,"context_line":"                default\u003d[],"},{"line_number":95,"context_line":"                item_type\u003dtypes.String(choices\u003d[\u0027gnocchi\u0027, \u0027prometheus\u0027]),"},{"line_number":96,"context_line":"                help\u003d\"Backend store used to store metrics\"),"},{"line_number":97,"context_line":"    cfg.StrOpt(\u0027alarm_backend\u0027,"},{"line_number":98,"context_line":"               default\u003d\u0027mysql\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"fabbcc9a_90f40bd9","line":95,"range":{"start_line":95,"start_character":14,"end_line":95,"end_character":74},"in_reply_to":"5b76e939_9c4501f7","updated":"2024-02-21 14:29:53.000000000","message":"I\u0027ll update that in the telemetry-operator PR, so that we don\u0027t have to worry about this later","commit_id":"9621ee95718cc3be977d57b5ed8d00c0c46553fe"}]}