)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"d71ecff17c512c4178d2935a0ccabed7d768f4ad","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added a server-pause test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Implement a Negative test for"},{"line_number":10,"context_line":"Pausing a already paused server."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic4bc35258e56f0758640d92d3338c9aa584f0a9a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"AAAAQn%2F%2Foa0%3D","line":9,"updated":"2013-07-26 05:59:41.000000000","message":"Please correct your commit message. Line breaks are not correctly.\n\nAlso there is a typo in your topic. For my point of view this it not a blueprint since it is just a small fix.","commit_id":"00e78668f8083f40a4ec520ec3480ccf6ee09672"}],"tempest/api/compute/servers/test_servers_negative.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"fb59a2a769074b7cae78f400369b3318d78969b4","unresolved":false,"context_lines":[{"line_number":105,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"},{"line_number":106,"context_line":"        self.client.delete_server(self.server_id)"},{"line_number":107,"context_line":"        self.client.wait_for_server_termination(self.server_id)"},{"line_number":108,"context_line":"        self.assertRaises(exceptions.UnprocessableEntity,"},{"line_number":109,"context_line":"                          self.client.pause_server,"},{"line_number":110,"context_line":"                          self.server_id)"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2Fyts%3D","line":108,"updated":"2013-07-23 06:27:38.000000000","message":"I am still unsure it is the expected documented exception.","commit_id":"3bd75534d0d13c5213e165dc5d05475e246fd7ba"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"fb59a2a769074b7cae78f400369b3318d78969b4","unresolved":false,"context_lines":[{"line_number":110,"context_line":"                          self.server_id)"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"    @attr(type\u003d\u0027negative\u0027)"},{"line_number":113,"context_line":"    def test_pause__paused_server(self):"},{"line_number":114,"context_line":"        # Pause a paused server"},{"line_number":115,"context_line":"        resp, server \u003d self.create_server()"},{"line_number":116,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2FysU%3D","line":113,"updated":"2013-07-23 06:27:38.000000000","message":"double __ .\n\nMay be the test_server_actions.py is a better place for double pause.\ntempest/api/compute/servers/test_server_actions.py","commit_id":"3bd75534d0d13c5213e165dc5d05475e246fd7ba"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"d6547b55bffe46f7e9ccf1e4b2d7d4031f87d59e","unresolved":false,"context_lines":[{"line_number":104,"context_line":"        resp, server \u003d self.create_server()"},{"line_number":105,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"},{"line_number":106,"context_line":"        self.client.delete_server(self.server_id)"},{"line_number":107,"context_line":"        self.client.wait_for_server_termination(self.server_id)"},{"line_number":108,"context_line":"        self.assertRaises(exceptions.UnprocessableEntity,"},{"line_number":109,"context_line":"                          self.client.pause_server,"},{"line_number":110,"context_line":"                          self.server_id)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2Fwvc%3D","line":107,"updated":"2013-07-23 17:53:42.000000000","message":"So I don\u0027t think there is any reason to create and then delete a server. You can get the same result by trying to pause a non-existent server.","commit_id":"9bd6b22434dd6c7c570cefad5973376d4bdebb6a"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"d6547b55bffe46f7e9ccf1e4b2d7d4031f87d59e","unresolved":false,"context_lines":[{"line_number":105,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"},{"line_number":106,"context_line":"        self.client.delete_server(self.server_id)"},{"line_number":107,"context_line":"        self.client.wait_for_server_termination(self.server_id)"},{"line_number":108,"context_line":"        self.assertRaises(exceptions.UnprocessableEntity,"},{"line_number":109,"context_line":"                          self.client.pause_server,"},{"line_number":110,"context_line":"                          self.server_id)"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2Fwvk%3D","line":108,"updated":"2013-07-23 17:53:42.000000000","message":"Regarding the the 422 comment from the previous version. So in general the 422 response is something that I feel shouldn\u0027t be used. (It\u0027s a WebDAV thing) but if that is what nova is returning than the test is fine. But, I\u0027d ask over on the -nova irc channel if they want to change it. With v3 coming out I\u0027m sure it\u0027ll be fixed there, so the answer is probably no.","commit_id":"9bd6b22434dd6c7c570cefad5973376d4bdebb6a"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"d6547b55bffe46f7e9ccf1e4b2d7d4031f87d59e","unresolved":false,"context_lines":[{"line_number":225,"context_line":""},{"line_number":226,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":227,"context_line":"    def test_delete_non_existent_server(self):"},{"line_number":228,"context_line":"        # Delete a non existent serveonsistent reviewing (afazeka"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.delete_server,"},{"line_number":231,"context_line":"                          \u0027999erra43\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2Fwxk%3D","line":228,"updated":"2013-07-23 17:53:42.000000000","message":"you mangled this comment here.","commit_id":"9bd6b22434dd6c7c570cefad5973376d4bdebb6a"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"22c602dee0ec4ac744b6724bbca5fa7229e06fb3","unresolved":false,"context_lines":[{"line_number":103,"context_line":"        # Pause a paused server."},{"line_number":104,"context_line":"        resp, server \u003d self.create_server()"},{"line_number":105,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"},{"line_number":106,"context_line":"        self.client.wait_for_server_status(self.server_id, \u0027ACTIVE\u0027)"},{"line_number":107,"context_line":"        self.client.pause_server(self.server_id)"},{"line_number":108,"context_line":"        self.client.wait_for_server_status(self.server_id, \u0027PAUSED\u0027)"},{"line_number":109,"context_line":"        self.assertRaises(exceptions.Duplicate,"}],"source_content_type":"text/x-python","patch_set":9,"id":"AAAAQn%2F%2Fqj8%3D","line":106,"updated":"2013-07-25 14:26:03.000000000","message":"Why not use self.create_server(wait_until\u003d\u0027ACTIVE\u0027) here instead of making the second wait call. \n\nNormally the only reason to do this is if there are steps to perform before it is set active. But, in this case you just want an active server so that you can pause it.","commit_id":"af0e4a4a8c5cd5c34f648c0de140a46e910de8b1"}]}