)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"ee815be7333c07503708d632d882f44c5a1cecfa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f3c890a8_ea42afb5","updated":"2023-08-13 06:54:58.000000000","message":"recheck\n\nUnrelated issues in cinder-tempest-plugin-lvm-nvmet-barbican and cinder-tempest-plugin-cbak-ceph","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"8ae8892f262229be92d60c8eeb36d3d2d2b6ac3f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fbc3ecc5_0052c3f7","updated":"2023-08-13 18:17:01.000000000","message":"recheck\n\nUnrelated issues in cinder-tempest-plugin-lvm-nvmet-barbican and cinder-tempest-plugin-cbak-ceph","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"7c132f9b180ffe3bf0323f711de9c2e38acc25a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"523a55e9_9960b716","updated":"2023-08-15 18:42:01.000000000","message":"recheck\n\nUnrelated issues in cinder-tempest-plugin-lvm-lio-barbican and cinder-tempest-plugin-lvm-nvmet-barbican","commit_id":"15a85eb7dd437cc9ef9411434b6ec90b3fdd1c7a"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"acc6408d4746b6c88c4592cbc9ac372c55fe1f50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"062b68dc_3be9d9e7","updated":"2023-08-16 11:51:05.000000000","message":"Thanks Yosi! LGTM!\nonly a minor comment","commit_id":"7752cc68e9086090248be34e5783885d886a475f"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"87288558cae5a89a4e2f64f16431743a248bbda4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"67b8a7b1_414937eb","updated":"2023-08-28 06:26:51.000000000","message":"run-NetApp CI","commit_id":"b42681daf98710688282a5295b88997ee664060a"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"a315a4b3e0687383f02b5af4a812a38791b89966","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f2ec5215_caa4af18","updated":"2023-08-27 14:58:31.000000000","message":"run-NetApp CI","commit_id":"b42681daf98710688282a5295b88997ee664060a"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"f8208bc2419a9dc4ee79bddafbb4f2e90d33ad9f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"aee863ea_702f5688","updated":"2024-06-09 10:16:22.000000000","message":"recheck\n\nfailure in base.py from rebase patch","commit_id":"edf13279bcc92959be59f2fd558b6836ae5295d9"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"b393d24bcb4a0cb078e119e28996361908133109","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c9a2745b_2d9248cd","updated":"2024-05-27 12:07:01.000000000","message":"recheck\n\nfailure in base.py from rebase patch","commit_id":"edf13279bcc92959be59f2fd558b6836ae5295d9"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"1a0568e2946c0b9950c8cba9abcba4c48a6bd285","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"582bf685_2e423ac2","updated":"2024-09-18 21:01:12.000000000","message":"Verified, looks good with exception of test skipped due to 2030939. Thanks Yosi.","commit_id":"8e4ba89653b1700ccfc8abbf60a7d6953c6f60d9"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"ac49730743b22877d2bda149f73eb74d09b2d1a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"918e5c39_81f9f11f","updated":"2024-06-10 12:10:29.000000000","message":"recheck\n\nunrelated issue in cinder-tempest-plugin-lvm-lio-barbican","commit_id":"8e4ba89653b1700ccfc8abbf60a7d6953c6f60d9"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"be569f784434e257d22d2395d4e2c347eaea506a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"0bfddd77_8487a307","updated":"2024-08-16 16:29:26.000000000","message":"recheck - need fresh logs","commit_id":"8e4ba89653b1700ccfc8abbf60a7d6953c6f60d9"}],"cinder_tempest_plugin/rbac/v3/base.py":[{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"6a022369ce581274f380e9d0ace836b601f0f212","unresolved":true,"context_lines":[{"line_number":129,"context_line":"        kwargs[\u0027name\u0027] \u003d data_utils.rand_name("},{"line_number":130,"context_line":"            VolumeV3RbacBaseTests.__name__ + \u0027-Snapshot\u0027"},{"line_number":131,"context_line":"        )"},{"line_number":132,"context_line":"        if metadata:"},{"line_number":133,"context_line":"            kwargs[\u0027metadata\u0027] \u003d metadata"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"        snapshot_id \u003d client.create_snapshot("},{"line_number":136,"context_line":"            volume_id\u003dvolume_id, **kwargs)[\u0027snapshot\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"36d5e37a_d99b383e","line":133,"range":{"start_line":132,"start_character":0,"end_line":133,"end_character":41},"updated":"2023-08-15 11:03:07.000000000","message":"Sorry I missed it.\nIf there is a default value you don\u0027t really need it.\nYou can use it directly:\nsnapshot_id \u003d client.create_snapshot(\n        volume_id\u003dvolume_id, metadata\u003dmetadata, **kwargs)[\u0027snapshot\u0027][\u0027id\u0027]","commit_id":"15a85eb7dd437cc9ef9411434b6ec90b3fdd1c7a"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"8a5320ce498ec2218ab06a110e6d045a30a3bb78","unresolved":true,"context_lines":[{"line_number":129,"context_line":"        kwargs[\u0027name\u0027] \u003d data_utils.rand_name("},{"line_number":130,"context_line":"            VolumeV3RbacBaseTests.__name__ + \u0027-Snapshot\u0027"},{"line_number":131,"context_line":"        )"},{"line_number":132,"context_line":"        if metadata:"},{"line_number":133,"context_line":"            kwargs[\u0027metadata\u0027] \u003d metadata"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"        snapshot_id \u003d client.create_snapshot("},{"line_number":136,"context_line":"            volume_id\u003dvolume_id, **kwargs)[\u0027snapshot\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"50151ff1_cccecf29","line":133,"range":{"start_line":132,"start_character":0,"end_line":133,"end_character":41},"in_reply_to":"36d5e37a_d99b383e","updated":"2023-08-15 18:39:55.000000000","message":"If I don\u0027t have any metadata, I don\u0027t want the dict (kwargs) to have the \"metadata\" key in it.","commit_id":"15a85eb7dd437cc9ef9411434b6ec90b3fdd1c7a"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"497023ac1edb420352f66ba0432249faceee5300","unresolved":true,"context_lines":[{"line_number":129,"context_line":"        kwargs[\u0027name\u0027] \u003d data_utils.rand_name("},{"line_number":130,"context_line":"            VolumeV3RbacBaseTests.__name__ + \u0027-Snapshot\u0027"},{"line_number":131,"context_line":"        )"},{"line_number":132,"context_line":"        if metadata:"},{"line_number":133,"context_line":"            kwargs[\u0027metadata\u0027] \u003d metadata"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"        snapshot_id \u003d client.create_snapshot("},{"line_number":136,"context_line":"            volume_id\u003dvolume_id, **kwargs)[\u0027snapshot\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"743b27bf_8575d328","line":133,"range":{"start_line":132,"start_character":0,"end_line":133,"end_character":41},"in_reply_to":"50151ff1_cccecf29","updated":"2023-08-15 19:25:40.000000000","message":"Why ? anyway, even when you don\u0027t pass the metadata key, it\u0027s passed in the request data as an empty dictionary.","commit_id":"15a85eb7dd437cc9ef9411434b6ec90b3fdd1c7a"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"65bf068c87f68c13681da51c9e79b2523a3f0fd0","unresolved":false,"context_lines":[{"line_number":129,"context_line":"        kwargs[\u0027name\u0027] \u003d data_utils.rand_name("},{"line_number":130,"context_line":"            VolumeV3RbacBaseTests.__name__ + \u0027-Snapshot\u0027"},{"line_number":131,"context_line":"        )"},{"line_number":132,"context_line":"        if metadata:"},{"line_number":133,"context_line":"            kwargs[\u0027metadata\u0027] \u003d metadata"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"        snapshot_id \u003d client.create_snapshot("},{"line_number":136,"context_line":"            volume_id\u003dvolume_id, **kwargs)[\u0027snapshot\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"e82e5abe_78392276","line":133,"range":{"start_line":132,"start_character":0,"end_line":133,"end_character":41},"in_reply_to":"743b27bf_8575d328","updated":"2023-08-16 08:20:29.000000000","message":"Done","commit_id":"15a85eb7dd437cc9ef9411434b6ec90b3fdd1c7a"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"acc6408d4746b6c88c4592cbc9ac372c55fe1f50","unresolved":true,"context_lines":[{"line_number":129,"context_line":"        kwargs[\u0027name\u0027] \u003d data_utils.rand_name("},{"line_number":130,"context_line":"            VolumeV3RbacBaseTests.__name__ + \u0027-Snapshot\u0027"},{"line_number":131,"context_line":"        )"},{"line_number":132,"context_line":"        kwargs[\u0027metadata\u0027] \u003d metadata"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        snapshot_id \u003d client.create_snapshot("},{"line_number":135,"context_line":"            volume_id\u003dvolume_id, **kwargs)[\u0027snapshot\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":6,"id":"c7946e5e_9f14b673","line":132,"range":{"start_line":132,"start_character":8,"end_line":132,"end_character":37},"updated":"2023-08-16 11:51:05.000000000","message":"I meant to remove this line as well and specify it directly:\n client.create_snapshot(metadata\u003dmetadata ...)","commit_id":"7752cc68e9086090248be34e5783885d886a475f"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"3268b8a8bf10a05b30e32c54a2ea2b5054072238","unresolved":false,"context_lines":[{"line_number":129,"context_line":"        kwargs[\u0027name\u0027] \u003d data_utils.rand_name("},{"line_number":130,"context_line":"            VolumeV3RbacBaseTests.__name__ + \u0027-Snapshot\u0027"},{"line_number":131,"context_line":"        )"},{"line_number":132,"context_line":"        kwargs[\u0027metadata\u0027] \u003d metadata"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        snapshot_id \u003d client.create_snapshot("},{"line_number":135,"context_line":"            volume_id\u003dvolume_id, **kwargs)[\u0027snapshot\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":6,"id":"120c8111_1c319cb7","line":132,"range":{"start_line":132,"start_character":8,"end_line":132,"end_character":37},"in_reply_to":"c7946e5e_9f14b673","updated":"2023-08-27 09:33:24.000000000","message":"Done","commit_id":"7752cc68e9086090248be34e5783885d886a475f"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"51c3f41336e35931b4584638c6477b348164faa8","unresolved":true,"context_lines":[{"line_number":121,"context_line":"            client: Client object"},{"line_number":122,"context_line":"            volume_id: ID of the volume"},{"line_number":123,"context_line":"            cleanup: Boolean if should delete the snapshot"},{"line_number":124,"context_line":"            metadata: Dict of ey-value pairs"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"        Returns:"},{"line_number":127,"context_line":"            ID of the created snapshot"}],"source_content_type":"text/x-python","patch_set":7,"id":"5f257435_82681de3","line":124,"range":{"start_line":124,"start_character":12,"end_line":124,"end_character":44},"updated":"2024-03-20 14:01:23.000000000","message":"I can\u0027t see this as a passed parameter","commit_id":"b42681daf98710688282a5295b88997ee664060a"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"82231d6ebcafdbf8ed46c45979541ac8bb484807","unresolved":false,"context_lines":[{"line_number":121,"context_line":"            client: Client object"},{"line_number":122,"context_line":"            volume_id: ID of the volume"},{"line_number":123,"context_line":"            cleanup: Boolean if should delete the snapshot"},{"line_number":124,"context_line":"            metadata: Dict of ey-value pairs"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"        Returns:"},{"line_number":127,"context_line":"            ID of the created snapshot"}],"source_content_type":"text/x-python","patch_set":7,"id":"6f16a31d_4d2fda4f","line":124,"range":{"start_line":124,"start_character":12,"end_line":124,"end_character":44},"in_reply_to":"5f257435_82681de3","updated":"2024-05-22 10:52:45.000000000","message":"Removed","commit_id":"b42681daf98710688282a5295b88997ee664060a"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"51c3f41336e35931b4584638c6477b348164faa8","unresolved":true,"context_lines":[{"line_number":129,"context_line":"        kwargs[\u0027name\u0027] \u003d data_utils.rand_name("},{"line_number":130,"context_line":"            VolumeV3RbacBaseTests.__name__ + \u0027-Snapshot\u0027"},{"line_number":131,"context_line":"        )"},{"line_number":132,"context_line":"        # kwargs[\u0027metadata\u0027] \u003d metadata"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        snapshot_id \u003d client.create_snapshot("},{"line_number":135,"context_line":"            volume_id\u003dvolume_id, **kwargs)[\u0027snapshot\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"23c4140b_b1c4fa77","line":132,"range":{"start_line":132,"start_character":8,"end_line":132,"end_character":39},"updated":"2024-03-20 14:01:23.000000000","message":"do you want to uncomment it or remove it?","commit_id":"b42681daf98710688282a5295b88997ee664060a"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"82231d6ebcafdbf8ed46c45979541ac8bb484807","unresolved":false,"context_lines":[{"line_number":129,"context_line":"        kwargs[\u0027name\u0027] \u003d data_utils.rand_name("},{"line_number":130,"context_line":"            VolumeV3RbacBaseTests.__name__ + \u0027-Snapshot\u0027"},{"line_number":131,"context_line":"        )"},{"line_number":132,"context_line":"        # kwargs[\u0027metadata\u0027] \u003d metadata"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        snapshot_id \u003d client.create_snapshot("},{"line_number":135,"context_line":"            volume_id\u003dvolume_id, **kwargs)[\u0027snapshot\u0027][\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"70c08469_abfb77d8","line":132,"range":{"start_line":132,"start_character":8,"end_line":132,"end_character":39},"in_reply_to":"23c4140b_b1c4fa77","updated":"2024-05-22 10:52:45.000000000","message":"Removed","commit_id":"b42681daf98710688282a5295b88997ee664060a"}],"cinder_tempest_plugin/rbac/v3/test_snapshots_metadata.py":[{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"d3e8335df34959de35332785202ceaeaaef916b2","unresolved":true,"context_lines":[{"line_number":39,"context_line":"        volume_id \u003d self.create_volume(client\u003dself.vol_other_client)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        # Create a snapshot"},{"line_number":42,"context_line":"        metadata \u003d {\"key1\": \"value1\","},{"line_number":43,"context_line":"                    \"key2\": \"value2\","},{"line_number":44,"context_line":"                    \"key3\": \"value3\"}"},{"line_number":45,"context_line":"        snapshot_id \u003d self.create_snapshot("},{"line_number":46,"context_line":"            client\u003dself.snap_other_client,"},{"line_number":47,"context_line":"            volume_id\u003dvolume_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"19c979fc_20bdf47c","line":44,"range":{"start_line":42,"start_character":0,"end_line":44,"end_character":37},"updated":"2023-08-14 08:04:21.000000000","message":"Since you use the same metadata in other methods, you can change it to class variable.","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"e091fdd8476488b71436fdfb7c48d0cef2909753","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        volume_id \u003d self.create_volume(client\u003dself.vol_other_client)"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        # Create a snapshot"},{"line_number":42,"context_line":"        metadata \u003d {\"key1\": \"value1\","},{"line_number":43,"context_line":"                    \"key2\": \"value2\","},{"line_number":44,"context_line":"                    \"key3\": \"value3\"}"},{"line_number":45,"context_line":"        snapshot_id \u003d self.create_snapshot("},{"line_number":46,"context_line":"            client\u003dself.snap_other_client,"},{"line_number":47,"context_line":"            volume_id\u003dvolume_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"702c95f6_a2c7c165","line":44,"range":{"start_line":42,"start_character":0,"end_line":44,"end_character":37},"in_reply_to":"19c979fc_20bdf47c","updated":"2023-08-14 11:33:17.000000000","message":"Done","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c4205acdfe11f4a196ef5ee43099a34977412d46","unresolved":true,"context_lines":[{"line_number":52,"context_line":"            method\u003d\u0027show_snapshot_metadata\u0027, expected_status\u003dexpected_status,"},{"line_number":53,"context_line":"            snapshot_id\u003dsnapshot_id"},{"line_number":54,"context_line":"        )[\u0027metadata\u0027]"},{"line_number":55,"context_line":"        self.assertThat("},{"line_number":56,"context_line":"            metadata.items(), matchers.ContainsAll(metadata.items()),"},{"line_number":57,"context_line":"            f\"Snapshot\u0027s metadata is {snapshot_metadata}\""},{"line_number":58,"context_line":"        )"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def _create_snapshot_metadata(self, expected_status):"},{"line_number":61,"context_line":"        \"\"\"Test create_snapshot_metadata operation."}],"source_content_type":"text/x-python","patch_set":2,"id":"8a4fe35f_b1d3af7a","line":58,"range":{"start_line":55,"start_character":0,"end_line":58,"end_character":9},"updated":"2023-08-14 08:48:05.000000000","message":"You verify permissions for the operation.\nYou don\u0027t need to verify that operation was successful.","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"e091fdd8476488b71436fdfb7c48d0cef2909753","unresolved":false,"context_lines":[{"line_number":52,"context_line":"            method\u003d\u0027show_snapshot_metadata\u0027, expected_status\u003dexpected_status,"},{"line_number":53,"context_line":"            snapshot_id\u003dsnapshot_id"},{"line_number":54,"context_line":"        )[\u0027metadata\u0027]"},{"line_number":55,"context_line":"        self.assertThat("},{"line_number":56,"context_line":"            metadata.items(), matchers.ContainsAll(metadata.items()),"},{"line_number":57,"context_line":"            f\"Snapshot\u0027s metadata is {snapshot_metadata}\""},{"line_number":58,"context_line":"        )"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def _create_snapshot_metadata(self, expected_status):"},{"line_number":61,"context_line":"        \"\"\"Test create_snapshot_metadata operation."}],"source_content_type":"text/x-python","patch_set":2,"id":"e61cd9fc_f9fa7d63","line":58,"range":{"start_line":55,"start_character":0,"end_line":58,"end_character":9},"in_reply_to":"8a4fe35f_b1d3af7a","updated":"2023-08-14 11:33:17.000000000","message":"Done","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"d3e8335df34959de35332785202ceaeaaef916b2","unresolved":true,"context_lines":[{"line_number":64,"context_line":"            expected_status: The expected HTTP response code"},{"line_number":65,"context_line":"        \"\"\""},{"line_number":66,"context_line":"        # Create a volume"},{"line_number":67,"context_line":"        volume_id \u003d self.create_volume(client\u003dself.vol_other_client)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        # Create a snapshot"},{"line_number":70,"context_line":"        metadata \u003d {\"key1\": \"value1\","},{"line_number":71,"context_line":"                    \"key2\": \"value2\","},{"line_number":72,"context_line":"                    \"key3\": \"value3\"}"},{"line_number":73,"context_line":"        snapshot_id \u003d self.create_snapshot("},{"line_number":74,"context_line":"            client\u003dself.snap_other_client,"},{"line_number":75,"context_line":"            volume_id\u003dvolume_id"},{"line_number":76,"context_line":"        )"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        self.do_request("},{"line_number":79,"context_line":"            method\u003d\u0027create_snapshot_metadata\u0027, expected_status\u003dexpected_status,"}],"source_content_type":"text/x-python","patch_set":2,"id":"c087bfd7_67d920c6","line":76,"range":{"start_line":67,"start_character":0,"end_line":76,"end_character":9},"updated":"2023-08-14 08:04:21.000000000","message":"This snippet is repeated in each method.\nI think it\u0027s a better to create a helper method for cleaner code.","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"e091fdd8476488b71436fdfb7c48d0cef2909753","unresolved":false,"context_lines":[{"line_number":64,"context_line":"            expected_status: The expected HTTP response code"},{"line_number":65,"context_line":"        \"\"\""},{"line_number":66,"context_line":"        # Create a volume"},{"line_number":67,"context_line":"        volume_id \u003d self.create_volume(client\u003dself.vol_other_client)"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"        # Create a snapshot"},{"line_number":70,"context_line":"        metadata \u003d {\"key1\": \"value1\","},{"line_number":71,"context_line":"                    \"key2\": \"value2\","},{"line_number":72,"context_line":"                    \"key3\": \"value3\"}"},{"line_number":73,"context_line":"        snapshot_id \u003d self.create_snapshot("},{"line_number":74,"context_line":"            client\u003dself.snap_other_client,"},{"line_number":75,"context_line":"            volume_id\u003dvolume_id"},{"line_number":76,"context_line":"        )"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        self.do_request("},{"line_number":79,"context_line":"            method\u003d\u0027create_snapshot_metadata\u0027, expected_status\u003dexpected_status,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1fcab93c_9b096160","line":76,"range":{"start_line":67,"start_character":0,"end_line":76,"end_character":9},"in_reply_to":"c087bfd7_67d920c6","updated":"2023-08-14 11:33:17.000000000","message":"Done","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c4205acdfe11f4a196ef5ee43099a34977412d46","unresolved":true,"context_lines":[{"line_number":79,"context_line":"            method\u003d\u0027create_snapshot_metadata\u0027, expected_status\u003dexpected_status,"},{"line_number":80,"context_line":"            snapshot_id\u003dsnapshot_id, metadata\u003dmetadata"},{"line_number":81,"context_line":"        )"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":84,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata("},{"line_number":85,"context_line":"                snapshot_id"},{"line_number":86,"context_line":"            )[\u0027metadata\u0027]"},{"line_number":87,"context_line":"            self.assertThat("},{"line_number":88,"context_line":"                body.items(), matchers.ContainsAll(metadata.items()),"},{"line_number":89,"context_line":"                \u0027Create snapshot metadata failed\u0027"},{"line_number":90,"context_line":"            )"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    def _update_snapshot_metadata(self, expected_status):"},{"line_number":93,"context_line":"        \"\"\"Test update_snapshot_metadata operation."}],"source_content_type":"text/x-python","patch_set":2,"id":"c5e44418_8ef6ac35","line":90,"range":{"start_line":82,"start_character":0,"end_line":90,"end_character":13},"updated":"2023-08-14 08:48:05.000000000","message":"Same!","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"e091fdd8476488b71436fdfb7c48d0cef2909753","unresolved":false,"context_lines":[{"line_number":79,"context_line":"            method\u003d\u0027create_snapshot_metadata\u0027, expected_status\u003dexpected_status,"},{"line_number":80,"context_line":"            snapshot_id\u003dsnapshot_id, metadata\u003dmetadata"},{"line_number":81,"context_line":"        )"},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":84,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata("},{"line_number":85,"context_line":"                snapshot_id"},{"line_number":86,"context_line":"            )[\u0027metadata\u0027]"},{"line_number":87,"context_line":"            self.assertThat("},{"line_number":88,"context_line":"                body.items(), matchers.ContainsAll(metadata.items()),"},{"line_number":89,"context_line":"                \u0027Create snapshot metadata failed\u0027"},{"line_number":90,"context_line":"            )"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"    def _update_snapshot_metadata(self, expected_status):"},{"line_number":93,"context_line":"        \"\"\"Test update_snapshot_metadata operation."}],"source_content_type":"text/x-python","patch_set":2,"id":"4c9cd2b2_cda039d8","line":90,"range":{"start_line":82,"start_character":0,"end_line":90,"end_character":13},"in_reply_to":"c5e44418_8ef6ac35","updated":"2023-08-14 11:33:17.000000000","message":"Done","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c4205acdfe11f4a196ef5ee43099a34977412d46","unresolved":true,"context_lines":[{"line_number":114,"context_line":"            snapshot_id\u003dsnapshot_id, metadata\u003dupdate"},{"line_number":115,"context_line":"        )"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":118,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata("},{"line_number":119,"context_line":"                snapshot_id"},{"line_number":120,"context_line":"            )[\u0027metadata\u0027]"},{"line_number":121,"context_line":"            self.assertThat("},{"line_number":122,"context_line":"                update.items(), matchers.ContainsAll(body.items()),"},{"line_number":123,"context_line":"                f\"Snapshot\u0027s metadata is {body}\""},{"line_number":124,"context_line":"            )"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    def _show_snapshot_metadata_item(self, expected_status):"},{"line_number":127,"context_line":"        \"\"\"Test show_snapshot_metadata_item operation"}],"source_content_type":"text/x-python","patch_set":2,"id":"e5469d7e_cb2e87c8","line":124,"range":{"start_line":117,"start_character":0,"end_line":124,"end_character":13},"updated":"2023-08-14 08:48:05.000000000","message":"Same!","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"e091fdd8476488b71436fdfb7c48d0cef2909753","unresolved":false,"context_lines":[{"line_number":114,"context_line":"            snapshot_id\u003dsnapshot_id, metadata\u003dupdate"},{"line_number":115,"context_line":"        )"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":118,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata("},{"line_number":119,"context_line":"                snapshot_id"},{"line_number":120,"context_line":"            )[\u0027metadata\u0027]"},{"line_number":121,"context_line":"            self.assertThat("},{"line_number":122,"context_line":"                update.items(), matchers.ContainsAll(body.items()),"},{"line_number":123,"context_line":"                f\"Snapshot\u0027s metadata is {body}\""},{"line_number":124,"context_line":"            )"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"    def _show_snapshot_metadata_item(self, expected_status):"},{"line_number":127,"context_line":"        \"\"\"Test show_snapshot_metadata_item operation"}],"source_content_type":"text/x-python","patch_set":2,"id":"1c18e98d_2b616cda","line":124,"range":{"start_line":117,"start_character":0,"end_line":124,"end_character":13},"in_reply_to":"e5469d7e_cb2e87c8","updated":"2023-08-14 11:33:17.000000000","message":"Done","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c4205acdfe11f4a196ef5ee43099a34977412d46","unresolved":true,"context_lines":[{"line_number":147,"context_line":"            expected_status\u003dexpected_status,"},{"line_number":148,"context_line":"            snapshot_id\u003dsnapshot_id, id\u003d\"key1\""},{"line_number":149,"context_line":"        )[\u0027meta\u0027]"},{"line_number":150,"context_line":"        self.assertThat("},{"line_number":151,"context_line":"            {\"key1\": metadata[\u0027key1\u0027]}.items(),"},{"line_number":152,"context_line":"            matchers.ContainsAll(snapshot_metadata.items()),"},{"line_number":153,"context_line":"            f\"Snapshot\u0027s metadata is {snapshot_metadata}\""},{"line_number":154,"context_line":"        )"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    def _delete_snapshot_metadata_item(self, expected_status):"},{"line_number":157,"context_line":"        \"\"\"Test delete_snapshot_metadata_item operation."}],"source_content_type":"text/x-python","patch_set":2,"id":"65cc2640_06121980","line":154,"range":{"start_line":150,"start_character":0,"end_line":154,"end_character":9},"updated":"2023-08-14 08:48:05.000000000","message":"Same!","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"e091fdd8476488b71436fdfb7c48d0cef2909753","unresolved":false,"context_lines":[{"line_number":147,"context_line":"            expected_status\u003dexpected_status,"},{"line_number":148,"context_line":"            snapshot_id\u003dsnapshot_id, id\u003d\"key1\""},{"line_number":149,"context_line":"        )[\u0027meta\u0027]"},{"line_number":150,"context_line":"        self.assertThat("},{"line_number":151,"context_line":"            {\"key1\": metadata[\u0027key1\u0027]}.items(),"},{"line_number":152,"context_line":"            matchers.ContainsAll(snapshot_metadata.items()),"},{"line_number":153,"context_line":"            f\"Snapshot\u0027s metadata is {snapshot_metadata}\""},{"line_number":154,"context_line":"        )"},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"    def _delete_snapshot_metadata_item(self, expected_status):"},{"line_number":157,"context_line":"        \"\"\"Test delete_snapshot_metadata_item operation."}],"source_content_type":"text/x-python","patch_set":2,"id":"5d65e0f1_3cd70d43","line":154,"range":{"start_line":150,"start_character":0,"end_line":154,"end_character":9},"in_reply_to":"65cc2640_06121980","updated":"2023-08-14 11:33:17.000000000","message":"Done","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c4205acdfe11f4a196ef5ee43099a34977412d46","unresolved":true,"context_lines":[{"line_number":180,"context_line":"            snapshot_id\u003dsnapshot_id, id\u003d\"key1\""},{"line_number":181,"context_line":"        )"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":184,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata("},{"line_number":185,"context_line":"                snapshot_id"},{"line_number":186,"context_line":"            )[\u0027metadata\u0027]"},{"line_number":187,"context_line":"            self.assertThat("},{"line_number":188,"context_line":"                expect.items(), matchers.ContainsAll(body.items()),"},{"line_number":189,"context_line":"                f\"Snapshot\u0027s metadata is {body}\""},{"line_number":190,"context_line":"            )"},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"    def _update_snapshot_metadata_item(self, expected_status):"},{"line_number":193,"context_line":"        \"\"\"Test update_snapshot_metadata_item operation."}],"source_content_type":"text/x-python","patch_set":2,"id":"6fdf0f5c_cec02819","line":190,"range":{"start_line":183,"start_character":0,"end_line":190,"end_character":13},"updated":"2023-08-14 08:48:05.000000000","message":"Same!","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"e091fdd8476488b71436fdfb7c48d0cef2909753","unresolved":false,"context_lines":[{"line_number":180,"context_line":"            snapshot_id\u003dsnapshot_id, id\u003d\"key1\""},{"line_number":181,"context_line":"        )"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":184,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata("},{"line_number":185,"context_line":"                snapshot_id"},{"line_number":186,"context_line":"            )[\u0027metadata\u0027]"},{"line_number":187,"context_line":"            self.assertThat("},{"line_number":188,"context_line":"                expect.items(), matchers.ContainsAll(body.items()),"},{"line_number":189,"context_line":"                f\"Snapshot\u0027s metadata is {body}\""},{"line_number":190,"context_line":"            )"},{"line_number":191,"context_line":""},{"line_number":192,"context_line":"    def _update_snapshot_metadata_item(self, expected_status):"},{"line_number":193,"context_line":"        \"\"\"Test update_snapshot_metadata_item operation."}],"source_content_type":"text/x-python","patch_set":2,"id":"0eb5481b_18f36913","line":190,"range":{"start_line":183,"start_character":0,"end_line":190,"end_character":13},"in_reply_to":"6fdf0f5c_cec02819","updated":"2023-08-14 11:33:17.000000000","message":"Done","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"c4205acdfe11f4a196ef5ee43099a34977412d46","unresolved":true,"context_lines":[{"line_number":217,"context_line":"            expected_status\u003dexpected_status,"},{"line_number":218,"context_line":"            snapshot_id\u003dsnapshot_id, id\u003d\"key3\", meta\u003dupdate_item"},{"line_number":219,"context_line":"        )"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":222,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata_item("},{"line_number":223,"context_line":"                snapshot_id\u003dsnapshot_id, id\u003d\"key3\")[\u0027meta\u0027]"},{"line_number":224,"context_line":"            self.assertThat("},{"line_number":225,"context_line":"                {\"key3\": expect[\u0027key3\u0027]}.items(),"},{"line_number":226,"context_line":"                matchers.ContainsAll(body.items()),"},{"line_number":227,"context_line":"                f\"Snapshot\u0027s metadata is {body}\""},{"line_number":228,"context_line":"            )"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata("},{"line_number":231,"context_line":"                snapshot_id"},{"line_number":232,"context_line":"            )[\u0027metadata\u0027]"},{"line_number":233,"context_line":"            self.assertThat(body.items(), matchers.ContainsAll(expect.items()))"},{"line_number":234,"context_line":""},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"class ProjectReaderTests(VolumeV3RbacSnapshotsMetadataTests):"}],"source_content_type":"text/x-python","patch_set":2,"id":"22638362_9718da46","line":233,"range":{"start_line":220,"start_character":0,"end_line":233,"end_character":79},"updated":"2023-08-14 08:48:05.000000000","message":"Same!","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"e091fdd8476488b71436fdfb7c48d0cef2909753","unresolved":false,"context_lines":[{"line_number":217,"context_line":"            expected_status\u003dexpected_status,"},{"line_number":218,"context_line":"            snapshot_id\u003dsnapshot_id, id\u003d\"key3\", meta\u003dupdate_item"},{"line_number":219,"context_line":"        )"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"        if expected_status !\u003d exceptions.Forbidden:"},{"line_number":222,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata_item("},{"line_number":223,"context_line":"                snapshot_id\u003dsnapshot_id, id\u003d\"key3\")[\u0027meta\u0027]"},{"line_number":224,"context_line":"            self.assertThat("},{"line_number":225,"context_line":"                {\"key3\": expect[\u0027key3\u0027]}.items(),"},{"line_number":226,"context_line":"                matchers.ContainsAll(body.items()),"},{"line_number":227,"context_line":"                f\"Snapshot\u0027s metadata is {body}\""},{"line_number":228,"context_line":"            )"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"            body \u003d self.snap_other_client.show_snapshot_metadata("},{"line_number":231,"context_line":"                snapshot_id"},{"line_number":232,"context_line":"            )[\u0027metadata\u0027]"},{"line_number":233,"context_line":"            self.assertThat(body.items(), matchers.ContainsAll(expect.items()))"},{"line_number":234,"context_line":""},{"line_number":235,"context_line":""},{"line_number":236,"context_line":"class ProjectReaderTests(VolumeV3RbacSnapshotsMetadataTests):"}],"source_content_type":"text/x-python","patch_set":2,"id":"063fcf61_62e0bbfe","line":233,"range":{"start_line":220,"start_character":0,"end_line":233,"end_character":79},"in_reply_to":"22638362_9718da46","updated":"2023-08-14 11:33:17.000000000","message":"Done","commit_id":"0d605d72af510be8a35d24c699a0dee6bf8f26ef"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"b29a5a2244a2aeb8a0590658fbe2cb6eeb8ec59b","unresolved":true,"context_lines":[{"line_number":31,"context_line":"        cls.vol_other_client \u003d cls.os_project_admin.volumes_client_latest"},{"line_number":32,"context_line":"        cls.snap_other_client \u003d cls.os_project_admin.snapshots_client_latest"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def _create_snapshot(self, with_metadata\u003dTrue):"},{"line_number":35,"context_line":"        # Create a volume"},{"line_number":36,"context_line":"        volume_id \u003d self.create_volume(client\u003dself.vol_other_client)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        # Create a snapshot"},{"line_number":39,"context_line":"        if with_metadata:"},{"line_number":40,"context_line":"            snapshot_id \u003d self.create_snapshot("},{"line_number":41,"context_line":"                client\u003dself.snap_other_client,"},{"line_number":42,"context_line":"                volume_id\u003dvolume_id,"},{"line_number":43,"context_line":"                metadata\u003dself.metadata"},{"line_number":44,"context_line":"            )"},{"line_number":45,"context_line":"        else:"},{"line_number":46,"context_line":"            snapshot_id \u003d self.create_snapshot("},{"line_number":47,"context_line":"                client\u003dself.snap_other_client,"},{"line_number":48,"context_line":"                volume_id\u003dvolume_id"},{"line_number":49,"context_line":"            )"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        return snapshot_id"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"28a2e87f_0ed1b67d","line":49,"range":{"start_line":34,"start_character":0,"end_line":49,"end_character":13},"updated":"2023-08-15 06:51:15.000000000","message":"def _create_snapshot(self, metadata\u003dNone):\n     snapshot_id \u003d self.create_snapshot(\n                client\u003dself.snap_other_client,\n                volume_id\u003dvolume_id,\n                metadata\u003dmetadata)\n                \n                \nself._create_snapshot(self.metadata)","commit_id":"11009ae3753b236019b2a0888492929558b7a132"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"888036f9c54e7a01cc6c718f1a66ddcf27fa3ad6","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        cls.vol_other_client \u003d cls.os_project_admin.volumes_client_latest"},{"line_number":32,"context_line":"        cls.snap_other_client \u003d cls.os_project_admin.snapshots_client_latest"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    def _create_snapshot(self, with_metadata\u003dTrue):"},{"line_number":35,"context_line":"        # Create a volume"},{"line_number":36,"context_line":"        volume_id \u003d self.create_volume(client\u003dself.vol_other_client)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        # Create a snapshot"},{"line_number":39,"context_line":"        if with_metadata:"},{"line_number":40,"context_line":"            snapshot_id \u003d self.create_snapshot("},{"line_number":41,"context_line":"                client\u003dself.snap_other_client,"},{"line_number":42,"context_line":"                volume_id\u003dvolume_id,"},{"line_number":43,"context_line":"                metadata\u003dself.metadata"},{"line_number":44,"context_line":"            )"},{"line_number":45,"context_line":"        else:"},{"line_number":46,"context_line":"            snapshot_id \u003d self.create_snapshot("},{"line_number":47,"context_line":"                client\u003dself.snap_other_client,"},{"line_number":48,"context_line":"                volume_id\u003dvolume_id"},{"line_number":49,"context_line":"            )"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        return snapshot_id"},{"line_number":52,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"a8eeb9f2_411de165","line":49,"range":{"start_line":34,"start_character":0,"end_line":49,"end_character":13},"in_reply_to":"28a2e87f_0ed1b67d","updated":"2023-08-15 07:37:04.000000000","message":"Done","commit_id":"11009ae3753b236019b2a0888492929558b7a132"}]}