)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15961,"name":"lisali","email":"xiaoyan.li@intel.com","username":"lisali"},"change_message_id":"c76d7a1fa57cf23deb4e96223ed9073683944675","unresolved":false,"context_lines":[{"line_number":15,"context_line":"consistencygroup is empy."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"This patch fix this problem by moving the creating of cgsnapshot object"},{"line_number":18,"context_line":"to after the test of whether consistencygroup is empty."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I09500ab1cf19b28591668a3fed6834d1a73ad176"},{"line_number":21,"context_line":"Closes-Bug: #1535979"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a5ae5dd_c7ab5ad5","line":18,"updated":"2016-02-04 03:24:47.000000000","message":"s/fix/fixes/","commit_id":"8c5f1484c5c2aa3c296dc7256bf5caf98ce2bd60"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"5505791521ec9465a2d51b055f4c8c2e9b9ebf89","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Accela Zhao \u003caccelazh@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-02-03 12:52:17 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Don\u0027t create cgsnapshot if empty consistencygroup"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If we create cgsnapshot from an empty consistencygroup, it fails"},{"line_number":10,"context_line":"with \"Consistency group is empty. No cgsnapshot will be created\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a5ae5dd_16ec458e","line":7,"updated":"2016-02-04 04:50:20.000000000","message":"You can use \u0027cg\u0027 instead of consistencygroup","commit_id":"c5103c6196c03ba727731324b7d9f9ff71ec7a18"},{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"59d30a98925d00a2074d8ebc8d355ca917e3d147","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Accela Zhao \u003caccelazh@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-02-03 12:52:17 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Don\u0027t create cgsnapshot if empty consistencygroup"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If we create cgsnapshot from an empty consistencygroup, it fails"},{"line_number":10,"context_line":"with \"Consistency group is empty. No cgsnapshot will be created\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a5ae5dd_160d2537","line":7,"updated":"2016-02-04 04:28:17.000000000","message":"nit: s/empty consistengcygroup/consistencygropu is empty/","commit_id":"c5103c6196c03ba727731324b7d9f9ff71ec7a18"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"d3028bd886b8cb9420efd3937c8989bd75621187","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Accela Zhao \u003caccelazh@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-02-03 12:52:17 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Don\u0027t create cgsnapshot if empty consistencygroup"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If we create cgsnapshot from an empty consistencygroup, it fails"},{"line_number":10,"context_line":"with \"Consistency group is empty. No cgsnapshot will be created\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5a5ae5dd_d63b0d37","line":7,"in_reply_to":"5a5ae5dd_160d2537","updated":"2016-02-04 04:47:48.000000000","message":"If I add \"is\", the title will be over length. But I think that should be OK.","commit_id":"c5103c6196c03ba727731324b7d9f9ff71ec7a18"}],"cinder/consistencygroup/api.py":[{"author":{"_account_id":15961,"name":"lisali","email":"xiaoyan.li@intel.com","username":"lisali"},"change_message_id":"c76d7a1fa57cf23deb4e96223ed9073683944675","unresolved":false,"context_lines":[{"line_number":713,"context_line":""},{"line_number":714,"context_line":"    def _create_cgsnapshot(self, context,"},{"line_number":715,"context_line":"                           group, name, description):"},{"line_number":716,"context_line":"        options \u003d {\u0027consistencygroup_id\u0027: group.id,"},{"line_number":717,"context_line":"                   \u0027user_id\u0027: context.user_id,"},{"line_number":718,"context_line":"                   \u0027project_id\u0027: context.project_id,"},{"line_number":719,"context_line":"                   \u0027status\u0027: \"creating\","}],"source_content_type":"text/x-python","patch_set":1,"id":"5a5ae5dd_3c35a5f6","line":716,"updated":"2016-02-04 03:24:47.000000000","message":"Better to move options before creating cgsnapshot","commit_id":"8c5f1484c5c2aa3c296dc7256bf5caf98ce2bd60"},{"author":{"_account_id":13900,"name":"Accela Zhao","email":"accelazh@gmail.com","username":"accelazh"},"change_message_id":"8b53f2882177f5b9016ecb41c91a2b32dea5d042","unresolved":false,"context_lines":[{"line_number":713,"context_line":""},{"line_number":714,"context_line":"    def _create_cgsnapshot(self, context,"},{"line_number":715,"context_line":"                           group, name, description):"},{"line_number":716,"context_line":"        options \u003d {\u0027consistencygroup_id\u0027: group.id,"},{"line_number":717,"context_line":"                   \u0027user_id\u0027: context.user_id,"},{"line_number":718,"context_line":"                   \u0027project_id\u0027: context.project_id,"},{"line_number":719,"context_line":"                   \u0027status\u0027: \"creating\","}],"source_content_type":"text/x-python","patch_set":1,"id":"5a5ae5dd_676dc697","line":716,"in_reply_to":"5a5ae5dd_3c35a5f6","updated":"2016-02-04 03:35:49.000000000","message":"Do you mean move it to line 731?","commit_id":"8c5f1484c5c2aa3c296dc7256bf5caf98ce2bd60"},{"author":{"_account_id":15961,"name":"lisali","email":"xiaoyan.li@intel.com","username":"lisali"},"change_message_id":"7e6242674c28af589ac0934e9ecde031b6e3e8e3","unresolved":false,"context_lines":[{"line_number":730,"context_line":"                   \u0027description\u0027: description}"},{"line_number":731,"context_line":""},{"line_number":732,"context_line":"        cgsnapshot \u003d None"},{"line_number":733,"context_line":"        cgsnapshot_id \u003d None"},{"line_number":734,"context_line":"        try:"},{"line_number":735,"context_line":"            cgsnapshot \u003d objects.CGSnapshot(context, **options)"},{"line_number":736,"context_line":"            cgsnapshot.create()"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a5ae5dd_16ebe5b1","line":733,"updated":"2016-02-04 04:44:46.000000000","message":"cgsnapshot_id \u003dNone is not needed.","commit_id":"c5103c6196c03ba727731324b7d9f9ff71ec7a18"}]}