)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"e54996a4ce8ab22bdc78afae6acc1009ba859d06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0d383611_3c504cc8","updated":"2026-09-30 07:46:25.000000000","message":"I think this is a very good idea. I remember myself wondering and being confused what\u0027s going on when seeing that error back in the days. Thank you.","commit_id":"46a75f045eeea45eb154264daecc1545562ee734"}],"telemetry_tempest_plugin/scenario/test_telemetry_integration.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"484faf51d5814c3b517823f66a5f01c1518051de","unresolved":false,"context_lines":[{"line_number":48,"context_line":"            # TODO(tkajinam): heat-tempest-plugin is required to load some"},{"line_number":49,"context_line":"            # options such as catalog_type. We should probably register"},{"line_number":50,"context_line":"            # the required options when the plugin is not loaded."},{"line_number":51,"context_line":"            import heat_tempest_plugin"},{"line_number":52,"context_line":"        except ImportError:"},{"line_number":53,"context_line":"            cls.skipException(\"heat-tempest-plugin is required\")"},{"line_number":54,"context_line":"        for name in [\"aodh\", \"gnocchi\", \"nova\", \"heat\","}],"source_content_type":"text/x-python","patch_set":1,"id":"7557afd9_bb14dd53","line":51,"updated":"2026-09-26 17:12:24.000000000","message":"pep8: F401 \u0027heat_tempest_plugin\u0027 imported but unused","commit_id":"2e4a8949d243c37a6b25b3d7f16a5a82829f1fcf"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"3fe199f3d349b92b4a529c6e60a9ece8a5568de6","unresolved":true,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    @classmethod"},{"line_number":58,"context_line":"    def _check_service(cls, name):"},{"line_number":59,"context_line":"        if not getattr(config.CONF.service_available, name):"},{"line_number":60,"context_line":"            raise cls.skipException(\"%s support is required\""},{"line_number":61,"context_line":"                                    % name.capitalize())"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5dd66103_f90b6e8b","line":59,"updated":"2026-09-29 09:54:58.000000000","message":"why remove the False as default?","commit_id":"46a75f045eeea45eb154264daecc1545562ee734"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"e54996a4ce8ab22bdc78afae6acc1009ba859d06","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    @classmethod"},{"line_number":58,"context_line":"    def _check_service(cls, name):"},{"line_number":59,"context_line":"        if not getattr(config.CONF.service_available, name):"},{"line_number":60,"context_line":"            raise cls.skipException(\"%s support is required\""},{"line_number":61,"context_line":"                                    % name.capitalize())"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"6f9b10d7_be5495a6","line":59,"in_reply_to":"30c9f11d_db13b85b","updated":"2026-09-30 07:46:25.000000000","message":"Acknowledged","commit_id":"46a75f045eeea45eb154264daecc1545562ee734"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"daed52c8a803e9c6347b7a5150e1f19d6eb52176","unresolved":true,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    @classmethod"},{"line_number":58,"context_line":"    def _check_service(cls, name):"},{"line_number":59,"context_line":"        if not getattr(config.CONF.service_available, name):"},{"line_number":60,"context_line":"            raise cls.skipException(\"%s support is required\""},{"line_number":61,"context_line":"                                    % name.capitalize())"},{"line_number":62,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"30c9f11d_db13b85b","line":59,"in_reply_to":"5dd66103_f90b6e8b","updated":"2026-09-29 15:25:15.000000000","message":"L50 already ensures that heat-tempest-plugin is installed, so the [service_available] heat option should be present.\nThe other options are from tempest and telemetry-tempest-plugin so should be always present.\n\nSo the fallback to False here is no longer useful and can hide missing options unexpectedly.","commit_id":"46a75f045eeea45eb154264daecc1545562ee734"}],"telemetry_tempest_plugin/scenario/test_telemetry_integration_prometheus.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"484faf51d5814c3b517823f66a5f01c1518051de","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            # TODO(tkajinam): heat-tempest-plugin is required to load some"},{"line_number":50,"context_line":"            # options such as catalog_type. We should probably register"},{"line_number":51,"context_line":"            # the required options when the plugin is not loaded."},{"line_number":52,"context_line":"            import heat_tempest_plugin"},{"line_number":53,"context_line":"        except ImportError:"},{"line_number":54,"context_line":"            cls.skipException(\"heat-tempest-plugin is required\")"},{"line_number":55,"context_line":"        for name in [\"aodh\", \"nova\", \"heat\","}],"source_content_type":"text/x-python","patch_set":1,"id":"fb8e0df9_d5046061","line":52,"updated":"2026-09-26 17:12:24.000000000","message":"pep8: F401 \u0027heat_tempest_plugin\u0027 imported but unused","commit_id":"2e4a8949d243c37a6b25b3d7f16a5a82829f1fcf"}]}
