)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Grace \u003cunnseo.park@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-08-09 04:39:36 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add documentation for snapshot metadata"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I94161e084d5412a4de67f765cfa6e7793b57e4a6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6204980d_e7ceda41","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":39},"updated":"2023-08-09 22:55:11.000000000","message":"This would be an incomplete commit message for what\u0027s being added.. \n\n\"Add support for share snapshot metadata\"\n\nperhaps? documentation additions are assumed to be done alongside the code additions","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Grace \u003cunnseo.park@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-08-09 04:39:36 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add documentation for snapshot metadata"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I94161e084d5412a4de67f765cfa6e7793b57e4a6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"2aa97b1b_870667f1","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":39},"in_reply_to":"6204980d_e7ceda41","updated":"2023-08-15 21:59:28.000000000","message":"Done","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"389c0b23_6fee8089","updated":"2023-08-09 22:55:11.000000000","message":"Thank you for working on this; please see comments inline.. \n\nin your next patch, please also include a release note calling out what\u0027s being added","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a9cc0f73_fb31366a","updated":"2023-08-15 21:59:28.000000000","message":"Thanks for submitting an update; there are some more issues. Please see comments inline. \n\nYou can run linting tests locally before submitting your patch here:\n\n```\nopenstacksdk $ tox -e pep8\n```","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"}],"doc/source/user/proxies/shared_file_system.rst":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":178,"context_line":"            delete_share_metadata"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Shared File System Snapshot Metadata"},{"line_number":181,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"List, Get, Create, Update, and Delete metadata for snapshots from the"},{"line_number":184,"context_line":"Shared File Systems service."}],"source_content_type":"text/x-rst","patch_set":2,"id":"ac0c6ab1_917bc39e","line":181,"range":{"start_line":181,"start_character":32,"end_line":181,"end_character":34},"updated":"2023-08-09 22:55:11.000000000","message":"extend underline to the length of the text above","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":false,"context_lines":[{"line_number":178,"context_line":"            delete_share_metadata"},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Shared File System Snapshot Metadata"},{"line_number":181,"context_line":"^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"List, Get, Create, Update, and Delete metadata for snapshots from the"},{"line_number":184,"context_line":"Shared File Systems service."}],"source_content_type":"text/x-rst","patch_set":2,"id":"a8c9b684_96624863","line":181,"range":{"start_line":181,"start_character":32,"end_line":181,"end_character":34},"in_reply_to":"ac0c6ab1_917bc39e","updated":"2023-08-15 21:59:28.000000000","message":"Done","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"}],"examples/shared_file_system/snapshot_metadata.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":30,"context_line":"    # Access metadata of snapshot"},{"line_number":31,"context_line":"    metadata \u003d returned_snapshot[\u0027metadata\u0027]"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    print(\"Get share metadata item given item key and snapshot id:\")"},{"line_number":34,"context_line":"    print(metadata[key])"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"7f8f2547_2aea5f14","line":33,"range":{"start_line":33,"start_character":15,"end_line":33,"end_character":20},"updated":"2023-08-09 22:55:11.000000000","message":"share snapshot","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":30,"context_line":"    # Access metadata of snapshot"},{"line_number":31,"context_line":"    metadata \u003d returned_snapshot[\u0027metadata\u0027]"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    print(\"Get share snpashot metadata item given item key and snapshot id:\")"},{"line_number":34,"context_line":"    print(metadata[key])"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"88fbd3d0_ef243332","line":33,"range":{"start_line":33,"start_character":21,"end_line":33,"end_character":29},"updated":"2023-08-15 21:59:28.000000000","message":"spelling: snapshot","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"}],"openstack/shared_file_system/v2/_proxy.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":1035,"context_line":"        \"\"\"Lists all metadata for a given snapshot."},{"line_number":1036,"context_line":""},{"line_number":1037,"context_line":"        :param snapshot ID: The ID of the snapshot."},{"line_number":1038,"context_line":"    "},{"line_number":1039,"context_line":"        :returns: One or more metadata key and value pairs as a dictionary of strings."},{"line_number":1040,"context_line":""},{"line_number":1041,"context_line":"        :rtype: "}],"source_content_type":"text/x-python","patch_set":2,"id":"8eac82e2_82a446f5","line":1038,"range":{"start_line":1038,"start_character":3,"end_line":1038,"end_character":4},"updated":"2023-08-09 22:55:11.000000000","message":"here and other places in this file, you have trailing white spaces. Could you please remove these? It may be hard to identify them on your visual studio IDE... here they are highlighted with a different color.. \n\nYou can also configure your IDE to automatically trim trailing whitespace: https://linux.how2shout.com/remove-trailing-spaces-automatically-in-visual-code-studio/","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":false,"context_lines":[{"line_number":1035,"context_line":"        \"\"\"Lists all metadata for a given snapshot."},{"line_number":1036,"context_line":""},{"line_number":1037,"context_line":"        :param snapshot ID: The ID of the snapshot."},{"line_number":1038,"context_line":"    "},{"line_number":1039,"context_line":"        :returns: One or more metadata key and value pairs as a dictionary of strings."},{"line_number":1040,"context_line":""},{"line_number":1041,"context_line":"        :rtype: "}],"source_content_type":"text/x-python","patch_set":2,"id":"06678aca_508ed32b","line":1038,"range":{"start_line":1038,"start_character":3,"end_line":1038,"end_character":4},"in_reply_to":"8eac82e2_82a446f5","updated":"2023-08-15 21:59:28.000000000","message":"Done","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":1036,"context_line":""},{"line_number":1037,"context_line":"        :param snapshot ID: The ID of the snapshot."},{"line_number":1038,"context_line":"    "},{"line_number":1039,"context_line":"        :returns: One or more metadata key and value pairs as a dictionary of strings."},{"line_number":1040,"context_line":""},{"line_number":1041,"context_line":"        :rtype: "},{"line_number":1042,"context_line":"            :class:`~openstack.shared_file_system.v2.share_snapshot.ShareSnapshot`"}],"source_content_type":"text/x-python","patch_set":2,"id":"2fd7f22a_8a242512","line":1039,"range":{"start_line":1039,"start_character":80,"end_line":1039,"end_character":85},"updated":"2023-08-09 22:55:11.000000000","message":"prevent long lines by wrapping extra text into the next line after 79 characters or under..","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":false,"context_lines":[{"line_number":1036,"context_line":""},{"line_number":1037,"context_line":"        :param snapshot ID: The ID of the snapshot."},{"line_number":1038,"context_line":"    "},{"line_number":1039,"context_line":"        :returns: One or more metadata key and value pairs as a dictionary of strings."},{"line_number":1040,"context_line":""},{"line_number":1041,"context_line":"        :rtype: "},{"line_number":1042,"context_line":"            :class:`~openstack.shared_file_system.v2.share_snapshot.ShareSnapshot`"}],"source_content_type":"text/x-python","patch_set":2,"id":"edc323bd_073e3928","line":1039,"range":{"start_line":1039,"start_character":80,"end_line":1039,"end_character":85},"in_reply_to":"2fd7f22a_8a242512","updated":"2023-08-15 21:59:28.000000000","message":"Done","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":1053,"context_line":"        :returns: A single metadata key and value pair"},{"line_number":1054,"context_line":""},{"line_number":1055,"context_line":"        :rtype:"},{"line_number":1056,"context_line":"            :class:`~openstack.shared_file_system.v2.snapshot.SnapshotMetadata`"},{"line_number":1057,"context_line":"        \"\"\""},{"line_number":1058,"context_line":"        snapshot \u003d self._get_resource(_share_snapshot.ShareSnapshot, snapshot_id)"},{"line_number":1059,"context_line":"        return snapshot.get_metadata_item(self, key)"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa50094f_27ced20c","line":1056,"range":{"start_line":1056,"start_character":21,"end_line":1056,"end_character":78},"updated":"2023-08-09 22:55:11.000000000","message":"openstack.shared_file_system.v2.share_snapshot.ShareSnapshot","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":1066,"context_line":""},{"line_number":1067,"context_line":"        :returns: One or more metadata key and value pairs as a dictionary of strings."},{"line_number":1068,"context_line":"        :rtype:"},{"line_number":1069,"context_line":"            :class:`~openstack.shared_file_system.v2.snapshot.SnapshotMetadata`"},{"line_number":1070,"context_line":"        \"\"\""},{"line_number":1071,"context_line":"        snapshot \u003d self._get_resource(_share_snapshot.ShareSnapshot, snapshot_id)"},{"line_number":1072,"context_line":"        return snapshot.set_metadata(self, metadata\u003dmetadata)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ced66a9f_44ea1bfe","line":1069,"range":{"start_line":1069,"start_character":21,"end_line":1069,"end_character":78},"updated":"2023-08-09 22:55:11.000000000","message":"openstack.shared_file_system.v2.share_snapshot.ShareSnapshot","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":1081,"context_line":""},{"line_number":1082,"context_line":"        :returns: One or more metadata key and value pairs as a dictionary of strings."},{"line_number":1083,"context_line":"        :rtype:"},{"line_number":1084,"context_line":"            :class:`~openstack.shared_file_system.v2.snapshot.SnapshotMetadata`"},{"line_number":1085,"context_line":"        \"\"\""},{"line_number":1086,"context_line":"        snapshot \u003d self._get_resource(_share_snapshot.ShareSnapshot, snapshot_id)"},{"line_number":1087,"context_line":"        return snapshot.set_metadata(self, metadata\u003dmetadata, replace\u003dreplace)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7cb5326b_d58775fb","line":1084,"range":{"start_line":1084,"start_character":22,"end_line":1084,"end_character":78},"updated":"2023-08-09 22:55:11.000000000","message":"openstack.shared_file_system.v2.share_snapshot.ShareSnapshot","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":1101,"context_line":"                snapshot.delete_metadata_item(self, key)"},{"line_number":1102,"context_line":"            except exceptions.NotFoundException:"},{"line_number":1103,"context_line":"                if not ignore_missing:"},{"line_number":1104,"context_line":"                    self._connection.log.info(\"Key %s not found.\", key)"},{"line_number":1105,"context_line":"                    keys_failed_to_delete.append(key)"},{"line_number":1106,"context_line":"            except exceptions.ForbiddenException:"},{"line_number":1107,"context_line":"                self._connection.log.info(\"Key %s cannot be deleted.\", key)"}],"source_content_type":"text/x-python","patch_set":2,"id":"fd41bafc_c827f4ec","line":1104,"range":{"start_line":1104,"start_character":41,"end_line":1104,"end_character":45},"updated":"2023-08-09 22:55:11.000000000","message":"error","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":1104,"context_line":"                    self._connection.log.info(\"Key %s not found.\", key)"},{"line_number":1105,"context_line":"                    keys_failed_to_delete.append(key)"},{"line_number":1106,"context_line":"            except exceptions.ForbiddenException:"},{"line_number":1107,"context_line":"                self._connection.log.info(\"Key %s cannot be deleted.\", key)"},{"line_number":1108,"context_line":"                keys_failed_to_delete.append(key)"},{"line_number":1109,"context_line":"            except exceptions.SDKException:"},{"line_number":1110,"context_line":"                self._connection.log.info(\"Failed to delete key %s.\", key)"}],"source_content_type":"text/x-python","patch_set":2,"id":"cafb8bfd_cb00f4ac","line":1107,"range":{"start_line":1107,"start_character":37,"end_line":1107,"end_character":41},"updated":"2023-08-09 22:55:11.000000000","message":"error","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":1107,"context_line":"                self._connection.log.info(\"Key %s cannot be deleted.\", key)"},{"line_number":1108,"context_line":"                keys_failed_to_delete.append(key)"},{"line_number":1109,"context_line":"            except exceptions.SDKException:"},{"line_number":1110,"context_line":"                self._connection.log.info(\"Failed to delete key %s.\", key)"},{"line_number":1111,"context_line":"                keys_failed_to_delete.append(key)"},{"line_number":1112,"context_line":"        if keys_failed_to_delete:"},{"line_number":1113,"context_line":"            raise exceptions.SDKException("}],"source_content_type":"text/x-python","patch_set":2,"id":"b3010df7_f3377f95","line":1110,"range":{"start_line":1110,"start_character":37,"end_line":1110,"end_character":41},"updated":"2023-08-09 22:55:11.000000000","message":"error","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1082,"context_line":"        return snapshot.fetch_metadata(self)"},{"line_number":1083,"context_line":""},{"line_number":1084,"context_line":"    def get_share_snapshot_metadata_item(self, snapshot_id, key):"},{"line_number":1085,"context_line":"        \"\"\"Retrieves a specific metadata item from a snapshot\u0027s metadata"},{"line_number":1086,"context_line":"            by its key."},{"line_number":1087,"context_line":""},{"line_number":1088,"context_line":"        :param snapshot_id: The ID of the snapshot"},{"line_number":1089,"context_line":"        :param key: The key of the snapshot metadata"}],"source_content_type":"text/x-python","patch_set":3,"id":"573a0227_c2db6941","line":1086,"range":{"start_line":1085,"start_character":11,"end_line":1086,"end_character":23},"updated":"2023-08-15 21:59:28.000000000","message":"the first line of a docstring cannot be a multi-line comment; perhaps change this to:\n\n\n```\n \"\"\"Retrieves a specific snapshot metadata item by key\n```","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1091,"context_line":"        :returns: A single metadata key and value pair"},{"line_number":1092,"context_line":""},{"line_number":1093,"context_line":"        :rtype:"},{"line_number":1094,"context_line":"            :class:`~openstack.shared_file_system.v2.share_snapshot.SnapshotMetadata`"},{"line_number":1095,"context_line":"        \"\"\""},{"line_number":1096,"context_line":"        snapshot \u003d self._get_resource(_share_snapshot.ShareSnapshot, snapshot_id)"},{"line_number":1097,"context_line":"        return snapshot.get_metadata_item(self, key)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3413e08d_cd153b50","line":1094,"range":{"start_line":1094,"start_character":0,"end_line":1094,"end_character":85},"updated":"2023-08-15 21:59:28.000000000","message":"there is no such resource; \n\nyou should set this to:\n\n:class:`~openstack.shared_file_system.v2.share_snapshot.ShareSnapshot`","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1105,"context_line":"        :returns: One or more metadata key and value pairs as a dictionary"},{"line_number":1106,"context_line":"            of strings."},{"line_number":1107,"context_line":"        :rtype:"},{"line_number":1108,"context_line":"            :class:`~openstack.shared_file_system.v2.share_snapshot.SnapshotMetadata`"},{"line_number":1109,"context_line":"        \"\"\""},{"line_number":1110,"context_line":"        snapshot \u003d self._get_resource(_share_snapshot.ShareSnapshot, snapshot_id)"},{"line_number":1111,"context_line":"        return snapshot.set_metadata(self, metadata\u003dmetadata)"}],"source_content_type":"text/x-python","patch_set":3,"id":"30305982_878119ea","line":1108,"range":{"start_line":1108,"start_character":12,"end_line":1108,"end_character":85},"updated":"2023-08-15 21:59:28.000000000","message":"same comment as above","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1107,"context_line":"        :rtype:"},{"line_number":1108,"context_line":"            :class:`~openstack.shared_file_system.v2.share_snapshot.SnapshotMetadata`"},{"line_number":1109,"context_line":"        \"\"\""},{"line_number":1110,"context_line":"        snapshot \u003d self._get_resource(_share_snapshot.ShareSnapshot, snapshot_id)"},{"line_number":1111,"context_line":"        return snapshot.set_metadata(self, metadata\u003dmetadata)"},{"line_number":1112,"context_line":""},{"line_number":1113,"context_line":"    def update_share_snapshot_metadata(self, snapshot_id, metadata, replace\u003dFalse):"}],"source_content_type":"text/x-python","patch_set":3,"id":"e72d8cfc_cba61c02","line":1110,"range":{"start_line":1110,"start_character":74,"end_line":1110,"end_character":81},"updated":"2023-08-15 21:59:28.000000000","message":"make sure not to exceed 79 characters. Please wrap into the next line:\n\n\n```\nsnapshot \u003d self._get_resource(_share_snapshot.ShareSnapshot, \n                              snapshot_id)\n```","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1110,"context_line":"        snapshot \u003d self._get_resource(_share_snapshot.ShareSnapshot, snapshot_id)"},{"line_number":1111,"context_line":"        return snapshot.set_metadata(self, metadata\u003dmetadata)"},{"line_number":1112,"context_line":""},{"line_number":1113,"context_line":"    def update_share_snapshot_metadata(self, snapshot_id, metadata, replace\u003dFalse):"},{"line_number":1114,"context_line":"        \"\"\"Replaces the metadata for a given snapshot with the metadata in the"},{"line_number":1115,"context_line":"            request object."},{"line_number":1116,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"78716c0e_631412f4","line":1113,"range":{"start_line":1113,"start_character":76,"end_line":1113,"end_character":83},"updated":"2023-08-15 21:59:28.000000000","message":"please wrap after 79 characters","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1111,"context_line":"        return snapshot.set_metadata(self, metadata\u003dmetadata)"},{"line_number":1112,"context_line":""},{"line_number":1113,"context_line":"    def update_share_snapshot_metadata(self, snapshot_id, metadata, replace\u003dFalse):"},{"line_number":1114,"context_line":"        \"\"\"Replaces the metadata for a given snapshot with the metadata in the"},{"line_number":1115,"context_line":"            request object."},{"line_number":1116,"context_line":""},{"line_number":1117,"context_line":"        :param snapshot_id: The ID of the snapshot to update"},{"line_number":1118,"context_line":"        :param metadata: The metadata to be updated"}],"source_content_type":"text/x-python","patch_set":3,"id":"9c643b3c_abc57a52","line":1115,"range":{"start_line":1114,"start_character":11,"end_line":1115,"end_character":27},"updated":"2023-08-15 21:59:28.000000000","message":"the title line of a docstring needs to be a single line:\n\n```\nReplaces metadata of a given snapshot\n```","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1127,"context_line":"        snapshot \u003d self._get_resource(_share_snapshot.ShareSnapshot, snapshot_id)"},{"line_number":1128,"context_line":"        return snapshot.set_metadata(self, metadata\u003dmetadata, replace\u003dreplace)"},{"line_number":1129,"context_line":""},{"line_number":1130,"context_line":"    def delete_share_snapshot_metadata(self, snapshot_id, keys, ignore_missing\u003dTrue):"},{"line_number":1131,"context_line":"        \"\"\"Deletes a single metadata item on a snapshot, identified by its key."},{"line_number":1132,"context_line":""},{"line_number":1133,"context_line":"        :param snapshot_id: The ID of the snapshot"}],"source_content_type":"text/x-python","patch_set":3,"id":"174fcc22_e22a912a","line":1130,"range":{"start_line":1130,"start_character":77,"end_line":1130,"end_character":85},"updated":"2023-08-15 21:59:28.000000000","message":"please wrap after 79 characters","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1142,"context_line":"                snapshot.delete_metadata_item(self, key)"},{"line_number":1143,"context_line":"            except exceptions.NotFoundException:"},{"line_number":1144,"context_line":"                if not ignore_missing:"},{"line_number":1145,"context_line":"                    self._connection.log.info(\"Key %s not found.\", key)"},{"line_number":1146,"context_line":"                    keys_failed_to_delete.append(key)"},{"line_number":1147,"context_line":"            except exceptions.ForbiddenException:"},{"line_number":1148,"context_line":"                self._connection.log.info(\"Key %s cannot be deleted.\", key)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5d00c007_db9fd93b","line":1145,"range":{"start_line":1145,"start_character":41,"end_line":1145,"end_character":45},"updated":"2023-08-15 21:59:28.000000000","message":"error","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1145,"context_line":"                    self._connection.log.info(\"Key %s not found.\", key)"},{"line_number":1146,"context_line":"                    keys_failed_to_delete.append(key)"},{"line_number":1147,"context_line":"            except exceptions.ForbiddenException:"},{"line_number":1148,"context_line":"                self._connection.log.info(\"Key %s cannot be deleted.\", key)"},{"line_number":1149,"context_line":"                keys_failed_to_delete.append(key)"},{"line_number":1150,"context_line":"            except exceptions.SDKException:"},{"line_number":1151,"context_line":"                self._connection.log.info(\"Failed to delete key %s.\", key)"}],"source_content_type":"text/x-python","patch_set":3,"id":"e6693b78_4755bba2","line":1148,"range":{"start_line":1148,"start_character":37,"end_line":1148,"end_character":41},"updated":"2023-08-15 21:59:28.000000000","message":"error","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":1148,"context_line":"                self._connection.log.info(\"Key %s cannot be deleted.\", key)"},{"line_number":1149,"context_line":"                keys_failed_to_delete.append(key)"},{"line_number":1150,"context_line":"            except exceptions.SDKException:"},{"line_number":1151,"context_line":"                self._connection.log.info(\"Failed to delete key %s.\", key)"},{"line_number":1152,"context_line":"                keys_failed_to_delete.append(key)"},{"line_number":1153,"context_line":"        if keys_failed_to_delete:"},{"line_number":1154,"context_line":"            raise exceptions.SDKException("}],"source_content_type":"text/x-python","patch_set":3,"id":"d54b253d_e4c2f383","line":1151,"range":{"start_line":1151,"start_character":37,"end_line":1151,"end_character":41},"updated":"2023-08-15 21:59:28.000000000","message":"replace \"info\" with \"error\" here and above","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"}],"openstack/tests/functional/shared_file_system/test_snapshot_metadata.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":42,"context_line":"        my_snapshot \u003d self.user_cloud.shared_file_system.create_share_snapshot("},{"line_number":43,"context_line":"            name\u003dself.SNAPSHOT_NAME,"},{"line_number":44,"context_line":"            share_id \u003d my_share.id,"},{"line_number":45,"context_line":"            description\u003dNone,"},{"line_number":46,"context_line":"            force\u003dTrue,"},{"line_number":47,"context_line":"        )"},{"line_number":48,"context_line":"        self.SNAPSHOT_ID \u003d my_snapshot.id"},{"line_number":49,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"6ded3c32_e65ed8fa","line":46,"range":{"start_line":45,"start_character":29,"end_line":46,"end_character":23},"updated":"2023-08-09 22:55:11.000000000","message":"not necessary","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":74,"context_line":"        )"},{"line_number":75,"context_line":"        self.assertIsNone(sot)"},{"line_number":76,"context_line":"        super(ShareSnapshotMetadataTest, self).tearDown()"},{"line_number":77,"context_line":"    "},{"line_number":78,"context_line":"    def test_create(self):"},{"line_number":79,"context_line":"        meta \u003d {\"foo\": \"bar\"}"},{"line_number":80,"context_line":"        created_snapshot \u003d ("}],"source_content_type":"text/x-python","patch_set":2,"id":"50488b73_2aa90054","line":77,"range":{"start_line":77,"start_character":3,"end_line":77,"end_character":4},"updated":"2023-08-09 22:55:11.000000000","message":"please remove trailing whitespace","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":93,"context_line":"            )"},{"line_number":94,"context_line":"        )"},{"line_number":95,"context_line":"        returned_snapshot \u003d ("},{"line_number":96,"context_line":"            self.user_cloud.shared_file_system.get_share_snapshot_metadata_item("},{"line_number":97,"context_line":"                self.SNAPSHOT_ID, \"foo\""},{"line_number":98,"context_line":"            )"},{"line_number":99,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":2,"id":"88e0c463_d32608b7","line":96,"range":{"start_line":96,"start_character":69,"end_line":96,"end_character":80},"updated":"2023-08-09 22:55:11.000000000","message":"don\u0027t exceed 79 characters; wrap long lines here, and everywhere else in this file\n\nmaybe do this; in the \"setUp()\" method, create a variable/alias for your client:\n\n```\n  self.sfs_client \u003d self.user_cloud.shared_file_system\n  \n```\n\n\nthis will let you have shorter lines:\n\n```\n  self.sfs_client.get_share_snapshot_metadata_item(self.SNAPSHOT_ID, \u0027foo\u0027)\n```","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":132,"context_line":"                created_snapshot, new_meta"},{"line_number":133,"context_line":"            )"},{"line_number":134,"context_line":"        )"},{"line_number":135,"context_line":"        self.assertEqual(updated_snapshot[\u0027metadata\u0027], new_meta)"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"        full_metadata \u003d self.user_cloud.shared_file_system.get_share_snapshot_metadata("},{"line_number":138,"context_line":"            created_snapshot"}],"source_content_type":"text/x-python","patch_set":2,"id":"08d4ec8d_105d4cb2","line":135,"range":{"start_line":135,"start_character":25,"end_line":135,"end_character":64},"updated":"2023-08-09 22:55:11.000000000","message":"flip the order here... \n\n```\nself.assertEqual(new_meta, updated_snapshot[\u0027metadata\u0027])\n```\n\njust a convention used by this project..","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":false,"context_lines":[{"line_number":132,"context_line":"                created_snapshot, new_meta"},{"line_number":133,"context_line":"            )"},{"line_number":134,"context_line":"        )"},{"line_number":135,"context_line":"        self.assertEqual(updated_snapshot[\u0027metadata\u0027], new_meta)"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"        full_metadata \u003d self.user_cloud.shared_file_system.get_share_snapshot_metadata("},{"line_number":138,"context_line":"            created_snapshot"}],"source_content_type":"text/x-python","patch_set":2,"id":"288546f4_13a514b3","line":135,"range":{"start_line":135,"start_character":25,"end_line":135,"end_character":64},"in_reply_to":"08d4ec8d_105d4cb2","updated":"2023-08-15 21:59:28.000000000","message":"Done","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":137,"context_line":"        full_metadata \u003d self.user_cloud.shared_file_system.get_share_snapshot_metadata("},{"line_number":138,"context_line":"            created_snapshot"},{"line_number":139,"context_line":"        )[\u0027metadata\u0027]"},{"line_number":140,"context_line":"        self.assertEqual(full_metadata, full_meta)"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        snapshot_with_deleted_metadata \u003d ("},{"line_number":143,"context_line":"            self.user_cloud.shared_file_system.update_share_snapshot_metadata("}],"source_content_type":"text/x-python","patch_set":2,"id":"11706171_d375e74d","line":140,"range":{"start_line":140,"start_character":25,"end_line":140,"end_character":49},"updated":"2023-08-09 22:55:11.000000000","message":"same, flip the order","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":false,"context_lines":[{"line_number":137,"context_line":"        full_metadata \u003d self.user_cloud.shared_file_system.get_share_snapshot_metadata("},{"line_number":138,"context_line":"            created_snapshot"},{"line_number":139,"context_line":"        )[\u0027metadata\u0027]"},{"line_number":140,"context_line":"        self.assertEqual(full_metadata, full_meta)"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"        snapshot_with_deleted_metadata \u003d ("},{"line_number":143,"context_line":"            self.user_cloud.shared_file_system.update_share_snapshot_metadata("}],"source_content_type":"text/x-python","patch_set":2,"id":"e856766b_6511238b","line":140,"range":{"start_line":140,"start_character":25,"end_line":140,"end_character":49},"in_reply_to":"11706171_d375e74d","updated":"2023-08-15 21:59:28.000000000","message":"Done","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":144,"context_line":"                updated_snapshot, empty_meta"},{"line_number":145,"context_line":"            )"},{"line_number":146,"context_line":"        )"},{"line_number":147,"context_line":"        self.assertEqual(snapshot_with_deleted_metadata[\u0027metadata\u0027], empty_meta)"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    def test_delete(self):"},{"line_number":150,"context_line":"        meta \u003d {\"foo\": \"bar\", \"newFoo\": \"newBar\"}"}],"source_content_type":"text/x-python","patch_set":2,"id":"81d638ff_0024a0d4","line":147,"range":{"start_line":147,"start_character":25,"end_line":147,"end_character":80},"updated":"2023-08-09 22:55:11.000000000","message":"same, flip the order","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":false,"context_lines":[{"line_number":144,"context_line":"                updated_snapshot, empty_meta"},{"line_number":145,"context_line":"            )"},{"line_number":146,"context_line":"        )"},{"line_number":147,"context_line":"        self.assertEqual(snapshot_with_deleted_metadata[\u0027metadata\u0027], empty_meta)"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    def test_delete(self):"},{"line_number":150,"context_line":"        meta \u003d {\"foo\": \"bar\", \"newFoo\": \"newBar\"}"}],"source_content_type":"text/x-python","patch_set":2,"id":"a4c65f18_44b8f152","line":147,"range":{"start_line":147,"start_character":25,"end_line":147,"end_character":80},"in_reply_to":"81d638ff_0024a0d4","updated":"2023-08-15 21:59:28.000000000","message":"Done","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[{"line_number":155,"context_line":"        )"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        self.user_cloud.shared_file_system.delete_share_snapshot_metadata("},{"line_number":158,"context_line":"            created_snapshot, [\"foo\"] # SNAPSHOT_ID"},{"line_number":159,"context_line":"        )"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"        deleted_snapshot \u003d self.user_cloud.shared_file_system.get_share_snapshot_metadata("}],"source_content_type":"text/x-python","patch_set":2,"id":"0a70b0be_86b722e3","line":158,"range":{"start_line":158,"start_character":38,"end_line":158,"end_character":51},"updated":"2023-08-09 22:55:11.000000000","message":"?","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":false,"context_lines":[{"line_number":155,"context_line":"        )"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        self.user_cloud.shared_file_system.delete_share_snapshot_metadata("},{"line_number":158,"context_line":"            created_snapshot, [\"foo\"] # SNAPSHOT_ID"},{"line_number":159,"context_line":"        )"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"        deleted_snapshot \u003d self.user_cloud.shared_file_system.get_share_snapshot_metadata("}],"source_content_type":"text/x-python","patch_set":2,"id":"ca5c6bc7_8028cbf6","line":158,"range":{"start_line":158,"start_character":38,"end_line":158,"end_character":51},"in_reply_to":"0a70b0be_86b722e3","updated":"2023-08-15 21:59:28.000000000","message":"Done","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"26d22185dc2444ed248f8e55383c31fc365156ed","unresolved":true,"context_lines":[{"line_number":93,"context_line":"            )"},{"line_number":94,"context_line":"        )"},{"line_number":95,"context_line":"        returned_snapshot \u003d ("},{"line_number":96,"context_line":"            self.sfs_client.get_share_snapshot_metadata_item(self.SNAPSHOT_ID, \u0027foo\u0027)"},{"line_number":97,"context_line":"        )"},{"line_number":98,"context_line":"        self.assertEqual("},{"line_number":99,"context_line":"            created_snapshot[\u0027metadata\u0027][\u0027foo\u0027], returned_snapshot[\u0027metadata\u0027][\u0027foo\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"adcffa24_a09ca2d7","line":96,"range":{"start_line":96,"start_character":12,"end_line":96,"end_character":85},"updated":"2023-08-15 21:59:28.000000000","message":"hmmm, this still doesn\u0027t fit in 79 characters; try:\n\n```\nreturned_snapshot \u003d (\n    self.sfs_client.get_share_snapshot_metadata_item(\n        self.SNAPSHOT_ID,\n        \u0027foo\u0027\n    )\n)\n```","commit_id":"01446b0e96df09c430a2601f4485e622ae85c208"}],"openstack/tests/unit/shared_file_system/v2/test_proxy.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"c026dca78b864030cd55b9907de0c8d75d7f75b7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"90075094_ae257d7a","updated":"2023-08-09 22:55:11.000000000","message":"make sure to adhere to 79 character limit; set it in your Visual Studio code so it can help you wrap lines automatically: use this https://marketplace.visualstudio.com/items?itemName\u003dstkb.rewrap","commit_id":"81b6111d01ce3e4606dd7911029865b51a7278d4"}]}
