)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"546647e2c9ed3a861ad5eead3123d5588be8cbc9","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I95f2dad6d5ae17f722202f4917b6ef460cd41e15"},{"line_number":10,"context_line":"Closes-Bug: #1176683"},{"line_number":11,"context_line":"recheck no bug"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAATn%2F%2F96M%3D","line":11,"updated":"2013-10-24 13:59:22.000000000","message":"this should be put in general comments of the review, not to commit message","commit_id":"68f0195ffcadb413b03178263b1ba041a6d3e30f"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"9a76ff08d5d588a93ba8cf31efcf53233c8951b4","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I95f2dad6d5ae17f722202f4917b6ef460cd41e15"},{"line_number":14,"context_line":"Closes-Bug: #1176683"},{"line_number":15,"context_line":"recheck no bug"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAATn%2F%2F6jA%3D","line":15,"updated":"2013-10-25 09:58:05.000000000","message":"please remove this line","commit_id":"e71c511e6a862aa904e6ba811f61185a61db2205"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"cb55d17e97e614615a42645c16f0ec43e1321d2d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Modified configuration group name to lowercase"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Most of the OpenStack project makes use of lower case for configuration group names. But, \u0027neutron\u0027 was having group names in either cases. Bug states it is fine to have group name in upper case for [DEFAULT] but needs to be changed for others. It was noted that [PLUMgridDirector] was having characters."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Text [PLUMgridDirector] changed as [plumgriddirector] in the updated files posted for review"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F%2F2WA%3D","line":9,"updated":"2013-10-27 16:36:12.000000000","message":"the last nit: could you please split this to several lines?","commit_id":"48d5bc345d2984f4c9bf5d131b79de64565b3048"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"8bf0a587c74db672151459f5d2865c81b2fe02d6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Modified configuration group name to lowercase"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Most of the OpenStack project makes use of lower case for configuration group names. But, \u0027neutron\u0027 was having group names in either cases. Bug states it is fine to have group name in upper case for [DEFAULT] but needs to be changed for others. It was noted that [PLUMgridDirector] was having characters."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Text [PLUMgridDirector] changed as [plumgriddirector] in the updated files posted for review"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F%2F2V0%3D","line":9,"in_reply_to":"AAAATn%2F%2F2WA%3D","updated":"2013-10-27 16:38:01.000000000","message":"by the way, it\u0027s oslo.config project that recommends using lower case section names.","commit_id":"48d5bc345d2984f4c9bf5d131b79de64565b3048"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"8bf0a587c74db672151459f5d2865c81b2fe02d6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Most of the OpenStack project makes use of lower case for configuration group names. But, \u0027neutron\u0027 was having group names in either cases. Bug states it is fine to have group name in upper case for [DEFAULT] but needs to be changed for others. It was noted that [PLUMgridDirector] was having characters."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Text [PLUMgridDirector] changed as [plumgriddirector] in the updated files posted for review"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I95f2dad6d5ae17f722202f4917b6ef460cd41e15"},{"line_number":14,"context_line":"Closes-Bug: #1176683"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F%2F2V4%3D","line":11,"updated":"2013-10-27 16:38:01.000000000","message":"also i think this line could be removed.","commit_id":"48d5bc345d2984f4c9bf5d131b79de64565b3048"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"ec4aa62c69a256f8daf717ed46519d32f4da0063","unresolved":false,"context_lines":[{"line_number":13,"context_line":"for others. It was noted that [PLUMgridDirector] was having"},{"line_number":14,"context_line":"characters."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"reverify no bug"},{"line_number":17,"context_line":"Change-Id: I95f2dad6d5ae17f722202f4917b6ef460cd41e15"},{"line_number":18,"context_line":"Closes-Bug: #1176683"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAATn%2F%2F1wE%3D","line":16,"updated":"2013-10-28 06:08:20.000000000","message":"please remove. \nthis should go in general comment section on review, it\u0027s not a part of the patch, but an instruction for the testing system to rerun the gate tests.","commit_id":"ed6d6c7a296528f802aec0741a6785f381e298e4"}]}