)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"b658edd409f1882b93394506c3e35a0d1b579e8d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"50dffc7b_a520b877","updated":"2026-07-07 14:37:54.000000000","message":"Let\u0027s type it correctly and ensure that it\u0027s always set. Doing it as a None like that let\u0027s people get away with not setting it. This way it\u0027s enforced properly.","commit_id":"ae0e1e0c85fa53f0c551dcb6e7583e6967d35c81"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"b45067f3e209ae29412506f048d5d04609bf80bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"27d61fe5_b7997b6a","in_reply_to":"50dffc7b_a520b877","updated":"2026-07-07 14:45:09.000000000","message":"Acknowledged","commit_id":"ae0e1e0c85fa53f0c551dcb6e7583e6967d35c81"}],"ironic/console/container/base.py":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"b658edd409f1882b93394506c3e35a0d1b579e8d","unresolved":true,"context_lines":[{"line_number":32,"context_line":"    \"\"\"Base class for console container provider APIs.\"\"\""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    # Provider name used in error messages, to be set by subclasses"},{"line_number":35,"context_line":"    provider_name \u003d None"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    @abc.abstractmethod"},{"line_number":38,"context_line":"    def start_container(self, task, app_name, app_info):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a6887d1_a86cb192","line":35,"updated":"2026-07-07 14:37:54.000000000","message":"```suggestion\n    @property\n    @abc.abstractmethod\n    def provider_name(self) -\u003e str:\n    \"\"\"Provider name used in error messages\"\"\"\n    ...\n```","commit_id":"ae0e1e0c85fa53f0c551dcb6e7583e6967d35c81"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"25c6ac067e7c9f7c9c32c5b0de2cae1614f02493","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    \"\"\"Base class for console container provider APIs.\"\"\""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    # Provider name used in error messages, to be set by subclasses"},{"line_number":35,"context_line":"    provider_name \u003d None"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    @abc.abstractmethod"},{"line_number":38,"context_line":"    def start_container(self, task, app_name, app_info):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3e7b6d15_aaeed1f0","line":35,"in_reply_to":"7a6887d1_a86cb192","updated":"2026-07-07 14:56:43.000000000","message":"Done","commit_id":"ae0e1e0c85fa53f0c551dcb6e7583e6967d35c81"}]}