)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"f9405c840de071d4d11b582df35f117c0dc8b7c6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Potter \u003cnathaniel.potter@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-24 22:04:38 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Move quota checks from manager to API"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently quotas are sometimes checked in cinder.volume.manager,"},{"line_number":10,"context_line":"and other times checked in cinder.volume.api. This patch is the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a8ffd7b_d796cb93","line":7,"updated":"2015-11-24 22:54:59.000000000","message":"It is a bit confusing that we have two patches with the same first line that are related to eachother. It would be helpful if we add some info to distinguish one from the other. Maybe adding \u0027part 2\u0027 or \u0027[2/2]\u0027?","commit_id":"ca792cb9f0cc2f2874e35798c53f3c10a7eda33e"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"93797e5a372883eef23011294460bb9f6854111e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Potter \u003cnathaniel.potter@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-24 22:04:38 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Move quota checks from manager to API"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently quotas are sometimes checked in cinder.volume.manager,"},{"line_number":10,"context_line":"and other times checked in cinder.volume.api. This patch is the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a8ffd7b_f7518f63","line":7,"in_reply_to":"9a8ffd7b_d796cb93","updated":"2015-11-24 23:00:55.000000000","message":"Yeah I was thinking this might have to happen, I just wasn\u0027t totally sure. Thanks!","commit_id":"ca792cb9f0cc2f2874e35798c53f3c10a7eda33e"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"f9405c840de071d4d11b582df35f117c0dc8b7c6","unresolved":false,"context_lines":[{"line_number":11,"context_line":"second of two commits that aim to make these checks more consistent"},{"line_number":12,"context_line":"by moving calls from the manager to the API. This way, we can give"},{"line_number":13,"context_line":"the user feedback before launching the RPC call."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"APIImpact"},{"line_number":16,"context_line":"Closes-bug: #1508249"},{"line_number":17,"context_line":"Change-Id: I081b4172d0c57f20239ace2ec3f2c16bb19d3d9b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a8ffd7b_5705fbbf","line":14,"updated":"2015-11-24 22:54:59.000000000","message":"It would be helpful to add Depends-On:\u003cgerrit change-id\u003e here.","commit_id":"ca792cb9f0cc2f2874e35798c53f3c10a7eda33e"},{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"93797e5a372883eef23011294460bb9f6854111e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"second of two commits that aim to make these checks more consistent"},{"line_number":12,"context_line":"by moving calls from the manager to the API. This way, we can give"},{"line_number":13,"context_line":"the user feedback before launching the RPC call."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"APIImpact"},{"line_number":16,"context_line":"Closes-bug: #1508249"},{"line_number":17,"context_line":"Change-Id: I081b4172d0c57f20239ace2ec3f2c16bb19d3d9b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a8ffd7b_375f1774","line":14,"in_reply_to":"9a8ffd7b_5705fbbf","updated":"2015-11-24 23:00:55.000000000","message":"Done","commit_id":"ca792cb9f0cc2f2874e35798c53f3c10a7eda33e"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"e58c3833314c6586ba0b747112024972d3dc5a3a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nate Potter \u003cnathaniel.potter@intel.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-24 23:00:59 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Move quota checks from manager to API Part 2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently quotas are sometimes checked in cinder.volume.manager,"},{"line_number":10,"context_line":"and other times checked in cinder.volume.api. This patch is the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"7a740942_9ae40a0e","line":7,"updated":"2015-12-03 18:47:02.000000000","message":"Please try to explain which calls you\u0027re affecting. These are changes in CGs calls, right?","commit_id":"299725dae1c327b9697748a850452751dfbe1d0d"}]}