)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16237,"name":"venkatamahesh","email":"venkatamaheshkotha@gmail.com","username":"venkatamahesh"},"change_message_id":"cfa9baf9c25cc65aac1be0f87dc9ea269dfbbe46","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-10-05 23:32:16 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add unit test for create_server, list_addresses, list_addresses_by_network."},{"line_number":8,"context_line":"Update delete_server function."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"We have a plan to migrate service clients to tempest-lib from Tempest."},{"line_number":11,"context_line":"However, we are not doing enough tests for service clients now."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a2fa921_9a4bb4b0","line":8,"updated":"2015-10-06 04:49:24.000000000","message":"Title is too long. minimize to \u003c\u003d 50 characters","commit_id":"5ed6e74c85383d0b49c28cb13b3e5bd83497d21d"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1a8e7740f4bf66a475eb7cc317972191442b4052","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We have a plan to migrate service clients to tempest-lib from Tempest."},{"line_number":10,"context_line":"However, we are not doing enough tests for service clients now."},{"line_number":11,"context_line":"This patch adds unit test for create_server, list_addresses, list_address_by_network,"},{"line_number":12,"context_line":"and delete_server. Update delete_server function. Update bytes_body parameter"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I5c2f73b87b033baa362360b6add6eabd02f54c38"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a2fa921_67511fd8","line":12,"updated":"2015-10-07 09:55:53.000000000","message":"the test of delete_server should not be changed as my previous comment.","commit_id":"7cfec757ef66e698213faee188f3b169a3ed1380"},{"author":{"_account_id":17866,"name":"Truong Le","email":"let@us.ibm.com","username":"truongle"},"change_message_id":"df6279ac8fdf4c9c10dca9d7c284be1ad6baa40d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We have a plan to migrate service clients to tempest-lib from Tempest."},{"line_number":10,"context_line":"However, we are not doing enough tests for service clients now."},{"line_number":11,"context_line":"This patch adds unit test for create_server, list_addresses, list_address_by_network,"},{"line_number":12,"context_line":"and delete_server. Update delete_server function. Update bytes_body parameter"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I5c2f73b87b033baa362360b6add6eabd02f54c38"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a2fa921_c5bb48e4","line":12,"in_reply_to":"7a2fa921_67511fd8","updated":"2015-10-12 06:29:13.000000000","message":"Done","commit_id":"7cfec757ef66e698213faee188f3b169a3ed1380"}],"tempest/tests/services/compute/test_servers_client.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0077bc99a49b0c1b991c0ded59e62bdd84b1a823","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    FAKE_SERVER_POST \u003d {\"server\": {"},{"line_number":91,"context_line":"        \"id\": \"616fb98f-46ca-475e-917e-2563e5a8cd19\","},{"line_number":92,"context_line":"        \"adminPass\": \"fake-admin-pass\","},{"line_number":93,"context_line":"        \"security_groups\": ["},{"line_number":94,"context_line":"            \u0027fake-security-group-1\u0027,"},{"line_number":95,"context_line":"            \u0027fake-security-group-2\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a2fa921_287fa5ab","line":92,"updated":"2015-10-06 13:34:58.000000000","message":"I guess Jordan will require \"¥u1234\" data as the previous review of the other patch.","commit_id":"9a8ee82190c84b2bdc5a6eaa4848f21c17b3ff1c"},{"author":{"_account_id":17866,"name":"Truong Le","email":"let@us.ibm.com","username":"truongle"},"change_message_id":"b718d41c1fb7cbbcb4a5dcb1042be8705943c441","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"    FAKE_SERVER_POST \u003d {\"server\": {"},{"line_number":91,"context_line":"        \"id\": \"616fb98f-46ca-475e-917e-2563e5a8cd19\","},{"line_number":92,"context_line":"        \"adminPass\": \"fake-admin-pass\","},{"line_number":93,"context_line":"        \"security_groups\": ["},{"line_number":94,"context_line":"            \u0027fake-security-group-1\u0027,"},{"line_number":95,"context_line":"            \u0027fake-security-group-2\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a2fa921_bec1a4e6","line":92,"in_reply_to":"7a2fa921_287fa5ab","updated":"2015-10-06 17:42:48.000000000","message":"I considered about this before making a commit; however, I thought that we mightn\u0027t need the \\u1234 in here since this is a password.","commit_id":"9a8ee82190c84b2bdc5a6eaa4848f21c17b3ff1c"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0077bc99a49b0c1b991c0ded59e62bdd84b1a823","unresolved":false,"context_lines":[{"line_number":163,"context_line":"        self.check_service_client_function("},{"line_number":164,"context_line":"            self.client.delete_server,"},{"line_number":165,"context_line":"            \u0027tempest.common.service_client.ServiceClient.delete\u0027,"},{"line_number":166,"context_line":"            {},"},{"line_number":167,"context_line":"            bytes_body,"},{"line_number":168,"context_line":"            status\u003d204,"},{"line_number":169,"context_line":"            server_id\u003dself.server_id"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a2fa921_68d91df3","line":166,"updated":"2015-10-06 13:34:58.000000000","message":"We don\u0027t need to change test_delete_server because bytes_body\u003dTrue test is meaningless.\n\nas line 166, the return body of delete_server is {}. So the body doesn\u0027t contain any string/byte data.","commit_id":"9a8ee82190c84b2bdc5a6eaa4848f21c17b3ff1c"},{"author":{"_account_id":17866,"name":"Truong Le","email":"let@us.ibm.com","username":"truongle"},"change_message_id":"b718d41c1fb7cbbcb4a5dcb1042be8705943c441","unresolved":false,"context_lines":[{"line_number":163,"context_line":"        self.check_service_client_function("},{"line_number":164,"context_line":"            self.client.delete_server,"},{"line_number":165,"context_line":"            \u0027tempest.common.service_client.ServiceClient.delete\u0027,"},{"line_number":166,"context_line":"            {},"},{"line_number":167,"context_line":"            bytes_body,"},{"line_number":168,"context_line":"            status\u003d204,"},{"line_number":169,"context_line":"            server_id\u003dself.server_id"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a2fa921_59d18acc","line":166,"in_reply_to":"7a2fa921_68d91df3","updated":"2015-10-06 17:42:48.000000000","message":"Agree! I\u0027ll change it regarding to your suggestion.","commit_id":"9a8ee82190c84b2bdc5a6eaa4848f21c17b3ff1c"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0077bc99a49b0c1b991c0ded59e62bdd84b1a823","unresolved":false,"context_lines":[{"line_number":183,"context_line":"            bytes_body,"},{"line_number":184,"context_line":"            status\u003d202,"},{"line_number":185,"context_line":"            name\u003d\u0027fake-name\u0027,"},{"line_number":186,"context_line":"            image_ref\u003d\u0027fake-image-ref\u0027,"},{"line_number":187,"context_line":"            flavor_ref\u003d\u0027fake-flavor-ref\u0027"},{"line_number":188,"context_line":"            )"},{"line_number":189,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7a2fa921_cb74eb9d","line":186,"updated":"2015-10-06 13:34:58.000000000","message":"here should be\n\n imageRef\u003d\u0027fake-image-ref\u0027\n\ntempest/api/compute/admin/test_servers.py is a good sample for seeing the usage of create_server method.","commit_id":"9a8ee82190c84b2bdc5a6eaa4848f21c17b3ff1c"},{"author":{"_account_id":17866,"name":"Truong Le","email":"let@us.ibm.com","username":"truongle"},"change_message_id":"b718d41c1fb7cbbcb4a5dcb1042be8705943c441","unresolved":false,"context_lines":[{"line_number":183,"context_line":"            bytes_body,"},{"line_number":184,"context_line":"            status\u003d202,"},{"line_number":185,"context_line":"            name\u003d\u0027fake-name\u0027,"},{"line_number":186,"context_line":"            image_ref\u003d\u0027fake-image-ref\u0027,"},{"line_number":187,"context_line":"            flavor_ref\u003d\u0027fake-flavor-ref\u0027"},{"line_number":188,"context_line":"            )"},{"line_number":189,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7a2fa921_d9935af9","line":186,"in_reply_to":"7a2fa921_cb74eb9d","updated":"2015-10-06 17:42:48.000000000","message":"Done","commit_id":"9a8ee82190c84b2bdc5a6eaa4848f21c17b3ff1c"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0077bc99a49b0c1b991c0ded59e62bdd84b1a823","unresolved":false,"context_lines":[{"line_number":184,"context_line":"            status\u003d202,"},{"line_number":185,"context_line":"            name\u003d\u0027fake-name\u0027,"},{"line_number":186,"context_line":"            image_ref\u003d\u0027fake-image-ref\u0027,"},{"line_number":187,"context_line":"            flavor_ref\u003d\u0027fake-flavor-ref\u0027"},{"line_number":188,"context_line":"            )"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"    def test_list_addresses_with_str_body(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a2fa921_2b60ffcd","line":187,"updated":"2015-10-06 13:34:58.000000000","message":"here should be\n\n flavorRef\u003d\u0027fake-flavor-ref\u0027","commit_id":"9a8ee82190c84b2bdc5a6eaa4848f21c17b3ff1c"},{"author":{"_account_id":17866,"name":"Truong Le","email":"let@us.ibm.com","username":"truongle"},"change_message_id":"b718d41c1fb7cbbcb4a5dcb1042be8705943c441","unresolved":false,"context_lines":[{"line_number":184,"context_line":"            status\u003d202,"},{"line_number":185,"context_line":"            name\u003d\u0027fake-name\u0027,"},{"line_number":186,"context_line":"            image_ref\u003d\u0027fake-image-ref\u0027,"},{"line_number":187,"context_line":"            flavor_ref\u003d\u0027fake-flavor-ref\u0027"},{"line_number":188,"context_line":"            )"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"    def test_list_addresses_with_str_body(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a2fa921_b9c1bee6","line":187,"in_reply_to":"7a2fa921_2b60ffcd","updated":"2015-10-06 17:42:48.000000000","message":"Done","commit_id":"9a8ee82190c84b2bdc5a6eaa4848f21c17b3ff1c"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1a8e7740f4bf66a475eb7cc317972191442b4052","unresolved":false,"context_lines":[{"line_number":158,"context_line":"            self.client.delete_server,"},{"line_number":159,"context_line":"            \u0027tempest.common.service_client.ServiceClient.delete\u0027,"},{"line_number":160,"context_line":"            {},"},{"line_number":161,"context_line":"            bytes_body,"},{"line_number":162,"context_line":"            status\u003d204,"},{"line_number":163,"context_line":"            server_id\u003dself.server_id"},{"line_number":164,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a2fa921_a7c0977d","line":161,"updated":"2015-10-07 09:55:53.000000000","message":"this bytes_body is not necessary to be added.","commit_id":"7cfec757ef66e698213faee188f3b169a3ed1380"},{"author":{"_account_id":17866,"name":"Truong Le","email":"let@us.ibm.com","username":"truongle"},"change_message_id":"df6279ac8fdf4c9c10dca9d7c284be1ad6baa40d","unresolved":false,"context_lines":[{"line_number":158,"context_line":"            self.client.delete_server,"},{"line_number":159,"context_line":"            \u0027tempest.common.service_client.ServiceClient.delete\u0027,"},{"line_number":160,"context_line":"            {},"},{"line_number":161,"context_line":"            bytes_body,"},{"line_number":162,"context_line":"            status\u003d204,"},{"line_number":163,"context_line":"            server_id\u003dself.server_id"},{"line_number":164,"context_line":"            )"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a2fa921_05f380d7","line":161,"in_reply_to":"7a2fa921_a7c0977d","updated":"2015-10-12 06:29:13.000000000","message":"Done","commit_id":"7cfec757ef66e698213faee188f3b169a3ed1380"}]}