)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16237,"name":"venkatamahesh","email":"venkatamaheshkotha@gmail.com","username":"venkatamahesh"},"change_message_id":"bd3ae6186a430be973aa7979ae1bffc65b4e4830","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tuan Nguyen \u003ctuan@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-09-29 14:27:51 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\"Partially implements blueprint migrate-service-clients-to-tempest-lib\""},{"line_number":8,"context_line":"Migrated quotas_client.py, test_quotas_client.py, quotas.py from tempest"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"This migrates the above files from tempest. This includes tempest commits:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_d4e5dfa3","line":7,"updated":"2015-09-30 00:39:30.000000000","message":"Leave a line gap. \nFor further reference \"https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure\"","commit_id":"6534758403abe6abae0aabc3ffcb2926e2f0e467"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"90ad39fd5ea12ad7b952f9e2c8431d0fbb5a483c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Tuan Nguyen \u003ctuan@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-09-29 14:27:51 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\"Partially implements blueprint migrate-service-clients-to-tempest-lib\""},{"line_number":8,"context_line":"Migrated quotas_client.py, test_quotas_client.py, quotas.py from tempest"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"This migrates the above files from tempest. This includes tempest commits:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_37677d4e","line":7,"in_reply_to":"9a1a9d01_d4e5dfa3","updated":"2015-09-30 01:27:41.000000000","message":"Yes, right.\nWe need to move the above line 7 to line 15.\n\nPlease see the other patch like https://review.openstack.org/#/c/228970/ as a sample.","commit_id":"6534758403abe6abae0aabc3ffcb2926e2f0e467"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"90ad39fd5ea12ad7b952f9e2c8431d0fbb5a483c","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2015-09-29 14:27:51 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\"Partially implements blueprint migrate-service-clients-to-tempest-lib\""},{"line_number":8,"context_line":"Migrated quotas_client.py, test_quotas_client.py, quotas.py from tempest"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"This migrates the above files from tempest. This includes tempest commits:"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_f75ca57e","line":8,"updated":"2015-09-30 01:27:41.000000000","message":"Here should be\n\n \tMigrated quotas_client.py from tempest\n\nfor clarifying what is main migrated module.","commit_id":"6534758403abe6abae0aabc3ffcb2926e2f0e467"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"90ad39fd5ea12ad7b952f9e2c8431d0fbb5a483c","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"This migrates the above files from tempest. This includes tempest commits:"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":" * tempest/services/compute/json/quotas_client.py: Idd8afb1bcfc1b0165b1e967a991d4705ff182249"},{"line_number":13,"context_line":" * tempest/tests/services/compute/test_quotas_client.py: I3ad6761651cec5e66012d08e6b63322f53286a5c"},{"line_number":14,"context_line":" * tempest/api_schema/response/compute/v2_1/quotas.py: Ica929c402a3d042ae751302384e68853eb28b405"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_b7538d6a","line":12,"updated":"2015-09-30 01:27:41.000000000","message":"Please remove paths like\n\n * quotas_client.py: Idd8afb1bcfc1b0165b1e967a991d4705ff182249","commit_id":"6534758403abe6abae0aabc3ffcb2926e2f0e467"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"90ad39fd5ea12ad7b952f9e2c8431d0fbb5a483c","unresolved":false,"context_lines":[{"line_number":10,"context_line":"This migrates the above files from tempest. This includes tempest commits:"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":" * tempest/services/compute/json/quotas_client.py: Idd8afb1bcfc1b0165b1e967a991d4705ff182249"},{"line_number":13,"context_line":" * tempest/tests/services/compute/test_quotas_client.py: I3ad6761651cec5e66012d08e6b63322f53286a5c"},{"line_number":14,"context_line":" * tempest/api_schema/response/compute/v2_1/quotas.py: Ica929c402a3d042ae751302384e68853eb28b405"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"to see the commit history for these files refer to the above Change-Ids in the tempest repository"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_574c71c7","line":13,"updated":"2015-09-30 01:27:41.000000000","message":"ditto","commit_id":"6534758403abe6abae0aabc3ffcb2926e2f0e467"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"90ad39fd5ea12ad7b952f9e2c8431d0fbb5a483c","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":" * tempest/services/compute/json/quotas_client.py: Idd8afb1bcfc1b0165b1e967a991d4705ff182249"},{"line_number":13,"context_line":" * tempest/tests/services/compute/test_quotas_client.py: I3ad6761651cec5e66012d08e6b63322f53286a5c"},{"line_number":14,"context_line":" * tempest/api_schema/response/compute/v2_1/quotas.py: Ica929c402a3d042ae751302384e68853eb28b405"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"to see the commit history for these files refer to the above Change-Ids in the tempest repository"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a1a9d01_17c91926","line":14,"updated":"2015-09-30 01:27:41.000000000","message":"ditto","commit_id":"6534758403abe6abae0aabc3ffcb2926e2f0e467"}],"tempest_lib/tests/services/compute/test_quotas_client.py":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"90ad39fd5ea12ad7b952f9e2c8431d0fbb5a483c","unresolved":false,"context_lines":[{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest_lib.tests import fake_auth_provider"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from tempest_lib.services.compute import quotas_client"},{"line_number":20,"context_line":"from tempest_lib.tests.services.compute import base"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a1a9d01_d7e5e1a3","line":18,"updated":"2015-09-30 01:27:41.000000000","message":"necessary to remove a blank line 18 because the imported modules are the same.","commit_id":"6534758403abe6abae0aabc3ffcb2926e2f0e467"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"5c38b1dd2c3b5c5101a9544534099947313e2688","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest_lib.tests import fake_auth_provider"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from tempest_lib.services.compute import quotas_client"},{"line_number":20,"context_line":"from tempest_lib.tests.services.compute import base"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_fdf1d295","line":17,"updated":"2015-09-30 12:56:01.000000000","message":"Can you check my previous comment?\n\nhttps://review.openstack.org/#/c/229120/1..2/tempest_lib/tests/services/compute/test_quotas_client.py","commit_id":"4b2ce7cef720701bbb58a1a7497d1d1e008fd573"},{"author":{"_account_id":8001,"name":"John Warren","email":"jswarren@us.ibm.com","username":"jswarren"},"change_message_id":"f2f5fa99b2773edd284641f569b4a4b5fd85d26b","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from tempest_lib.tests import fake_auth_provider"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from tempest_lib.services.compute import quotas_client"},{"line_number":20,"context_line":"from tempest_lib.tests.services.compute import base"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a1a9d01_dde3aed7","line":19,"updated":"2015-09-30 12:55:34.000000000","message":"Imports from the project need to be grouped together.  This means that line 17 should go between lines 19 and 20 and the blank line 16 should be removed.  See http://docs.openstack.org/developer/hacking/#import-order-template","commit_id":"4b2ce7cef720701bbb58a1a7497d1d1e008fd573"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"61773e92a4d334171cc191a176a5912b4167e519","unresolved":false,"context_lines":[{"line_number":12,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":13,"context_line":"#    under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":"from tempest_lib.services.compute import quotas_client"},{"line_number":17,"context_line":"from tempest_lib.tests import fake_auth_provider"},{"line_number":18,"context_line":"from tempest_lib.tests.services.compute import base"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a1a9d01_fcac1c37","line":15,"updated":"2015-10-02 07:33:51.000000000","message":"need to add a new line for separating standard lib and local module import.","commit_id":"694b38735d94b8a7262f406a17b0545c2cc6a6cc"}]}