)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"9a6d365eadcabf61e11c9ee1b45931d44b2a95d9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     jiamin \u003cshljia@cn.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-07-23 03:40:39 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"update volumes/snapshots metadata in create_cgsnapshot and"},{"line_number":8,"context_line":"create_consistencygroup_from_src"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"volumes/snapshots metadata returned by the driver will be not updated into"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dada55a8_72621718","line":7,"updated":"2016-07-23 14:28:11.000000000","message":"This is too long and not really descriptive enough for a summary line. Maybe change it to be something like \"Add missing volume metadata update\"?","commit_id":"dfc9114ee43a813d1bc4295264a8105ba1da8281"},{"author":{"_account_id":17564,"name":"jiamin","email":"shljia@cn.ibm.com","username":"jiamin"},"change_message_id":"96970927d4a89cf1a44ddbdbc16b5a8d43f397cf","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     jiamin \u003cshljia@cn.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-07-23 03:40:39 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"update volumes/snapshots metadata in create_cgsnapshot and"},{"line_number":8,"context_line":"create_consistencygroup_from_src"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"volumes/snapshots metadata returned by the driver will be not updated into"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dada55a8_370fe8e2","line":7,"in_reply_to":"dada55a8_72621718","updated":"2016-07-24 09:09:02.000000000","message":"Thanks Sean.","commit_id":"dfc9114ee43a813d1bc4295264a8105ba1da8281"}],"cinder/volume/manager.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"9a6d365eadcabf61e11c9ee1b45931d44b2a95d9","unresolved":false,"context_lines":[{"line_number":2523,"context_line":""},{"line_number":2524,"context_line":"            if volumes_model_update:"},{"line_number":2525,"context_line":"                for update in volumes_model_update:"},{"line_number":2526,"context_line":"                    if update.get(\u0027metadata\u0027, None):"},{"line_number":2527,"context_line":"                        metadata \u003d update.pop(\u0027metadata\u0027)"},{"line_number":2528,"context_line":"                        self.db.volume_metadata_update(context,"},{"line_number":2529,"context_line":"                                                       update[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"dada55a8_f27607d7","line":2526,"range":{"start_line":2526,"start_character":44,"end_line":2526,"end_character":50},"updated":"2016-07-23 14:28:11.000000000","message":"None not needed as that is the default return value from .get()","commit_id":"dfc9114ee43a813d1bc4295264a8105ba1da8281"},{"author":{"_account_id":17564,"name":"jiamin","email":"shljia@cn.ibm.com","username":"jiamin"},"change_message_id":"96970927d4a89cf1a44ddbdbc16b5a8d43f397cf","unresolved":false,"context_lines":[{"line_number":2523,"context_line":""},{"line_number":2524,"context_line":"            if volumes_model_update:"},{"line_number":2525,"context_line":"                for update in volumes_model_update:"},{"line_number":2526,"context_line":"                    if update.get(\u0027metadata\u0027, None):"},{"line_number":2527,"context_line":"                        metadata \u003d update.pop(\u0027metadata\u0027)"},{"line_number":2528,"context_line":"                        self.db.volume_metadata_update(context,"},{"line_number":2529,"context_line":"                                                       update[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"dada55a8_f70430c7","line":2526,"range":{"start_line":2526,"start_character":44,"end_line":2526,"end_character":50},"in_reply_to":"dada55a8_f27607d7","updated":"2016-07-24 09:09:02.000000000","message":"Thanks Sean,I have changed it.","commit_id":"dfc9114ee43a813d1bc4295264a8105ba1da8281"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"9a6d365eadcabf61e11c9ee1b45931d44b2a95d9","unresolved":false,"context_lines":[{"line_number":2982,"context_line":"                    # Update db for snapshot."},{"line_number":2983,"context_line":"                    # NOTE(xyang): snapshots is a list of snapshot objects."},{"line_number":2984,"context_line":"                    # snapshots_model_update should be a list of dicts."},{"line_number":2985,"context_line":"                    if snap_model.get(\u0027metadata\u0027, None):"},{"line_number":2986,"context_line":"                        metadata \u003d snap_model.pop(\u0027metadata\u0027)"},{"line_number":2987,"context_line":"                        self.db.snapshot_metadata_update(context,"},{"line_number":2988,"context_line":"                                                         snap_model[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"dada55a8_524ef396","line":2985,"range":{"start_line":2985,"start_character":48,"end_line":2985,"end_character":54},"updated":"2016-07-23 14:28:11.000000000","message":"Same.","commit_id":"dfc9114ee43a813d1bc4295264a8105ba1da8281"},{"author":{"_account_id":17564,"name":"jiamin","email":"shljia@cn.ibm.com","username":"jiamin"},"change_message_id":"96970927d4a89cf1a44ddbdbc16b5a8d43f397cf","unresolved":false,"context_lines":[{"line_number":2982,"context_line":"                    # Update db for snapshot."},{"line_number":2983,"context_line":"                    # NOTE(xyang): snapshots is a list of snapshot objects."},{"line_number":2984,"context_line":"                    # snapshots_model_update should be a list of dicts."},{"line_number":2985,"context_line":"                    if snap_model.get(\u0027metadata\u0027, None):"},{"line_number":2986,"context_line":"                        metadata \u003d snap_model.pop(\u0027metadata\u0027)"},{"line_number":2987,"context_line":"                        self.db.snapshot_metadata_update(context,"},{"line_number":2988,"context_line":"                                                         snap_model[\u0027id\u0027],"}],"source_content_type":"text/x-python","patch_set":1,"id":"dada55a8_d7076cc7","line":2985,"range":{"start_line":2985,"start_character":48,"end_line":2985,"end_character":54},"in_reply_to":"dada55a8_524ef396","updated":"2016-07-24 09:09:02.000000000","message":"Thanks Sean.","commit_id":"dfc9114ee43a813d1bc4295264a8105ba1da8281"}]}