)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12712,"name":"Gleb Stepanov","email":"gstepanov@mirantis.com","username":"stgleb"},"change_message_id":"a8e8115161097f60ffbfed1681636a393f3efffb","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-08-11 06:59:10 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Should be used \u0027-\u0027 instead of \u0027_\u0027 in the prompt message of"},{"line_number":8,"context_line":"qos-bandwidth-limit-rule-create command"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Modify \u0027Must provide max_kbps or max_burst_kbps option.\u0027 to"},{"line_number":11,"context_line":"\u0027Must provide max-kbps or max-burst-kbps option.\u0027 in the prompt"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9ad45d7e_81a7e9bb","line":8,"updated":"2016-08-11 13:11:13.000000000","message":"I think you should not dive in details insinde commit message header.\n\nFix prompt message or\nChange prompt message for qos-bandwidth-limit-rule-create\n\nBut i dont think that it is important to mention that you have changed \u0027-\u0027 instead of \u0027_\u0027 in commit message header.","commit_id":"732e8d29b264cb58d691810ae3ebc1eee88ae286"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"5aab049757d300c972e23046b791b270201b5ebb","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix prompt message for qos-bandwidth-limit-rule-create"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Modify \u0027Must provide max_kbps or max_burst_kbps option.\u0027 to"},{"line_number":10,"context_line":"\u0027Must provide max-kbps or max-burst-kbps option.\u0027 in the prompt"},{"line_number":11,"context_line":"message of qos-bandwidth-limit-rule-create command."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I97f57fb56030d032274270d73bdb66fddf0f58f9"},{"line_number":14,"context_line":"Closes-Bug: #1597635"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1ac06dbe_a6fda80a","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":51},"updated":"2016-08-24 12:04:16.000000000","message":"Generally, the commit message should describe why we need this change. This commit message just describes what is changed.\nThe reason is that error message should mention the actual arguments. The actual arguments are not underscore version but hyphen version.","commit_id":"f8ed21286b8d4ab3bed7c70aa5b99235881cf5b0"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"1fb03be74b790781687a703b1a9ab67cda6b7c79","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix prompt message for qos-bandwidth-limit-rule-create"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Modify \u0027Must provide max_kbps or max_burst_kbps option.\u0027 to"},{"line_number":10,"context_line":"\u0027Must provide max-kbps or max-burst-kbps option.\u0027 in the prompt"},{"line_number":11,"context_line":"message of qos-bandwidth-limit-rule-create command."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I97f57fb56030d032274270d73bdb66fddf0f58f9"},{"line_number":14,"context_line":"Closes-Bug: #1597635"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1ac06dbe_a6b808ea","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":51},"in_reply_to":"1ac06dbe_a6fda80a","updated":"2016-08-24 12:09:10.000000000","message":"Done","commit_id":"f8ed21286b8d4ab3bed7c70aa5b99235881cf5b0"}],"neutronclient/neutron/v2_0/qos/bandwidth_limit_rule.py":[{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"2323c0d824a291bf5f673bcdca4fa3ea885f8be9","unresolved":false,"context_lines":[{"line_number":27,"context_line":"def add_bandwidth_limit_arguments(parser):"},{"line_number":28,"context_line":"    parser.add_argument("},{"line_number":29,"context_line":"        \u0027--max-kbps\u0027,"},{"line_number":30,"context_line":"        help\u003d_(\u0027max bandwidth in kbps.\u0027))"},{"line_number":31,"context_line":"    parser.add_argument("},{"line_number":32,"context_line":"        \u0027--max-burst-kbps\u0027,"},{"line_number":33,"context_line":"        help\u003d_(\u0027max burst bandwidth in kbps.\u0027))"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_5eed4fee","line":30,"updated":"2016-07-05 08:21:20.000000000","message":"How about saying \u0027Maximum\u0027 instead of \u0027max\u0027 here? Better not to use short notation in the help msg..","commit_id":"109b45b02a4302667ac402791390771f24b280b7"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"2323c0d824a291bf5f673bcdca4fa3ea885f8be9","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        help\u003d_(\u0027max bandwidth in kbps.\u0027))"},{"line_number":31,"context_line":"    parser.add_argument("},{"line_number":32,"context_line":"        \u0027--max-burst-kbps\u0027,"},{"line_number":33,"context_line":"        help\u003d_(\u0027max burst bandwidth in kbps.\u0027))"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"def update_bandwidth_limit_args2body(parsed_args, body):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_9ebdf7d9","line":33,"updated":"2016-07-05 08:21:20.000000000","message":"ditto","commit_id":"109b45b02a4302667ac402791390771f24b280b7"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"110ac1ae5a9464e716806211cbf62ad1451b97b6","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        help\u003d_(\u0027Maximum bandwidth in kbps.\u0027))"},{"line_number":31,"context_line":"    parser.add_argument("},{"line_number":32,"context_line":"        \u0027--max-burst-kbps\u0027,"},{"line_number":33,"context_line":"        help\u003d_(\u0027Maximumm burst bandwidth in kbps.\u0027))"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"def update_bandwidth_limit_args2body(parsed_args, body):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_8d045418","line":33,"range":{"start_line":33,"start_character":23,"end_line":33,"end_character":24},"updated":"2016-07-06 16:37:28.000000000","message":"typo","commit_id":"184fe4d2f43ff8d5245572ddf85560149c122a6f"}]}