)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"6644b90d2d0940934bb5e5602d706ba2234c907e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nir Magnezi \u003cnmagnezi@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-09-01 12:58:10 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding port status validation upon interface addition to a router"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iad1a61c025161914f323fc032cbc9dfb51838a9a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa98f980_3b82c094","line":7,"updated":"2014-09-02 02:35:05.000000000","message":"We need more explanation. This says only \u0027what is changed\u0027. But we need \u0027why do we need this patch\u0027.\nPlease write the reason why we need this patch.","commit_id":"8970caa31c0068f45e010cee43f5720cc6ec6702"},{"author":{"_account_id":6579,"name":"Nir Magnezi","email":"nmagnezi@redhat.com","username":"nmagnezi"},"change_message_id":"67767f714204034f36152969089a45decc0495f9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nir Magnezi \u003cnmagnezi@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-09-01 12:58:10 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adding port status validation upon interface addition to a router"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iad1a61c025161914f323fc032cbc9dfb51838a9a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa98f980_a4dbddeb","line":7,"in_reply_to":"fa98f980_3b82c094","updated":"2014-09-02 09:16:16.000000000","message":"Done","commit_id":"8970caa31c0068f45e010cee43f5720cc6ec6702"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"62512e112ecfbb8e359607d1d1ee4b1e733258df","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Adds status check for router Port in scenarios"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Verifies port status is active (hence, functional) upon interface addition to a router."},{"line_number":10,"context_line":"future patches will verify that port status for other events such as floating IP association and dis-association."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Iad1a61c025161914f323fc032cbc9dfb51838a9a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"fa98f980_225ed376","line":10,"updated":"2014-09-03 06:30:15.000000000","message":"I think these lines are too long in one line.\n\n Subsequent lines should be wrapped at 72 characters.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_GIT_commit_message_structure\n\nI don\u0027t understand why the gate allowed this, though..","commit_id":"1040891112a118cd6b02d128be721ffaa838a6c6"}]}