)]}'
{"tempest/api/compute/servers/test_servers_negative.py":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"8f9e48b03f97e6401468f95b4c9657d48cd04dd8","unresolved":false,"context_lines":[{"line_number":253,"context_line":"        # Get a non existent server details"},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"        self.assertRaises(exceptions.NotFound, self.client.get_server,"},{"line_number":256,"context_line":"                          \u0027999erra43\u0027)"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":259,"context_line":"    def test_stop_non_existent_server(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FB0Y%3D","line":256,"updated":"2013-08-08 07:43:50.000000000","message":"see below - maybe fix it here too.","commit_id":"fde2a61c3c4427731e6dfacc7ab25b69a6f3d4f6"},{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"8f9e48b03f97e6401468f95b4c9657d48cd04dd8","unresolved":false,"context_lines":[{"line_number":260,"context_line":"        # Stop a non existent server"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"        self.assertRaises(exceptions.NotFound, self.servers_client.stop,"},{"line_number":263,"context_line":"                          \u0027999erra43\u0027)"},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":266,"context_line":"    def test_pause_non_existent_server(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FB0g%3D","line":263,"updated":"2013-08-08 07:43:50.000000000","message":"I know that it\u0027s very unlikely that this id is already taken but could we just create a random name for it? like in https://review.openstack.org/#/c/40049/","commit_id":"fde2a61c3c4427731e6dfacc7ab25b69a6f3d4f6"},{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"ff96a247228f198873b40e24d4b8728a39b31594","unresolved":false,"context_lines":[{"line_number":258,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":259,"context_line":"    def test_stop_non_existent_server(self):"},{"line_number":260,"context_line":"        # Stop a non existent server"},{"line_number":261,"context_line":"        non_exist_id \u003d rand_name(\u0027server\u0027)"},{"line_number":262,"context_line":"        self.assertRaises(exceptions.NotFound, self.servers_client.stop,"},{"line_number":263,"context_line":"                          non_exist_id)"},{"line_number":264,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAQn%2F%2FBVs%3D","line":261,"updated":"2013-08-08 11:26:17.000000000","message":"perhaps it would be better set it to something like \"non_existant_server\" and not something that could actually possibly exist  due to another test running in parallel?","commit_id":"7b4ff4eadef9c8b9ded3deca7c3e28648efb9924"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"56c999e14c1297a138fbe85dacbedd9868399b31","unresolved":false,"context_lines":[{"line_number":258,"context_line":"    @attr(type\u003d[\u0027negative\u0027, \u0027gate\u0027])"},{"line_number":259,"context_line":"    def test_stop_non_existent_server(self):"},{"line_number":260,"context_line":"        # Stop a non existent server"},{"line_number":261,"context_line":"        non_exist_id \u003d rand_name(\u0027server\u0027)"},{"line_number":262,"context_line":"        self.assertRaises(exceptions.NotFound, self.servers_client.stop,"},{"line_number":263,"context_line":"                          non_exist_id)"},{"line_number":264,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"AAAAQn%2F%2FBTQ%3D","line":261,"in_reply_to":"AAAAQn%2F%2FBVs%3D","updated":"2013-08-08 11:34:56.000000000","message":"anju, you could also avoid completely rand_name and pass some arbitrary string as per test_get_non_existent_server; actually, it could be worth passing the very same string","commit_id":"7b4ff4eadef9c8b9ded3deca7c3e28648efb9924"}]}