)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3fb2ddec3aa147f5f058ba85425c0f6ee7a500c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c65214a7_5a10a56e","updated":"2022-02-08 20:28:00.000000000","message":"recheck","commit_id":"ac79fee5de09305e540313b7a35caec35ec15262"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"98850525575b91c605dc7400a04240f12102977f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"48387334_e2cbcb3f","updated":"2022-03-04 21:02:45.000000000","message":"recheck","commit_id":"2a0be5c71516581acd79a59fb819be648fd2559c"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"6cd5a4722b5d7bcae4982d4632f794ffc450d6bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"dd82e515_0f2628e0","updated":"2022-06-23 19:44:55.000000000","message":"recheck lvm test failures","commit_id":"11bd7a342ae5a8f5a1f07a74167f6e85bac89124"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"a948d41c057c7df42c99252e2fced2b482f6b1a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"85364947_3af4245f","updated":"2022-08-31 12:56:20.000000000","message":"Additional comment.","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"9d9340f802e52f997b31163cdc12d501c0c20432","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"396eb07c_7330fad5","updated":"2022-08-26 15:56:56.000000000","message":"Looks good, with additional comments. Failures in non-voting cephfs-nfs are unrelated.","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"75e1e617_fb1db06d","updated":"2022-08-25 07:52:37.000000000","message":"Thanks Ashley!","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"eb1071d1b077552ceca33f56bb73a69cc005cf1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"f89990b6_281003a1","updated":"2022-09-09 19:23:50.000000000","message":"recheck\nIssues in the core patch were fixed","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"d4492777af22f2eeb4fb0469ac38d394ce01f66c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"12c612e3_eff3b77c","updated":"2022-09-06 20:33:54.000000000","message":"recheck\nLogs are one month old - Let\u0027s get a new clean result :)","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"64f554491db48eb8f4d79e496e69cfde74b84c0f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"5f6a212e_3abe9ca5","updated":"2022-09-09 19:31:13.000000000","message":"Just fixed up the microversion stuff because the server was bumped","commit_id":"1a53e017a8a8a1a6cfbcc0be7d2d219024e6fcd9"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"7fd012f94a58f63eea09fcc4dad599cb91b4e94a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"267eb485_bdef4754","updated":"2022-09-21 07:23:56.000000000","message":"LGTM.","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"df40f7eb71b7f79999f0cec5ea5708dda4ca5b74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"5b5e2127_b97822d3","updated":"2022-09-22 19:26:49.000000000","message":"Overall looks good! Some minor comments that I wouldn\u0027t mind being addressed in a follow-up change. Thanks!","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f057efd26b7f262862f83d77d61dc10cfd47cbe8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"6377cde6_8d8d752e","updated":"2022-09-22 22:44:18.000000000","message":"Thanks Ashley, some comments inline.. ","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"0054f3a73815c8b0cf6e03bc14373121a2fcfe15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"b1e6095f_2e657d6e","updated":"2022-09-21 07:24:05.000000000","message":"recheck","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"043feb0a3e066233512ac5f30940d620d13f5779","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"915d71ef_a37db2ad","updated":"2022-09-28 23:01:24.000000000","message":"Waiting on CI and a response regarding some test case deletions","commit_id":"c9ba890aea50e1d0419ca3368e4143538e9e4cc9"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"49a13da5b45296575ed2b3e1d67241ab7810a436","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"7eb6f03a_4f67c260","updated":"2022-09-28 22:03:07.000000000","message":"recheck\nKnown test failure","commit_id":"c9ba890aea50e1d0419ca3368e4143538e9e4cc9"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"5a11f5f390f629f032be1800a34125b52b125b20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"56f69b22_7b574933","updated":"2022-10-10 19:46:00.000000000","message":"Getting back after some more investigation. Please let me know if you have questions","commit_id":"3f894934108758ef9d8181945a1424325708e994"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"c62be3dca4697759ea8c08a641900fbd7968f026","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"31e195ad_ac4453b7","updated":"2022-10-07 19:58:35.000000000","message":"Hello, I see this is failing in the rbac tests and I believe this is an issue in the rbac tests themselves.\nWe are currently sending the request with do_request in the rbac tests [1], when we could be using the new methods implemented in the json_v2_client in this change\nThis may be something this change will need to address too, but I have a feeling that it would not possibly be appropriate considering that one change should not address two issues at once. Let\u0027s discuss it further here or on IRC :)\n\n\n[1] https://github.com/openstack/manila-tempest-plugin/blob/master/manila_tempest_tests/tests/rbac/test_shares.py#L308","commit_id":"3f894934108758ef9d8181945a1424325708e994"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d1eeae955209160fadbe6f1f4b67a2a42260063","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"c923bfb1_972af81d","updated":"2022-10-06 18:51:53.000000000","message":"recheck tests failed to run","commit_id":"3f894934108758ef9d8181945a1424325708e994"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"5a11f5f390f629f032be1800a34125b52b125b20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"1a259287_bd173566","in_reply_to":"31e195ad_ac4453b7","updated":"2022-10-10 19:46:00.000000000","message":"\u003e Hello, I see this is failing in the rbac tests and I believe this is an issue in the rbac tests themselves.\n\u003e We are currently sending the request with do_request in the rbac tests [1], when we could be using the new methods implemented in the json_v2_client in this change\n\u003e This may be something this change will need to address too, but I have a feeling that it would not possibly be appropriate considering that one change should not address two issues at once. Let\u0027s discuss it further here or on IRC :)\n\u003e \n\u003e \n\u003e [1] https://github.com/openstack/manila-tempest-plugin/blob/master/manila_tempest_tests/tests/rbac/test_shares.py#L308\n\nCircling back to this - I think I was mistaken, let me expand:\n\nThis change is currently doing some overrides in the metadata methods (in the json client). The method previously sent the requests for shares metadata in a certain way. Our metadata APIs have been changing over the past few months, and this change is implementing new tests, modifying the way we do requests to reflect our APIs in a smart and reusable way for shares, snapshots and some future resources that can come to have metadata.\n\nThis change [1] introduced some tests in the past weeks. These RBAC tests also contain a set of metadata tests for shares, which are referring to the old way we used to send requests, making the RBAC tests to fail. In order to have the tests passing again, we will need to take some actions:\n\n- Rebase this change\n\nThis will help to reproduce issues in a local environment and also helps with keeping the change up-to-date\n\n- Modify the RBAC share metadata tests\n\nThe tests doing requests like this [2] should be modified to used the methods being re-written in the json v2 client.\n\nI believe that should solve the issue :)\n\n[1] https://review.opendev.org/c/openstack/manila-tempest-plugin/+/805938\n[2] https://github.com/openstack/manila-tempest-plugin/blob/master/manila_tempest_tests/tests/rbac/test_shares.py#L308-L310","commit_id":"3f894934108758ef9d8181945a1424325708e994"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"e0200a9221e230543579094cd2a476797560f40a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"99b984ce_b9927922","in_reply_to":"56f69b22_7b574933","updated":"2022-10-19 09:21:05.000000000","message":"Hi Ashley,\n\nAs Carlos mentioned the rbac tests use the new implementation of metadata actions of v2 client.\nSo I guess all you need is to replace \u0027share_id\u0027 argument by \u0027resource_id\u0027, for instance:\n\n        self.do_request(\n            \u0027delete_metadata\u0027, expected_status\u003d200, share_id\u003dshare[\u0027id\u0027],\n            key\u003d\u0027key\u0027)\n\nChange to:\n\n        self.do_request(\n            \u0027delete_metadata\u0027, expected_status\u003d200, resource_id\u003dshare[\u0027id\u0027],\n            key\u003d\u0027key\u0027)","commit_id":"3f894934108758ef9d8181945a1424325708e994"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"6d93bb94_44c64bfc","updated":"2022-10-19 15:06:46.000000000","message":"Thanks Liron, Carlos for your comments! ","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"9aed2abf_f9872048","updated":"2022-10-19 15:34:53.000000000","message":"We need to change it in other places.","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":30,"id":"8ed3aef3_91b612c5","updated":"2022-10-20 10:58:52.000000000","message":"Thanks Ashely,\nPlease see bellow very small comments.","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"55c19277e18fc4ff988cf6a9dcb85a604e191b77","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"cf67f098_c16e7dfc","updated":"2022-11-08 18:59:21.000000000","message":"LGTM; CI is happy\nThank you, Ashley! :)","commit_id":"c45cb4b3d25e87ec24df07683c6b90977091534f"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"94dbee9a306422a70fd7091d6c2f3beee2b24077","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"8acaf839_6341b814","updated":"2022-11-10 10:58:50.000000000","message":"Thanks Ashley for the hard work!","commit_id":"c45cb4b3d25e87ec24df07683c6b90977091534f"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1d37587435a0d48d45099356bad93be4cfb4e6aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"9cd1e21f_29226a9f","updated":"2022-11-02 15:02:14.000000000","message":"Thanks for the help Liron!","commit_id":"c45cb4b3d25e87ec24df07683c6b90977091534f"},{"author":{"_account_id":30407,"name":"haixin","email":"haixin_haixin@qq.com","username":"haixin"},"change_message_id":"3c2501731d2f1f833b29f0ae9ea18921c9779e7e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"57beec36_8bdd15e9","updated":"2022-11-16 05:44:43.000000000","message":"also LGTM. let\u0027s merge this patch 😊","commit_id":"c45cb4b3d25e87ec24df07683c6b90977091534f"}],"manila_tempest_tests/services/share/v2/json/shares_client.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"df40f7eb71b7f79999f0cec5ea5708dda4ca5b74","unresolved":true,"context_lines":[{"line_number":531,"context_line":"            }"},{"line_number":532,"context_line":"        }"},{"line_number":533,"context_line":"        if utils.is_microversion_ge(version, \"2.73\"):"},{"line_number":534,"context_line":"            metadata \u003d metadata or {}"},{"line_number":535,"context_line":"            post_body[\"snapshot\"][\"metadata\"] \u003d metadata"},{"line_number":536,"context_line":""},{"line_number":537,"context_line":"        body \u003d json.dumps(post_body)"}],"source_content_type":"text/x-python","patch_set":24,"id":"ada74aeb_5741882e","line":534,"range":{"start_line":534,"start_character":12,"end_line":534,"end_character":37},"updated":"2022-09-22 19:26:49.000000000","message":"is metadata a mandatory item to send in the body now? If it is not, we could remove this and do:\nif utils.is_microversion_ge(version, \"2.73\") and metadata:\n    post_body[\"snapshot\"][\"metadata\"] \u003d metadata","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d7f2d9d6cd4b364916995b3a215d444c149d68c","unresolved":false,"context_lines":[{"line_number":531,"context_line":"            }"},{"line_number":532,"context_line":"        }"},{"line_number":533,"context_line":"        if utils.is_microversion_ge(version, \"2.73\"):"},{"line_number":534,"context_line":"            metadata \u003d metadata or {}"},{"line_number":535,"context_line":"            post_body[\"snapshot\"][\"metadata\"] \u003d metadata"},{"line_number":536,"context_line":""},{"line_number":537,"context_line":"        body \u003d json.dumps(post_body)"}],"source_content_type":"text/x-python","patch_set":24,"id":"82b55b58_89c82553","line":534,"range":{"start_line":534,"start_character":12,"end_line":534,"end_character":37},"in_reply_to":"ada74aeb_5741882e","updated":"2022-09-29 15:09:11.000000000","message":"Done","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"df40f7eb71b7f79999f0cec5ea5708dda4ca5b74","unresolved":true,"context_lines":[{"line_number":2082,"context_line":"    def _update_metadata(self, resource, resource_id, metadata\u003dNone,"},{"line_number":2083,"context_line":"                         method\u003d\"post\", parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2084,"context_line":"        if parent_resource is None:"},{"line_number":2085,"context_line":"            uri \u003d \"%s/%s/metadata\" % (resource, resource_id)"},{"line_number":2086,"context_line":"        else:"},{"line_number":2087,"context_line":"            uri \u003d (\"%s/%s/%s/%s/metadata\""},{"line_number":2088,"context_line":"                   % (parent_resource, parent_id, resource, resource_id))"}],"source_content_type":"text/x-python","patch_set":24,"id":"976f0891_90bf768c","line":2085,"range":{"start_line":2085,"start_character":17,"end_line":2085,"end_character":60},"updated":"2022-09-22 19:26:49.000000000","message":"f strings would possibly make it more readable\nf\u0027{resource}/{resource_id}/metadata\u0027\nhere and in the below occurrences :)","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d7f2d9d6cd4b364916995b3a215d444c149d68c","unresolved":false,"context_lines":[{"line_number":2082,"context_line":"    def _update_metadata(self, resource, resource_id, metadata\u003dNone,"},{"line_number":2083,"context_line":"                         method\u003d\"post\", parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2084,"context_line":"        if parent_resource is None:"},{"line_number":2085,"context_line":"            uri \u003d \"%s/%s/metadata\" % (resource, resource_id)"},{"line_number":2086,"context_line":"        else:"},{"line_number":2087,"context_line":"            uri \u003d (\"%s/%s/%s/%s/metadata\""},{"line_number":2088,"context_line":"                   % (parent_resource, parent_id, resource, resource_id))"}],"source_content_type":"text/x-python","patch_set":24,"id":"32bf1417_1eebdd42","line":2085,"range":{"start_line":2085,"start_character":17,"end_line":2085,"end_character":60},"in_reply_to":"976f0891_90bf768c","updated":"2022-09-29 15:09:11.000000000","message":"Done","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":2125,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2126,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2127,"context_line":""},{"line_number":2128,"context_line":"    def get_metadata(self, resource_id, resource\u003dNone, parent_resource\u003dNone,"},{"line_number":2129,"context_line":"                     parent_id\u003dNone):"},{"line_number":2130,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2131,"context_line":"        if parent_resource is None:"}],"source_content_type":"text/x-python","patch_set":28,"id":"a5e1577d_2ac08170","line":2128,"range":{"start_line":2128,"start_character":27,"end_line":2128,"end_character":38},"updated":"2022-10-19 14:50:22.000000000","message":"I guess the reason for the failures in the s-rbac tests is because the \"resource_id\" argument is not compatible with v1 which is \"share_id\":\n\nhttps://github.com/openstack/manila-tempest-plugin/blob/master/manila_tempest_tests/services/share/json/shares_client.py#L444","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2097,"context_line":"        body \u003d json.loads(body)"},{"line_number":2098,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2099,"context_line":""},{"line_number":2100,"context_line":"    def set_metadata(self, resource_id, metadata\u003dNone, resource\u003dNone,"},{"line_number":2101,"context_line":"                     parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2102,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2103,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"}],"source_content_type":"text/x-python","patch_set":30,"id":"79d1d5d3_04f25f83","line":2100,"range":{"start_line":2100,"start_character":64,"end_line":2100,"end_character":68},"updated":"2022-10-20 10:58:52.000000000","message":"\u0027share\u0027","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2097,"context_line":"        body \u003d json.loads(body)"},{"line_number":2098,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2099,"context_line":""},{"line_number":2100,"context_line":"    def set_metadata(self, resource_id, metadata\u003dNone, resource\u003dNone,"},{"line_number":2101,"context_line":"                     parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2102,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2103,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"}],"source_content_type":"text/x-python","patch_set":30,"id":"9be4e5d2_48b0b6a7","line":2100,"range":{"start_line":2100,"start_character":64,"end_line":2100,"end_character":68},"in_reply_to":"79d1d5d3_04f25f83","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2099,"context_line":""},{"line_number":2100,"context_line":"    def set_metadata(self, resource_id, metadata\u003dNone, resource\u003dNone,"},{"line_number":2101,"context_line":"                     parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2102,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2103,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"},{"line_number":2104,"context_line":"                                     method\u003d\"post\","},{"line_number":2105,"context_line":"                                     parent_resource\u003dparent_resource,"}],"source_content_type":"text/x-python","patch_set":30,"id":"b95f055e_8c86e024","line":2102,"range":{"start_line":2102,"start_character":19,"end_line":2102,"end_character":27},"updated":"2022-10-20 10:58:52.000000000","message":"Sorry for this small comment!\nSince it refers to a single resource it\u0027s better to call it a \u0027share\u0027.\nBut actually you can remove this line and define \"share\" as a default parameter as you can see in the bellow comment.","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2099,"context_line":""},{"line_number":2100,"context_line":"    def set_metadata(self, resource_id, metadata\u003dNone, resource\u003dNone,"},{"line_number":2101,"context_line":"                     parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2102,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2103,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"},{"line_number":2104,"context_line":"                                     method\u003d\"post\","},{"line_number":2105,"context_line":"                                     parent_resource\u003dparent_resource,"}],"source_content_type":"text/x-python","patch_set":30,"id":"e32e6fc6_0d8d7c97","line":2102,"range":{"start_line":2102,"start_character":19,"end_line":2102,"end_character":27},"in_reply_to":"b95f055e_8c86e024","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2105,"context_line":"                                     parent_resource\u003dparent_resource,"},{"line_number":2106,"context_line":"                                     parent_id\u003dparent_id)"},{"line_number":2107,"context_line":""},{"line_number":2108,"context_line":"    def update_all_metadata(self, resource_id, metadata\u003dNone, resource\u003dNone,"},{"line_number":2109,"context_line":"                            parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2110,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2111,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"}],"source_content_type":"text/x-python","patch_set":30,"id":"60e56b43_def9bfc2","line":2108,"range":{"start_line":2108,"start_character":71,"end_line":2108,"end_character":75},"updated":"2022-10-20 10:58:52.000000000","message":"\u0027share\u0027","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2105,"context_line":"                                     parent_resource\u003dparent_resource,"},{"line_number":2106,"context_line":"                                     parent_id\u003dparent_id)"},{"line_number":2107,"context_line":""},{"line_number":2108,"context_line":"    def update_all_metadata(self, resource_id, metadata\u003dNone, resource\u003dNone,"},{"line_number":2109,"context_line":"                            parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2110,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2111,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"}],"source_content_type":"text/x-python","patch_set":30,"id":"1d34283c_51c02ce5","line":2108,"range":{"start_line":2108,"start_character":71,"end_line":2108,"end_character":75},"in_reply_to":"60e56b43_def9bfc2","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2107,"context_line":""},{"line_number":2108,"context_line":"    def update_all_metadata(self, resource_id, metadata\u003dNone, resource\u003dNone,"},{"line_number":2109,"context_line":"                            parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2110,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2111,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"},{"line_number":2112,"context_line":"                                     method\u003d\"put\","},{"line_number":2113,"context_line":"                                     parent_resource\u003dparent_resource,"}],"source_content_type":"text/x-python","patch_set":30,"id":"90172572_92e11ab1","line":2110,"range":{"start_line":2110,"start_character":8,"end_line":2110,"end_character":61},"updated":"2022-10-20 10:58:52.000000000","message":"Same!","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2107,"context_line":""},{"line_number":2108,"context_line":"    def update_all_metadata(self, resource_id, metadata\u003dNone, resource\u003dNone,"},{"line_number":2109,"context_line":"                            parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2110,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2111,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"},{"line_number":2112,"context_line":"                                     method\u003d\"put\","},{"line_number":2113,"context_line":"                                     parent_resource\u003dparent_resource,"}],"source_content_type":"text/x-python","patch_set":30,"id":"0b3f78e1_40bb682d","line":2110,"range":{"start_line":2110,"start_character":8,"end_line":2110,"end_character":61},"in_reply_to":"90172572_92e11ab1","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2113,"context_line":"                                     parent_resource\u003dparent_resource,"},{"line_number":2114,"context_line":"                                     parent_id\u003dparent_id)"},{"line_number":2115,"context_line":""},{"line_number":2116,"context_line":"    def delete_metadata(self, resource_id, key, resource\u003dNone,"},{"line_number":2117,"context_line":"                        parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2118,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2119,"context_line":"        if parent_resource is None:"}],"source_content_type":"text/x-python","patch_set":30,"id":"6c50b9f3_d210c261","line":2116,"range":{"start_line":2116,"start_character":57,"end_line":2116,"end_character":61},"updated":"2022-10-20 10:58:52.000000000","message":"\u0027share\u0027","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2113,"context_line":"                                     parent_resource\u003dparent_resource,"},{"line_number":2114,"context_line":"                                     parent_id\u003dparent_id)"},{"line_number":2115,"context_line":""},{"line_number":2116,"context_line":"    def delete_metadata(self, resource_id, key, resource\u003dNone,"},{"line_number":2117,"context_line":"                        parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2118,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2119,"context_line":"        if parent_resource is None:"}],"source_content_type":"text/x-python","patch_set":30,"id":"2a33b4d4_4844af0f","line":2116,"range":{"start_line":2116,"start_character":57,"end_line":2116,"end_character":61},"in_reply_to":"6c50b9f3_d210c261","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2115,"context_line":""},{"line_number":2116,"context_line":"    def delete_metadata(self, resource_id, key, resource\u003dNone,"},{"line_number":2117,"context_line":"                        parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2118,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2119,"context_line":"        if parent_resource is None:"},{"line_number":2120,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2121,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":30,"id":"a0995260_35f5398f","line":2118,"range":{"start_line":2118,"start_character":8,"end_line":2118,"end_character":61},"updated":"2022-10-20 10:58:52.000000000","message":"same!","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2115,"context_line":""},{"line_number":2116,"context_line":"    def delete_metadata(self, resource_id, key, resource\u003dNone,"},{"line_number":2117,"context_line":"                        parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2118,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2119,"context_line":"        if parent_resource is None:"},{"line_number":2120,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2121,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":30,"id":"91059021_6b1ccfe6","line":2118,"range":{"start_line":2118,"start_character":8,"end_line":2118,"end_character":61},"in_reply_to":"a0995260_35f5398f","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2117,"context_line":"                        parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2118,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2119,"context_line":"        if parent_resource is None:"},{"line_number":2120,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2121,"context_line":"        else:"},{"line_number":2122,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2123,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata/{key}\u0027)"}],"source_content_type":"text/x-python","patch_set":30,"id":"5a6fdaf1_4401f930","line":2120,"range":{"start_line":2120,"start_character":20,"end_line":2120,"end_character":31},"updated":"2022-10-20 10:58:52.000000000","message":"{resource}s/\nSince \u0027s\u0027 was removed","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2117,"context_line":"                        parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2118,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2119,"context_line":"        if parent_resource is None:"},{"line_number":2120,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2121,"context_line":"        else:"},{"line_number":2122,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2123,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata/{key}\u0027)"}],"source_content_type":"text/x-python","patch_set":30,"id":"a0549b47_a42eac31","line":2120,"range":{"start_line":2120,"start_character":20,"end_line":2120,"end_character":31},"in_reply_to":"5a6fdaf1_4401f930","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2120,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2121,"context_line":"        else:"},{"line_number":2122,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2123,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata/{key}\u0027)"},{"line_number":2124,"context_line":"        resp, body \u003d self.delete(uri)"},{"line_number":2125,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2126,"context_line":"        return rest_client.ResponseBody(resp, body)"}],"source_content_type":"text/x-python","patch_set":30,"id":"0305b274_7d97ea1c","line":2123,"range":{"start_line":2123,"start_character":22,"end_line":2123,"end_character":33},"updated":"2022-10-20 10:58:52.000000000","message":"Same!","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2120,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2121,"context_line":"        else:"},{"line_number":2122,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2123,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata/{key}\u0027)"},{"line_number":2124,"context_line":"        resp, body \u003d self.delete(uri)"},{"line_number":2125,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2126,"context_line":"        return rest_client.ResponseBody(resp, body)"}],"source_content_type":"text/x-python","patch_set":30,"id":"911bb631_8a7e9630","line":2123,"range":{"start_line":2123,"start_character":22,"end_line":2123,"end_character":33},"in_reply_to":"0305b274_7d97ea1c","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2125,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2126,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2127,"context_line":""},{"line_number":2128,"context_line":"    def get_metadata(self, resource_id, resource\u003dNone, parent_resource\u003dNone,"},{"line_number":2129,"context_line":"                     parent_id\u003dNone):"},{"line_number":2130,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2131,"context_line":"        if parent_resource is None:"}],"source_content_type":"text/x-python","patch_set":30,"id":"61112fd9_aba29552","line":2128,"range":{"start_line":2128,"start_character":49,"end_line":2128,"end_character":53},"updated":"2022-10-20 10:58:52.000000000","message":"\u0027share\u0027","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2125,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2126,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2127,"context_line":""},{"line_number":2128,"context_line":"    def get_metadata(self, resource_id, resource\u003dNone, parent_resource\u003dNone,"},{"line_number":2129,"context_line":"                     parent_id\u003dNone):"},{"line_number":2130,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2131,"context_line":"        if parent_resource is None:"}],"source_content_type":"text/x-python","patch_set":30,"id":"9e4a3919_a3bf36cb","line":2128,"range":{"start_line":2128,"start_character":49,"end_line":2128,"end_character":53},"in_reply_to":"61112fd9_aba29552","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2127,"context_line":""},{"line_number":2128,"context_line":"    def get_metadata(self, resource_id, resource\u003dNone, parent_resource\u003dNone,"},{"line_number":2129,"context_line":"                     parent_id\u003dNone):"},{"line_number":2130,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2131,"context_line":"        if parent_resource is None:"},{"line_number":2132,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2133,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":30,"id":"f3b6f8c3_59c37cc2","line":2130,"range":{"start_line":2130,"start_character":8,"end_line":2130,"end_character":61},"updated":"2022-10-20 10:58:52.000000000","message":"Same!","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2127,"context_line":""},{"line_number":2128,"context_line":"    def get_metadata(self, resource_id, resource\u003dNone, parent_resource\u003dNone,"},{"line_number":2129,"context_line":"                     parent_id\u003dNone):"},{"line_number":2130,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2131,"context_line":"        if parent_resource is None:"},{"line_number":2132,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2133,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":30,"id":"a73f6e4d_978a8feb","line":2130,"range":{"start_line":2130,"start_character":8,"end_line":2130,"end_character":61},"in_reply_to":"f3b6f8c3_59c37cc2","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2129,"context_line":"                     parent_id\u003dNone):"},{"line_number":2130,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2131,"context_line":"        if parent_resource is None:"},{"line_number":2132,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2133,"context_line":"        else:"},{"line_number":2134,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2135,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata\u0027)"}],"source_content_type":"text/x-python","patch_set":30,"id":"c1891f8f_db7778d5","line":2132,"range":{"start_line":2132,"start_character":12,"end_line":2132,"end_character":54},"updated":"2022-10-20 10:58:52.000000000","message":"Same!","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2129,"context_line":"                     parent_id\u003dNone):"},{"line_number":2130,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2131,"context_line":"        if parent_resource is None:"},{"line_number":2132,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2133,"context_line":"        else:"},{"line_number":2134,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2135,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata\u0027)"}],"source_content_type":"text/x-python","patch_set":30,"id":"149a1d0b_facf773c","line":2132,"range":{"start_line":2132,"start_character":12,"end_line":2132,"end_character":54},"in_reply_to":"c1891f8f_db7778d5","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2132,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2133,"context_line":"        else:"},{"line_number":2134,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2135,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata\u0027)"},{"line_number":2136,"context_line":"        resp, body \u003d self.get(uri)"},{"line_number":2137,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2138,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":30,"id":"b4fde52a_92eac2e1","line":2135,"range":{"start_line":2135,"start_character":22,"end_line":2135,"end_character":32},"updated":"2022-10-20 10:58:52.000000000","message":"Same!","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2132,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2133,"context_line":"        else:"},{"line_number":2134,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2135,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata\u0027)"},{"line_number":2136,"context_line":"        resp, body \u003d self.get(uri)"},{"line_number":2137,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2138,"context_line":"        body \u003d json.loads(body)"}],"source_content_type":"text/x-python","patch_set":30,"id":"d529b11f_e5577152","line":2135,"range":{"start_line":2135,"start_character":22,"end_line":2135,"end_character":32},"in_reply_to":"b4fde52a_92eac2e1","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2140,"context_line":""},{"line_number":2141,"context_line":"    def get_metadata_item(self, resource_id, key, resource\u003dNone,"},{"line_number":2142,"context_line":"                          parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2143,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2144,"context_line":"        if parent_resource is None:"},{"line_number":2145,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2146,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":30,"id":"77f11d7c_3cfa7357","line":2143,"range":{"start_line":2143,"start_character":8,"end_line":2143,"end_character":61},"updated":"2022-10-20 10:58:52.000000000","message":"Same!","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2140,"context_line":""},{"line_number":2141,"context_line":"    def get_metadata_item(self, resource_id, key, resource\u003dNone,"},{"line_number":2142,"context_line":"                          parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2143,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2144,"context_line":"        if parent_resource is None:"},{"line_number":2145,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2146,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":30,"id":"20ce03ec_9396794a","line":2143,"range":{"start_line":2143,"start_character":8,"end_line":2143,"end_character":61},"in_reply_to":"77f11d7c_3cfa7357","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2142,"context_line":"                          parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2143,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2144,"context_line":"        if parent_resource is None:"},{"line_number":2145,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2146,"context_line":"        else:"},{"line_number":2147,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2148,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata/{key}\u0027)"}],"source_content_type":"text/x-python","patch_set":30,"id":"0d6ad509_94a1d2cb","line":2145,"range":{"start_line":2145,"start_character":20,"end_line":2145,"end_character":31},"updated":"2022-10-20 10:58:52.000000000","message":"Same!","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2142,"context_line":"                          parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2143,"context_line":"        resource \u003d \u0027shares\u0027 if resource is None else resource"},{"line_number":2144,"context_line":"        if parent_resource is None:"},{"line_number":2145,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2146,"context_line":"        else:"},{"line_number":2147,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2148,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata/{key}\u0027)"}],"source_content_type":"text/x-python","patch_set":30,"id":"8f7ef1c6_c0ffda17","line":2145,"range":{"start_line":2145,"start_character":20,"end_line":2145,"end_character":31},"in_reply_to":"0d6ad509_94a1d2cb","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":2145,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2146,"context_line":"        else:"},{"line_number":2147,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2148,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata/{key}\u0027)"},{"line_number":2149,"context_line":"        resp, body \u003d self.get(uri)"},{"line_number":2150,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2151,"context_line":"        return self._parse_resp(body)"}],"source_content_type":"text/x-python","patch_set":30,"id":"08e46ac9_e4acf0e9","line":2148,"range":{"start_line":2148,"start_character":22,"end_line":2148,"end_character":33},"updated":"2022-10-20 10:58:52.000000000","message":"Same!","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":2145,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata/{key}\u0027"},{"line_number":2146,"context_line":"        else:"},{"line_number":2147,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2148,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata/{key}\u0027)"},{"line_number":2149,"context_line":"        resp, body \u003d self.get(uri)"},{"line_number":2150,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2151,"context_line":"        return self._parse_resp(body)"}],"source_content_type":"text/x-python","patch_set":30,"id":"331e9b63_a1cb0e0b","line":2148,"range":{"start_line":2148,"start_character":22,"end_line":2148,"end_character":33},"in_reply_to":"08e46ac9_e4acf0e9","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"7cd46050bbd4396cfad41aad7d2d442824a5ff15","unresolved":true,"context_lines":[{"line_number":2081,"context_line":"    def _update_metadata(self, resource, resource_id, metadata\u003dNone,"},{"line_number":2082,"context_line":"                         method\u003d\"post\", parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2083,"context_line":"        if parent_resource is None:"},{"line_number":2084,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2085,"context_line":"        else:"},{"line_number":2086,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2087,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata\u0027)"}],"source_content_type":"text/x-python","patch_set":33,"id":"cdc9d373_f51409d9","line":2084,"range":{"start_line":2084,"start_character":20,"end_line":2084,"end_character":31},"updated":"2022-10-23 09:13:19.000000000","message":"{resource}s/","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1d37587435a0d48d45099356bad93be4cfb4e6aa","unresolved":false,"context_lines":[{"line_number":2081,"context_line":"    def _update_metadata(self, resource, resource_id, metadata\u003dNone,"},{"line_number":2082,"context_line":"                         method\u003d\"post\", parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2083,"context_line":"        if parent_resource is None:"},{"line_number":2084,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2085,"context_line":"        else:"},{"line_number":2086,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2087,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata\u0027)"}],"source_content_type":"text/x-python","patch_set":33,"id":"0f6e7724_c3bb49c2","line":2084,"range":{"start_line":2084,"start_character":20,"end_line":2084,"end_character":31},"in_reply_to":"cdc9d373_f51409d9","updated":"2022-11-02 15:02:14.000000000","message":"Done","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"7cd46050bbd4396cfad41aad7d2d442824a5ff15","unresolved":true,"context_lines":[{"line_number":2084,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2085,"context_line":"        else:"},{"line_number":2086,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2087,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata\u0027)"},{"line_number":2088,"context_line":"        if metadata is None:"},{"line_number":2089,"context_line":"            metadata \u003d {}"},{"line_number":2090,"context_line":"        post_body \u003d {\"metadata\": metadata}"}],"source_content_type":"text/x-python","patch_set":33,"id":"777ec914_9300075f","line":2087,"range":{"start_line":2087,"start_character":22,"end_line":2087,"end_character":33},"updated":"2022-10-23 09:13:19.000000000","message":"{resource}s/","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1d37587435a0d48d45099356bad93be4cfb4e6aa","unresolved":false,"context_lines":[{"line_number":2084,"context_line":"            uri \u003d f\u0027{resource}/{resource_id}/metadata\u0027"},{"line_number":2085,"context_line":"        else:"},{"line_number":2086,"context_line":"            uri \u003d (f\u0027{parent_resource}/{parent_id}\u0027"},{"line_number":2087,"context_line":"                   f\u0027/{resource}/{resource_id}/metadata\u0027)"},{"line_number":2088,"context_line":"        if metadata is None:"},{"line_number":2089,"context_line":"            metadata \u003d {}"},{"line_number":2090,"context_line":"        post_body \u003d {\"metadata\": metadata}"}],"source_content_type":"text/x-python","patch_set":33,"id":"1e0e2572_8adcc44c","line":2087,"range":{"start_line":2087,"start_character":22,"end_line":2087,"end_character":33},"in_reply_to":"777ec914_9300075f","updated":"2022-11-02 15:02:14.000000000","message":"Done","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"7cd46050bbd4396cfad41aad7d2d442824a5ff15","unresolved":true,"context_lines":[{"line_number":2097,"context_line":"        body \u003d json.loads(body)"},{"line_number":2098,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2099,"context_line":""},{"line_number":2100,"context_line":"    def set_metadata(self, resource_id, metadata\u003dNone, resource\u003d\u0027shares\u0027,"},{"line_number":2101,"context_line":"                     parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2102,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"},{"line_number":2103,"context_line":"                                     method\u003d\"post\","}],"source_content_type":"text/x-python","patch_set":33,"id":"d6f476d6_8ad98f30","line":2100,"range":{"start_line":2100,"start_character":65,"end_line":2100,"end_character":71},"updated":"2022-10-23 09:13:19.000000000","message":"share","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1d37587435a0d48d45099356bad93be4cfb4e6aa","unresolved":false,"context_lines":[{"line_number":2097,"context_line":"        body \u003d json.loads(body)"},{"line_number":2098,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2099,"context_line":""},{"line_number":2100,"context_line":"    def set_metadata(self, resource_id, metadata\u003dNone, resource\u003d\u0027shares\u0027,"},{"line_number":2101,"context_line":"                     parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2102,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"},{"line_number":2103,"context_line":"                                     method\u003d\"post\","}],"source_content_type":"text/x-python","patch_set":33,"id":"bac8e71e_60711204","line":2100,"range":{"start_line":2100,"start_character":65,"end_line":2100,"end_character":71},"in_reply_to":"d6f476d6_8ad98f30","updated":"2022-11-02 15:02:14.000000000","message":"Done","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"7cd46050bbd4396cfad41aad7d2d442824a5ff15","unresolved":true,"context_lines":[{"line_number":2105,"context_line":"                                     parent_id\u003dparent_id)"},{"line_number":2106,"context_line":""},{"line_number":2107,"context_line":"    def update_all_metadata(self, resource_id, metadata\u003dNone,"},{"line_number":2108,"context_line":"                            resource\u003d\u0027shares\u0027, parent_resource\u003dNone,"},{"line_number":2109,"context_line":"                            parent_id\u003dNone):"},{"line_number":2110,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"},{"line_number":2111,"context_line":"                                     method\u003d\"put\","}],"source_content_type":"text/x-python","patch_set":33,"id":"a5711488_51577032","line":2108,"range":{"start_line":2108,"start_character":38,"end_line":2108,"end_character":44},"updated":"2022-10-23 09:13:19.000000000","message":"share","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1d37587435a0d48d45099356bad93be4cfb4e6aa","unresolved":false,"context_lines":[{"line_number":2105,"context_line":"                                     parent_id\u003dparent_id)"},{"line_number":2106,"context_line":""},{"line_number":2107,"context_line":"    def update_all_metadata(self, resource_id, metadata\u003dNone,"},{"line_number":2108,"context_line":"                            resource\u003d\u0027shares\u0027, parent_resource\u003dNone,"},{"line_number":2109,"context_line":"                            parent_id\u003dNone):"},{"line_number":2110,"context_line":"        return self._update_metadata(resource, resource_id, metadata,"},{"line_number":2111,"context_line":"                                     method\u003d\"put\","}],"source_content_type":"text/x-python","patch_set":33,"id":"b96c0f34_9b7b6532","line":2108,"range":{"start_line":2108,"start_character":38,"end_line":2108,"end_character":44},"in_reply_to":"a5711488_51577032","updated":"2022-11-02 15:02:14.000000000","message":"Done","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"7cd46050bbd4396cfad41aad7d2d442824a5ff15","unresolved":true,"context_lines":[{"line_number":2112,"context_line":"                                     parent_resource\u003dparent_resource,"},{"line_number":2113,"context_line":"                                     parent_id\u003dparent_id)"},{"line_number":2114,"context_line":""},{"line_number":2115,"context_line":"    def delete_metadata(self, resource_id, key, resource\u003d\u0027shares\u0027,"},{"line_number":2116,"context_line":"                        parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2117,"context_line":"        if parent_resource is None:"},{"line_number":2118,"context_line":"            uri \u003d f\u0027{resource}s/{resource_id}/metadata/{key}\u0027"}],"source_content_type":"text/x-python","patch_set":33,"id":"f8097e58_c3512c1d","line":2115,"range":{"start_line":2115,"start_character":58,"end_line":2115,"end_character":64},"updated":"2022-10-23 09:13:19.000000000","message":"share","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1d37587435a0d48d45099356bad93be4cfb4e6aa","unresolved":false,"context_lines":[{"line_number":2112,"context_line":"                                     parent_resource\u003dparent_resource,"},{"line_number":2113,"context_line":"                                     parent_id\u003dparent_id)"},{"line_number":2114,"context_line":""},{"line_number":2115,"context_line":"    def delete_metadata(self, resource_id, key, resource\u003d\u0027shares\u0027,"},{"line_number":2116,"context_line":"                        parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2117,"context_line":"        if parent_resource is None:"},{"line_number":2118,"context_line":"            uri \u003d f\u0027{resource}s/{resource_id}/metadata/{key}\u0027"}],"source_content_type":"text/x-python","patch_set":33,"id":"412a84d2_6fb9a566","line":2115,"range":{"start_line":2115,"start_character":58,"end_line":2115,"end_character":64},"in_reply_to":"f8097e58_c3512c1d","updated":"2022-11-02 15:02:14.000000000","message":"Done","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"7cd46050bbd4396cfad41aad7d2d442824a5ff15","unresolved":true,"context_lines":[{"line_number":2123,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2124,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2125,"context_line":""},{"line_number":2126,"context_line":"    def get_metadata(self, resource_id, resource\u003d\u0027shares\u0027,"},{"line_number":2127,"context_line":"                     parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2128,"context_line":"        if parent_resource is None:"},{"line_number":2129,"context_line":"            uri \u003d f\u0027{resource}s/{resource_id}/metadata\u0027"}],"source_content_type":"text/x-python","patch_set":33,"id":"dbfcc50a_e29afa07","line":2126,"range":{"start_line":2126,"start_character":50,"end_line":2126,"end_character":56},"updated":"2022-10-23 09:13:19.000000000","message":"share","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1d37587435a0d48d45099356bad93be4cfb4e6aa","unresolved":false,"context_lines":[{"line_number":2123,"context_line":"        self.expected_success(200, resp.status)"},{"line_number":2124,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2125,"context_line":""},{"line_number":2126,"context_line":"    def get_metadata(self, resource_id, resource\u003d\u0027shares\u0027,"},{"line_number":2127,"context_line":"                     parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2128,"context_line":"        if parent_resource is None:"},{"line_number":2129,"context_line":"            uri \u003d f\u0027{resource}s/{resource_id}/metadata\u0027"}],"source_content_type":"text/x-python","patch_set":33,"id":"8a80c606_6da6f59f","line":2126,"range":{"start_line":2126,"start_character":50,"end_line":2126,"end_character":56},"in_reply_to":"dbfcc50a_e29afa07","updated":"2022-11-02 15:02:14.000000000","message":"Done","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"7cd46050bbd4396cfad41aad7d2d442824a5ff15","unresolved":true,"context_lines":[{"line_number":2135,"context_line":"        body \u003d json.loads(body)"},{"line_number":2136,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2137,"context_line":""},{"line_number":2138,"context_line":"    def get_metadata_item(self, resource_id, key, resource\u003d\u0027shares\u0027,"},{"line_number":2139,"context_line":"                          parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2140,"context_line":"        if parent_resource is None:"},{"line_number":2141,"context_line":"            uri \u003d f\u0027{resource}s/{resource_id}/metadata/{key}\u0027"}],"source_content_type":"text/x-python","patch_set":33,"id":"49c106d3_a83ddba9","line":2138,"range":{"start_line":2138,"start_character":60,"end_line":2138,"end_character":66},"updated":"2022-10-23 09:13:19.000000000","message":"share","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"1d37587435a0d48d45099356bad93be4cfb4e6aa","unresolved":false,"context_lines":[{"line_number":2135,"context_line":"        body \u003d json.loads(body)"},{"line_number":2136,"context_line":"        return rest_client.ResponseBody(resp, body)"},{"line_number":2137,"context_line":""},{"line_number":2138,"context_line":"    def get_metadata_item(self, resource_id, key, resource\u003d\u0027shares\u0027,"},{"line_number":2139,"context_line":"                          parent_resource\u003dNone, parent_id\u003dNone):"},{"line_number":2140,"context_line":"        if parent_resource is None:"},{"line_number":2141,"context_line":"            uri \u003d f\u0027{resource}s/{resource_id}/metadata/{key}\u0027"}],"source_content_type":"text/x-python","patch_set":33,"id":"86f52788_c2c3825d","line":2138,"range":{"start_line":2138,"start_character":60,"end_line":2138,"end_character":66},"in_reply_to":"49c106d3_a83ddba9","updated":"2022-11-02 15:02:14.000000000","message":"Done","commit_id":"51122c6a9e238f2fc72568ce30aa25c95a97a6c6"}],"manila_tempest_tests/tests/api/admin/test_snapshot_manage.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f057efd26b7f262862f83d77d61dc10cfd47cbe8","unresolved":true,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"    @decorators.idempotent_id(\u00275fb65a19-fb73-4b5a-9210-010f93e0304f\u0027)"},{"line_number":139,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_BACKEND)"},{"line_number":140,"context_line":"    @ddt.data(\u00272.12\u0027, \u00272.16\u0027, \u00272.73\u0027, CONF.share.max_api_microversion)"},{"line_number":141,"context_line":"    def test_manage_different_versions(self, version):"},{"line_number":142,"context_line":"        \"\"\"Run snapshot manage test for multiple versions."},{"line_number":143,"context_line":""}],"source_content_type":"text/x-python","patch_set":24,"id":"ebb70ac7_d1828f29","line":140,"range":{"start_line":140,"start_character":14,"end_line":140,"end_character":69},"updated":"2022-09-22 22:44:18.000000000","message":"deduplicate these.. https://github.com/openstack/manila-tempest-plugin/blob/d62b5dcf51e8b9824ddc279a73ee4d03ee23d0b2/manila_tempest_tests/utils.py#L30-L36","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d7f2d9d6cd4b364916995b3a215d444c149d68c","unresolved":false,"context_lines":[{"line_number":137,"context_line":""},{"line_number":138,"context_line":"    @decorators.idempotent_id(\u00275fb65a19-fb73-4b5a-9210-010f93e0304f\u0027)"},{"line_number":139,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_BACKEND)"},{"line_number":140,"context_line":"    @ddt.data(\u00272.12\u0027, \u00272.16\u0027, \u00272.73\u0027, CONF.share.max_api_microversion)"},{"line_number":141,"context_line":"    def test_manage_different_versions(self, version):"},{"line_number":142,"context_line":"        \"\"\"Run snapshot manage test for multiple versions."},{"line_number":143,"context_line":""}],"source_content_type":"text/x-python","patch_set":24,"id":"42c03c76_033c7a13","line":140,"range":{"start_line":140,"start_character":14,"end_line":140,"end_character":69},"in_reply_to":"ebb70ac7_d1828f29","updated":"2022-09-29 15:09:11.000000000","message":"Done","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"}],"manila_tempest_tests/tests/api/test_metadata_negative.py":[{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    @ddt.data(True, False)"},{"line_number":30,"context_line":"    def test_try_set_metadata_to_unexisting_share(self, is_v2_client):"},{"line_number":31,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":32,"context_line":"        client \u003d (self.shares_v2_client if is_v2_client else"},{"line_number":33,"context_line":"                  self.shares_v2_client)"},{"line_number":34,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":35,"context_line":"                          client.set_metadata,"},{"line_number":36,"context_line":"                          \"wrong_share_id\", md)"}],"source_content_type":"text/x-python","patch_set":20,"id":"a894ad13_d661f09a","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":40},"updated":"2022-08-25 07:52:37.000000000","message":"This condition is redundant it\u0027ll always be self.shares_v2_client no matter what the value of is_v2_client.\nI think can just you v2 client.\nSo you can remove this line and remove ddt decorator.","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    @ddt.data(True, False)"},{"line_number":30,"context_line":"    def test_try_set_metadata_to_unexisting_share(self, is_v2_client):"},{"line_number":31,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":32,"context_line":"        client \u003d (self.shares_v2_client if is_v2_client else"},{"line_number":33,"context_line":"                  self.shares_v2_client)"},{"line_number":34,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":35,"context_line":"                          client.set_metadata,"},{"line_number":36,"context_line":"                          \"wrong_share_id\", md)"}],"source_content_type":"text/x-python","patch_set":20,"id":"48f274bc_89f7b3de","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":40},"in_reply_to":"a894ad13_d661f09a","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":32,"context_line":"        client \u003d (self.shares_v2_client if is_v2_client else"},{"line_number":33,"context_line":"                  self.shares_v2_client)"},{"line_number":34,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":35,"context_line":"                          client.set_metadata,"},{"line_number":36,"context_line":"                          \"wrong_share_id\", md)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    @decorators.idempotent_id(\u00277df0acd7-03f8-45c4-8c72-eb6932af70b1\u0027)"}],"source_content_type":"text/x-python","patch_set":20,"id":"7fea1830_997ac515","line":35,"range":{"start_line":35,"start_character":26,"end_line":35,"end_character":32},"updated":"2022-08-25 07:52:37.000000000","message":"self.shares_v2_client","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        client \u003d (self.shares_v2_client if is_v2_client else"},{"line_number":33,"context_line":"                  self.shares_v2_client)"},{"line_number":34,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":35,"context_line":"                          client.set_metadata,"},{"line_number":36,"context_line":"                          \"wrong_share_id\", md)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    @decorators.idempotent_id(\u00277df0acd7-03f8-45c4-8c72-eb6932af70b1\u0027)"}],"source_content_type":"text/x-python","patch_set":20,"id":"ac0d2f23_bce6047f","line":35,"range":{"start_line":35,"start_character":26,"end_line":35,"end_character":32},"in_reply_to":"7fea1830_997ac515","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":40,"context_line":"    @ddt.data(True, False)"},{"line_number":41,"context_line":"    def test_try_update_all_metadata_for_unexisting_share(self, is_v2_client):"},{"line_number":42,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":43,"context_line":"        client \u003d (self.shares_v2_client if is_v2_client else"},{"line_number":44,"context_line":"                  self.shares_v2_client)"},{"line_number":45,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":46,"context_line":"                          client.update_all_metadata,"},{"line_number":47,"context_line":"                          \"wrong_share_id\", md)"}],"source_content_type":"text/x-python","patch_set":20,"id":"5997a9b1_6c870a59","line":44,"range":{"start_line":43,"start_character":0,"end_line":44,"end_character":40},"updated":"2022-08-25 07:52:37.000000000","message":"Same here","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":40,"context_line":"    @ddt.data(True, False)"},{"line_number":41,"context_line":"    def test_try_update_all_metadata_for_unexisting_share(self, is_v2_client):"},{"line_number":42,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":43,"context_line":"        client \u003d (self.shares_v2_client if is_v2_client else"},{"line_number":44,"context_line":"                  self.shares_v2_client)"},{"line_number":45,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":46,"context_line":"                          client.update_all_metadata,"},{"line_number":47,"context_line":"                          \"wrong_share_id\", md)"}],"source_content_type":"text/x-python","patch_set":20,"id":"865f25b3_509bf5e7","line":44,"range":{"start_line":43,"start_character":0,"end_line":44,"end_character":40},"in_reply_to":"5997a9b1_6c870a59","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f057efd26b7f262862f83d77d61dc10cfd47cbe8","unresolved":true,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"@ddt.ddt"},{"line_number":25,"context_line":"class SharesMetadataAPIOnlyNegativeTest(base.BaseSharesTest):"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    @decorators.idempotent_id(\u002722aecf50-0d98-4b97-82b8-599559f7692f\u0027)"},{"line_number":28,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API)"},{"line_number":29,"context_line":"    def test_try_set_metadata_to_unexisting_share(self):"}],"source_content_type":"text/x-python","patch_set":24,"id":"16b77374_a7dd49ca","line":26,"updated":"2022-09-22 22:44:18.000000000","message":"Why are we removing these test cases?","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"043feb0a3e066233512ac5f30940d620d13f5779","unresolved":true,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"@ddt.ddt"},{"line_number":25,"context_line":"class SharesMetadataAPIOnlyNegativeTest(base.BaseSharesTest):"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    @decorators.idempotent_id(\u002722aecf50-0d98-4b97-82b8-599559f7692f\u0027)"},{"line_number":28,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API)"},{"line_number":29,"context_line":"    def test_try_set_metadata_to_unexisting_share(self):"}],"source_content_type":"text/x-python","patch_set":24,"id":"346c6902_8b561685","line":26,"in_reply_to":"16b77374_a7dd49ca","updated":"2022-09-28 23:01:24.000000000","message":"Reiterating this question","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d7f2d9d6cd4b364916995b3a215d444c149d68c","unresolved":true,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"@ddt.ddt"},{"line_number":25,"context_line":"class SharesMetadataAPIOnlyNegativeTest(base.BaseSharesTest):"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    @decorators.idempotent_id(\u002722aecf50-0d98-4b97-82b8-599559f7692f\u0027)"},{"line_number":28,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API)"},{"line_number":29,"context_line":"    def test_try_set_metadata_to_unexisting_share(self):"}],"source_content_type":"text/x-python","patch_set":24,"id":"f35c49a0_0f17415f","line":26,"in_reply_to":"16b77374_a7dd49ca","updated":"2022-09-29 15:09:11.000000000","message":"Removed what test cases? here Ive refactored to account for the changes ive made to the shares_client file and ensuring it always uses v2 client","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"b6436e8d91e6211de94738b1a05d424ccf24c4d6","unresolved":true,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"@ddt.ddt"},{"line_number":25,"context_line":"class SharesMetadataAPIOnlyNegativeTest(base.BaseSharesTest):"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    @decorators.idempotent_id(\u002722aecf50-0d98-4b97-82b8-599559f7692f\u0027)"},{"line_number":28,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API)"},{"line_number":29,"context_line":"    def test_try_set_metadata_to_unexisting_share(self):"}],"source_content_type":"text/x-python","patch_set":24,"id":"bdb3b180_c3a835ba","line":26,"in_reply_to":"f35c49a0_0f17415f","updated":"2022-09-29 17:09:06.000000000","message":"These tests were running twice - once with the v1 client and the v2 client -- would it be a problem to leave the v1 tests alone? the interfaces aren\u0027t removed: https://opendev.org/openstack/manila-tempest-plugin/src/commit/5e559df86912a5dcc8a071a64cac65e55189695a/manila_tempest_tests/services/share/json/shares_client.py#L433-L453","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"}],"manila_tempest_tests/tests/api/test_snapshot_metadata.py":[{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    @decorators.idempotent_id(\u00275d537913-ce6f-4771-beb2-84e2390b06d3\u0027)"},{"line_number":73,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":74,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":75,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":76,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":77,"context_line":"    def test_set_metadata_in_snapshot_creation(self):"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"}],"source_content_type":"text/x-python","patch_set":20,"id":"9c2f809a_14b8954b","line":76,"range":{"start_line":74,"start_character":0,"end_line":76,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Since this class test snapshot metadata, you should move these conditions under skip_checks classmethod.\n\n@classmethod\ndef skip_checks(cls):\n    super(ShareSnapshotMetadataTest, cls).skip_checks()\n    utils.check_skip_if_microversion_not_supported(\"2.72\")        \n    if not CONF.share.run_snapshot_tests:\n        raise cls.skipException(\u0027Snapshot tests are disabled.\u0027)","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":71,"context_line":""},{"line_number":72,"context_line":"    @decorators.idempotent_id(\u00275d537913-ce6f-4771-beb2-84e2390b06d3\u0027)"},{"line_number":73,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":74,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":75,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":76,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":77,"context_line":"    def test_set_metadata_in_snapshot_creation(self):"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"}],"source_content_type":"text/x-python","patch_set":20,"id":"ffca7a07_bb3168d0","line":76,"range":{"start_line":74,"start_character":0,"end_line":76,"end_character":57},"in_reply_to":"9c2f809a_14b8954b","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    @decorators.idempotent_id(\u00277cbdf3c5-fb72-4ea5-9e60-ba50bad68ee9\u0027)"},{"line_number":90,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":91,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":92,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":93,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":94,"context_line":"    def test_set_get_delete_metadata(self):"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        md \u003d {u\"key3\": u\"value3\", u\"key4\": u\"value4\", u\"key.5.1\": u\"value.5\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"ebc43f38_590bc0ab","line":93,"range":{"start_line":91,"start_character":0,"end_line":93,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":88,"context_line":""},{"line_number":89,"context_line":"    @decorators.idempotent_id(\u00277cbdf3c5-fb72-4ea5-9e60-ba50bad68ee9\u0027)"},{"line_number":90,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":91,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":92,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":93,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":94,"context_line":"    def test_set_get_delete_metadata(self):"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        md \u003d {u\"key3\": u\"value3\", u\"key4\": u\"value4\", u\"key.5.1\": u\"value.5\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"1be64b05_6e27db36","line":93,"range":{"start_line":91,"start_character":0,"end_line":93,"end_character":57},"in_reply_to":"ebc43f38_590bc0ab","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    @decorators.idempotent_id(\u002723ec837d-1b50-499c-bbb9-a7bde843c9e8\u0027)"},{"line_number":121,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":122,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":123,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":124,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":125,"context_line":"    def test_set_metadata_not_delete_pre_metadata(self):"},{"line_number":126,"context_line":"        md1 \u003d {u\"key9\": u\"value9\", u\"key10\": u\"value10\", }"},{"line_number":127,"context_line":"        md2 \u003d {u\"key11\": u\"value11\", u\"key12\": u\"value12\", }"}],"source_content_type":"text/x-python","patch_set":20,"id":"05702c02_cdc42548","line":124,"range":{"start_line":122,"start_character":0,"end_line":124,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    @decorators.idempotent_id(\u002723ec837d-1b50-499c-bbb9-a7bde843c9e8\u0027)"},{"line_number":121,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":122,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":123,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":124,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":125,"context_line":"    def test_set_metadata_not_delete_pre_metadata(self):"},{"line_number":126,"context_line":"        md1 \u003d {u\"key9\": u\"value9\", u\"key10\": u\"value10\", }"},{"line_number":127,"context_line":"        md2 \u003d {u\"key11\": u\"value11\", u\"key12\": u\"value12\", }"}],"source_content_type":"text/x-python","patch_set":20,"id":"8f0edfb5_2ae9fe04","line":124,"range":{"start_line":122,"start_character":0,"end_line":124,"end_character":57},"in_reply_to":"05702c02_cdc42548","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    @decorators.idempotent_id(\u0027b7a00be5-3dd1-4d25-8723-c662581c923f\u0027)"},{"line_number":163,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":164,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":165,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":166,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":167,"context_line":"    def test_set_metadata_key_already_exist(self):"},{"line_number":168,"context_line":"        md1 \u003d {u\"key9\": u\"value9\", u\"key10\": u\"value10\", }"},{"line_number":169,"context_line":"        md2 \u003d {u\"key9\": u\"value13\", u\"key11\": u\"value11\", }"}],"source_content_type":"text/x-python","patch_set":20,"id":"bb345e33_63274f9d","line":166,"range":{"start_line":164,"start_character":0,"end_line":166,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    @decorators.idempotent_id(\u0027b7a00be5-3dd1-4d25-8723-c662581c923f\u0027)"},{"line_number":163,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":164,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":165,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":166,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":167,"context_line":"    def test_set_metadata_key_already_exist(self):"},{"line_number":168,"context_line":"        md1 \u003d {u\"key9\": u\"value9\", u\"key10\": u\"value10\", }"},{"line_number":169,"context_line":"        md2 \u003d {u\"key9\": u\"value13\", u\"key11\": u\"value11\", }"}],"source_content_type":"text/x-python","patch_set":20,"id":"b0e01c7f_0a44d4fd","line":166,"range":{"start_line":164,"start_character":0,"end_line":166,"end_character":57},"in_reply_to":"bb345e33_63274f9d","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":201,"context_line":""},{"line_number":202,"context_line":"    @decorators.idempotent_id(\u002790120310-07a9-43f4-9d5e-38d0a3f2f5bb\u0027)"},{"line_number":203,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":204,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":205,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":206,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":207,"context_line":"    def test_set_and_update_metadata_by_key(self):"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"        md1 \u003d {u\"key5\": u\"value5\", u\"key6\": u\"value6\", }"}],"source_content_type":"text/x-python","patch_set":20,"id":"e63c0df3_d84530bf","line":206,"range":{"start_line":204,"start_character":0,"end_line":206,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":201,"context_line":""},{"line_number":202,"context_line":"    @decorators.idempotent_id(\u002790120310-07a9-43f4-9d5e-38d0a3f2f5bb\u0027)"},{"line_number":203,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":204,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":205,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":206,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":207,"context_line":"    def test_set_and_update_metadata_by_key(self):"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"        md1 \u003d {u\"key5\": u\"value5\", u\"key6\": u\"value6\", }"}],"source_content_type":"text/x-python","patch_set":20,"id":"2e5b3764_42a36f43","line":206,"range":{"start_line":204,"start_character":0,"end_line":206,"end_character":57},"in_reply_to":"e63c0df3_d84530bf","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":228,"context_line":"    # change ids"},{"line_number":229,"context_line":"    @decorators.idempotent_id(\u00278963b7ae-db3a-476e-b0c7-29023e7aa321\u0027)"},{"line_number":230,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":231,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":232,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":233,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":234,"context_line":"    def test_set_metadata_min_size_key(self):"},{"line_number":235,"context_line":"        data \u003d {\"k\": \"value\"}"},{"line_number":236,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"91a91f42_e989dd6f","line":233,"range":{"start_line":231,"start_character":0,"end_line":233,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":228,"context_line":"    # change ids"},{"line_number":229,"context_line":"    @decorators.idempotent_id(\u00278963b7ae-db3a-476e-b0c7-29023e7aa321\u0027)"},{"line_number":230,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":231,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":232,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":233,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":234,"context_line":"    def test_set_metadata_min_size_key(self):"},{"line_number":235,"context_line":"        data \u003d {\"k\": \"value\"}"},{"line_number":236,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"bcab6eb2_ed2da69e","line":233,"range":{"start_line":231,"start_character":0,"end_line":233,"end_character":57},"in_reply_to":"91a91f42_e989dd6f","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    @decorators.idempotent_id(\u0027dc226070-5820-4df2-a30a-9dfb2f037a4b\u0027)"},{"line_number":245,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":246,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":247,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":248,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":249,"context_line":"    def test_set_metadata_max_size_key(self):"},{"line_number":250,"context_line":"        max_key \u003d \"k\" * 255"},{"line_number":251,"context_line":"        data \u003d {max_key: \"value\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"523af993_b2c51f93","line":248,"range":{"start_line":246,"start_character":0,"end_line":248,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    @decorators.idempotent_id(\u0027dc226070-5820-4df2-a30a-9dfb2f037a4b\u0027)"},{"line_number":245,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":246,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":247,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":248,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":249,"context_line":"    def test_set_metadata_max_size_key(self):"},{"line_number":250,"context_line":"        max_key \u003d \"k\" * 255"},{"line_number":251,"context_line":"        data \u003d {max_key: \"value\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"eec0ebce_3b3db847","line":248,"range":{"start_line":246,"start_character":0,"end_line":248,"end_character":57},"in_reply_to":"523af993_b2c51f93","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":260,"context_line":""},{"line_number":261,"context_line":"    @decorators.idempotent_id(\u0027940c283f-4f43-4122-86e8-32230da81886\u0027)"},{"line_number":262,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":263,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":264,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":265,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":266,"context_line":"    def test_set_metadata_min_size_value(self):"},{"line_number":267,"context_line":"        data \u003d {\"key\": \"v\"}"},{"line_number":268,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"519bf562_f69ae764","line":265,"range":{"start_line":263,"start_character":0,"end_line":265,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":260,"context_line":""},{"line_number":261,"context_line":"    @decorators.idempotent_id(\u0027940c283f-4f43-4122-86e8-32230da81886\u0027)"},{"line_number":262,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":263,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":264,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":265,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":266,"context_line":"    def test_set_metadata_min_size_value(self):"},{"line_number":267,"context_line":"        data \u003d {\"key\": \"v\"}"},{"line_number":268,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"8a022426_0f3caa4f","line":265,"range":{"start_line":263,"start_character":0,"end_line":265,"end_character":57},"in_reply_to":"519bf562_f69ae764","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    @decorators.idempotent_id(\u002785c480bc-0ffa-43e1-bc0a-284c5641996d\u0027)"},{"line_number":277,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":278,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":279,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":280,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":281,"context_line":"    def test_set_metadata_max_size_value(self):"},{"line_number":282,"context_line":"        max_value \u003d \"v\" * 1023"},{"line_number":283,"context_line":"        data \u003d {\"key\": max_value}"}],"source_content_type":"text/x-python","patch_set":20,"id":"5db2b612_bef6fb8b","line":280,"range":{"start_line":278,"start_character":0,"end_line":280,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    @decorators.idempotent_id(\u002785c480bc-0ffa-43e1-bc0a-284c5641996d\u0027)"},{"line_number":277,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":278,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":279,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":280,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":281,"context_line":"    def test_set_metadata_max_size_value(self):"},{"line_number":282,"context_line":"        max_value \u003d \"v\" * 1023"},{"line_number":283,"context_line":"        data \u003d {\"key\": max_value}"}],"source_content_type":"text/x-python","patch_set":20,"id":"f1ca88ed_476e6b9f","line":280,"range":{"start_line":278,"start_character":0,"end_line":280,"end_character":57},"in_reply_to":"5db2b612_bef6fb8b","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":291,"context_line":""},{"line_number":292,"context_line":"    @decorators.idempotent_id(\u0027c42335ae-ee90-4b73-b022-51c0a9bc301d\u0027)"},{"line_number":293,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":294,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":295,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":296,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":297,"context_line":"    def test_upd_metadata_min_size_key(self):"},{"line_number":298,"context_line":"        data \u003d {\"k\": \"value\"}"},{"line_number":299,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"57187c79_6106eca6","line":296,"range":{"start_line":294,"start_character":0,"end_line":296,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":291,"context_line":""},{"line_number":292,"context_line":"    @decorators.idempotent_id(\u0027c42335ae-ee90-4b73-b022-51c0a9bc301d\u0027)"},{"line_number":293,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":294,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":295,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":296,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":297,"context_line":"    def test_upd_metadata_min_size_key(self):"},{"line_number":298,"context_line":"        data \u003d {\"k\": \"value\"}"},{"line_number":299,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"bec63a51_25176d48","line":296,"range":{"start_line":294,"start_character":0,"end_line":296,"end_character":57},"in_reply_to":"57187c79_6106eca6","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    @decorators.idempotent_id(\u00271b5f06b0-bbff-49d1-8a4b-6e912039e2ba\u0027)"},{"line_number":308,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":309,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":310,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":311,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":312,"context_line":"    def test_upd_metadata_max_size_key(self):"},{"line_number":313,"context_line":"        max_key \u003d \"k\" * 255"},{"line_number":314,"context_line":"        data \u003d {max_key: \"value\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"7d1e5e3d_5d7de1d7","line":311,"range":{"start_line":309,"start_character":0,"end_line":311,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":306,"context_line":""},{"line_number":307,"context_line":"    @decorators.idempotent_id(\u00271b5f06b0-bbff-49d1-8a4b-6e912039e2ba\u0027)"},{"line_number":308,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":309,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":310,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":311,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":312,"context_line":"    def test_upd_metadata_max_size_key(self):"},{"line_number":313,"context_line":"        max_key \u003d \"k\" * 255"},{"line_number":314,"context_line":"        data \u003d {max_key: \"value\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"90da7905_9f86a1fe","line":311,"range":{"start_line":309,"start_character":0,"end_line":311,"end_character":57},"in_reply_to":"7d1e5e3d_5d7de1d7","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":322,"context_line":""},{"line_number":323,"context_line":"    @decorators.idempotent_id(\u0027849fdcd4-9b4c-4aea-833a-240d7d06966b\u0027)"},{"line_number":324,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":325,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":326,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":327,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":328,"context_line":"    def test_upd_metadata_min_size_value(self):"},{"line_number":329,"context_line":"        data \u003d {\"key\": \"v\"}"},{"line_number":330,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"8b6c97a8_6f2a6853","line":327,"range":{"start_line":325,"start_character":0,"end_line":327,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":322,"context_line":""},{"line_number":323,"context_line":"    @decorators.idempotent_id(\u0027849fdcd4-9b4c-4aea-833a-240d7d06966b\u0027)"},{"line_number":324,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":325,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":326,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":327,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":328,"context_line":"    def test_upd_metadata_min_size_value(self):"},{"line_number":329,"context_line":"        data \u003d {\"key\": \"v\"}"},{"line_number":330,"context_line":""}],"source_content_type":"text/x-python","patch_set":20,"id":"5bc0503b_d4ef1a74","line":327,"range":{"start_line":325,"start_character":0,"end_line":327,"end_character":57},"in_reply_to":"8b6c97a8_6f2a6853","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":337,"context_line":""},{"line_number":338,"context_line":"    @decorators.idempotent_id(\u0027fdfbe469-6403-41de-b909-c4c13fc57407\u0027)"},{"line_number":339,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":340,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":341,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":342,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":343,"context_line":"    def test_upd_metadata_max_size_value(self):"},{"line_number":344,"context_line":"        max_value \u003d \"v\" * 1023"},{"line_number":345,"context_line":"        data \u003d {\"key\": max_value}"}],"source_content_type":"text/x-python","patch_set":20,"id":"d9934866_c38d20e0","line":342,"range":{"start_line":340,"start_character":0,"end_line":342,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":337,"context_line":""},{"line_number":338,"context_line":"    @decorators.idempotent_id(\u0027fdfbe469-6403-41de-b909-c4c13fc57407\u0027)"},{"line_number":339,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":340,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":341,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":342,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":343,"context_line":"    def test_upd_metadata_max_size_value(self):"},{"line_number":344,"context_line":"        max_value \u003d \"v\" * 1023"},{"line_number":345,"context_line":"        data \u003d {\"key\": max_value}"}],"source_content_type":"text/x-python","patch_set":20,"id":"5f07fee3_5be459d3","line":342,"range":{"start_line":340,"start_character":0,"end_line":342,"end_character":57},"in_reply_to":"d9934866_c38d20e0","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"df40f7eb71b7f79999f0cec5ea5708dda4ca5b74","unresolved":true,"context_lines":[{"line_number":179,"context_line":"            snapshot[\u0027id\u0027], md2, resource\u003d\"snapshots\")"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"        # verify metadata"},{"line_number":182,"context_line":"        md \u003d {u\"key9\": u\"value13\", u\"key10\": u\"value10\","},{"line_number":183,"context_line":"              u\"key11\": u\"value11\"}"},{"line_number":184,"context_line":"        self._verify_snapshot_metadata(snapshot, md)"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"        # delete metadata"}],"source_content_type":"text/x-python","patch_set":24,"id":"dbad4cea_630dde3c","line":183,"range":{"start_line":182,"start_character":8,"end_line":183,"end_character":35},"updated":"2022-09-22 19:26:49.000000000","message":"md1.update(md2) should already do what is expected, please see:\nhttps://paste.opendev.org/show/b6ccQOGncKdHOVEz5YQe/","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d7f2d9d6cd4b364916995b3a215d444c149d68c","unresolved":false,"context_lines":[{"line_number":179,"context_line":"            snapshot[\u0027id\u0027], md2, resource\u003d\"snapshots\")"},{"line_number":180,"context_line":""},{"line_number":181,"context_line":"        # verify metadata"},{"line_number":182,"context_line":"        md \u003d {u\"key9\": u\"value13\", u\"key10\": u\"value10\","},{"line_number":183,"context_line":"              u\"key11\": u\"value11\"}"},{"line_number":184,"context_line":"        self._verify_snapshot_metadata(snapshot, md)"},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"        # delete metadata"}],"source_content_type":"text/x-python","patch_set":24,"id":"9f618ffd_0c375d24","line":183,"range":{"start_line":182,"start_character":8,"end_line":183,"end_character":35},"in_reply_to":"dbad4cea_630dde3c","updated":"2022-09-29 15:09:11.000000000","message":"Done","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f057efd26b7f262862f83d77d61dc10cfd47cbe8","unresolved":true,"context_lines":[{"line_number":216,"context_line":"        # verify metadata"},{"line_number":217,"context_line":"        self._verify_snapshot_metadata(snapshot, md2)"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"    # change ids"},{"line_number":220,"context_line":"    @decorators.idempotent_id(\u00278963b7ae-db3a-476e-b0c7-29023e7aa321\u0027)"},{"line_number":221,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":222,"context_line":"    def test_set_metadata_min_size_key(self):"}],"source_content_type":"text/x-python","patch_set":24,"id":"ff35c119_844392b1","line":219,"range":{"start_line":219,"start_character":4,"end_line":219,"end_character":16},"updated":"2022-09-22 22:44:18.000000000","message":"what does this mean?","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d7f2d9d6cd4b364916995b3a215d444c149d68c","unresolved":false,"context_lines":[{"line_number":216,"context_line":"        # verify metadata"},{"line_number":217,"context_line":"        self._verify_snapshot_metadata(snapshot, md2)"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"    # change ids"},{"line_number":220,"context_line":"    @decorators.idempotent_id(\u00278963b7ae-db3a-476e-b0c7-29023e7aa321\u0027)"},{"line_number":221,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":222,"context_line":"    def test_set_metadata_min_size_key(self):"}],"source_content_type":"text/x-python","patch_set":24,"id":"1003c87f_66001f2d","line":219,"range":{"start_line":219,"start_character":4,"end_line":219,"end_character":16},"in_reply_to":"ff35c119_844392b1","updated":"2022-09-29 15:09:11.000000000","message":"leftover notes from when i started working on the patch no worries","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"1c717268357a8d698828d53a25b157c2b50f1d1c","unresolved":true,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        # get metadata of snapshot"},{"line_number":66,"context_line":"        metadata \u003d self.shares_v2_client.get_metadata("},{"line_number":67,"context_line":"            snapshot[\u0027id\u0027], resource\u003d\"snapshots\")[\u0027metadata\u0027]"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        # verify metadata"},{"line_number":70,"context_line":"        self.assertEqual(md, metadata)"}],"source_content_type":"text/x-python","patch_set":30,"id":"a80fd1d7_a0a6f406","line":67,"range":{"start_line":67,"start_character":38,"end_line":67,"end_character":47},"updated":"2022-10-20 10:58:52.000000000","message":"snapshot\nYou\u0027ll have to change it all over the places.\nSorry again, but with IDE it\u0027s very easy.","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"db6447db3d6d5195eced737f6285b51492a3da89","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        # get metadata of snapshot"},{"line_number":66,"context_line":"        metadata \u003d self.shares_v2_client.get_metadata("},{"line_number":67,"context_line":"            snapshot[\u0027id\u0027], resource\u003d\"snapshots\")[\u0027metadata\u0027]"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        # verify metadata"},{"line_number":70,"context_line":"        self.assertEqual(md, metadata)"}],"source_content_type":"text/x-python","patch_set":30,"id":"dee5da2c_39a8c7c9","line":67,"range":{"start_line":67,"start_character":38,"end_line":67,"end_character":47},"in_reply_to":"a80fd1d7_a0a6f406","updated":"2022-10-21 14:08:28.000000000","message":"Done","commit_id":"5df2433fdf492647a0880b0bc27ef08e3a86ee2d"}],"manila_tempest_tests/tests/api/test_snapshot_metadata_negative.py":[{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"9d9340f802e52f997b31163cdc12d501c0c20432","unresolved":true,"context_lines":[{"line_number":32,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":33,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":34,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":35,"context_line":"    def test_try_set_metadata_to_unexisting_snapshot(self):"},{"line_number":36,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":37,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":38,"context_line":"                          self.shares_v2_client .set_metadata,"}],"source_content_type":"text/x-python","patch_set":20,"id":"7f62b3af_d953a649","line":35,"range":{"start_line":35,"start_character":8,"end_line":35,"end_character":52},"updated":"2022-08-26 15:56:56.000000000","message":"Is this intended to test metadata for the share created from snapshot? If so you may consider to name the test as such and target the child share.","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"a948d41c057c7df42c99252e2fced2b482f6b1a9","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":33,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":34,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":35,"context_line":"    def test_try_set_metadata_to_unexisting_snapshot(self):"},{"line_number":36,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":37,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":38,"context_line":"                          self.shares_v2_client .set_metadata,"}],"source_content_type":"text/x-python","patch_set":20,"id":"582dfd98_be223f70","line":35,"range":{"start_line":35,"start_character":8,"end_line":35,"end_character":52},"in_reply_to":"7f62b3af_d953a649","updated":"2022-08-31 12:56:20.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"a948d41c057c7df42c99252e2fced2b482f6b1a9","unresolved":true,"context_lines":[{"line_number":35,"context_line":"    def test_try_set_metadata_to_unexisting_snapshot(self):"},{"line_number":36,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":37,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":38,"context_line":"                          self.shares_v2_client .set_metadata,"},{"line_number":39,"context_line":"                          \"wrong_share_id\", md, resource\u003d\"snapshots\")"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    @decorators.idempotent_id(\u002703a7f6e9-de8b-4669-87e1-b179308b477d\u0027)"}],"source_content_type":"text/x-python","patch_set":20,"id":"95656ec0_ae25307c","line":38,"range":{"start_line":38,"start_character":47,"end_line":38,"end_character":48},"updated":"2022-08-31 12:56:20.000000000","message":"Extra spcae.","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    def test_try_set_metadata_to_unexisting_snapshot(self):"},{"line_number":36,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":37,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":38,"context_line":"                          self.shares_v2_client .set_metadata,"},{"line_number":39,"context_line":"                          \"wrong_share_id\", md, resource\u003d\"snapshots\")"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    @decorators.idempotent_id(\u002703a7f6e9-de8b-4669-87e1-b179308b477d\u0027)"}],"source_content_type":"text/x-python","patch_set":20,"id":"197e58d2_c5fd6ec0","line":38,"range":{"start_line":38,"start_character":47,"end_line":38,"end_character":48},"in_reply_to":"95656ec0_ae25307c","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"9d9340f802e52f997b31163cdc12d501c0c20432","unresolved":true,"context_lines":[{"line_number":43,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":44,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":45,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":46,"context_line":"    def test_try_update_all_metadata_for_unexisting_share(self):"},{"line_number":47,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":48,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":49,"context_line":"                          self.shares_v2_client.update_all_metadata,"}],"source_content_type":"text/x-python","patch_set":20,"id":"160f399b_c3b99c16","line":46,"range":{"start_line":46,"start_character":8,"end_line":46,"end_character":57},"updated":"2022-08-26 15:56:56.000000000","message":"Ditto as above.","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":25,"context_line":"LATEST_MICROVERSION \u003d CONF.share.max_api_microversion"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"@ddt.ddt"},{"line_number":29,"context_line":"class ShareSnapshotMetadataAPIOnlyNegativeTest(base.BaseSharesTest):"},{"line_number":30,"context_line":"    @decorators.idempotent_id(\u00278be4773b-6af9-413f-97e2-8acdb6149e7a\u0027)"},{"line_number":31,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API)"},{"line_number":32,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":33,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":34,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":35,"context_line":"    def test_try_set_metadata_to_unexisting_snapshot(self):"},{"line_number":36,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":37,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":38,"context_line":"                          self.shares_v2_client .set_metadata,"},{"line_number":39,"context_line":"                          \"wrong_share_id\", md, resource\u003d\"snapshots\")"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    @decorators.idempotent_id(\u002703a7f6e9-de8b-4669-87e1-b179308b477d\u0027)"},{"line_number":42,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API)"},{"line_number":43,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":44,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":45,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":46,"context_line":"    def test_try_update_all_metadata_for_unexisting_share(self):"},{"line_number":47,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":48,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":49,"context_line":"                          self.shares_v2_client.update_all_metadata,"},{"line_number":50,"context_line":"                          \"wrong_share_id\", md, resource\u003d\"snapshots\")"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":20,"id":"538fb26b_0119c94c","line":50,"range":{"start_line":28,"start_character":0,"end_line":50,"end_character":69},"updated":"2022-08-25 07:52:37.000000000","message":"Please move these tests under \"ShareSnapshotMetadataNegativeTest\" I think it\u0027s redundant to create an isolated class for api tests. It\u0027s enough to mark the tests as \"api\".","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":25,"context_line":"LATEST_MICROVERSION \u003d CONF.share.max_api_microversion"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"@ddt.ddt"},{"line_number":29,"context_line":"class ShareSnapshotMetadataAPIOnlyNegativeTest(base.BaseSharesTest):"},{"line_number":30,"context_line":"    @decorators.idempotent_id(\u00278be4773b-6af9-413f-97e2-8acdb6149e7a\u0027)"},{"line_number":31,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API)"},{"line_number":32,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":33,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":34,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":35,"context_line":"    def test_try_set_metadata_to_unexisting_snapshot(self):"},{"line_number":36,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":37,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":38,"context_line":"                          self.shares_v2_client .set_metadata,"},{"line_number":39,"context_line":"                          \"wrong_share_id\", md, resource\u003d\"snapshots\")"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    @decorators.idempotent_id(\u002703a7f6e9-de8b-4669-87e1-b179308b477d\u0027)"},{"line_number":42,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API)"},{"line_number":43,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":44,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":45,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":46,"context_line":"    def test_try_update_all_metadata_for_unexisting_share(self):"},{"line_number":47,"context_line":"        md \u003d {u\"key1\": u\"value1\", u\"key2\": u\"value2\", }"},{"line_number":48,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":49,"context_line":"                          self.shares_v2_client.update_all_metadata,"},{"line_number":50,"context_line":"                          \"wrong_share_id\", md, resource\u003d\"snapshots\")"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":20,"id":"1750e6ee_81bb4ada","line":50,"range":{"start_line":28,"start_character":0,"end_line":50,"end_character":69},"in_reply_to":"538fb26b_0119c94c","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"9d9340f802e52f997b31163cdc12d501c0c20432","unresolved":true,"context_lines":[{"line_number":53,"context_line":"@ddt.ddt"},{"line_number":54,"context_line":"class ShareSnapshotMetadataNegativeTest(base.BaseSharesMixedTest):"},{"line_number":55,"context_line":"    @classmethod"},{"line_number":56,"context_line":"    def resource_setup(cls):"},{"line_number":57,"context_line":"        super(ShareSnapshotMetadataNegativeTest, cls).resource_setup()"},{"line_number":58,"context_line":"        # create share_type"},{"line_number":59,"context_line":"        extra_specs \u003d {}"}],"source_content_type":"text/x-python","patch_set":20,"id":"e470d846_4401eed3","line":56,"range":{"start_line":56,"start_character":8,"end_line":56,"end_character":22},"updated":"2022-08-26 15:56:56.000000000","message":"This is similar to the ShareSnapshotMetadataTest resouce_setup, may consider sharing the method.","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    @decorators.idempotent_id(\u0027ef0afcc8-7b12-41bc-8aa1-4916ad4b4560\u0027)"},{"line_number":91,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":92,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":93,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":94,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":95,"context_line":"    def test_try_set_metadata_with_empty_key(self):"},{"line_number":96,"context_line":"        self.assertRaises(lib_exc.BadRequest,"},{"line_number":97,"context_line":"                          self.shares_v2_client.set_metadata,"}],"source_content_type":"text/x-python","patch_set":20,"id":"1eab4f08_75ea102b","line":94,"range":{"start_line":92,"start_character":0,"end_line":94,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"@classmethod\ndef skip_checks(cls):\n    super(ShareSnapshotMetadataNegativeTest, cls).skip_checks()\n    utils.check_skip_if_microversion_not_supported(\"2.72\")        \n    if not CONF.share.run_snapshot_tests:\n        raise cls.skipException(\u0027Snapshot tests are disabled.\u0027)","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    @decorators.idempotent_id(\u0027ef0afcc8-7b12-41bc-8aa1-4916ad4b4560\u0027)"},{"line_number":91,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":92,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":93,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":94,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":95,"context_line":"    def test_try_set_metadata_with_empty_key(self):"},{"line_number":96,"context_line":"        self.assertRaises(lib_exc.BadRequest,"},{"line_number":97,"context_line":"                          self.shares_v2_client.set_metadata,"}],"source_content_type":"text/x-python","patch_set":20,"id":"5a859fb3_1bd072e0","line":94,"range":{"start_line":92,"start_character":0,"end_line":94,"end_character":57},"in_reply_to":"1eab4f08_75ea102b","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    @decorators.idempotent_id(\u00279f2aee7c-ebd6-4516-87f7-bd85453d74c9\u0027)"},{"line_number":101,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":102,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":103,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":104,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":105,"context_line":"    def test_try_upd_metadata_with_empty_key(self):"},{"line_number":106,"context_line":"        self.assertRaises(lib_exc.BadRequest,"},{"line_number":107,"context_line":"                          self.shares_v2_client.update_all_metadata,"}],"source_content_type":"text/x-python","patch_set":20,"id":"29de48cd_69121632","line":104,"range":{"start_line":102,"start_character":0,"end_line":104,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    @decorators.idempotent_id(\u00279f2aee7c-ebd6-4516-87f7-bd85453d74c9\u0027)"},{"line_number":101,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":102,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":103,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":104,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":105,"context_line":"    def test_try_upd_metadata_with_empty_key(self):"},{"line_number":106,"context_line":"        self.assertRaises(lib_exc.BadRequest,"},{"line_number":107,"context_line":"                          self.shares_v2_client.update_all_metadata,"}],"source_content_type":"text/x-python","patch_set":20,"id":"c149d0f0_88d8923c","line":104,"range":{"start_line":102,"start_character":0,"end_line":104,"end_character":57},"in_reply_to":"29de48cd_69121632","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    @decorators.idempotent_id(\u0027ef61255e-462c-49fe-8e94-ff3afafcccb3\u0027)"},{"line_number":111,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":112,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":113,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":114,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":115,"context_line":"    def test_try_set_metadata_with_too_big_key(self):"},{"line_number":116,"context_line":"        too_big_key \u003d \"x\" * 256"},{"line_number":117,"context_line":"        md \u003d {too_big_key: \"value\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"78a85a2c_c4666971","line":114,"range":{"start_line":112,"start_character":0,"end_line":114,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    @decorators.idempotent_id(\u0027ef61255e-462c-49fe-8e94-ff3afafcccb3\u0027)"},{"line_number":111,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":112,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":113,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":114,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":115,"context_line":"    def test_try_set_metadata_with_too_big_key(self):"},{"line_number":116,"context_line":"        too_big_key \u003d \"x\" * 256"},{"line_number":117,"context_line":"        md \u003d {too_big_key: \"value\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"3de1bbbc_232bfa23","line":114,"range":{"start_line":112,"start_character":0,"end_line":114,"end_character":57},"in_reply_to":"78a85a2c_c4666971","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    @decorators.idempotent_id(\u0027f896f354-6179-4abb-b0c5-7b7dc96f0870\u0027)"},{"line_number":123,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":124,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":125,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":126,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":127,"context_line":"    def test_try_upd_metadata_with_too_big_key(self):"},{"line_number":128,"context_line":"        too_big_key \u003d \"x\" * 256"},{"line_number":129,"context_line":"        md \u003d {too_big_key: \"value\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"011c0bc9_56010268","line":126,"range":{"start_line":124,"start_character":0,"end_line":126,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    @decorators.idempotent_id(\u0027f896f354-6179-4abb-b0c5-7b7dc96f0870\u0027)"},{"line_number":123,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":124,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":125,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":126,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":127,"context_line":"    def test_try_upd_metadata_with_too_big_key(self):"},{"line_number":128,"context_line":"        too_big_key \u003d \"x\" * 256"},{"line_number":129,"context_line":"        md \u003d {too_big_key: \"value\"}"}],"source_content_type":"text/x-python","patch_set":20,"id":"4ecfd4ce_b234953b","line":126,"range":{"start_line":124,"start_character":0,"end_line":126,"end_character":57},"in_reply_to":"011c0bc9_56010268","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":133,"context_line":""},{"line_number":134,"context_line":"    @decorators.idempotent_id(\u00271bf97c18-27df-4618-94f4-224d1a98bc0c\u0027)"},{"line_number":135,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":136,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":137,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":138,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":139,"context_line":"    def test_try_set_metadata_with_too_big_value(self):"},{"line_number":140,"context_line":"        too_big_value \u003d \"x\" * 1024"},{"line_number":141,"context_line":"        md \u003d {\"key\": too_big_value}"}],"source_content_type":"text/x-python","patch_set":20,"id":"2c650b0c_56661683","line":138,"range":{"start_line":136,"start_character":0,"end_line":138,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":133,"context_line":""},{"line_number":134,"context_line":"    @decorators.idempotent_id(\u00271bf97c18-27df-4618-94f4-224d1a98bc0c\u0027)"},{"line_number":135,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":136,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":137,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":138,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":139,"context_line":"    def test_try_set_metadata_with_too_big_value(self):"},{"line_number":140,"context_line":"        too_big_value \u003d \"x\" * 1024"},{"line_number":141,"context_line":"        md \u003d {\"key\": too_big_value}"}],"source_content_type":"text/x-python","patch_set":20,"id":"db5faa5e_de65fae5","line":138,"range":{"start_line":136,"start_character":0,"end_line":138,"end_character":57},"in_reply_to":"2c650b0c_56661683","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"    @decorators.idempotent_id(\u00272b9e08fa-b35d-4bfe-9137-e59ab50bd9ef\u0027)"},{"line_number":147,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":148,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":149,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":150,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":151,"context_line":"    def test_try_upd_metadata_with_too_big_value(self):"},{"line_number":152,"context_line":"        too_big_value \u003d \"x\" * 1024"},{"line_number":153,"context_line":"        md \u003d {\"key\": too_big_value}"}],"source_content_type":"text/x-python","patch_set":20,"id":"3713c6bf_aeda5ff8","line":150,"range":{"start_line":148,"start_character":0,"end_line":150,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"    @decorators.idempotent_id(\u00272b9e08fa-b35d-4bfe-9137-e59ab50bd9ef\u0027)"},{"line_number":147,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":148,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":149,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":150,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":151,"context_line":"    def test_try_upd_metadata_with_too_big_value(self):"},{"line_number":152,"context_line":"        too_big_value \u003d \"x\" * 1024"},{"line_number":153,"context_line":"        md \u003d {\"key\": too_big_value}"}],"source_content_type":"text/x-python","patch_set":20,"id":"606cd52b_1b8a245b","line":150,"range":{"start_line":148,"start_character":0,"end_line":150,"end_character":57},"in_reply_to":"3713c6bf_aeda5ff8","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"02d7ad5c37703743506d322dd1bed681422886f9","unresolved":true,"context_lines":[{"line_number":157,"context_line":""},{"line_number":158,"context_line":"    @decorators.idempotent_id(\u00279afb381d-c48c-4c2c-a5b5-42463daef5a2\u0027)"},{"line_number":159,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":160,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":161,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":162,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":163,"context_line":"    def test_try_delete_unexisting_metadata(self):"},{"line_number":164,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":165,"context_line":"                          self.shares_v2_client.delete_metadata,"}],"source_content_type":"text/x-python","patch_set":20,"id":"a6094e4e_2bd6a424","line":162,"range":{"start_line":160,"start_character":0,"end_line":162,"end_character":57},"updated":"2022-08-25 07:52:37.000000000","message":"Same","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"dc6e0b1372731fad39241c7c0bbd415404154a32","unresolved":false,"context_lines":[{"line_number":157,"context_line":""},{"line_number":158,"context_line":"    @decorators.idempotent_id(\u00279afb381d-c48c-4c2c-a5b5-42463daef5a2\u0027)"},{"line_number":159,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":160,"context_line":"    @utils.skip_if_microversion_not_supported(\"2.72\")"},{"line_number":161,"context_line":"    @testtools.skipUnless(CONF.share.run_snapshot_tests,"},{"line_number":162,"context_line":"                          \"Snapshot tests are disabled.\")"},{"line_number":163,"context_line":"    def test_try_delete_unexisting_metadata(self):"},{"line_number":164,"context_line":"        self.assertRaises(lib_exc.NotFound,"},{"line_number":165,"context_line":"                          self.shares_v2_client.delete_metadata,"}],"source_content_type":"text/x-python","patch_set":20,"id":"b4b77bcc_9b53e215","line":162,"range":{"start_line":160,"start_character":0,"end_line":162,"end_character":57},"in_reply_to":"a6094e4e_2bd6a424","updated":"2022-09-16 18:46:38.000000000","message":"Done","commit_id":"5823514681dd89e8ecdcedeaec2aa81c469ee3ce"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f057efd26b7f262862f83d77d61dc10cfd47cbe8","unresolved":true,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    @decorators.idempotent_id(\u00279f2aee7c-ebd6-4516-87f7-bd85453d74c9\u0027)"},{"line_number":95,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":96,"context_line":"    def test_try_upd_metadata_with_empty_key(self):"},{"line_number":97,"context_line":"        self.assertRaises(lib_exc.BadRequest,"},{"line_number":98,"context_line":"                          self.shares_v2_client.update_all_metadata,"},{"line_number":99,"context_line":"                          self.snap_id, {\"\": \"value\"}, resource\u003d\"snapshots\")"}],"source_content_type":"text/x-python","patch_set":24,"id":"a3345b1b_f35892d3","line":96,"range":{"start_line":96,"start_character":17,"end_line":96,"end_character":20},"updated":"2022-09-22 22:44:18.000000000","message":"nit: update","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d7f2d9d6cd4b364916995b3a215d444c149d68c","unresolved":false,"context_lines":[{"line_number":93,"context_line":""},{"line_number":94,"context_line":"    @decorators.idempotent_id(\u00279f2aee7c-ebd6-4516-87f7-bd85453d74c9\u0027)"},{"line_number":95,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":96,"context_line":"    def test_try_upd_metadata_with_empty_key(self):"},{"line_number":97,"context_line":"        self.assertRaises(lib_exc.BadRequest,"},{"line_number":98,"context_line":"                          self.shares_v2_client.update_all_metadata,"},{"line_number":99,"context_line":"                          self.snap_id, {\"\": \"value\"}, resource\u003d\"snapshots\")"}],"source_content_type":"text/x-python","patch_set":24,"id":"a95abaed_d9c2978e","line":96,"range":{"start_line":96,"start_character":17,"end_line":96,"end_character":20},"in_reply_to":"a3345b1b_f35892d3","updated":"2022-09-29 15:09:11.000000000","message":"Done","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f057efd26b7f262862f83d77d61dc10cfd47cbe8","unresolved":true,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    @decorators.idempotent_id(\u0027f896f354-6179-4abb-b0c5-7b7dc96f0870\u0027)"},{"line_number":111,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":112,"context_line":"    def test_try_upd_metadata_with_too_big_key(self):"},{"line_number":113,"context_line":"        too_big_key \u003d \"x\" * 256"},{"line_number":114,"context_line":"        md \u003d {too_big_key: \"value\"}"},{"line_number":115,"context_line":"        self.assertRaises(lib_exc.BadRequest,"}],"source_content_type":"text/x-python","patch_set":24,"id":"98b3dfe8_5102cd22","line":112,"range":{"start_line":112,"start_character":17,"end_line":112,"end_character":20},"updated":"2022-09-22 22:44:18.000000000","message":"same","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d7f2d9d6cd4b364916995b3a215d444c149d68c","unresolved":false,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    @decorators.idempotent_id(\u0027f896f354-6179-4abb-b0c5-7b7dc96f0870\u0027)"},{"line_number":111,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":112,"context_line":"    def test_try_upd_metadata_with_too_big_key(self):"},{"line_number":113,"context_line":"        too_big_key \u003d \"x\" * 256"},{"line_number":114,"context_line":"        md \u003d {too_big_key: \"value\"}"},{"line_number":115,"context_line":"        self.assertRaises(lib_exc.BadRequest,"}],"source_content_type":"text/x-python","patch_set":24,"id":"cef8ec20_bba127ac","line":112,"range":{"start_line":112,"start_character":17,"end_line":112,"end_character":20},"in_reply_to":"98b3dfe8_5102cd22","updated":"2022-09-29 15:09:11.000000000","message":"Done","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f057efd26b7f262862f83d77d61dc10cfd47cbe8","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @decorators.idempotent_id(\u00272b9e08fa-b35d-4bfe-9137-e59ab50bd9ef\u0027)"},{"line_number":129,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":130,"context_line":"    def test_try_upd_metadata_with_too_big_value(self):"},{"line_number":131,"context_line":"        too_big_value \u003d \"x\" * 1024"},{"line_number":132,"context_line":"        md \u003d {\"key\": too_big_value}"},{"line_number":133,"context_line":"        self.assertRaises(lib_exc.BadRequest,"}],"source_content_type":"text/x-python","patch_set":24,"id":"2cbb2343_6aed182f","line":130,"range":{"start_line":130,"start_character":17,"end_line":130,"end_character":20},"updated":"2022-09-22 22:44:18.000000000","message":"same","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"7d7f2d9d6cd4b364916995b3a215d444c149d68c","unresolved":false,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    @decorators.idempotent_id(\u00272b9e08fa-b35d-4bfe-9137-e59ab50bd9ef\u0027)"},{"line_number":129,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":130,"context_line":"    def test_try_upd_metadata_with_too_big_value(self):"},{"line_number":131,"context_line":"        too_big_value \u003d \"x\" * 1024"},{"line_number":132,"context_line":"        md \u003d {\"key\": too_big_value}"},{"line_number":133,"context_line":"        self.assertRaises(lib_exc.BadRequest,"}],"source_content_type":"text/x-python","patch_set":24,"id":"338e3e7e_a0ac8dcd","line":130,"range":{"start_line":130,"start_character":17,"end_line":130,"end_character":20},"in_reply_to":"2cbb2343_6aed182f","updated":"2022-09-29 15:09:11.000000000","message":"Done","commit_id":"96e64457642c45d0fca6f9820ef88d5d111e3921"}],"manila_tempest_tests/tests/rbac/test_shares.py":[{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":273,"context_line":"        share \u003d self.create_share("},{"line_number":274,"context_line":"            self.share_member_client, self.share_type[\u0027id\u0027])"},{"line_number":275,"context_line":"        self.do_request("},{"line_number":276,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":277,"context_line":"            share_id\u003dshare[\u0027id\u0027],"},{"line_number":278,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":279,"context_line":""}],"source_content_type":"text/x-python","patch_set":28,"id":"47a8c918_179fe068","line":276,"range":{"start_line":276,"start_character":49,"end_line":276,"end_character":77},"updated":"2022-10-19 14:50:22.000000000","message":"This is wrong.\nIt should uses the persona client.\nself.do_request uses it as default.\nThe client is pointed to v2.","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":273,"context_line":"        share \u003d self.create_share("},{"line_number":274,"context_line":"            self.share_member_client, self.share_type[\u0027id\u0027])"},{"line_number":275,"context_line":"        self.do_request("},{"line_number":276,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":277,"context_line":"            share_id\u003dshare[\u0027id\u0027],"},{"line_number":278,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":279,"context_line":""}],"source_content_type":"text/x-python","patch_set":28,"id":"e99eded0_b72463da","line":276,"range":{"start_line":276,"start_character":49,"end_line":276,"end_character":77},"in_reply_to":"47a8c918_179fe068","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":274,"context_line":"            self.share_member_client, self.share_type[\u0027id\u0027])"},{"line_number":275,"context_line":"        self.do_request("},{"line_number":276,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":277,"context_line":"            share_id\u003dshare[\u0027id\u0027],"},{"line_number":278,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":28,"id":"1f4bdd7e_37bd0897","line":277,"range":{"start_line":277,"start_character":12,"end_line":277,"end_character":20},"updated":"2022-10-19 14:50:22.000000000","message":"You need to change it to \"resource_id\"","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":274,"context_line":"            self.share_member_client, self.share_type[\u0027id\u0027])"},{"line_number":275,"context_line":"        self.do_request("},{"line_number":276,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":277,"context_line":"            share_id\u003dshare[\u0027id\u0027],"},{"line_number":278,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":279,"context_line":""},{"line_number":280,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":28,"id":"26d2b91d_56857518","line":277,"range":{"start_line":277,"start_character":12,"end_line":277,"end_character":20},"in_reply_to":"1f4bdd7e_37bd0897","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":280,"context_line":"        alt_share \u003d self.create_share("},{"line_number":281,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"},{"line_number":282,"context_line":"        self.do_request("},{"line_number":283,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":284,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"    @decorators.idempotent_id(\u00272d91e97e-d0e5-4112-8b22-60cd4659586c\u0027)"}],"source_content_type":"text/x-python","patch_set":28,"id":"b19d98a9_0c93c21c","line":283,"range":{"start_line":283,"start_character":49,"end_line":283,"end_character":77},"updated":"2022-10-19 14:50:22.000000000","message":"Please remove it.","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":280,"context_line":"        alt_share \u003d self.create_share("},{"line_number":281,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"},{"line_number":282,"context_line":"        self.do_request("},{"line_number":283,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":284,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"    @decorators.idempotent_id(\u00272d91e97e-d0e5-4112-8b22-60cd4659586c\u0027)"}],"source_content_type":"text/x-python","patch_set":28,"id":"a22175b6_71cae1d3","line":283,"range":{"start_line":283,"start_character":49,"end_line":283,"end_character":77},"in_reply_to":"b19d98a9_0c93c21c","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":281,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"},{"line_number":282,"context_line":"        self.do_request("},{"line_number":283,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":284,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"    @decorators.idempotent_id(\u00272d91e97e-d0e5-4112-8b22-60cd4659586c\u0027)"},{"line_number":287,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":28,"id":"bb0dfc77_29b910ac","line":284,"range":{"start_line":284,"start_character":12,"end_line":284,"end_character":20},"updated":"2022-10-19 14:50:22.000000000","message":"resource_id","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":281,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"},{"line_number":282,"context_line":"        self.do_request("},{"line_number":283,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":284,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":285,"context_line":""},{"line_number":286,"context_line":"    @decorators.idempotent_id(\u00272d91e97e-d0e5-4112-8b22-60cd4659586c\u0027)"},{"line_number":287,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":28,"id":"7eee259f_0cc0c901","line":284,"range":{"start_line":284,"start_character":12,"end_line":284,"end_character":20},"in_reply_to":"bb0dfc77_29b910ac","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":291,"context_line":"            self.share_member_client, self.share_type[\u0027id\u0027],"},{"line_number":292,"context_line":"            metadata\u003dmetadata)"},{"line_number":293,"context_line":"        self.do_request("},{"line_number":294,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":295,"context_line":"            share_id\u003dshare[\u0027id\u0027])"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":28,"id":"4ba7127d_477436a0","line":294,"range":{"start_line":294,"start_character":49,"end_line":294,"end_character":77},"updated":"2022-10-19 14:50:22.000000000","message":"Please remove it.","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":291,"context_line":"            self.share_member_client, self.share_type[\u0027id\u0027],"},{"line_number":292,"context_line":"            metadata\u003dmetadata)"},{"line_number":293,"context_line":"        self.do_request("},{"line_number":294,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":295,"context_line":"            share_id\u003dshare[\u0027id\u0027])"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":28,"id":"9de403d7_ba808074","line":294,"range":{"start_line":294,"start_character":49,"end_line":294,"end_character":77},"in_reply_to":"4ba7127d_477436a0","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":292,"context_line":"            metadata\u003dmetadata)"},{"line_number":293,"context_line":"        self.do_request("},{"line_number":294,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":295,"context_line":"            share_id\u003dshare[\u0027id\u0027])"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        alt_share \u003d self.create_share("},{"line_number":298,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":28,"id":"73fd2996_6989a720","line":295,"range":{"start_line":295,"start_character":12,"end_line":295,"end_character":20},"updated":"2022-10-19 14:50:22.000000000","message":"resource_id","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":292,"context_line":"            metadata\u003dmetadata)"},{"line_number":293,"context_line":"        self.do_request("},{"line_number":294,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":295,"context_line":"            share_id\u003dshare[\u0027id\u0027])"},{"line_number":296,"context_line":""},{"line_number":297,"context_line":"        alt_share \u003d self.create_share("},{"line_number":298,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":28,"id":"21c72039_c555d81d","line":295,"range":{"start_line":295,"start_character":12,"end_line":295,"end_character":20},"in_reply_to":"73fd2996_6989a720","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":298,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027],"},{"line_number":299,"context_line":"            metadata\u003dmetadata)"},{"line_number":300,"context_line":"        self.do_request("},{"line_number":301,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":302,"context_line":"            share_id\u003dalt_share[\u0027id\u0027])"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    @decorators.idempotent_id(\u00274cd807d6-bac4-4d0f-a207-c84dfe77f032\u0027)"}],"source_content_type":"text/x-python","patch_set":28,"id":"227e87f1_acda0e80","line":301,"range":{"start_line":301,"start_character":49,"end_line":301,"end_character":77},"updated":"2022-10-19 14:50:22.000000000","message":"Please remove it.","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":298,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027],"},{"line_number":299,"context_line":"            metadata\u003dmetadata)"},{"line_number":300,"context_line":"        self.do_request("},{"line_number":301,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":302,"context_line":"            share_id\u003dalt_share[\u0027id\u0027])"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    @decorators.idempotent_id(\u00274cd807d6-bac4-4d0f-a207-c84dfe77f032\u0027)"}],"source_content_type":"text/x-python","patch_set":28,"id":"9b06616f_74460b47","line":301,"range":{"start_line":301,"start_character":49,"end_line":301,"end_character":77},"in_reply_to":"227e87f1_acda0e80","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":299,"context_line":"            metadata\u003dmetadata)"},{"line_number":300,"context_line":"        self.do_request("},{"line_number":301,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":302,"context_line":"            share_id\u003dalt_share[\u0027id\u0027])"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    @decorators.idempotent_id(\u00274cd807d6-bac4-4d0f-a207-c84dfe77f032\u0027)"},{"line_number":305,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":28,"id":"fc2e8c79_9411d6b8","line":302,"range":{"start_line":302,"start_character":12,"end_line":302,"end_character":20},"updated":"2022-10-19 14:50:22.000000000","message":"resource_id","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":299,"context_line":"            metadata\u003dmetadata)"},{"line_number":300,"context_line":"        self.do_request("},{"line_number":301,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, client\u003dself.shares_v2_client,"},{"line_number":302,"context_line":"            share_id\u003dalt_share[\u0027id\u0027])"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    @decorators.idempotent_id(\u00274cd807d6-bac4-4d0f-a207-c84dfe77f032\u0027)"},{"line_number":305,"context_line":"    @tc.attr(base.TAG_POSITIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":28,"id":"4c5eea67_481b4645","line":302,"range":{"start_line":302,"start_character":12,"end_line":302,"end_character":20},"in_reply_to":"fc2e8c79_9411d6b8","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":309,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":310,"context_line":"        self.do_request("},{"line_number":311,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003d200,"},{"line_number":312,"context_line":"            client\u003dself.shares_v2_client, share_id\u003dshare[\u0027id\u0027],"},{"line_number":313,"context_line":"            key\u003d\u0027key\u0027)"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":28,"id":"322504b4_ba790b36","line":312,"range":{"start_line":312,"start_character":12,"end_line":312,"end_character":40},"updated":"2022-10-19 14:50:22.000000000","message":"Please remove it.","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":309,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":310,"context_line":"        self.do_request("},{"line_number":311,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003d200,"},{"line_number":312,"context_line":"            client\u003dself.shares_v2_client, share_id\u003dshare[\u0027id\u0027],"},{"line_number":313,"context_line":"            key\u003d\u0027key\u0027)"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":28,"id":"66465a1d_c2caa1c4","line":312,"range":{"start_line":312,"start_character":42,"end_line":312,"end_character":50},"updated":"2022-10-19 14:50:22.000000000","message":"resource_id","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":309,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":310,"context_line":"        self.do_request("},{"line_number":311,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003d200,"},{"line_number":312,"context_line":"            client\u003dself.shares_v2_client, share_id\u003dshare[\u0027id\u0027],"},{"line_number":313,"context_line":"            key\u003d\u0027key\u0027)"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":28,"id":"0f1f8063_ca04c383","line":312,"range":{"start_line":312,"start_character":12,"end_line":312,"end_character":40},"in_reply_to":"322504b4_ba790b36","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":309,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":310,"context_line":"        self.do_request("},{"line_number":311,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003d200,"},{"line_number":312,"context_line":"            client\u003dself.shares_v2_client, share_id\u003dshare[\u0027id\u0027],"},{"line_number":313,"context_line":"            key\u003d\u0027key\u0027)"},{"line_number":314,"context_line":""},{"line_number":315,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":28,"id":"0d53f506_75b59b70","line":312,"range":{"start_line":312,"start_character":42,"end_line":312,"end_character":50},"in_reply_to":"66465a1d_c2caa1c4","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"43fc616d1a904d446d1da6fd78e62e52248f0d3e","unresolved":true,"context_lines":[{"line_number":317,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":318,"context_line":"        self.do_request("},{"line_number":319,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003d200,"},{"line_number":320,"context_line":"            client\u003dself.shares_v2_client,"},{"line_number":321,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], key\u003d\u0027key\u0027)"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":""}],"source_content_type":"text/x-python","patch_set":28,"id":"ff6a7ca3_4c521855","line":320,"range":{"start_line":320,"start_character":12,"end_line":320,"end_character":40},"updated":"2022-10-19 14:50:22.000000000","message":"Please remove it.","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"9abc0781c835fd6a27dcb32bfae55a87c96a00df","unresolved":false,"context_lines":[{"line_number":317,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":318,"context_line":"        self.do_request("},{"line_number":319,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003d200,"},{"line_number":320,"context_line":"            client\u003dself.shares_v2_client,"},{"line_number":321,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], key\u003d\u0027key\u0027)"},{"line_number":322,"context_line":""},{"line_number":323,"context_line":""}],"source_content_type":"text/x-python","patch_set":28,"id":"d2716ece_d4eafc86","line":320,"range":{"start_line":320,"start_character":12,"end_line":320,"end_character":40},"in_reply_to":"ff6a7ca3_4c521855","updated":"2022-10-19 15:06:46.000000000","message":"Done","commit_id":"489833db2f58700356a5cd58e423d9d7d994a8f3"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":468,"context_line":"        share \u003d self.create_share("},{"line_number":469,"context_line":"            self.client, self.share_type[\u0027id\u0027])"},{"line_number":470,"context_line":"        self.do_request("},{"line_number":471,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, share_id\u003dshare[\u0027id\u0027],"},{"line_number":472,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":29,"id":"7e313041_54c44973","line":471,"range":{"start_line":471,"start_character":49,"end_line":471,"end_character":57},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":468,"context_line":"        share \u003d self.create_share("},{"line_number":469,"context_line":"            self.client, self.share_type[\u0027id\u0027])"},{"line_number":470,"context_line":"        self.do_request("},{"line_number":471,"context_line":"            \u0027set_metadata\u0027, expected_status\u003d200, share_id\u003dshare[\u0027id\u0027],"},{"line_number":472,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":473,"context_line":""},{"line_number":474,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":29,"id":"3d529d12_e368a6d9","line":471,"range":{"start_line":471,"start_character":49,"end_line":471,"end_character":57},"in_reply_to":"7e313041_54c44973","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":475,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"},{"line_number":476,"context_line":"        self.do_request("},{"line_number":477,"context_line":"            \u0027set_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":478,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":479,"context_line":""},{"line_number":480,"context_line":"    @decorators.idempotent_id(\u0027a69a2b85-3374-4621-83a9-89937ddb520b\u0027)"},{"line_number":481,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":29,"id":"b4621989_dc00ce23","line":478,"range":{"start_line":478,"start_character":12,"end_line":478,"end_character":20},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":475,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"},{"line_number":476,"context_line":"        self.do_request("},{"line_number":477,"context_line":"            \u0027set_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":478,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":479,"context_line":""},{"line_number":480,"context_line":"    @decorators.idempotent_id(\u0027a69a2b85-3374-4621-83a9-89937ddb520b\u0027)"},{"line_number":481,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":29,"id":"cd1cd570_b5da9573","line":478,"range":{"start_line":478,"start_character":12,"end_line":478,"end_character":20},"in_reply_to":"b4621989_dc00ce23","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":485,"context_line":"        share \u003d self.create_share(share_client, self.share_type[\u0027id\u0027],"},{"line_number":486,"context_line":"                                  metadata\u003dmetadata)"},{"line_number":487,"context_line":"        self.do_request("},{"line_number":488,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, share_id\u003dshare[\u0027id\u0027])"},{"line_number":489,"context_line":""},{"line_number":490,"context_line":"        alt_share \u003d self.create_share("},{"line_number":491,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":29,"id":"e2069dfc_43e6ca88","line":488,"range":{"start_line":488,"start_character":49,"end_line":488,"end_character":57},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":485,"context_line":"        share \u003d self.create_share(share_client, self.share_type[\u0027id\u0027],"},{"line_number":486,"context_line":"                                  metadata\u003dmetadata)"},{"line_number":487,"context_line":"        self.do_request("},{"line_number":488,"context_line":"            \u0027get_metadata\u0027, expected_status\u003d200, share_id\u003dshare[\u0027id\u0027])"},{"line_number":489,"context_line":""},{"line_number":490,"context_line":"        alt_share \u003d self.create_share("},{"line_number":491,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":29,"id":"994b7035_5baffe9d","line":488,"range":{"start_line":488,"start_character":49,"end_line":488,"end_character":57},"in_reply_to":"e2069dfc_43e6ca88","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":492,"context_line":"            metadata\u003dmetadata)"},{"line_number":493,"context_line":"        self.do_request("},{"line_number":494,"context_line":"            \u0027get_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":495,"context_line":"            share_id\u003dalt_share[\u0027id\u0027])"},{"line_number":496,"context_line":""},{"line_number":497,"context_line":"    @decorators.idempotent_id(\u0027bea5518a-338e-494d-9034-1d03658ed58b\u0027)"},{"line_number":498,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":29,"id":"c2f83273_df43aa5c","line":495,"range":{"start_line":495,"start_character":12,"end_line":495,"end_character":20},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":492,"context_line":"            metadata\u003dmetadata)"},{"line_number":493,"context_line":"        self.do_request("},{"line_number":494,"context_line":"            \u0027get_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":495,"context_line":"            share_id\u003dalt_share[\u0027id\u0027])"},{"line_number":496,"context_line":""},{"line_number":497,"context_line":"    @decorators.idempotent_id(\u0027bea5518a-338e-494d-9034-1d03658ed58b\u0027)"},{"line_number":498,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":29,"id":"1eeabde9_7c48ca9a","line":495,"range":{"start_line":495,"start_character":12,"end_line":495,"end_character":20},"in_reply_to":"c2f83273_df43aa5c","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":500,"context_line":"        share \u003d self.create_share("},{"line_number":501,"context_line":"            self.client, self.share_type[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":502,"context_line":"        self.do_request("},{"line_number":503,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003d200, share_id\u003dshare[\u0027id\u0027],"},{"line_number":504,"context_line":"            key\u003d\u0027key\u0027)"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":29,"id":"9acc6ba9_b2bc6edf","line":503,"range":{"start_line":503,"start_character":52,"end_line":503,"end_character":60},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":500,"context_line":"        share \u003d self.create_share("},{"line_number":501,"context_line":"            self.client, self.share_type[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":502,"context_line":"        self.do_request("},{"line_number":503,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003d200, share_id\u003dshare[\u0027id\u0027],"},{"line_number":504,"context_line":"            key\u003d\u0027key\u0027)"},{"line_number":505,"context_line":""},{"line_number":506,"context_line":"        alt_share \u003d self.create_share("}],"source_content_type":"text/x-python","patch_set":29,"id":"fc8640fa_f5c94f81","line":503,"range":{"start_line":503,"start_character":52,"end_line":503,"end_character":60},"in_reply_to":"9acc6ba9_b2bc6edf","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":508,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":509,"context_line":"        self.do_request("},{"line_number":510,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":511,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], key\u003d\u0027key\u0027)"},{"line_number":512,"context_line":""},{"line_number":513,"context_line":""},{"line_number":514,"context_line":"class TestProjectReaderTestsNFS(TestProjectMemberTestsNFS):"}],"source_content_type":"text/x-python","patch_set":29,"id":"0baee242_4419a1e3","line":511,"range":{"start_line":511,"start_character":12,"end_line":511,"end_character":20},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":508,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":509,"context_line":"        self.do_request("},{"line_number":510,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":511,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], key\u003d\u0027key\u0027)"},{"line_number":512,"context_line":""},{"line_number":513,"context_line":""},{"line_number":514,"context_line":"class TestProjectReaderTestsNFS(TestProjectMemberTestsNFS):"}],"source_content_type":"text/x-python","patch_set":29,"id":"bad90f8c_a9084e05","line":511,"range":{"start_line":511,"start_character":12,"end_line":511,"end_character":20},"in_reply_to":"0baee242_4419a1e3","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":631,"context_line":"            self.share_member_client, self.share_type[\u0027id\u0027])"},{"line_number":632,"context_line":"        self.do_request("},{"line_number":633,"context_line":"            \u0027set_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":634,"context_line":"            share_id\u003dshare[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":635,"context_line":""},{"line_number":636,"context_line":"        alt_share \u003d self.create_share("},{"line_number":637,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":29,"id":"1b1deb61_c93c54ec","line":634,"range":{"start_line":634,"start_character":12,"end_line":634,"end_character":20},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":631,"context_line":"            self.share_member_client, self.share_type[\u0027id\u0027])"},{"line_number":632,"context_line":"        self.do_request("},{"line_number":633,"context_line":"            \u0027set_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":634,"context_line":"            share_id\u003dshare[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":635,"context_line":""},{"line_number":636,"context_line":"        alt_share \u003d self.create_share("},{"line_number":637,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":29,"id":"6d851bcd_149ac179","line":634,"range":{"start_line":634,"start_character":12,"end_line":634,"end_character":20},"in_reply_to":"1b1deb61_c93c54ec","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":637,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"},{"line_number":638,"context_line":"        self.do_request("},{"line_number":639,"context_line":"            \u0027set_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":640,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":641,"context_line":""},{"line_number":642,"context_line":"    @decorators.idempotent_id(\u002728cacc77-556f-4707-ba2b-5ef3e56d6ef9\u0027)"},{"line_number":643,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":29,"id":"5b0d5e2f_b9e9baaf","line":640,"range":{"start_line":640,"start_character":12,"end_line":640,"end_character":20},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":637,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027])"},{"line_number":638,"context_line":"        self.do_request("},{"line_number":639,"context_line":"            \u0027set_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":640,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":641,"context_line":""},{"line_number":642,"context_line":"    @decorators.idempotent_id(\u002728cacc77-556f-4707-ba2b-5ef3e56d6ef9\u0027)"},{"line_number":643,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"}],"source_content_type":"text/x-python","patch_set":29,"id":"abfb296e_74ded158","line":640,"range":{"start_line":640,"start_character":12,"end_line":640,"end_character":20},"in_reply_to":"5b0d5e2f_b9e9baaf","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":652,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":653,"context_line":"        self.do_request("},{"line_number":654,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":655,"context_line":"            share_id\u003dshare[\u0027id\u0027], key\u003d\u0027key\u0027)"},{"line_number":656,"context_line":""},{"line_number":657,"context_line":"        alt_share \u003d self.create_share("},{"line_number":658,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":29,"id":"34da7723_c20c9d9b","line":655,"range":{"start_line":655,"start_character":12,"end_line":655,"end_character":20},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":652,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":653,"context_line":"        self.do_request("},{"line_number":654,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":655,"context_line":"            share_id\u003dshare[\u0027id\u0027], key\u003d\u0027key\u0027)"},{"line_number":656,"context_line":""},{"line_number":657,"context_line":"        alt_share \u003d self.create_share("},{"line_number":658,"context_line":"            self.alt_project_share_v2_client, self.share_type[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":29,"id":"28e6c980_7a5e125d","line":655,"range":{"start_line":655,"start_character":12,"end_line":655,"end_character":20},"in_reply_to":"34da7723_c20c9d9b","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"2acca92143bf0219d9cae56407ae9081159f04e3","unresolved":true,"context_lines":[{"line_number":659,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":660,"context_line":"        self.do_request("},{"line_number":661,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":662,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], key\u003d\u0027key\u0027)"},{"line_number":663,"context_line":""},{"line_number":664,"context_line":""},{"line_number":665,"context_line":"class TestProjectAdminTestsCEPHFS(TestProjectAdminTestsNFS):"}],"source_content_type":"text/x-python","patch_set":29,"id":"c171a5c2_7633c663","line":662,"range":{"start_line":662,"start_character":12,"end_line":662,"end_character":20},"updated":"2022-10-19 15:34:53.000000000","message":"resource_id","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"},{"author":{"_account_id":32594,"name":"Ashley Rodriguez","email":"ashrod98@redhat.com","username":"ashrod98"},"change_message_id":"3c7d5e73eecf852b529109358fc9d3a158b5df20","unresolved":false,"context_lines":[{"line_number":659,"context_line":"            metadata\u003d{\u0027key\u0027: \u0027value\u0027})"},{"line_number":660,"context_line":"        self.do_request("},{"line_number":661,"context_line":"            \u0027delete_metadata\u0027, expected_status\u003dlib_exc.Forbidden,"},{"line_number":662,"context_line":"            share_id\u003dalt_share[\u0027id\u0027], key\u003d\u0027key\u0027)"},{"line_number":663,"context_line":""},{"line_number":664,"context_line":""},{"line_number":665,"context_line":"class TestProjectAdminTestsCEPHFS(TestProjectAdminTestsNFS):"}],"source_content_type":"text/x-python","patch_set":29,"id":"a59033dd_8e2ce27e","line":662,"range":{"start_line":662,"start_character":12,"end_line":662,"end_character":20},"in_reply_to":"c171a5c2_7633c663","updated":"2022-10-19 20:55:23.000000000","message":"Done","commit_id":"14b8dec8fedc3eba7290ae554605ba296b2185c1"}]}