)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12024,"name":"Tong Liu","email":"tongl@vmware.com","username":"tong101"},"change_message_id":"f64a19f1c9973431a4dd68c70bbd2fe9434a1cc3","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix test_lock_unlock_server test for admin role"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When this test was run with admin role, it failed"},{"line_number":10,"context_line":"on this assertRaises because it didnt actually raise"},{"line_number":11,"context_line":"that exception, and it was causing a hurdle in getting"},{"line_number":12,"context_line":"this test run with admin role assigned to tempest user."},{"line_number":13,"context_line":"There seems to be no apparent reason why this condition"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba0121b8_7fa7ce0a","line":10,"updated":"2016-04-01 03:11:53.000000000","message":"s/didnt/didn\u0027t","commit_id":"6796e07fdcbe2915d018af28147ed242375481cf"},{"author":{"_account_id":9152,"name":"Castulo J. Martinez","email":"castulo.martinez@intel.com","username":"cas"},"change_message_id":"d8ffb4b8485be757cae4588635bcbefb3f3c5ed2","unresolved":false,"context_lines":[{"line_number":10,"context_line":"on this assertRaises because it didnt actually raise"},{"line_number":11,"context_line":"that exception, and it was causing a hurdle in getting"},{"line_number":12,"context_line":"this test run with admin role assigned to tempest user."},{"line_number":13,"context_line":"There seems to be no apparent reason why this condition"},{"line_number":14,"context_line":"is put here, after removing this assertRaise, test was"},{"line_number":15,"context_line":"passed succesffully with admin role."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ic36f0039c32293f5abbc477eb18c018f6f33fd52"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba0121b8_5b8c777f","line":14,"range":{"start_line":13,"start_character":0,"end_line":14,"end_character":54},"updated":"2016-04-01 18:56:49.000000000","message":"I agree with Tong Liu that is better to leave the assertRaise in there, so just as a reminder, you should update the commit message too.","commit_id":"6796e07fdcbe2915d018af28147ed242375481cf"},{"author":{"_account_id":12024,"name":"Tong Liu","email":"tongl@vmware.com","username":"tong101"},"change_message_id":"f64a19f1c9973431a4dd68c70bbd2fe9434a1cc3","unresolved":false,"context_lines":[{"line_number":11,"context_line":"that exception, and it was causing a hurdle in getting"},{"line_number":12,"context_line":"this test run with admin role assigned to tempest user."},{"line_number":13,"context_line":"There seems to be no apparent reason why this condition"},{"line_number":14,"context_line":"is put here, after removing this assertRaise, test was"},{"line_number":15,"context_line":"passed succesffully with admin role."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ic36f0039c32293f5abbc477eb18c018f6f33fd52"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba0121b8_ff8abe62","line":14,"updated":"2016-04-01 03:11:53.000000000","message":"s/, after/. After","commit_id":"6796e07fdcbe2915d018af28147ed242375481cf"}],"tempest/api/compute/servers/test_server_actions.py":[{"author":{"_account_id":12024,"name":"Tong Liu","email":"tongl@vmware.com","username":"tong101"},"change_message_id":"f64a19f1c9973431a4dd68c70bbd2fe9434a1cc3","unresolved":false,"context_lines":[{"line_number":478,"context_line":"        self.client.lock_server(self.server_id)"},{"line_number":479,"context_line":"        self.addCleanup(self.client.unlock_server, self.server_id)"},{"line_number":480,"context_line":"        server \u003d self.client.show_server(self.server_id)[\u0027server\u0027]"},{"line_number":481,"context_line":"        self.assertEqual(server[\u0027status\u0027], \u0027ACTIVE\u0027)"},{"line_number":482,"context_line":"        self.client.unlock_server(self.server_id)"},{"line_number":483,"context_line":"        self.client.stop_server(self.server_id)"},{"line_number":484,"context_line":"        waiters.wait_for_server_status(self.client, self.server_id, \u0027SHUTOFF\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba0121b8_1fe7b240","line":481,"updated":"2016-04-01 03:11:53.000000000","message":"It explicitly tests the server cannot be stopped by non-admin user. I think you can add a condition to skip it is admin role instead of removing it totally.","commit_id":"6796e07fdcbe2915d018af28147ed242375481cf"}]}