)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b69f31df_6a60822c","updated":"2022-06-09 05:30:37.000000000","message":"IMO there is still some unused code left in policy.py that needs to be removed.","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"df6e92eeb280ec6e48b0eab1f8c2e4cead6ea05c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"708e1c69_213e318e","updated":"2022-06-09 09:50:50.000000000","message":"@abhishek, I have addressed all your comments and updated the new patch set. Thanks!\n","commit_id":"ca474d23f91eceeef1c0d40fde5a54a9cf4aa90a"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"9497d82232b64c2befcd4b7944532cbc4ee51eea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a32d8634_e45c16d3","updated":"2022-06-09 10:00:20.000000000","message":"Sorry I didn\u0027t highlighted it earlier assuming it has been removed.","commit_id":"ca474d23f91eceeef1c0d40fde5a54a9cf4aa90a"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"e014a9b14c233e448cf7183642be686824eac920","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ff5e1c27_65ba3a00","updated":"2022-06-09 13:39:52.000000000","message":"Looks good to me now,thank you!","commit_id":"e537d1a7c1e400e6bb11631b44af30d49708e444"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"79dd06115b2557deb8c2f5cd445161cc9c5987d0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"933fa0f3_822cad65","updated":"2022-06-09 14:31:52.000000000","message":"This is my favorite patch of the week 😊\n\nOne minor naming thing inline, which isn\u0027t critical, but if you\u0027re going to respin anyway it would be nice to change. Otherwise, seems pretty straightforward to me.","commit_id":"e537d1a7c1e400e6bb11631b44af30d49708e444"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"3a71c9a69ebcf2ec15f247a39b9195c67496fd3d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"81718a92_9193df09","updated":"2022-06-13 14:13:26.000000000","message":"This looks far better now, sorry for not looking more carefully at test_gateway test module earlier.","commit_id":"1e23248a1e388f0fb597eb0eac53b9575e78feee"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"cc8e3383bb6d917e4cc4ce65422eec0a21e92724","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"69742438_64d718ad","updated":"2022-06-14 12:03:26.000000000","message":"Looks perfect now, thank you!","commit_id":"3790cfd4a11f6229fdcaa178db63e3572cf91100"}],"glance/api/policy.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":125,"context_line":"        _ENFORCER \u003d Enforcer()"},{"line_number":126,"context_line":"    return _ENFORCER"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"class ImageRepoProxy(glance.domain.proxy.Repo):"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"    def __init__(self, image_repo, context, policy):"},{"line_number":132,"context_line":"        self.context \u003d context"},{"line_number":133,"context_line":"        self.policy \u003d policy"},{"line_number":134,"context_line":"        self.image_repo \u003d image_repo"},{"line_number":135,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":136,"context_line":"        super(ImageRepoProxy, self).__init__(image_repo,"},{"line_number":137,"context_line":"                                             item_proxy_class\u003dImageProxy,"},{"line_number":138,"context_line":"                                             item_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"def _enforce_image_visibility(policy, context, visibility, target):"}],"source_content_type":"text/x-python","patch_set":2,"id":"c9191262_e0e2b077","line":138,"range":{"start_line":128,"start_character":0,"end_line":138,"end_character":76},"updated":"2022-06-09 05:30:37.000000000","message":"is this still used? I think this code is unreachable as well.","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"df6e92eeb280ec6e48b0eab1f8c2e4cead6ea05c","unresolved":false,"context_lines":[{"line_number":125,"context_line":"        _ENFORCER \u003d Enforcer()"},{"line_number":126,"context_line":"    return _ENFORCER"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"class ImageRepoProxy(glance.domain.proxy.Repo):"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"    def __init__(self, image_repo, context, policy):"},{"line_number":132,"context_line":"        self.context \u003d context"},{"line_number":133,"context_line":"        self.policy \u003d policy"},{"line_number":134,"context_line":"        self.image_repo \u003d image_repo"},{"line_number":135,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":136,"context_line":"        super(ImageRepoProxy, self).__init__(image_repo,"},{"line_number":137,"context_line":"                                             item_proxy_class\u003dImageProxy,"},{"line_number":138,"context_line":"                                             item_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"def _enforce_image_visibility(policy, context, visibility, target):"}],"source_content_type":"text/x-python","patch_set":2,"id":"a35a9ecc_8cfc511f","line":138,"range":{"start_line":128,"start_character":0,"end_line":138,"end_character":76},"in_reply_to":"7dd64354_58ebee11","updated":"2022-06-09 09:50:50.000000000","message":"Done","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":true,"context_lines":[{"line_number":125,"context_line":"        _ENFORCER \u003d Enforcer()"},{"line_number":126,"context_line":"    return _ENFORCER"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"class ImageRepoProxy(glance.domain.proxy.Repo):"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"    def __init__(self, image_repo, context, policy):"},{"line_number":132,"context_line":"        self.context \u003d context"},{"line_number":133,"context_line":"        self.policy \u003d policy"},{"line_number":134,"context_line":"        self.image_repo \u003d image_repo"},{"line_number":135,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":136,"context_line":"        super(ImageRepoProxy, self).__init__(image_repo,"},{"line_number":137,"context_line":"                                             item_proxy_class\u003dImageProxy,"},{"line_number":138,"context_line":"                                             item_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":""},{"line_number":141,"context_line":"def _enforce_image_visibility(policy, context, visibility, target):"}],"source_content_type":"text/x-python","patch_set":2,"id":"7dd64354_58ebee11","line":138,"range":{"start_line":128,"start_character":0,"end_line":138,"end_character":76},"in_reply_to":"c9191262_e0e2b077","updated":"2022-06-09 07:46:30.000000000","message":"not really, it is just mocked in test_gateway.py.\nI will remove this.","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":145,"context_line":"        policy.enforce(context, \u0027communitize_image\u0027, target)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"class ImageProxy(glance.domain.proxy.Image):"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    def __init__(self, image, context, policy):"},{"line_number":151,"context_line":"        self.image \u003d image"},{"line_number":152,"context_line":"        self.target \u003d ImageTarget(image)"},{"line_number":153,"context_line":"        self.context \u003d context"},{"line_number":154,"context_line":"        self.policy \u003d policy"},{"line_number":155,"context_line":"        super(ImageProxy, self).__init__(image)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"class ImageMemberProxy(glance.domain.proxy.ImageMember):"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3cb2e396_01ba9f0a","line":156,"range":{"start_line":148,"start_character":0,"end_line":156,"end_character":0},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":true,"context_lines":[{"line_number":145,"context_line":"        policy.enforce(context, \u0027communitize_image\u0027, target)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"class ImageProxy(glance.domain.proxy.Image):"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    def __init__(self, image, context, policy):"},{"line_number":151,"context_line":"        self.image \u003d image"},{"line_number":152,"context_line":"        self.target \u003d ImageTarget(image)"},{"line_number":153,"context_line":"        self.context \u003d context"},{"line_number":154,"context_line":"        self.policy \u003d policy"},{"line_number":155,"context_line":"        super(ImageProxy, self).__init__(image)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"class ImageMemberProxy(glance.domain.proxy.ImageMember):"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"67269d25_ec2e29e2","line":156,"range":{"start_line":148,"start_character":0,"end_line":156,"end_character":0},"in_reply_to":"3cb2e396_01ba9f0a","updated":"2022-06-09 07:46:30.000000000","message":"This is called from ImageTarget,\nhttps://review.opendev.org/c/openstack/glance/+/845114/2/glance/api/policy.py#264","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"5ce329865d9ca20ccba794d06b784fd2dccdb47e","unresolved":true,"context_lines":[{"line_number":145,"context_line":"        policy.enforce(context, \u0027communitize_image\u0027, target)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"class ImageProxy(glance.domain.proxy.Image):"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    def __init__(self, image, context, policy):"},{"line_number":151,"context_line":"        self.image \u003d image"},{"line_number":152,"context_line":"        self.target \u003d ImageTarget(image)"},{"line_number":153,"context_line":"        self.context \u003d context"},{"line_number":154,"context_line":"        self.policy \u003d policy"},{"line_number":155,"context_line":"        super(ImageProxy, self).__init__(image)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"class ImageMemberProxy(glance.domain.proxy.ImageMember):"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"e2804d0e_16b80ffb","line":156,"range":{"start_line":148,"start_character":0,"end_line":156,"end_character":0},"in_reply_to":"67269d25_ec2e29e2","updated":"2022-06-09 07:51:24.000000000","message":"I think you can directly use \u0027glance.domain.proxy.Image\u0027 there and remove this class","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"df6e92eeb280ec6e48b0eab1f8c2e4cead6ea05c","unresolved":false,"context_lines":[{"line_number":145,"context_line":"        policy.enforce(context, \u0027communitize_image\u0027, target)"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"class ImageProxy(glance.domain.proxy.Image):"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    def __init__(self, image, context, policy):"},{"line_number":151,"context_line":"        self.image \u003d image"},{"line_number":152,"context_line":"        self.target \u003d ImageTarget(image)"},{"line_number":153,"context_line":"        self.context \u003d context"},{"line_number":154,"context_line":"        self.policy \u003d policy"},{"line_number":155,"context_line":"        super(ImageProxy, self).__init__(image)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"class ImageMemberProxy(glance.domain.proxy.ImageMember):"},{"line_number":159,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"c8eead1f_455d2274","line":156,"range":{"start_line":148,"start_character":0,"end_line":156,"end_character":0},"in_reply_to":"e2804d0e_16b80ffb","updated":"2022-06-09 09:50:50.000000000","message":"Done","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":154,"context_line":"        self.policy \u003d policy"},{"line_number":155,"context_line":"        super(ImageProxy, self).__init__(image)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"class ImageMemberProxy(glance.domain.proxy.ImageMember):"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    def __init__(self, image_member, context, policy):"},{"line_number":161,"context_line":"        super(ImageMemberProxy, self).__init__(image_member)"},{"line_number":162,"context_line":"        self.image_member \u003d image_member"},{"line_number":163,"context_line":"        self.context \u003d context"},{"line_number":164,"context_line":"        self.policy \u003d policy"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"class ImageFactoryProxy(glance.domain.proxy.ImageFactory):"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"    def __init__(self, image_factory, context, policy):"},{"line_number":170,"context_line":"        self.image_factory \u003d image_factory"},{"line_number":171,"context_line":"        self.context \u003d context"},{"line_number":172,"context_line":"        self.policy \u003d policy"},{"line_number":173,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":174,"context_line":"        super(ImageFactoryProxy, self).__init__(image_factory,"},{"line_number":175,"context_line":"                                                proxy_class\u003dImageProxy,"},{"line_number":176,"context_line":"                                                proxy_kwargs\u003dproxy_kwargs)"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"class ImageMemberFactoryProxy(glance.domain.proxy.ImageMembershipFactory):"}],"source_content_type":"text/x-python","patch_set":2,"id":"98121d26_86acd608","line":176,"range":{"start_line":157,"start_character":0,"end_line":176,"end_character":74},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":154,"context_line":"        self.policy \u003d policy"},{"line_number":155,"context_line":"        super(ImageProxy, self).__init__(image)"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"class ImageMemberProxy(glance.domain.proxy.ImageMember):"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"    def __init__(self, image_member, context, policy):"},{"line_number":161,"context_line":"        super(ImageMemberProxy, self).__init__(image_member)"},{"line_number":162,"context_line":"        self.image_member \u003d image_member"},{"line_number":163,"context_line":"        self.context \u003d context"},{"line_number":164,"context_line":"        self.policy \u003d policy"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":""},{"line_number":167,"context_line":"class ImageFactoryProxy(glance.domain.proxy.ImageFactory):"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"    def __init__(self, image_factory, context, policy):"},{"line_number":170,"context_line":"        self.image_factory \u003d image_factory"},{"line_number":171,"context_line":"        self.context \u003d context"},{"line_number":172,"context_line":"        self.policy \u003d policy"},{"line_number":173,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":174,"context_line":"        super(ImageFactoryProxy, self).__init__(image_factory,"},{"line_number":175,"context_line":"                                                proxy_class\u003dImageProxy,"},{"line_number":176,"context_line":"                                                proxy_kwargs\u003dproxy_kwargs)"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"class ImageMemberFactoryProxy(glance.domain.proxy.ImageMembershipFactory):"}],"source_content_type":"text/x-python","patch_set":2,"id":"6d93a3a7_8424ca99","line":176,"range":{"start_line":157,"start_character":0,"end_line":176,"end_character":74},"in_reply_to":"98121d26_86acd608","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":176,"context_line":"                                                proxy_kwargs\u003dproxy_kwargs)"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"class ImageMemberFactoryProxy(glance.domain.proxy.ImageMembershipFactory):"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    def __init__(self, member_factory, context, policy):"},{"line_number":182,"context_line":"        super(ImageMemberFactoryProxy, self).__init__("},{"line_number":183,"context_line":"            member_factory,"},{"line_number":184,"context_line":"            proxy_class\u003dImageMemberProxy,"},{"line_number":185,"context_line":"            proxy_kwargs\u003d{\u0027context\u0027: context, \u0027policy\u0027: policy})"},{"line_number":186,"context_line":""},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"class ImageMemberRepoProxy(glance.domain.proxy.Repo):"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"    def __init__(self, member_repo, image, context, policy):"},{"line_number":191,"context_line":"        self.member_repo \u003d member_repo"},{"line_number":192,"context_line":"        self.image \u003d image"},{"line_number":193,"context_line":"        self.target \u003d ImageTarget(image)"},{"line_number":194,"context_line":"        self.context \u003d context"},{"line_number":195,"context_line":"        self.policy \u003d policy"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"class TaskProxy(glance.domain.proxy.Task):"}],"source_content_type":"text/x-python","patch_set":2,"id":"41820559_75f1ea17","line":195,"range":{"start_line":179,"start_character":0,"end_line":195,"end_character":28},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":176,"context_line":"                                                proxy_kwargs\u003dproxy_kwargs)"},{"line_number":177,"context_line":""},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"class ImageMemberFactoryProxy(glance.domain.proxy.ImageMembershipFactory):"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"    def __init__(self, member_factory, context, policy):"},{"line_number":182,"context_line":"        super(ImageMemberFactoryProxy, self).__init__("},{"line_number":183,"context_line":"            member_factory,"},{"line_number":184,"context_line":"            proxy_class\u003dImageMemberProxy,"},{"line_number":185,"context_line":"            proxy_kwargs\u003d{\u0027context\u0027: context, \u0027policy\u0027: policy})"},{"line_number":186,"context_line":""},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"class ImageMemberRepoProxy(glance.domain.proxy.Repo):"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"    def __init__(self, member_repo, image, context, policy):"},{"line_number":191,"context_line":"        self.member_repo \u003d member_repo"},{"line_number":192,"context_line":"        self.image \u003d image"},{"line_number":193,"context_line":"        self.target \u003d ImageTarget(image)"},{"line_number":194,"context_line":"        self.context \u003d context"},{"line_number":195,"context_line":"        self.policy \u003d policy"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"class TaskProxy(glance.domain.proxy.Task):"}],"source_content_type":"text/x-python","patch_set":2,"id":"c4912734_19631ea7","line":195,"range":{"start_line":179,"start_character":0,"end_line":195,"end_character":28},"in_reply_to":"41820559_75f1ea17","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":195,"context_line":"        self.policy \u003d policy"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"class TaskProxy(glance.domain.proxy.Task):"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"    def __init__(self, task, context, policy):"},{"line_number":201,"context_line":"        self.task \u003d task"}],"source_content_type":"text/x-python","patch_set":2,"id":"363e0b47_11725ecf","line":198,"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":195,"context_line":"        self.policy \u003d policy"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"class TaskProxy(glance.domain.proxy.Task):"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"    def __init__(self, task, context, policy):"},{"line_number":201,"context_line":"        self.task \u003d task"}],"source_content_type":"text/x-python","patch_set":2,"id":"b69cdd54_fb60902e","line":198,"in_reply_to":"363e0b47_11725ecf","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":195,"context_line":"        self.policy \u003d policy"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"class TaskProxy(glance.domain.proxy.Task):"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"    def __init__(self, task, context, policy):"},{"line_number":201,"context_line":"        self.task \u003d task"},{"line_number":202,"context_line":"        self.context \u003d context"},{"line_number":203,"context_line":"        self.policy \u003d policy"},{"line_number":204,"context_line":"        super(TaskProxy, self).__init__(task)"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"class TaskStubProxy(glance.domain.proxy.TaskStub):"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"    def __init__(self, task_stub, context, policy):"},{"line_number":210,"context_line":"        self.task_stub \u003d task_stub"},{"line_number":211,"context_line":"        self.context \u003d context"},{"line_number":212,"context_line":"        self.policy \u003d policy"},{"line_number":213,"context_line":"        super(TaskStubProxy, self).__init__(task_stub)"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"class TaskRepoProxy(glance.domain.proxy.TaskRepo):"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"    def __init__(self, task_repo, context, task_policy):"},{"line_number":219,"context_line":"        self.context \u003d context"},{"line_number":220,"context_line":"        self.policy \u003d task_policy"},{"line_number":221,"context_line":"        self.task_repo \u003d task_repo"},{"line_number":222,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":223,"context_line":"        super(TaskRepoProxy,"},{"line_number":224,"context_line":"              self).__init__(task_repo,"},{"line_number":225,"context_line":"                             task_proxy_class\u003dTaskProxy,"},{"line_number":226,"context_line":"                             task_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"class TaskStubRepoProxy(glance.domain.proxy.TaskStubRepo):"}],"source_content_type":"text/x-python","patch_set":2,"id":"0ef25d64_5b136d8b","line":226,"range":{"start_line":198,"start_character":0,"end_line":226,"end_character":60},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":195,"context_line":"        self.policy \u003d policy"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":""},{"line_number":198,"context_line":"class TaskProxy(glance.domain.proxy.Task):"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"    def __init__(self, task, context, policy):"},{"line_number":201,"context_line":"        self.task \u003d task"},{"line_number":202,"context_line":"        self.context \u003d context"},{"line_number":203,"context_line":"        self.policy \u003d policy"},{"line_number":204,"context_line":"        super(TaskProxy, self).__init__(task)"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":""},{"line_number":207,"context_line":"class TaskStubProxy(glance.domain.proxy.TaskStub):"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"    def __init__(self, task_stub, context, policy):"},{"line_number":210,"context_line":"        self.task_stub \u003d task_stub"},{"line_number":211,"context_line":"        self.context \u003d context"},{"line_number":212,"context_line":"        self.policy \u003d policy"},{"line_number":213,"context_line":"        super(TaskStubProxy, self).__init__(task_stub)"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":""},{"line_number":216,"context_line":"class TaskRepoProxy(glance.domain.proxy.TaskRepo):"},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"    def __init__(self, task_repo, context, task_policy):"},{"line_number":219,"context_line":"        self.context \u003d context"},{"line_number":220,"context_line":"        self.policy \u003d task_policy"},{"line_number":221,"context_line":"        self.task_repo \u003d task_repo"},{"line_number":222,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":223,"context_line":"        super(TaskRepoProxy,"},{"line_number":224,"context_line":"              self).__init__(task_repo,"},{"line_number":225,"context_line":"                             task_proxy_class\u003dTaskProxy,"},{"line_number":226,"context_line":"                             task_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"class TaskStubRepoProxy(glance.domain.proxy.TaskStubRepo):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f3681f7_b14f8b32","line":226,"range":{"start_line":198,"start_character":0,"end_line":226,"end_character":60},"in_reply_to":"0ef25d64_5b136d8b","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":226,"context_line":"                             task_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"class TaskStubRepoProxy(glance.domain.proxy.TaskStubRepo):"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    def __init__(self, task_stub_repo, context, task_policy):"},{"line_number":232,"context_line":"        self.context \u003d context"},{"line_number":233,"context_line":"        self.policy \u003d task_policy"},{"line_number":234,"context_line":"        self.task_stub_repo \u003d task_stub_repo"},{"line_number":235,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":236,"context_line":"        super(TaskStubRepoProxy,"},{"line_number":237,"context_line":"              self).__init__(task_stub_repo,"},{"line_number":238,"context_line":"                             task_stub_proxy_class\u003dTaskStubProxy,"},{"line_number":239,"context_line":"                             task_stub_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"class TaskFactoryProxy(glance.domain.proxy.TaskFactory):"},{"line_number":243,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"b3bd4b59_c570a472","line":240,"range":{"start_line":229,"start_character":0,"end_line":240,"end_character":0},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":226,"context_line":"                             task_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"class TaskStubRepoProxy(glance.domain.proxy.TaskStubRepo):"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    def __init__(self, task_stub_repo, context, task_policy):"},{"line_number":232,"context_line":"        self.context \u003d context"},{"line_number":233,"context_line":"        self.policy \u003d task_policy"},{"line_number":234,"context_line":"        self.task_stub_repo \u003d task_stub_repo"},{"line_number":235,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":236,"context_line":"        super(TaskStubRepoProxy,"},{"line_number":237,"context_line":"              self).__init__(task_stub_repo,"},{"line_number":238,"context_line":"                             task_stub_proxy_class\u003dTaskStubProxy,"},{"line_number":239,"context_line":"                             task_stub_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"class TaskFactoryProxy(glance.domain.proxy.TaskFactory):"},{"line_number":243,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"efc2db0d_f04496c8","line":240,"range":{"start_line":229,"start_character":0,"end_line":240,"end_character":0},"in_reply_to":"b3bd4b59_c570a472","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":239,"context_line":"                             task_stub_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"class TaskFactoryProxy(glance.domain.proxy.TaskFactory):"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    def __init__(self, task_factory, context, policy):"},{"line_number":245,"context_line":"        self.task_factory \u003d task_factory"},{"line_number":246,"context_line":"        self.context \u003d context"},{"line_number":247,"context_line":"        self.policy \u003d policy"},{"line_number":248,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":249,"context_line":"        super(TaskFactoryProxy, self).__init__("},{"line_number":250,"context_line":"            task_factory,"},{"line_number":251,"context_line":"            task_proxy_class\u003dTaskProxy,"},{"line_number":252,"context_line":"            task_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"class ImageTarget(abc.Mapping):"}],"source_content_type":"text/x-python","patch_set":2,"id":"a0136dac_f33292c5","line":252,"range":{"start_line":242,"start_character":0,"end_line":252,"end_character":43},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":239,"context_line":"                             task_stub_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":240,"context_line":""},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"class TaskFactoryProxy(glance.domain.proxy.TaskFactory):"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    def __init__(self, task_factory, context, policy):"},{"line_number":245,"context_line":"        self.task_factory \u003d task_factory"},{"line_number":246,"context_line":"        self.context \u003d context"},{"line_number":247,"context_line":"        self.policy \u003d policy"},{"line_number":248,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":249,"context_line":"        super(TaskFactoryProxy, self).__init__("},{"line_number":250,"context_line":"            task_factory,"},{"line_number":251,"context_line":"            task_proxy_class\u003dTaskProxy,"},{"line_number":252,"context_line":"            task_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":253,"context_line":""},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"class ImageTarget(abc.Mapping):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a754f34_d2870dba","line":252,"range":{"start_line":242,"start_character":0,"end_line":252,"end_character":43},"in_reply_to":"a0136dac_f33292c5","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":320,"context_line":""},{"line_number":321,"context_line":"        return transforms.get(key, key)"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"# Metadef Namespace classes"},{"line_number":325,"context_line":"class MetadefNamespaceProxy(glance.domain.proxy.MetadefNamespace):"},{"line_number":326,"context_line":""},{"line_number":327,"context_line":"    def __init__(self, namespace, context, policy):"},{"line_number":328,"context_line":"        self.namespace_input \u003d namespace"},{"line_number":329,"context_line":"        self.context \u003d context"},{"line_number":330,"context_line":"        self.policy \u003d policy"},{"line_number":331,"context_line":"        super(MetadefNamespaceProxy, self).__init__(namespace)"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"class MetadefNamespaceRepoProxy(glance.domain.proxy.MetadefNamespaceRepo):"}],"source_content_type":"text/x-python","patch_set":2,"id":"19618724_6dfa4db6","line":331,"range":{"start_line":323,"start_character":0,"end_line":331,"end_character":62},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":320,"context_line":""},{"line_number":321,"context_line":"        return transforms.get(key, key)"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"# Metadef Namespace classes"},{"line_number":325,"context_line":"class MetadefNamespaceProxy(glance.domain.proxy.MetadefNamespace):"},{"line_number":326,"context_line":""},{"line_number":327,"context_line":"    def __init__(self, namespace, context, policy):"},{"line_number":328,"context_line":"        self.namespace_input \u003d namespace"},{"line_number":329,"context_line":"        self.context \u003d context"},{"line_number":330,"context_line":"        self.policy \u003d policy"},{"line_number":331,"context_line":"        super(MetadefNamespaceProxy, self).__init__(namespace)"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"class MetadefNamespaceRepoProxy(glance.domain.proxy.MetadefNamespaceRepo):"}],"source_content_type":"text/x-python","patch_set":2,"id":"bb1c5a4c_65f04704","line":331,"range":{"start_line":323,"start_character":0,"end_line":331,"end_character":62},"in_reply_to":"19618724_6dfa4db6","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":330,"context_line":"        self.policy \u003d policy"},{"line_number":331,"context_line":"        super(MetadefNamespaceProxy, self).__init__(namespace)"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"class MetadefNamespaceRepoProxy(glance.domain.proxy.MetadefNamespaceRepo):"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"    def __init__(self, namespace_repo, context, namespace_policy):"},{"line_number":337,"context_line":"        self.context \u003d context"},{"line_number":338,"context_line":"        self.policy \u003d namespace_policy"},{"line_number":339,"context_line":"        self.namespace_repo \u003d namespace_repo"},{"line_number":340,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":341,"context_line":"        super(MetadefNamespaceRepoProxy,"},{"line_number":342,"context_line":"              self).__init__(namespace_repo,"},{"line_number":343,"context_line":"                             namespace_proxy_class\u003dMetadefNamespaceProxy,"},{"line_number":344,"context_line":"                             namespace_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"class MetadefNamespaceFactoryProxy("}],"source_content_type":"text/x-python","patch_set":2,"id":"9eaea47d_3e0c7351","line":344,"range":{"start_line":333,"start_character":0,"end_line":344,"end_character":65},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":330,"context_line":"        self.policy \u003d policy"},{"line_number":331,"context_line":"        super(MetadefNamespaceProxy, self).__init__(namespace)"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"class MetadefNamespaceRepoProxy(glance.domain.proxy.MetadefNamespaceRepo):"},{"line_number":335,"context_line":""},{"line_number":336,"context_line":"    def __init__(self, namespace_repo, context, namespace_policy):"},{"line_number":337,"context_line":"        self.context \u003d context"},{"line_number":338,"context_line":"        self.policy \u003d namespace_policy"},{"line_number":339,"context_line":"        self.namespace_repo \u003d namespace_repo"},{"line_number":340,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":341,"context_line":"        super(MetadefNamespaceRepoProxy,"},{"line_number":342,"context_line":"              self).__init__(namespace_repo,"},{"line_number":343,"context_line":"                             namespace_proxy_class\u003dMetadefNamespaceProxy,"},{"line_number":344,"context_line":"                             namespace_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"class MetadefNamespaceFactoryProxy("}],"source_content_type":"text/x-python","patch_set":2,"id":"2c5320eb_bc5e1582","line":344,"range":{"start_line":333,"start_character":0,"end_line":344,"end_character":65},"in_reply_to":"9eaea47d_3e0c7351","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":344,"context_line":"                             namespace_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"class MetadefNamespaceFactoryProxy("},{"line_number":348,"context_line":"        glance.domain.proxy.MetadefNamespaceFactory):"},{"line_number":349,"context_line":""},{"line_number":350,"context_line":"    def __init__(self, meta_namespace_factory, context, policy):"},{"line_number":351,"context_line":"        self.meta_namespace_factory \u003d meta_namespace_factory"},{"line_number":352,"context_line":"        self.context \u003d context"},{"line_number":353,"context_line":"        self.policy \u003d policy"},{"line_number":354,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":355,"context_line":"        super(MetadefNamespaceFactoryProxy, self).__init__("},{"line_number":356,"context_line":"            meta_namespace_factory,"},{"line_number":357,"context_line":"            meta_namespace_proxy_class\u003dMetadefNamespaceProxy,"},{"line_number":358,"context_line":"            meta_namespace_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":""},{"line_number":361,"context_line":"# Metadef Object classes"}],"source_content_type":"text/x-python","patch_set":2,"id":"708deb49_1f189134","line":358,"range":{"start_line":347,"start_character":0,"end_line":358,"end_character":53},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":344,"context_line":"                             namespace_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":345,"context_line":""},{"line_number":346,"context_line":""},{"line_number":347,"context_line":"class MetadefNamespaceFactoryProxy("},{"line_number":348,"context_line":"        glance.domain.proxy.MetadefNamespaceFactory):"},{"line_number":349,"context_line":""},{"line_number":350,"context_line":"    def __init__(self, meta_namespace_factory, context, policy):"},{"line_number":351,"context_line":"        self.meta_namespace_factory \u003d meta_namespace_factory"},{"line_number":352,"context_line":"        self.context \u003d context"},{"line_number":353,"context_line":"        self.policy \u003d policy"},{"line_number":354,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":355,"context_line":"        super(MetadefNamespaceFactoryProxy, self).__init__("},{"line_number":356,"context_line":"            meta_namespace_factory,"},{"line_number":357,"context_line":"            meta_namespace_proxy_class\u003dMetadefNamespaceProxy,"},{"line_number":358,"context_line":"            meta_namespace_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":""},{"line_number":361,"context_line":"# Metadef Object classes"}],"source_content_type":"text/x-python","patch_set":2,"id":"fcc4e32f_9ac32c6b","line":358,"range":{"start_line":347,"start_character":0,"end_line":358,"end_character":53},"in_reply_to":"708deb49_1f189134","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":357,"context_line":"            meta_namespace_proxy_class\u003dMetadefNamespaceProxy,"},{"line_number":358,"context_line":"            meta_namespace_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":""},{"line_number":361,"context_line":"# Metadef Object classes"},{"line_number":362,"context_line":"class MetadefObjectProxy(glance.domain.proxy.MetadefObject):"},{"line_number":363,"context_line":""},{"line_number":364,"context_line":"    def __init__(self, meta_object, context, policy):"},{"line_number":365,"context_line":"        self.meta_object \u003d meta_object"},{"line_number":366,"context_line":"        self.context \u003d context"},{"line_number":367,"context_line":"        self.policy \u003d policy"},{"line_number":368,"context_line":"        super(MetadefObjectProxy, self).__init__(meta_object)"},{"line_number":369,"context_line":""},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"class MetadefObjectRepoProxy(glance.domain.proxy.MetadefObjectRepo):"}],"source_content_type":"text/x-python","patch_set":2,"id":"59857dca_5505b819","line":368,"range":{"start_line":360,"start_character":0,"end_line":368,"end_character":61},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":357,"context_line":"            meta_namespace_proxy_class\u003dMetadefNamespaceProxy,"},{"line_number":358,"context_line":"            meta_namespace_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":359,"context_line":""},{"line_number":360,"context_line":""},{"line_number":361,"context_line":"# Metadef Object classes"},{"line_number":362,"context_line":"class MetadefObjectProxy(glance.domain.proxy.MetadefObject):"},{"line_number":363,"context_line":""},{"line_number":364,"context_line":"    def __init__(self, meta_object, context, policy):"},{"line_number":365,"context_line":"        self.meta_object \u003d meta_object"},{"line_number":366,"context_line":"        self.context \u003d context"},{"line_number":367,"context_line":"        self.policy \u003d policy"},{"line_number":368,"context_line":"        super(MetadefObjectProxy, self).__init__(meta_object)"},{"line_number":369,"context_line":""},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"class MetadefObjectRepoProxy(glance.domain.proxy.MetadefObjectRepo):"}],"source_content_type":"text/x-python","patch_set":2,"id":"5181779b_6cf10f25","line":368,"range":{"start_line":360,"start_character":0,"end_line":368,"end_character":61},"in_reply_to":"59857dca_5505b819","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":368,"context_line":"        super(MetadefObjectProxy, self).__init__(meta_object)"},{"line_number":369,"context_line":""},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"class MetadefObjectRepoProxy(glance.domain.proxy.MetadefObjectRepo):"},{"line_number":372,"context_line":""},{"line_number":373,"context_line":"    def __init__(self, object_repo, context, object_policy):"},{"line_number":374,"context_line":"        self.context \u003d context"},{"line_number":375,"context_line":"        self.policy \u003d object_policy"},{"line_number":376,"context_line":"        self.object_repo \u003d object_repo"},{"line_number":377,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":378,"context_line":"        super(MetadefObjectRepoProxy,"},{"line_number":379,"context_line":"              self).__init__(object_repo,"},{"line_number":380,"context_line":"                             object_proxy_class\u003dMetadefObjectProxy,"},{"line_number":381,"context_line":"                             object_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":""},{"line_number":384,"context_line":"class MetadefObjectFactoryProxy(glance.domain.proxy.MetadefObjectFactory):"},{"line_number":385,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"e7904be0_95fb535a","line":382,"range":{"start_line":371,"start_character":0,"end_line":382,"end_character":0},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":368,"context_line":"        super(MetadefObjectProxy, self).__init__(meta_object)"},{"line_number":369,"context_line":""},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"class MetadefObjectRepoProxy(glance.domain.proxy.MetadefObjectRepo):"},{"line_number":372,"context_line":""},{"line_number":373,"context_line":"    def __init__(self, object_repo, context, object_policy):"},{"line_number":374,"context_line":"        self.context \u003d context"},{"line_number":375,"context_line":"        self.policy \u003d object_policy"},{"line_number":376,"context_line":"        self.object_repo \u003d object_repo"},{"line_number":377,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":378,"context_line":"        super(MetadefObjectRepoProxy,"},{"line_number":379,"context_line":"              self).__init__(object_repo,"},{"line_number":380,"context_line":"                             object_proxy_class\u003dMetadefObjectProxy,"},{"line_number":381,"context_line":"                             object_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":""},{"line_number":384,"context_line":"class MetadefObjectFactoryProxy(glance.domain.proxy.MetadefObjectFactory):"},{"line_number":385,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"f86ecfdf_8b73638d","line":382,"range":{"start_line":371,"start_character":0,"end_line":382,"end_character":0},"in_reply_to":"e7904be0_95fb535a","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":380,"context_line":"                             object_proxy_class\u003dMetadefObjectProxy,"},{"line_number":381,"context_line":"                             object_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":""},{"line_number":384,"context_line":"class MetadefObjectFactoryProxy(glance.domain.proxy.MetadefObjectFactory):"},{"line_number":385,"context_line":""},{"line_number":386,"context_line":"    def __init__(self, meta_object_factory, context, policy):"},{"line_number":387,"context_line":"        self.meta_object_factory \u003d meta_object_factory"},{"line_number":388,"context_line":"        self.context \u003d context"},{"line_number":389,"context_line":"        self.policy \u003d policy"},{"line_number":390,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":391,"context_line":"        super(MetadefObjectFactoryProxy, self).__init__("},{"line_number":392,"context_line":"            meta_object_factory,"},{"line_number":393,"context_line":"            meta_object_proxy_class\u003dMetadefObjectProxy,"},{"line_number":394,"context_line":"            meta_object_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":395,"context_line":""},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"# Metadef ResourceType classes"},{"line_number":398,"context_line":"class MetadefResourceTypeProxy(glance.domain.proxy.MetadefResourceType):"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"    def __init__(self, meta_resource_type, context, policy):"},{"line_number":401,"context_line":"        self.meta_resource_type \u003d meta_resource_type"},{"line_number":402,"context_line":"        self.context \u003d context"},{"line_number":403,"context_line":"        self.policy \u003d policy"},{"line_number":404,"context_line":"        super(MetadefResourceTypeProxy, self).__init__(meta_resource_type)"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":""},{"line_number":407,"context_line":"class MetadefResourceTypeRepoProxy("}],"source_content_type":"text/x-python","patch_set":2,"id":"71dc22b1_5e29e62a","line":404,"range":{"start_line":383,"start_character":0,"end_line":404,"end_character":74},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":380,"context_line":"                             object_proxy_class\u003dMetadefObjectProxy,"},{"line_number":381,"context_line":"                             object_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":""},{"line_number":384,"context_line":"class MetadefObjectFactoryProxy(glance.domain.proxy.MetadefObjectFactory):"},{"line_number":385,"context_line":""},{"line_number":386,"context_line":"    def __init__(self, meta_object_factory, context, policy):"},{"line_number":387,"context_line":"        self.meta_object_factory \u003d meta_object_factory"},{"line_number":388,"context_line":"        self.context \u003d context"},{"line_number":389,"context_line":"        self.policy \u003d policy"},{"line_number":390,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":391,"context_line":"        super(MetadefObjectFactoryProxy, self).__init__("},{"line_number":392,"context_line":"            meta_object_factory,"},{"line_number":393,"context_line":"            meta_object_proxy_class\u003dMetadefObjectProxy,"},{"line_number":394,"context_line":"            meta_object_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":395,"context_line":""},{"line_number":396,"context_line":""},{"line_number":397,"context_line":"# Metadef ResourceType classes"},{"line_number":398,"context_line":"class MetadefResourceTypeProxy(glance.domain.proxy.MetadefResourceType):"},{"line_number":399,"context_line":""},{"line_number":400,"context_line":"    def __init__(self, meta_resource_type, context, policy):"},{"line_number":401,"context_line":"        self.meta_resource_type \u003d meta_resource_type"},{"line_number":402,"context_line":"        self.context \u003d context"},{"line_number":403,"context_line":"        self.policy \u003d policy"},{"line_number":404,"context_line":"        super(MetadefResourceTypeProxy, self).__init__(meta_resource_type)"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":""},{"line_number":407,"context_line":"class MetadefResourceTypeRepoProxy("}],"source_content_type":"text/x-python","patch_set":2,"id":"0639d522_47b66e0d","line":404,"range":{"start_line":383,"start_character":0,"end_line":404,"end_character":74},"in_reply_to":"71dc22b1_5e29e62a","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":404,"context_line":"        super(MetadefResourceTypeProxy, self).__init__(meta_resource_type)"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":""},{"line_number":407,"context_line":"class MetadefResourceTypeRepoProxy("},{"line_number":408,"context_line":"        glance.domain.proxy.MetadefResourceTypeRepo):"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"    def __init__(self, resource_type_repo, context, resource_type_policy):"},{"line_number":411,"context_line":"        self.context \u003d context"},{"line_number":412,"context_line":"        self.policy \u003d resource_type_policy"},{"line_number":413,"context_line":"        self.resource_type_repo \u003d resource_type_repo"},{"line_number":414,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":415,"context_line":"        super(MetadefResourceTypeRepoProxy, self).__init__("},{"line_number":416,"context_line":"            resource_type_repo,"},{"line_number":417,"context_line":"            resource_type_proxy_class\u003dMetadefResourceTypeProxy,"},{"line_number":418,"context_line":"            resource_type_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":""},{"line_number":421,"context_line":"class MetadefResourceTypeFactoryProxy("}],"source_content_type":"text/x-python","patch_set":2,"id":"fdf97ed9_c3b11f41","line":418,"range":{"start_line":407,"start_character":0,"end_line":418,"end_character":52},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":404,"context_line":"        super(MetadefResourceTypeProxy, self).__init__(meta_resource_type)"},{"line_number":405,"context_line":""},{"line_number":406,"context_line":""},{"line_number":407,"context_line":"class MetadefResourceTypeRepoProxy("},{"line_number":408,"context_line":"        glance.domain.proxy.MetadefResourceTypeRepo):"},{"line_number":409,"context_line":""},{"line_number":410,"context_line":"    def __init__(self, resource_type_repo, context, resource_type_policy):"},{"line_number":411,"context_line":"        self.context \u003d context"},{"line_number":412,"context_line":"        self.policy \u003d resource_type_policy"},{"line_number":413,"context_line":"        self.resource_type_repo \u003d resource_type_repo"},{"line_number":414,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":415,"context_line":"        super(MetadefResourceTypeRepoProxy, self).__init__("},{"line_number":416,"context_line":"            resource_type_repo,"},{"line_number":417,"context_line":"            resource_type_proxy_class\u003dMetadefResourceTypeProxy,"},{"line_number":418,"context_line":"            resource_type_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":""},{"line_number":421,"context_line":"class MetadefResourceTypeFactoryProxy("}],"source_content_type":"text/x-python","patch_set":2,"id":"8c8e8fad_77166c0a","line":418,"range":{"start_line":407,"start_character":0,"end_line":418,"end_character":52},"in_reply_to":"fdf97ed9_c3b11f41","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":417,"context_line":"            resource_type_proxy_class\u003dMetadefResourceTypeProxy,"},{"line_number":418,"context_line":"            resource_type_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":""},{"line_number":421,"context_line":"class MetadefResourceTypeFactoryProxy("},{"line_number":422,"context_line":"        glance.domain.proxy.MetadefResourceTypeFactory):"},{"line_number":423,"context_line":""},{"line_number":424,"context_line":"    def __init__(self, resource_type_factory, context, policy):"},{"line_number":425,"context_line":"        self.resource_type_factory \u003d resource_type_factory"},{"line_number":426,"context_line":"        self.context \u003d context"},{"line_number":427,"context_line":"        self.policy \u003d policy"},{"line_number":428,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":429,"context_line":"        super(MetadefResourceTypeFactoryProxy, self).__init__("},{"line_number":430,"context_line":"            resource_type_factory,"},{"line_number":431,"context_line":"            resource_type_proxy_class\u003dMetadefResourceTypeProxy,"},{"line_number":432,"context_line":"            resource_type_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":433,"context_line":""},{"line_number":434,"context_line":""},{"line_number":435,"context_line":"# Metadef namespace properties classes"},{"line_number":436,"context_line":"class MetadefPropertyProxy(glance.domain.proxy.MetadefProperty):"},{"line_number":437,"context_line":""},{"line_number":438,"context_line":"    def __init__(self, namespace_property, context, policy):"},{"line_number":439,"context_line":"        self.namespace_property \u003d namespace_property"},{"line_number":440,"context_line":"        self.context \u003d context"},{"line_number":441,"context_line":"        self.policy \u003d policy"},{"line_number":442,"context_line":"        super(MetadefPropertyProxy, self).__init__(namespace_property)"},{"line_number":443,"context_line":""},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"class MetadefPropertyRepoProxy(glance.domain.proxy.MetadefPropertyRepo):"},{"line_number":446,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"f43784d4_6b48f07b","line":443,"range":{"start_line":420,"start_character":0,"end_line":443,"end_character":0},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":417,"context_line":"            resource_type_proxy_class\u003dMetadefResourceTypeProxy,"},{"line_number":418,"context_line":"            resource_type_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":419,"context_line":""},{"line_number":420,"context_line":""},{"line_number":421,"context_line":"class MetadefResourceTypeFactoryProxy("},{"line_number":422,"context_line":"        glance.domain.proxy.MetadefResourceTypeFactory):"},{"line_number":423,"context_line":""},{"line_number":424,"context_line":"    def __init__(self, resource_type_factory, context, policy):"},{"line_number":425,"context_line":"        self.resource_type_factory \u003d resource_type_factory"},{"line_number":426,"context_line":"        self.context \u003d context"},{"line_number":427,"context_line":"        self.policy \u003d policy"},{"line_number":428,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":429,"context_line":"        super(MetadefResourceTypeFactoryProxy, self).__init__("},{"line_number":430,"context_line":"            resource_type_factory,"},{"line_number":431,"context_line":"            resource_type_proxy_class\u003dMetadefResourceTypeProxy,"},{"line_number":432,"context_line":"            resource_type_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":433,"context_line":""},{"line_number":434,"context_line":""},{"line_number":435,"context_line":"# Metadef namespace properties classes"},{"line_number":436,"context_line":"class MetadefPropertyProxy(glance.domain.proxy.MetadefProperty):"},{"line_number":437,"context_line":""},{"line_number":438,"context_line":"    def __init__(self, namespace_property, context, policy):"},{"line_number":439,"context_line":"        self.namespace_property \u003d namespace_property"},{"line_number":440,"context_line":"        self.context \u003d context"},{"line_number":441,"context_line":"        self.policy \u003d policy"},{"line_number":442,"context_line":"        super(MetadefPropertyProxy, self).__init__(namespace_property)"},{"line_number":443,"context_line":""},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"class MetadefPropertyRepoProxy(glance.domain.proxy.MetadefPropertyRepo):"},{"line_number":446,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"c5c5020f_c5b9ba51","line":443,"range":{"start_line":420,"start_character":0,"end_line":443,"end_character":0},"in_reply_to":"f43784d4_6b48f07b","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":442,"context_line":"        super(MetadefPropertyProxy, self).__init__(namespace_property)"},{"line_number":443,"context_line":""},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"class MetadefPropertyRepoProxy(glance.domain.proxy.MetadefPropertyRepo):"},{"line_number":446,"context_line":""},{"line_number":447,"context_line":"    def __init__(self, property_repo, context, object_policy):"},{"line_number":448,"context_line":"        self.context \u003d context"},{"line_number":449,"context_line":"        self.policy \u003d object_policy"},{"line_number":450,"context_line":"        self.property_repo \u003d property_repo"},{"line_number":451,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":452,"context_line":"        super(MetadefPropertyRepoProxy, self).__init__("},{"line_number":453,"context_line":"            property_repo,"},{"line_number":454,"context_line":"            property_proxy_class\u003dMetadefPropertyProxy,"},{"line_number":455,"context_line":"            property_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"class MetadefPropertyFactoryProxy(glance.domain.proxy.MetadefPropertyFactory):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9c825ae9_796dda3c","line":455,"range":{"start_line":445,"start_character":0,"end_line":455,"end_character":47},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":442,"context_line":"        super(MetadefPropertyProxy, self).__init__(namespace_property)"},{"line_number":443,"context_line":""},{"line_number":444,"context_line":""},{"line_number":445,"context_line":"class MetadefPropertyRepoProxy(glance.domain.proxy.MetadefPropertyRepo):"},{"line_number":446,"context_line":""},{"line_number":447,"context_line":"    def __init__(self, property_repo, context, object_policy):"},{"line_number":448,"context_line":"        self.context \u003d context"},{"line_number":449,"context_line":"        self.policy \u003d object_policy"},{"line_number":450,"context_line":"        self.property_repo \u003d property_repo"},{"line_number":451,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":452,"context_line":"        super(MetadefPropertyRepoProxy, self).__init__("},{"line_number":453,"context_line":"            property_repo,"},{"line_number":454,"context_line":"            property_proxy_class\u003dMetadefPropertyProxy,"},{"line_number":455,"context_line":"            property_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"class MetadefPropertyFactoryProxy(glance.domain.proxy.MetadefPropertyFactory):"}],"source_content_type":"text/x-python","patch_set":2,"id":"39abc3a9_648826c4","line":455,"range":{"start_line":445,"start_character":0,"end_line":455,"end_character":47},"in_reply_to":"9c825ae9_796dda3c","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":454,"context_line":"            property_proxy_class\u003dMetadefPropertyProxy,"},{"line_number":455,"context_line":"            property_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"class MetadefPropertyFactoryProxy(glance.domain.proxy.MetadefPropertyFactory):"},{"line_number":459,"context_line":""},{"line_number":460,"context_line":"    def __init__(self, namespace_property_factory, context, policy):"},{"line_number":461,"context_line":"        self.namespace_property_factory \u003d namespace_property_factory"},{"line_number":462,"context_line":"        self.context \u003d context"},{"line_number":463,"context_line":"        self.policy \u003d policy"},{"line_number":464,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":465,"context_line":"        super(MetadefPropertyFactoryProxy, self).__init__("},{"line_number":466,"context_line":"            namespace_property_factory,"},{"line_number":467,"context_line":"            property_proxy_class\u003dMetadefPropertyProxy,"},{"line_number":468,"context_line":"            property_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":469,"context_line":""},{"line_number":470,"context_line":""},{"line_number":471,"context_line":"# Metadef Tag classes"},{"line_number":472,"context_line":"class MetadefTagProxy(glance.domain.proxy.MetadefTag):"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"    def __init__(self, meta_tag, context, policy):"},{"line_number":475,"context_line":"        self.context \u003d context"},{"line_number":476,"context_line":"        self.policy \u003d policy"},{"line_number":477,"context_line":"        super(MetadefTagProxy, self).__init__(meta_tag)"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":""},{"line_number":480,"context_line":"class MetadefTagRepoProxy(glance.domain.proxy.MetadefTagRepo):"}],"source_content_type":"text/x-python","patch_set":2,"id":"343f972a_7d46fe0b","line":477,"range":{"start_line":457,"start_character":0,"end_line":477,"end_character":55},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":454,"context_line":"            property_proxy_class\u003dMetadefPropertyProxy,"},{"line_number":455,"context_line":"            property_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":456,"context_line":""},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"class MetadefPropertyFactoryProxy(glance.domain.proxy.MetadefPropertyFactory):"},{"line_number":459,"context_line":""},{"line_number":460,"context_line":"    def __init__(self, namespace_property_factory, context, policy):"},{"line_number":461,"context_line":"        self.namespace_property_factory \u003d namespace_property_factory"},{"line_number":462,"context_line":"        self.context \u003d context"},{"line_number":463,"context_line":"        self.policy \u003d policy"},{"line_number":464,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":465,"context_line":"        super(MetadefPropertyFactoryProxy, self).__init__("},{"line_number":466,"context_line":"            namespace_property_factory,"},{"line_number":467,"context_line":"            property_proxy_class\u003dMetadefPropertyProxy,"},{"line_number":468,"context_line":"            property_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":469,"context_line":""},{"line_number":470,"context_line":""},{"line_number":471,"context_line":"# Metadef Tag classes"},{"line_number":472,"context_line":"class MetadefTagProxy(glance.domain.proxy.MetadefTag):"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"    def __init__(self, meta_tag, context, policy):"},{"line_number":475,"context_line":"        self.context \u003d context"},{"line_number":476,"context_line":"        self.policy \u003d policy"},{"line_number":477,"context_line":"        super(MetadefTagProxy, self).__init__(meta_tag)"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":""},{"line_number":480,"context_line":"class MetadefTagRepoProxy(glance.domain.proxy.MetadefTagRepo):"}],"source_content_type":"text/x-python","patch_set":2,"id":"14c97739_229cafe5","line":477,"range":{"start_line":457,"start_character":0,"end_line":477,"end_character":55},"in_reply_to":"343f972a_7d46fe0b","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":476,"context_line":"        self.policy \u003d policy"},{"line_number":477,"context_line":"        super(MetadefTagProxy, self).__init__(meta_tag)"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":""},{"line_number":480,"context_line":"class MetadefTagRepoProxy(glance.domain.proxy.MetadefTagRepo):"},{"line_number":481,"context_line":""},{"line_number":482,"context_line":"    def __init__(self, tag_repo, context, tag_policy):"},{"line_number":483,"context_line":"        self.context \u003d context"},{"line_number":484,"context_line":"        self.policy \u003d tag_policy"},{"line_number":485,"context_line":"        self.tag_repo \u003d tag_repo"},{"line_number":486,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":487,"context_line":"        super(MetadefTagRepoProxy,"},{"line_number":488,"context_line":"              self).__init__(tag_repo,"},{"line_number":489,"context_line":"                             tag_proxy_class\u003dMetadefTagProxy,"},{"line_number":490,"context_line":"                             tag_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":491,"context_line":""},{"line_number":492,"context_line":""},{"line_number":493,"context_line":"class MetadefTagFactoryProxy(glance.domain.proxy.MetadefTagFactory):"}],"source_content_type":"text/x-python","patch_set":2,"id":"70063c54_bf115417","line":490,"range":{"start_line":479,"start_character":0,"end_line":490,"end_character":59},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":476,"context_line":"        self.policy \u003d policy"},{"line_number":477,"context_line":"        super(MetadefTagProxy, self).__init__(meta_tag)"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":""},{"line_number":480,"context_line":"class MetadefTagRepoProxy(glance.domain.proxy.MetadefTagRepo):"},{"line_number":481,"context_line":""},{"line_number":482,"context_line":"    def __init__(self, tag_repo, context, tag_policy):"},{"line_number":483,"context_line":"        self.context \u003d context"},{"line_number":484,"context_line":"        self.policy \u003d tag_policy"},{"line_number":485,"context_line":"        self.tag_repo \u003d tag_repo"},{"line_number":486,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":487,"context_line":"        super(MetadefTagRepoProxy,"},{"line_number":488,"context_line":"              self).__init__(tag_repo,"},{"line_number":489,"context_line":"                             tag_proxy_class\u003dMetadefTagProxy,"},{"line_number":490,"context_line":"                             tag_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":491,"context_line":""},{"line_number":492,"context_line":""},{"line_number":493,"context_line":"class MetadefTagFactoryProxy(glance.domain.proxy.MetadefTagFactory):"}],"source_content_type":"text/x-python","patch_set":2,"id":"bd15099a_7eb22536","line":490,"range":{"start_line":479,"start_character":0,"end_line":490,"end_character":59},"in_reply_to":"70063c54_bf115417","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"585332a6209f3ccaf0c73a7cb589cbeaf99cb020","unresolved":true,"context_lines":[{"line_number":490,"context_line":"                             tag_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":491,"context_line":""},{"line_number":492,"context_line":""},{"line_number":493,"context_line":"class MetadefTagFactoryProxy(glance.domain.proxy.MetadefTagFactory):"},{"line_number":494,"context_line":""},{"line_number":495,"context_line":"    def __init__(self, meta_tag_factory, context, policy):"},{"line_number":496,"context_line":"        self.meta_tag_factory \u003d meta_tag_factory"},{"line_number":497,"context_line":"        self.context \u003d context"},{"line_number":498,"context_line":"        self.policy \u003d policy"},{"line_number":499,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":500,"context_line":"        super(MetadefTagFactoryProxy, self).__init__("},{"line_number":501,"context_line":"            meta_tag_factory,"},{"line_number":502,"context_line":"            meta_tag_proxy_class\u003dMetadefTagProxy,"},{"line_number":503,"context_line":"            meta_tag_proxy_kwargs\u003dproxy_kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"f3f796e2_07a0d1db","line":503,"range":{"start_line":493,"start_character":0,"end_line":503,"end_character":47},"updated":"2022-06-09 05:30:37.000000000","message":"ditto","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"a6f67ca9f9463708fdb3a6c1dd63d1967be85823","unresolved":false,"context_lines":[{"line_number":490,"context_line":"                             tag_proxy_kwargs\u003dproxy_kwargs)"},{"line_number":491,"context_line":""},{"line_number":492,"context_line":""},{"line_number":493,"context_line":"class MetadefTagFactoryProxy(glance.domain.proxy.MetadefTagFactory):"},{"line_number":494,"context_line":""},{"line_number":495,"context_line":"    def __init__(self, meta_tag_factory, context, policy):"},{"line_number":496,"context_line":"        self.meta_tag_factory \u003d meta_tag_factory"},{"line_number":497,"context_line":"        self.context \u003d context"},{"line_number":498,"context_line":"        self.policy \u003d policy"},{"line_number":499,"context_line":"        proxy_kwargs \u003d {\u0027context\u0027: self.context, \u0027policy\u0027: self.policy}"},{"line_number":500,"context_line":"        super(MetadefTagFactoryProxy, self).__init__("},{"line_number":501,"context_line":"            meta_tag_factory,"},{"line_number":502,"context_line":"            meta_tag_proxy_class\u003dMetadefTagProxy,"},{"line_number":503,"context_line":"            meta_tag_proxy_kwargs\u003dproxy_kwargs)"}],"source_content_type":"text/x-python","patch_set":2,"id":"c243c567_b0ff6cb2","line":503,"range":{"start_line":493,"start_character":0,"end_line":503,"end_character":47},"in_reply_to":"f3f796e2_07a0d1db","updated":"2022-06-09 07:46:30.000000000","message":"Ack","commit_id":"1f13913d7c0fb3048bc5306653b6a613143ca5ab"}],"glance/gateway.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"9497d82232b64c2befcd4b7944532cbc4ee51eea","unresolved":true,"context_lines":[{"line_number":35,"context_line":"        self.notifier \u003d notifier or glance.notifier.Notifier()"},{"line_number":36,"context_line":"        self.policy \u003d policy_enforcer or policy.Enforcer()"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def get_image_factory(self, context, authorization_layer\u003dFalse):"},{"line_number":39,"context_line":"        factory \u003d glance.domain.ImageFactory()"},{"line_number":40,"context_line":"        factory \u003d glance.location.ImageFactoryProxy("},{"line_number":41,"context_line":"            factory, context, self.store_api, self.store_utils)"}],"source_content_type":"text/x-python","patch_set":3,"id":"ca8a8dea_c8bafae8","line":38,"range":{"start_line":38,"start_character":41,"end_line":38,"end_character":66},"updated":"2022-06-09 10:00:20.000000000","message":"why this parameter is still here?\nI think you can get rid of this as well now, there is no use of keeping it anymore.","commit_id":"ca474d23f91eceeef1c0d40fde5a54a9cf4aa90a"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"67e32ffe47a5a4ed4bb93a051767f6bc401a24e2","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        self.notifier \u003d notifier or glance.notifier.Notifier()"},{"line_number":36,"context_line":"        self.policy \u003d policy_enforcer or policy.Enforcer()"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def get_image_factory(self, context, authorization_layer\u003dFalse):"},{"line_number":39,"context_line":"        factory \u003d glance.domain.ImageFactory()"},{"line_number":40,"context_line":"        factory \u003d glance.location.ImageFactoryProxy("},{"line_number":41,"context_line":"            factory, context, self.store_api, self.store_utils)"}],"source_content_type":"text/x-python","patch_set":3,"id":"71790282_9e5b1e88","line":38,"range":{"start_line":38,"start_character":41,"end_line":38,"end_character":66},"in_reply_to":"ca8a8dea_c8bafae8","updated":"2022-06-09 12:02:56.000000000","message":"Done","commit_id":"ca474d23f91eceeef1c0d40fde5a54a9cf4aa90a"}],"glance/tests/unit/test_gateway.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"79dd06115b2557deb8c2f5cd445161cc9c5987d0","unresolved":true,"context_lines":[{"line_number":31,"context_line":"        self.gateway \u003d gateway.Gateway()"},{"line_number":32,"context_line":"        self.context \u003d mock.sentinel.context"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def test_get_repo_without_auth(self):"},{"line_number":35,"context_line":"        repo \u003d self.gateway.get_repo(self.context)"},{"line_number":36,"context_line":"        self.assertIsInstance(repo, notifier.ImageRepoProxy)"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"b393442b_24e1c2a5","line":34,"range":{"start_line":34,"start_character":21,"end_line":34,"end_character":34},"updated":"2022-06-09 14:31:52.000000000","message":"While this is technically true, these \"_without_auth\" versions were to test the different behavior of passing auth_layer\u003dFalse when it was not the default. Now that it\u0027s not an option, I think these should really be just \"test_get_repo()\" (etc) and not have the suffix attached to each one.\n\nIt\u0027s a minor thing, but...","commit_id":"e537d1a7c1e400e6bb11631b44af30d49708e444"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"246f1a72e815ac95463a90b7b28d39a442a631f8","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        self.gateway \u003d gateway.Gateway()"},{"line_number":32,"context_line":"        self.context \u003d mock.sentinel.context"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def test_get_repo_without_auth(self):"},{"line_number":35,"context_line":"        repo \u003d self.gateway.get_repo(self.context)"},{"line_number":36,"context_line":"        self.assertIsInstance(repo, notifier.ImageRepoProxy)"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"adb48e98_4db75838","line":34,"range":{"start_line":34,"start_character":21,"end_line":34,"end_character":34},"in_reply_to":"b393442b_24e1c2a5","updated":"2022-06-13 07:05:29.000000000","message":"Ack","commit_id":"e537d1a7c1e400e6bb11631b44af30d49708e444"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"3a71c9a69ebcf2ec15f247a39b9195c67496fd3d","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    def setUp(self):"},{"line_number":30,"context_line":"        super(TestGateway, self).setUp()"},{"line_number":31,"context_line":"        self.gateway \u003d gateway.Gateway()"},{"line_number":32,"context_line":"        self.context \u003d mock.sentinel.context"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    @mock.patch(\u0027glance.domain.TaskExecutorFactory\u0027)"},{"line_number":35,"context_line":"    def test_get_task_executor_factory(self, mock_factory):"},{"line_number":36,"context_line":"        @mock.patch.object(self.gateway, \u0027get_task_repo\u0027)"},{"line_number":37,"context_line":"        @mock.patch.object(self.gateway, \u0027get_repo\u0027)"},{"line_number":38,"context_line":"        @mock.patch.object(self.gateway, \u0027get_image_factory\u0027)"},{"line_number":39,"context_line":"        def _test(mock_gif, mock_gr, mock_gtr):"},{"line_number":40,"context_line":"            self.gateway.get_task_executor_factory(self.context)"},{"line_number":41,"context_line":"            mock_gtr.assert_called_once_with("},{"line_number":42,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":43,"context_line":"            mock_gr.assert_called_once_with("},{"line_number":44,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":45,"context_line":"            mock_gif.assert_called_once_with("},{"line_number":46,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":47,"context_line":"            mock_factory.assert_called_once_with("},{"line_number":48,"context_line":"                mock_gtr.return_value,"},{"line_number":49,"context_line":"                mock_gr.return_value,"},{"line_number":50,"context_line":"                mock_gif.return_value,"},{"line_number":51,"context_line":"                admin_repo\u003dNone)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        _test()"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @mock.patch(\u0027glance.domain.TaskExecutorFactory\u0027)"},{"line_number":56,"context_line":"    def test_get_task_executor_factory_with_admin(self, mock_factory):"},{"line_number":57,"context_line":"        @mock.patch.object(self.gateway, \u0027get_task_repo\u0027)"},{"line_number":58,"context_line":"        @mock.patch.object(self.gateway, \u0027get_repo\u0027)"},{"line_number":59,"context_line":"        @mock.patch.object(self.gateway, \u0027get_image_factory\u0027)"},{"line_number":60,"context_line":"        def _test(mock_gif, mock_gr, mock_gtr):"},{"line_number":61,"context_line":"            mock_gr.side_effect \u003d [mock.sentinel.image_repo,"},{"line_number":62,"context_line":"                                   mock.sentinel.admin_repo]"},{"line_number":63,"context_line":"            self.gateway.get_task_executor_factory("},{"line_number":64,"context_line":"                self.context,"},{"line_number":65,"context_line":"                admin_context\u003dmock.sentinel.admin_context)"},{"line_number":66,"context_line":"            mock_gtr.assert_called_once_with("},{"line_number":67,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":68,"context_line":"            mock_gr.assert_has_calls(["},{"line_number":69,"context_line":"                mock.call(self.context, authorization_layer\u003dTrue),"},{"line_number":70,"context_line":"                mock.call(mock.sentinel.admin_context,"},{"line_number":71,"context_line":"                          authorization_layer\u003dTrue),"},{"line_number":72,"context_line":"            ])"},{"line_number":73,"context_line":"            mock_gif.assert_called_once_with("},{"line_number":74,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":75,"context_line":"            mock_factory.assert_called_once_with("},{"line_number":76,"context_line":"                mock_gtr.return_value,"},{"line_number":77,"context_line":"                mock.sentinel.image_repo,"},{"line_number":78,"context_line":"                mock_gif.return_value,"},{"line_number":79,"context_line":"                admin_repo\u003dmock.sentinel.admin_repo)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        _test()"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    @mock.patch(\u0027glance.api.policy.ImageRepoProxy\u0027)"},{"line_number":84,"context_line":"    def test_get_repo(self, mock_proxy):"}],"source_content_type":"text/x-python","patch_set":6,"id":"c88307d4_70cb5f54","side":"PARENT","line":81,"range":{"start_line":32,"start_character":44,"end_line":81,"end_character":15},"updated":"2022-06-13 14:13:26.000000000","message":"Sorry Pranali,\n\nI overlooked these in previous patches, but you need to keep these tests and just remove authorization_layer\u003dTrue from it.","commit_id":"ebef97d776eb1982b948658eaf20566853dc70a9"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"ad2b8bae72ccd721bdc7077067bbc1e75f8d34a7","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    def setUp(self):"},{"line_number":30,"context_line":"        super(TestGateway, self).setUp()"},{"line_number":31,"context_line":"        self.gateway \u003d gateway.Gateway()"},{"line_number":32,"context_line":"        self.context \u003d mock.sentinel.context"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    @mock.patch(\u0027glance.domain.TaskExecutorFactory\u0027)"},{"line_number":35,"context_line":"    def test_get_task_executor_factory(self, mock_factory):"},{"line_number":36,"context_line":"        @mock.patch.object(self.gateway, \u0027get_task_repo\u0027)"},{"line_number":37,"context_line":"        @mock.patch.object(self.gateway, \u0027get_repo\u0027)"},{"line_number":38,"context_line":"        @mock.patch.object(self.gateway, \u0027get_image_factory\u0027)"},{"line_number":39,"context_line":"        def _test(mock_gif, mock_gr, mock_gtr):"},{"line_number":40,"context_line":"            self.gateway.get_task_executor_factory(self.context)"},{"line_number":41,"context_line":"            mock_gtr.assert_called_once_with("},{"line_number":42,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":43,"context_line":"            mock_gr.assert_called_once_with("},{"line_number":44,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":45,"context_line":"            mock_gif.assert_called_once_with("},{"line_number":46,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":47,"context_line":"            mock_factory.assert_called_once_with("},{"line_number":48,"context_line":"                mock_gtr.return_value,"},{"line_number":49,"context_line":"                mock_gr.return_value,"},{"line_number":50,"context_line":"                mock_gif.return_value,"},{"line_number":51,"context_line":"                admin_repo\u003dNone)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        _test()"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @mock.patch(\u0027glance.domain.TaskExecutorFactory\u0027)"},{"line_number":56,"context_line":"    def test_get_task_executor_factory_with_admin(self, mock_factory):"},{"line_number":57,"context_line":"        @mock.patch.object(self.gateway, \u0027get_task_repo\u0027)"},{"line_number":58,"context_line":"        @mock.patch.object(self.gateway, \u0027get_repo\u0027)"},{"line_number":59,"context_line":"        @mock.patch.object(self.gateway, \u0027get_image_factory\u0027)"},{"line_number":60,"context_line":"        def _test(mock_gif, mock_gr, mock_gtr):"},{"line_number":61,"context_line":"            mock_gr.side_effect \u003d [mock.sentinel.image_repo,"},{"line_number":62,"context_line":"                                   mock.sentinel.admin_repo]"},{"line_number":63,"context_line":"            self.gateway.get_task_executor_factory("},{"line_number":64,"context_line":"                self.context,"},{"line_number":65,"context_line":"                admin_context\u003dmock.sentinel.admin_context)"},{"line_number":66,"context_line":"            mock_gtr.assert_called_once_with("},{"line_number":67,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":68,"context_line":"            mock_gr.assert_has_calls(["},{"line_number":69,"context_line":"                mock.call(self.context, authorization_layer\u003dTrue),"},{"line_number":70,"context_line":"                mock.call(mock.sentinel.admin_context,"},{"line_number":71,"context_line":"                          authorization_layer\u003dTrue),"},{"line_number":72,"context_line":"            ])"},{"line_number":73,"context_line":"            mock_gif.assert_called_once_with("},{"line_number":74,"context_line":"                self.context, authorization_layer\u003dTrue)"},{"line_number":75,"context_line":"            mock_factory.assert_called_once_with("},{"line_number":76,"context_line":"                mock_gtr.return_value,"},{"line_number":77,"context_line":"                mock.sentinel.image_repo,"},{"line_number":78,"context_line":"                mock_gif.return_value,"},{"line_number":79,"context_line":"                admin_repo\u003dmock.sentinel.admin_repo)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        _test()"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"    @mock.patch(\u0027glance.api.policy.ImageRepoProxy\u0027)"},{"line_number":84,"context_line":"    def test_get_repo(self, mock_proxy):"}],"source_content_type":"text/x-python","patch_set":6,"id":"752f8b01_bdc555f7","side":"PARENT","line":81,"range":{"start_line":32,"start_character":44,"end_line":81,"end_character":15},"in_reply_to":"c88307d4_70cb5f54","updated":"2022-06-14 10:33:21.000000000","message":"ohh no problem, I should have checked it carefully.","commit_id":"ebef97d776eb1982b948658eaf20566853dc70a9"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"3a71c9a69ebcf2ec15f247a39b9195c67496fd3d","unresolved":true,"context_lines":[{"line_number":117,"context_line":"        self.assertIsInstance(factory,"},{"line_number":118,"context_line":"                              property_protections.ProtectedImageFactoryProxy)"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    @mock.patch(\u0027glance.api.policy.ImageFactoryProxy\u0027)"},{"line_number":121,"context_line":"    def test_get_image_factory_policy_layer(self, mock_pif):"},{"line_number":122,"context_line":"        self.gateway.get_image_factory(self.context, authorization_layer\u003dFalse)"},{"line_number":123,"context_line":"        mock_pif.assert_not_called()"},{"line_number":124,"context_line":"        self.gateway.get_image_factory(self.context)"},{"line_number":125,"context_line":"        self.assertTrue(mock_pif.called)"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"    def test_get_repo_member_property(self):"},{"line_number":128,"context_line":"        \"\"\"Test that the image.member property is propagated all the way from"}],"source_content_type":"text/x-python","patch_set":6,"id":"25d046af_8f7ac04e","side":"PARENT","line":125,"range":{"start_line":120,"start_character":4,"end_line":125,"end_character":40},"updated":"2022-06-13 14:13:26.000000000","message":"I think same goes with this.","commit_id":"ebef97d776eb1982b948658eaf20566853dc70a9"}],"glance/tests/unit/v2/test_metadef_resources.py":[{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"27b64c3d84eb7b82fb93928d773dd935fc74e49d","unresolved":true,"context_lines":[{"line_number":26,"context_line":"from glance.api.v2 import metadef_properties as properties"},{"line_number":27,"context_line":"from glance.api.v2 import metadef_resource_types as resource_types"},{"line_number":28,"context_line":"from glance.api.v2 import metadef_tags as tags"},{"line_number":29,"context_line":"import glance.gateway"},{"line_number":30,"context_line":"from glance.tests.unit import base"},{"line_number":31,"context_line":"import glance.tests.unit.utils as unit_test_utils"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"eadfaaeb_645c72b2","side":"PARENT","line":29,"updated":"2022-06-09 13:57:03.000000000","message":"Is this removed by mistake?","commit_id":"3840fa8d3fc2a6744744a35149a92b2c1ee37701"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"246f1a72e815ac95463a90b7b28d39a442a631f8","unresolved":true,"context_lines":[{"line_number":26,"context_line":"from glance.api.v2 import metadef_properties as properties"},{"line_number":27,"context_line":"from glance.api.v2 import metadef_resource_types as resource_types"},{"line_number":28,"context_line":"from glance.api.v2 import metadef_tags as tags"},{"line_number":29,"context_line":"import glance.gateway"},{"line_number":30,"context_line":"from glance.tests.unit import base"},{"line_number":31,"context_line":"import glance.tests.unit.utils as unit_test_utils"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"f6bb6154_7172366a","side":"PARENT","line":29,"in_reply_to":"eadfaaeb_645c72b2","updated":"2022-06-13 07:05:29.000000000","message":"nope, initially i thought the below 2 tests are not required.\nNow modified those.","commit_id":"3840fa8d3fc2a6744744a35149a92b2c1ee37701"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"22d9386d3016ada2ff6dab10171b88588dbd6638","unresolved":true,"context_lines":[{"line_number":444,"context_line":"                                              policy_enforcer\u003dself.policy)"},{"line_number":445,"context_line":"        req \u003d unit_test_utils.get_fake_request(roles\u003d[\u0027admin\u0027])"},{"line_number":446,"context_line":"        namespace \u003d namespaces.Namespace()"},{"line_number":447,"context_line":"        namespace.namespace \u003d NAMESPACE4"},{"line_number":448,"context_line":"        namespace \u003d self.namespace_controller.create(req, namespace)"},{"line_number":449,"context_line":""},{"line_number":450,"context_line":"        ns_repo \u003d fake_gateway.get_metadef_namespace_repo(req.context)"}],"source_content_type":"text/x-python","patch_set":5,"id":"e86c6e5f_c78c10cc","line":447,"range":{"start_line":447,"start_character":30,"end_line":447,"end_character":40},"updated":"2022-06-13 09:02:32.000000000","message":"Any specific reason changing this to NAMESPACE4 here and not in below test at line 466?","commit_id":"33fe7d5aebee61f95e69a00986068bad9cede11e"},{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"ee63e8ec8145079eeba269d1a44bab9d7873afb7","unresolved":true,"context_lines":[{"line_number":444,"context_line":"                                              policy_enforcer\u003dself.policy)"},{"line_number":445,"context_line":"        req \u003d unit_test_utils.get_fake_request(roles\u003d[\u0027admin\u0027])"},{"line_number":446,"context_line":"        namespace \u003d namespaces.Namespace()"},{"line_number":447,"context_line":"        namespace.namespace \u003d NAMESPACE4"},{"line_number":448,"context_line":"        namespace \u003d self.namespace_controller.create(req, namespace)"},{"line_number":449,"context_line":""},{"line_number":450,"context_line":"        ns_repo \u003d fake_gateway.get_metadef_namespace_repo(req.context)"}],"source_content_type":"text/x-python","patch_set":5,"id":"8fd7b01b_9bed56ae","line":447,"range":{"start_line":447,"start_character":30,"end_line":447,"end_character":40},"in_reply_to":"e86c6e5f_c78c10cc","updated":"2022-06-13 10:53:43.000000000","message":"ahuu nothing specific, i accidentally changed it. will change it to the older one.","commit_id":"33fe7d5aebee61f95e69a00986068bad9cede11e"}]}