)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"eee75140a36bc155bce71ae6f40cef05087308be","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Zainub Wahid \u003czainub.wahid@pivot-tech.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-03 08:06:50 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"neutron"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Changes bug:"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba5da102_fc482fb5","line":7,"updated":"2016-11-03 09:41:57.000000000","message":"Please see https://wiki.openstack.org/wiki/GitCommitMessages\n\nYou need to give an explanation what the change is and why.\n\n(the code change you made is correct). just please fix the commit message","commit_id":"8c140a353055fb3f462a0ff51c376bfa2f89aaee"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"403aa60b9991d742d7a0fc816c7e7099b8e80044","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9a629dbe_0b10bba7","updated":"2016-11-08 19:31:45.000000000","message":"Please don\u0027t forget to structure correctly your commit message, like the one we\u0027ve been working in https://review.openstack.org/#/c/393150/","commit_id":"f128982336c76841a749e28a194ebbf824c69677"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"8a70fb1488f682bf64ff38400216f6dbf5de4d9d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Zainub Wahid \u003czainub.wahid@pivot-tech.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-11-09 05:42:21 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Neutron allowed_address_pair"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use assertCount and assertItemsCount both to solve the problem."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: 391724"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a77a97e_55a5919f","line":9,"range":{"start_line":7,"start_character":0,"end_line":9,"end_character":63},"updated":"2016-11-18 01:25:05.000000000","message":"Description doesn\u0027t correspond with introduced changes in this patch.","commit_id":"8d95cd6197d2e57981e4078dc3f3f23b292878ea"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"8a70fb1488f682bf64ff38400216f6dbf5de4d9d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use assertCount and assertItemsCount both to solve the problem."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: 391724"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Id5ed970152b048e8dd9f902e05f00c7abd86fc90"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7a77a97e_b5718546","line":11,"range":{"start_line":11,"start_character":12,"end_line":11,"end_character":18},"updated":"2016-11-18 01:25:05.000000000","message":"Bug ID is incorrect","commit_id":"8d95cd6197d2e57981e4078dc3f3f23b292878ea"}],"neutron/tests/tempest/api/test_allowed_address_pair.py":[{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"75254ea6bf210276bf92864024b41b9f11c47c01","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import netaddr"},{"line_number":17,"context_line":"from tempest import test"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from neutron.tests.tempest.api import base"},{"line_number":21,"context_line":"from neutron.tests.tempest import config"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ba5da102_ce6959c0","line":19,"updated":"2016-10-31 20:47:14.000000000","message":"According to the OpenStack hacking guidelines (http://docs.openstack.org/developer/hacking/#import-order-template) this additional blank line is not needed","commit_id":"a6034dbbe4dd238fd3466705dc7660c4621a838b"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"403aa60b9991d742d7a0fc816c7e7099b8e80044","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import netaddr"},{"line_number":17,"context_line":"from neutron.tests.tempest.api import base"},{"line_number":18,"context_line":"from neutron.tests.tempest import config"},{"line_number":19,"context_line":"from tempest import test"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"class AllowedAddressPairTestJSON(base.BaseNetworkTest):"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a629dbe_0bb43bcf","line":19,"updated":"2016-11-08 19:31:45.000000000","message":"This change goes against the recommended way to structure the imports. You need to have 3 sections, each separated by an empty line:\n\nPython stdlib imports in human alphabetical order\n\u003cempty line\u003e\nthird-party lib imports in human alphabetical order\n\u003cempty line\u003e\nNeutron imports in human alphabetical order\n\u003cempty line\u003e\n\nYou can see an example here: http://docs.openstack.org/developer/hacking/#real-world-import-order-examples\n\nIn the case of this module, we are not importing anything from the Python stdlib, so we only have the last two sections.\n\nSince tempest is not part of Neutron (it is another OpenStack project), it belongs in the third-party lib imports, where we are also importing netaddr (Line 16 above)","commit_id":"f128982336c76841a749e28a194ebbf824c69677"},{"author":{"_account_id":14611,"name":"dasm","display_name":"dasm","email":"dsmigiel@redhat.com","username":"dasm"},"change_message_id":"8a70fb1488f682bf64ff38400216f6dbf5de4d9d","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from neutron.tests.tempest.api import base"},{"line_number":19,"context_line":"from neutron.tests.tempest import config"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"from tempest import test"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"class AllowedAddressPairTestJSON(base.BaseNetworkTest):"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_15e5d9da","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":24},"updated":"2016-11-18 01:25:05.000000000","message":"This is incorrect. Please refer again to Miguel\u0027s comment. `tempest` is third-party module, so should be grouped with netaddr","commit_id":"8d95cd6197d2e57981e4078dc3f3f23b292878ea"}]}