)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"22a656dfae1d72b5f539345744171d708f211072","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7303b5b4_598f015e","updated":"2023-08-17 19:10:08.000000000","message":"A couple of nitpicks","commit_id":"32247654668ba8da36ba7e91dd1ee4754a230f0e"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"76e69f5bd118e9ab6bbf345d7f3a3962c9fed0c8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"eea99ce4_2349ecb5","updated":"2023-11-07 07:00:17.000000000","message":"Thank you!","commit_id":"4bc559c4501dd2989cb779089c41e7677630d77b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"a5926a00efbf3a5e71b5370b696659c2f5dba204","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ee29516b_842bea60","updated":"2023-11-08 15:37:21.000000000","message":"Already had +W. Carrying that forward after a trivial enough rebase","commit_id":"78ef009a3aebe43583d6b93e65f53c49a1d818a6"}],"openstackclient/image/v2/metadef_objects.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"22a656dfae1d72b5f539345744171d708f211072","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#   Copyright 2012-2013 OpenStack Foundation"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#   Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#   not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"30648a85_af485438","line":1,"range":{"start_line":1,"start_character":34,"end_line":1,"end_character":44},"updated":"2023-08-17 19:10:08.000000000","message":"Probably \"2023 Red Hat, Inc\" :)","commit_id":"32247654668ba8da36ba7e91dd1ee4754a230f0e"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"df831c48d610ddb02446358d70c62f23d8c74283","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#   Copyright 2012-2013 OpenStack Foundation"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#   Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#   not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"8536af0c_ae84b7a4","line":1,"range":{"start_line":1,"start_character":34,"end_line":1,"end_character":44},"in_reply_to":"30648a85_af485438","updated":"2023-08-22 08:36:49.000000000","message":"Done","commit_id":"32247654668ba8da36ba7e91dd1ee4754a230f0e"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"22a656dfae1d72b5f539345744171d708f211072","unresolved":true,"context_lines":[{"line_number":29,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"def _format_object(md_object):"},{"line_number":33,"context_line":"    info \u003d {}"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    fields_to_show \u003d ["}],"source_content_type":"text/x-python","patch_set":2,"id":"bdf290c2_54f1cc88","line":32,"range":{"start_line":32,"start_character":4,"end_line":32,"end_character":18},"updated":"2023-08-17 19:10:08.000000000","message":"Is this function going to be used in multiples places in the future? If not, it could be a static function in ShowMetadefObjects.","commit_id":"32247654668ba8da36ba7e91dd1ee4754a230f0e"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"bb3c2c96ff1edbf019888b113708ec41475e97b4","unresolved":false,"context_lines":[{"line_number":29,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"def _format_object(md_object):"},{"line_number":33,"context_line":"    info \u003d {}"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    fields_to_show \u003d ["}],"source_content_type":"text/x-python","patch_set":2,"id":"1050bf85_027aca32","line":32,"range":{"start_line":32,"start_character":4,"end_line":32,"end_character":18},"in_reply_to":"baecf41f_441b78aa","updated":"2023-10-27 13:34:07.000000000","message":"Done","commit_id":"32247654668ba8da36ba7e91dd1ee4754a230f0e"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"df831c48d610ddb02446358d70c62f23d8c74283","unresolved":true,"context_lines":[{"line_number":29,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"def _format_object(md_object):"},{"line_number":33,"context_line":"    info \u003d {}"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    fields_to_show \u003d ["}],"source_content_type":"text/x-python","patch_set":2,"id":"baecf41f_441b78aa","line":32,"range":{"start_line":32,"start_character":4,"end_line":32,"end_character":18},"in_reply_to":"bdf290c2_54f1cc88","updated":"2023-08-22 08:36:49.000000000","message":"Yes, we will be using it for other functionalities as well","commit_id":"32247654668ba8da36ba7e91dd1ee4754a230f0e"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"397d73c318977e963232cfadb08a41f479e92d74","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from openstackclient.i18n import _"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"_formatters \u003d {"},{"line_number":26,"context_line":"    \u0027tags\u0027: format_columns.ListColumn,"},{"line_number":27,"context_line":"}"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"f2c7b884_8fe4c43b","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":11},"updated":"2023-09-27 06:35:38.000000000","message":"I don\u0027t see this is used anywhere in this file","commit_id":"6095d50a7add200b18dec234aa781c3fd47b75f7"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"1f7f89fb015e753f88c076a8f69508d7a274d0e0","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from openstackclient.i18n import _"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"_formatters \u003d {"},{"line_number":26,"context_line":"    \u0027tags\u0027: format_columns.ListColumn,"},{"line_number":27,"context_line":"}"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3c16f123_3efd4cd2","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":11},"in_reply_to":"343e928e_8defc2a9","updated":"2023-09-29 05:20:55.000000000","message":"Ack","commit_id":"6095d50a7add200b18dec234aa781c3fd47b75f7"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"42cf5790174998a70a22074ddb155b99e12e9fc4","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"from openstackclient.i18n import _"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"_formatters \u003d {"},{"line_number":26,"context_line":"    \u0027tags\u0027: format_columns.ListColumn,"},{"line_number":27,"context_line":"}"},{"line_number":28,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"343e928e_8defc2a9","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":11},"in_reply_to":"f2c7b884_8fe4c43b","updated":"2023-09-28 00:52:54.000000000","message":"+1. The tests still work if this variable is removed. A lot of files have such a variable, so I think this may come in handy in the future, but for now it should be removed.","commit_id":"6095d50a7add200b18dec234aa781c3fd47b75f7"}],"openstackclient/tests/unit/image/v2/fakes.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f5be528938e19f29f2abbc4a6c6fd0832996becc","unresolved":true,"context_lines":[{"line_number":239,"context_line":"    return tasks"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"class FakeMetadefNamespaceClient:"},{"line_number":243,"context_line":"    def __init__(self, **kwargs):"},{"line_number":244,"context_line":"        self.create_metadef_namespace \u003d mock.Mock()"},{"line_number":245,"context_line":"        self.delete_metadef_namespace \u003d mock.Mock()"},{"line_number":246,"context_line":"        self.metadef_namespaces \u003d mock.Mock()"},{"line_number":247,"context_line":"        self.get_metadef_namespace \u003d mock.Mock()"},{"line_number":248,"context_line":"        self.update_metadef_namespace \u003d mock.Mock()"},{"line_number":249,"context_line":""},{"line_number":250,"context_line":"        self.get_metadef_object \u003d mock.Mock()"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"        self.auth_token \u003d kwargs[\u0027token\u0027]"},{"line_number":253,"context_line":"        self.management_url \u003d kwargs[\u0027endpoint\u0027]"},{"line_number":254,"context_line":"        self.version \u003d 2.0"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"class TestMetadefNamespaces(utils.TestCommand):"},{"line_number":258,"context_line":"    def setUp(self):"},{"line_number":259,"context_line":"        super().setUp()"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"        self.app.client_manager.image \u003d FakeMetadefNamespaceClient("},{"line_number":262,"context_line":"            endpoint\u003dfakes.AUTH_URL,"},{"line_number":263,"context_line":"            token\u003dfakes.AUTH_TOKEN,"},{"line_number":264,"context_line":"        )"},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"        self.app.client_manager.identity \u003d identity_fakes.FakeIdentityv3Client("},{"line_number":267,"context_line":"            endpoint\u003dfakes.AUTH_URL,"},{"line_number":268,"context_line":"            token\u003dfakes.AUTH_TOKEN,"},{"line_number":269,"context_line":"        )"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"def create_one_metadef_namespace(attrs\u003dNone):"}],"source_content_type":"text/x-python","patch_set":5,"id":"acaf6807_c36135d9","line":269,"range":{"start_line":242,"start_character":0,"end_line":269,"end_character":9},"updated":"2023-10-06 10:39:52.000000000","message":"As I noted on https://review.opendev.org/c/openstack/python-openstackclient/+/878631, you don\u0027t need this. We already have the `FakeClientMixin` to create a specced image proxy API mock and we should be using that.","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"8d5d607bdf982629a96c2a1928b2cfd2d047cc47","unresolved":false,"context_lines":[{"line_number":239,"context_line":"    return tasks"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"class FakeMetadefNamespaceClient:"},{"line_number":243,"context_line":"    def __init__(self, **kwargs):"},{"line_number":244,"context_line":"        self.create_metadef_namespace \u003d mock.Mock()"},{"line_number":245,"context_line":"        self.delete_metadef_namespace \u003d mock.Mock()"},{"line_number":246,"context_line":"        self.metadef_namespaces \u003d mock.Mock()"},{"line_number":247,"context_line":"        self.get_metadef_namespace \u003d mock.Mock()"},{"line_number":248,"context_line":"        self.update_metadef_namespace \u003d mock.Mock()"},{"line_number":249,"context_line":""},{"line_number":250,"context_line":"        self.get_metadef_object \u003d mock.Mock()"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"        self.auth_token \u003d kwargs[\u0027token\u0027]"},{"line_number":253,"context_line":"        self.management_url \u003d kwargs[\u0027endpoint\u0027]"},{"line_number":254,"context_line":"        self.version \u003d 2.0"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"class TestMetadefNamespaces(utils.TestCommand):"},{"line_number":258,"context_line":"    def setUp(self):"},{"line_number":259,"context_line":"        super().setUp()"},{"line_number":260,"context_line":""},{"line_number":261,"context_line":"        self.app.client_manager.image \u003d FakeMetadefNamespaceClient("},{"line_number":262,"context_line":"            endpoint\u003dfakes.AUTH_URL,"},{"line_number":263,"context_line":"            token\u003dfakes.AUTH_TOKEN,"},{"line_number":264,"context_line":"        )"},{"line_number":265,"context_line":""},{"line_number":266,"context_line":"        self.app.client_manager.identity \u003d identity_fakes.FakeIdentityv3Client("},{"line_number":267,"context_line":"            endpoint\u003dfakes.AUTH_URL,"},{"line_number":268,"context_line":"            token\u003dfakes.AUTH_TOKEN,"},{"line_number":269,"context_line":"        )"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":""},{"line_number":272,"context_line":"def create_one_metadef_namespace(attrs\u003dNone):"}],"source_content_type":"text/x-python","patch_set":5,"id":"ef945810_5e4bb97e","line":269,"range":{"start_line":242,"start_character":0,"end_line":269,"end_character":9},"in_reply_to":"acaf6807_c36135d9","updated":"2023-10-09 07:49:33.000000000","message":"Done","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"}],"openstackclient/tests/unit/image/v2/test_metadef_objects.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"22a656dfae1d72b5f539345744171d708f211072","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#   Copyright 2012-2013 OpenStack Foundation"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#   Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#   not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"ef305554_d0522073","line":1,"range":{"start_line":1,"start_character":34,"end_line":1,"end_character":44},"updated":"2023-08-17 19:10:08.000000000","message":"Ditto","commit_id":"32247654668ba8da36ba7e91dd1ee4754a230f0e"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"df831c48d610ddb02446358d70c62f23d8c74283","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#   Copyright 2012-2013 OpenStack Foundation"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"#   Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"#   not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"d2c7fc52_d3587158","line":1,"range":{"start_line":1,"start_character":34,"end_line":1,"end_character":44},"in_reply_to":"ef305554_d0522073","updated":"2023-08-22 08:36:49.000000000","message":"Done","commit_id":"32247654668ba8da36ba7e91dd1ee4754a230f0e"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f5be528938e19f29f2abbc4a6c6fd0832996becc","unresolved":true,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from openstackclient.image.v2 import metadef_objects"},{"line_number":17,"context_line":"from openstackclient.tests.unit.image.v2 import fakes as md_objects_fakes"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TestMetadefObjects(md_objects_fakes.TestMetadefNamespaces):"}],"source_content_type":"text/x-python","patch_set":5,"id":"c1facc9f_4e268162","line":17,"range":{"start_line":17,"start_character":53,"end_line":17,"end_character":73},"updated":"2023-10-06 10:39:52.000000000","message":"nit: do you really need the alias (the `as md_objects_fakes` bit)? `fakes` is shorter and there\u0027s nothing metadef objects-specific about the module...","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"8d5d607bdf982629a96c2a1928b2cfd2d047cc47","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from openstackclient.image.v2 import metadef_objects"},{"line_number":17,"context_line":"from openstackclient.tests.unit.image.v2 import fakes as md_objects_fakes"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TestMetadefObjects(md_objects_fakes.TestMetadefNamespaces):"}],"source_content_type":"text/x-python","patch_set":5,"id":"28fa5afc_59cd4e8f","line":17,"range":{"start_line":17,"start_character":53,"end_line":17,"end_character":73},"in_reply_to":"c1facc9f_4e268162","updated":"2023-10-09 07:49:33.000000000","message":"Done","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f5be528938e19f29f2abbc4a6c6fd0832996becc","unresolved":true,"context_lines":[{"line_number":17,"context_line":"from openstackclient.tests.unit.image.v2 import fakes as md_objects_fakes"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TestMetadefObjects(md_objects_fakes.TestMetadefNamespaces):"},{"line_number":21,"context_line":"    def setUp(self):"},{"line_number":22,"context_line":"        super().setUp()"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"d58dae05_f923a49c","line":20,"range":{"start_line":20,"start_character":25,"end_line":20,"end_character":63},"updated":"2023-10-06 10:39:52.000000000","message":"As I said in the previous file, you shouldn\u0027t need a custom base class. Use `fakes.TestImagev2` like everything else.","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"8d5d607bdf982629a96c2a1928b2cfd2d047cc47","unresolved":false,"context_lines":[{"line_number":17,"context_line":"from openstackclient.tests.unit.image.v2 import fakes as md_objects_fakes"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class TestMetadefObjects(md_objects_fakes.TestMetadefNamespaces):"},{"line_number":21,"context_line":"    def setUp(self):"},{"line_number":22,"context_line":"        super().setUp()"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"c9390bd7_b84add59","line":20,"range":{"start_line":20,"start_character":25,"end_line":20,"end_character":63},"in_reply_to":"d58dae05_f923a49c","updated":"2023-10-09 07:49:33.000000000","message":"Done","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f5be528938e19f29f2abbc4a6c6fd0832996becc","unresolved":true,"context_lines":[{"line_number":22,"context_line":"        super().setUp()"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"        # Get shortcuts to mocked image client"},{"line_number":25,"context_line":"        self.client \u003d self.app.client_manager.image"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"        # Get shortcut to the Mocks in identity client"},{"line_number":28,"context_line":"        self.project_mock \u003d self.app.client_manager.identity.projects"}],"source_content_type":"text/x-python","patch_set":5,"id":"bcdcce5f_171832d8","line":25,"updated":"2023-10-06 10:39:52.000000000","message":"This is configured by the `FakeClientMixin` used by `fakes.TestImagev2` so you get it for free by using that. Note that the client is stored as `image_client` though, not `client`, so you need to do `s/\\\u003cself.client\\\u003e/\\\u003cself.image_client\\\u003e/` in this file.","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"8d5d607bdf982629a96c2a1928b2cfd2d047cc47","unresolved":false,"context_lines":[{"line_number":22,"context_line":"        super().setUp()"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"        # Get shortcuts to mocked image client"},{"line_number":25,"context_line":"        self.client \u003d self.app.client_manager.image"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"        # Get shortcut to the Mocks in identity client"},{"line_number":28,"context_line":"        self.project_mock \u003d self.app.client_manager.identity.projects"}],"source_content_type":"text/x-python","patch_set":5,"id":"b050c17f_4d917575","line":25,"in_reply_to":"bcdcce5f_171832d8","updated":"2023-10-09 07:49:33.000000000","message":"Done","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f5be528938e19f29f2abbc4a6c6fd0832996becc","unresolved":true,"context_lines":[{"line_number":24,"context_line":"        # Get shortcuts to mocked image client"},{"line_number":25,"context_line":"        self.client \u003d self.app.client_manager.image"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"        # Get shortcut to the Mocks in identity client"},{"line_number":28,"context_line":"        self.project_mock \u003d self.app.client_manager.identity.projects"},{"line_number":29,"context_line":"        self.project_mock.reset_mock()"},{"line_number":30,"context_line":"        self.domain_mock \u003d self.app.client_manager.identity.domains"},{"line_number":31,"context_line":"        self.domain_mock.reset_mock()"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"class TestMetadefObjectsShow(TestMetadefObjects):"}],"source_content_type":"text/x-python","patch_set":5,"id":"2788919a_808e8ee1","line":31,"range":{"start_line":27,"start_character":0,"end_line":31,"end_character":37},"updated":"2023-10-06 10:39:52.000000000","message":"Do you need these? What\u0027s using them?","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"8d5d607bdf982629a96c2a1928b2cfd2d047cc47","unresolved":false,"context_lines":[{"line_number":24,"context_line":"        # Get shortcuts to mocked image client"},{"line_number":25,"context_line":"        self.client \u003d self.app.client_manager.image"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"        # Get shortcut to the Mocks in identity client"},{"line_number":28,"context_line":"        self.project_mock \u003d self.app.client_manager.identity.projects"},{"line_number":29,"context_line":"        self.project_mock.reset_mock()"},{"line_number":30,"context_line":"        self.domain_mock \u003d self.app.client_manager.identity.domains"},{"line_number":31,"context_line":"        self.domain_mock.reset_mock()"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"class TestMetadefObjectsShow(TestMetadefObjects):"}],"source_content_type":"text/x-python","patch_set":5,"id":"d7651fd4_b02e54bc","line":31,"range":{"start_line":27,"start_character":0,"end_line":31,"end_character":37},"in_reply_to":"2788919a_808e8ee1","updated":"2023-10-09 07:49:33.000000000","message":"Ack, Not needed","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"}],"setup.cfg":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f5be528938e19f29f2abbc4a6c6fd0832996becc","unresolved":true,"context_lines":[{"line_number":396,"context_line":"    image_metadef_namespace_list \u003d openstackclient.image.v2.metadef_namespaces:ListMetadefNameSpaces"},{"line_number":397,"context_line":"    image_metadef_namespace_set \u003d openstackclient.image.v2.metadef_namespaces:SetMetadefNameSpace"},{"line_number":398,"context_line":"    image_metadef_namespace_show \u003d openstackclient.image.v2.metadef_namespaces:ShowMetadefNameSpace"},{"line_number":399,"context_line":"    image_metadef_object_show \u003d openstackclient.image.v2.metadef_objects:ShowMetadefObjects"},{"line_number":400,"context_line":""},{"line_number":401,"context_line":"    image_metadef_resource_type_list \u003d openstackclient.image.v2.metadef_resource_types:ListMetadefResourceTypes"},{"line_number":402,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":5,"id":"c5808d7f_a8a735b1","line":399,"updated":"2023-10-06 10:39:52.000000000","message":"nit: this should be in a separate section, so can you add a newline before this?","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"8d5d607bdf982629a96c2a1928b2cfd2d047cc47","unresolved":false,"context_lines":[{"line_number":396,"context_line":"    image_metadef_namespace_list \u003d openstackclient.image.v2.metadef_namespaces:ListMetadefNameSpaces"},{"line_number":397,"context_line":"    image_metadef_namespace_set \u003d openstackclient.image.v2.metadef_namespaces:SetMetadefNameSpace"},{"line_number":398,"context_line":"    image_metadef_namespace_show \u003d openstackclient.image.v2.metadef_namespaces:ShowMetadefNameSpace"},{"line_number":399,"context_line":"    image_metadef_object_show \u003d openstackclient.image.v2.metadef_objects:ShowMetadefObjects"},{"line_number":400,"context_line":""},{"line_number":401,"context_line":"    image_metadef_resource_type_list \u003d openstackclient.image.v2.metadef_resource_types:ListMetadefResourceTypes"},{"line_number":402,"context_line":""}],"source_content_type":"text/x-ttcn-cfg","patch_set":5,"id":"83cdeda8_51501d78","line":399,"in_reply_to":"c5808d7f_a8a735b1","updated":"2023-10-09 07:49:33.000000000","message":"Done","commit_id":"5439a04bcd4f33d31bb602a03bebe0df481cf390"}]}