)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6a42a30b29e35eb58256b6e548d06ca464b1e0e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8fa37ea7_a149426d","updated":"2025-09-25 10:06:24.000000000","message":"Looking good. One nit inline","commit_id":"a11e1c3a8770aa91faabf6f81f9328927af013c5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f35f9ffd6db1064d2d2f41e8f661c3d7ec1ee6bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8d62e910_f1ad1c83","updated":"2025-11-12 12:56:09.000000000","message":"Two further comments inline.\n\nCould you please also mark comments as resolved when you\u0027ve address them 🙏","commit_id":"1ec7301fbc43c0bcdf708f193455116d4c9f4bda"}],"openstackclient/tests/functional/image/v2/test_metadef_objects.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6a42a30b29e35eb58256b6e548d06ca464b1e0e0","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    def test_metadef_objects(self):"},{"line_number":29,"context_line":"        # CREATE"},{"line_number":30,"context_line":"        created \u003d self.openstack("},{"line_number":31,"context_line":"            f\"image metadef object create --namespace {self.ns_name} {self.obj_name} -f json\","},{"line_number":32,"context_line":"            parse_output\u003dTrue,"},{"line_number":33,"context_line":"        )"},{"line_number":34,"context_line":"        self.addCleanup("}],"source_content_type":"text/x-python","patch_set":2,"id":"63a4179c_1184fe9f","line":31,"range":{"start_line":31,"start_character":84,"end_line":31,"end_character":92},"updated":"2025-09-25 10:06:24.000000000","message":"`-f json` is redundant when you pass `parse_output\u003dTrue` since that appends exactly that to the command. Can you drop this here and below?","commit_id":"a11e1c3a8770aa91faabf6f81f9328927af013c5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f35f9ffd6db1064d2d2f41e8f661c3d7ec1ee6bc","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    def test_metadef_objects(self):"},{"line_number":29,"context_line":"        # CREATE"},{"line_number":30,"context_line":"        created \u003d self.openstack("},{"line_number":31,"context_line":"            f\"image metadef object create --namespace {self.ns_name} {self.obj_name} -f json\","},{"line_number":32,"context_line":"            parse_output\u003dTrue,"},{"line_number":33,"context_line":"        )"},{"line_number":34,"context_line":"        self.addCleanup("}],"source_content_type":"text/x-python","patch_set":2,"id":"6e0e00bb_7ba6c4c2","line":31,"range":{"start_line":31,"start_character":84,"end_line":31,"end_character":92},"in_reply_to":"63a4179c_1184fe9f","updated":"2025-11-12 12:56:09.000000000","message":"Done","commit_id":"a11e1c3a8770aa91faabf6f81f9328927af013c5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f35f9ffd6db1064d2d2f41e8f661c3d7ec1ee6bc","unresolved":true,"context_lines":[{"line_number":28,"context_line":"    def test_metadef_objects(self):"},{"line_number":29,"context_line":"        # CREATE"},{"line_number":30,"context_line":"        created \u003d self.openstack("},{"line_number":31,"context_line":"            f\"image metadef object create --namespace {self.ns_name} {self.obj_name}\","},{"line_number":32,"context_line":"            parse_output\u003dTrue,"},{"line_number":33,"context_line":"        )"},{"line_number":34,"context_line":"        self.addCleanup("}],"source_content_type":"text/x-python","patch_set":3,"id":"aeda9045_5717e489","line":31,"updated":"2025-11-12 12:56:09.000000000","message":"Could you wrap this at \u003c\u003d 79 characters?","commit_id":"1ec7301fbc43c0bcdf708f193455116d4c9f4bda"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"191ed74c1dba312f45bfae405e3c323538fb3ed5","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    def test_metadef_objects(self):"},{"line_number":29,"context_line":"        # CREATE"},{"line_number":30,"context_line":"        created \u003d self.openstack("},{"line_number":31,"context_line":"            f\"image metadef object create --namespace {self.ns_name} {self.obj_name}\","},{"line_number":32,"context_line":"            parse_output\u003dTrue,"},{"line_number":33,"context_line":"        )"},{"line_number":34,"context_line":"        self.addCleanup("}],"source_content_type":"text/x-python","patch_set":3,"id":"fbe7a2fd_b18a9327","line":31,"in_reply_to":"aeda9045_5717e489","updated":"2026-01-21 12:43:39.000000000","message":"Done","commit_id":"1ec7301fbc43c0bcdf708f193455116d4c9f4bda"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f35f9ffd6db1064d2d2f41e8f661c3d7ec1ee6bc","unresolved":true,"context_lines":[{"line_number":42,"context_line":"        # UPDATE"},{"line_number":43,"context_line":"        new_name \u003d f\"{self.obj_name}-updated\""},{"line_number":44,"context_line":"        self.openstack("},{"line_number":45,"context_line":"            f\"image metadef object update {self.ns_name} {self.obj_name} --name {new_name}\""},{"line_number":46,"context_line":"        )"},{"line_number":47,"context_line":"        self.obj_name \u003d new_name"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"a7e7f1fe_e1ecbbee","line":45,"updated":"2025-11-12 12:56:09.000000000","message":"ditto","commit_id":"1ec7301fbc43c0bcdf708f193455116d4c9f4bda"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"191ed74c1dba312f45bfae405e3c323538fb3ed5","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        # UPDATE"},{"line_number":43,"context_line":"        new_name \u003d f\"{self.obj_name}-updated\""},{"line_number":44,"context_line":"        self.openstack("},{"line_number":45,"context_line":"            f\"image metadef object update {self.ns_name} {self.obj_name} --name {new_name}\""},{"line_number":46,"context_line":"        )"},{"line_number":47,"context_line":"        self.obj_name \u003d new_name"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9d0b516e_409beaba","line":45,"in_reply_to":"a7e7f1fe_e1ecbbee","updated":"2026-01-21 12:43:39.000000000","message":"Done","commit_id":"1ec7301fbc43c0bcdf708f193455116d4c9f4bda"}]}
