)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"a0f660ad55ce12c849c2ac9c044e463d9d3063d2","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update \"test_unmanage_manage_snapshot\" test to support some params"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The patch validates snapshot manage while using the following parameter:"},{"line_number":10,"context_line":"name, description and metadata."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: If36aa709cd5329e559a47b0d3bfc9115f0c6b633"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df140735_f305f946","line":9,"range":{"start_line":9,"start_character":62,"end_line":9,"end_character":71},"updated":"2017-06-06 15:53:00.000000000","message":"parameters","commit_id":"f8bb6d5afbc3dd730c9c80dc22243d20e7b33550"}],"tempest/api/volume/admin/test_snapshot_manage.py":[{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"b7192cff4aa3174b985f25ff8d5755acd148b896","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        # Manage the snapshot"},{"line_number":69,"context_line":"        snapshot_ref \u003d \u0027_snapshot-%s\u0027 % snapshot[\u0027id\u0027]"},{"line_number":70,"context_line":"        new_snapshot \u003d self.admin_snapshot_manage_client.manage_snapshot("},{"line_number":71,"context_line":"            volume_id\u003dvolume[\u0027id\u0027],"},{"line_number":72,"context_line":"            ref\u003d{\u0027source-name\u0027: snapshot_ref}, name\u003dname,"},{"line_number":73,"context_line":"            description\u003ddescription, metadata\u003dmetadata)[\u0027snapshot\u0027]"},{"line_number":74,"context_line":"        self.addCleanup(self.delete_snapshot, new_snapshot[\u0027id\u0027],"},{"line_number":75,"context_line":"                        self.admin_snapshots_client)"},{"line_number":76,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_0c3a7a33","line":73,"range":{"start_line":70,"start_character":0,"end_line":73,"end_character":67},"updated":"2017-06-07 01:47:25.000000000","message":"nit: this can be compressed to 3 lines:\n        new_snapshot \u003d self.admin_snapshot_manage_client.manage_snapshot(\n            volume_id\u003dvolume[\u0027id\u0027], ref\u003d{\u0027source-name\u0027: snapshot_ref},\n            name\u003dname, description\u003ddescription, metadata\u003dmetadata)[\u0027snapshot\u0027]","commit_id":"f8bb6d5afbc3dd730c9c80dc22243d20e7b33550"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"f5265a51eb8432fb489dbc61b354fa5567086477","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"        # Manage the snapshot"},{"line_number":69,"context_line":"        snapshot_ref \u003d \u0027_snapshot-%s\u0027 % snapshot[\u0027id\u0027]"},{"line_number":70,"context_line":"        new_snapshot \u003d self.admin_snapshot_manage_client.manage_snapshot("},{"line_number":71,"context_line":"            volume_id\u003dvolume[\u0027id\u0027],"},{"line_number":72,"context_line":"            ref\u003d{\u0027source-name\u0027: snapshot_ref}, name\u003dname,"},{"line_number":73,"context_line":"            description\u003ddescription, metadata\u003dmetadata)[\u0027snapshot\u0027]"},{"line_number":74,"context_line":"        self.addCleanup(self.delete_snapshot, new_snapshot[\u0027id\u0027],"},{"line_number":75,"context_line":"                        self.admin_snapshots_client)"},{"line_number":76,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_9bf35d74","line":73,"range":{"start_line":70,"start_character":0,"end_line":73,"end_character":67},"in_reply_to":"df140735_0c3a7a33","updated":"2017-06-07 06:38:33.000000000","message":"Thanks for your comments, done","commit_id":"f8bb6d5afbc3dd730c9c80dc22243d20e7b33550"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"a0f660ad55ce12c849c2ac9c044e463d9d3063d2","unresolved":false,"context_lines":[{"line_number":79,"context_line":"                                                new_snapshot[\u0027id\u0027],"},{"line_number":80,"context_line":"                                                \u0027available\u0027)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        # Verify the managed snapshot has the expected parent volume"},{"line_number":83,"context_line":"        self.assertEqual(new_snapshot[\u0027volume_id\u0027], volume[\u0027id\u0027])"},{"line_number":84,"context_line":"        self.assertEqual(name, new_snapshot[\u0027name\u0027])"},{"line_number":85,"context_line":"        self.assertEqual(description, new_snapshot[\u0027description\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_73c0c9e1","line":82,"updated":"2017-06-06 15:53:00.000000000","message":"and params","commit_id":"f8bb6d5afbc3dd730c9c80dc22243d20e7b33550"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"f5265a51eb8432fb489dbc61b354fa5567086477","unresolved":false,"context_lines":[{"line_number":79,"context_line":"                                                new_snapshot[\u0027id\u0027],"},{"line_number":80,"context_line":"                                                \u0027available\u0027)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        # Verify the managed snapshot has the expected parent volume"},{"line_number":83,"context_line":"        self.assertEqual(new_snapshot[\u0027volume_id\u0027], volume[\u0027id\u0027])"},{"line_number":84,"context_line":"        self.assertEqual(name, new_snapshot[\u0027name\u0027])"},{"line_number":85,"context_line":"        self.assertEqual(description, new_snapshot[\u0027description\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_7be8897b","line":82,"in_reply_to":"df140735_73c0c9e1","updated":"2017-06-07 06:38:33.000000000","message":"Done","commit_id":"f8bb6d5afbc3dd730c9c80dc22243d20e7b33550"},{"author":{"_account_id":20190,"name":"zhufl","email":"zhu.fanglei@zte.com.cn","username":"zhufl"},"change_message_id":"b7192cff4aa3174b985f25ff8d5755acd148b896","unresolved":false,"context_lines":[{"line_number":80,"context_line":"                                                \u0027available\u0027)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        # Verify the managed snapshot has the expected parent volume"},{"line_number":83,"context_line":"        self.assertEqual(new_snapshot[\u0027volume_id\u0027], volume[\u0027id\u0027])"},{"line_number":84,"context_line":"        self.assertEqual(name, new_snapshot[\u0027name\u0027])"},{"line_number":85,"context_line":"        self.assertEqual(description, new_snapshot[\u0027description\u0027])"},{"line_number":86,"context_line":"        self.assertEqual(metadata, new_snapshot[\u0027metadata\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_ec4626c1","line":83,"updated":"2017-06-07 01:47:25.000000000","message":"better to add show_snapshot before check","commit_id":"f8bb6d5afbc3dd730c9c80dc22243d20e7b33550"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"f5265a51eb8432fb489dbc61b354fa5567086477","unresolved":false,"context_lines":[{"line_number":80,"context_line":"                                                \u0027available\u0027)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"        # Verify the managed snapshot has the expected parent volume"},{"line_number":83,"context_line":"        self.assertEqual(new_snapshot[\u0027volume_id\u0027], volume[\u0027id\u0027])"},{"line_number":84,"context_line":"        self.assertEqual(name, new_snapshot[\u0027name\u0027])"},{"line_number":85,"context_line":"        self.assertEqual(description, new_snapshot[\u0027description\u0027])"},{"line_number":86,"context_line":"        self.assertEqual(metadata, new_snapshot[\u0027metadata\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_5bedc589","line":83,"in_reply_to":"df140735_ec4626c1","updated":"2017-06-07 06:38:33.000000000","message":"Done","commit_id":"f8bb6d5afbc3dd730c9c80dc22243d20e7b33550"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"a67c6b5b9c1bdec2ed5e544d0fa1f0b4f008543f","unresolved":false,"context_lines":[{"line_number":78,"context_line":"                                                new_snapshot[\u0027id\u0027],"},{"line_number":79,"context_line":"                                                \u0027available\u0027)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        # Verify the managed snapshot have the expected parent volume"},{"line_number":82,"context_line":"        # and the expected field values."},{"line_number":83,"context_line":"        new_snap_info \u003d self.admin_snapshots_client.show_snapshot("},{"line_number":84,"context_line":"            new_snapshot[\u0027id\u0027])[\u0027snapshot\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f231b9d_fdc23fe9","line":81,"updated":"2017-06-12 07:48:12.000000000","message":"I don\u0027t understand you changed \u0027has\u0027 to \u0027have\u0027 here.","commit_id":"8267e6eee1417862cb76d8dc76a609c4d00a7309"},{"author":{"_account_id":19262,"name":"Liron Kuchlani","email":"lkuchlan@redhat.com","username":"lkuchlan"},"change_message_id":"e2b0226135993692f76e88f08a39aa89eeab2b52","unresolved":false,"context_lines":[{"line_number":78,"context_line":"                                                new_snapshot[\u0027id\u0027],"},{"line_number":79,"context_line":"                                                \u0027available\u0027)"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"        # Verify the managed snapshot have the expected parent volume"},{"line_number":82,"context_line":"        # and the expected field values."},{"line_number":83,"context_line":"        new_snap_info \u003d self.admin_snapshots_client.show_snapshot("},{"line_number":84,"context_line":"            new_snapshot[\u0027id\u0027])[\u0027snapshot\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f231b9d_7d218fb4","line":81,"in_reply_to":"7f231b9d_fdc23fe9","updated":"2017-06-12 07:51:10.000000000","message":"Me too :)","commit_id":"8267e6eee1417862cb76d8dc76a609c4d00a7309"}]}