)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"532071f9acf7f00885ca7f96bb793bdd0876c5b8","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Vida Haririan \u003cvhariria@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-02-02 15:59:05 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[WIP]Added test test_create_snapshot_over_quota_limit"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This test case validates the condition of creating a share"},{"line_number":10,"context_line":"from napshot with quota over limit."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"f6cca4fa_62dd5113","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":5},"updated":"2021-02-03 15:21:02.000000000","message":"You can remove it.","commit_id":"e8d092f95b21a48132fe9197fb623dfdf2271179"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"532071f9acf7f00885ca7f96bb793bdd0876c5b8","unresolved":true,"context_lines":[{"line_number":7,"context_line":"[WIP]Added test test_create_snapshot_over_quota_limit"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This test case validates the condition of creating a share"},{"line_number":10,"context_line":"from napshot with quota over limit."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib5f20982155029e9a3c0183eae218ae6d1b88e49"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"572a64bb_8f23bbcf","line":10,"range":{"start_line":10,"start_character":5,"end_line":10,"end_character":12},"updated":"2021-02-03 15:21:02.000000000","message":"snapshot","commit_id":"e8d092f95b21a48132fe9197fb623dfdf2271179"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"cf4e2c7140d561018a630b134c9a025f49620873","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4ce3a4b5_2d536120","updated":"2022-01-11 19:40:30.000000000","message":"Thanks, updated the script per snapshot extra specs changes.","commit_id":"7c09b6984334eca1b32a34a19e4de20cfa7e901c"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"1a88b6e4fd56f79d4b4140323cd1ba35151e5d3d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"93d19505_a782adae","updated":"2022-01-13 14:13:53.000000000","message":"LGTM, good test case to have! Thanks, Vida","commit_id":"4b20d449fff58ca62ee1c94ab491a0d7e53b6851"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"53129fd81d44595018d834c0010aec567cb38a53","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7a49c1b3_1eec4bc9","updated":"2022-01-12 07:58:13.000000000","message":"LGTM, thanks Vida!","commit_id":"4b20d449fff58ca62ee1c94ab491a0d7e53b6851"}],"manila_tempest_tests/tests/api/admin/test_quotas_negative.py":[{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"35ebc1682aa6bb1268fff61b2eb5fb3363385a4f","unresolved":true,"context_lines":[{"line_number":431,"context_line":"        CONF.share.run_snapshot_tests, \u0027Snapshot tests are disabled.\u0027)"},{"line_number":432,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":433,"context_line":"    def test_create_snapshot_over_quota_limit(self):"},{"line_number":434,"context_line":"        share \u003d self.create_share(share_type_id\u003dself.share_type_id)"},{"line_number":435,"context_line":""},{"line_number":436,"context_line":"        # Update snapshot quota"},{"line_number":437,"context_line":"        self.update_quotas(self.tenant_id, snapshots\u003d1)"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa4a748a_7ce3bf99","line":434,"range":{"start_line":434,"start_character":8,"end_line":434,"end_character":67},"updated":"2022-01-02 10:17:04.000000000","message":"Hi Vida,\nThe test fails due to incapability.\nSince [1] parch was merged, we have to create a new share type and setup snapshot extra specs.\nSo please add the following lines:\n        extra_specs \u003d {\u0027snapshot_support\u0027: True}\n        share_type \u003d self.create_share_type(extra_specs\u003dextra_specs)\n        share \u003d self.create_share(share_type_id\u003dshare_type[\u0027id\u0027])\n\n\n\n[1] https://review.opendev.org/c/openstack/manila-tempest-plugin/+/785307","commit_id":"7c09b6984334eca1b32a34a19e4de20cfa7e901c"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"cf4e2c7140d561018a630b134c9a025f49620873","unresolved":false,"context_lines":[{"line_number":431,"context_line":"        CONF.share.run_snapshot_tests, \u0027Snapshot tests are disabled.\u0027)"},{"line_number":432,"context_line":"    @tc.attr(base.TAG_NEGATIVE, base.TAG_API_WITH_BACKEND)"},{"line_number":433,"context_line":"    def test_create_snapshot_over_quota_limit(self):"},{"line_number":434,"context_line":"        share \u003d self.create_share(share_type_id\u003dself.share_type_id)"},{"line_number":435,"context_line":""},{"line_number":436,"context_line":"        # Update snapshot quota"},{"line_number":437,"context_line":"        self.update_quotas(self.tenant_id, snapshots\u003d1)"}],"source_content_type":"text/x-python","patch_set":4,"id":"c39dd7f1_64daa1d9","line":434,"range":{"start_line":434,"start_character":8,"end_line":434,"end_character":67},"in_reply_to":"fa4a748a_7ce3bf99","updated":"2022-01-11 19:40:30.000000000","message":"Done","commit_id":"7c09b6984334eca1b32a34a19e4de20cfa7e901c"}]}
