)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"17be25d9891c9fbd6ee6e685eaa440d40f252c0a","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"ConsistencyGroup object was missing relationships that were defined on"},{"line_number":10,"context_line":"the SQLAlchemy model as backrefs. This includes volumes and cgsnapshots"},{"line_number":11,"context_line":"fields. This commit adds fields representing these relationships."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ia74997ddfed1e472874b6892c2fcaa208b7a9a3c"},{"line_number":14,"context_line":"Partial-Bug: 1509012"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa69d971_b19edf05","line":11,"updated":"2015-12-29 21:30:20.000000000","message":"For the same reason, should snapshots be added to the Volume object?  I\u0027m just trying to understand when it is needed and when it is not.\n\n    volume \u003d relationship(Volume, backref\u003d\"snapshots\",\n                          foreign_keys\u003dvolume_id,\n                          primaryjoin\u003d\u0027Snapshot.volume_id \u003d\u003d Volume.id\u0027)","commit_id":"b3f37476546d6170368138c6fe1e9bb176536ef8"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"4a7f73038ca55d51804edd76b2abae3b5d0ecc7f","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"ConsistencyGroup object was missing relationships that were defined on"},{"line_number":10,"context_line":"the SQLAlchemy model as backrefs. This includes volumes and cgsnapshots"},{"line_number":11,"context_line":"fields. This commit adds fields representing these relationships."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ia74997ddfed1e472874b6892c2fcaa208b7a9a3c"},{"line_number":14,"context_line":"Partial-Bug: 1509012"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa69d971_f8451317","line":11,"in_reply_to":"fa69d971_b19edf05","updated":"2015-12-30 07:55:17.000000000","message":"Your understanding is correct and `snapshots` field is already there [1]. There were several patches adding missing relationships.\n\n[1] https://github.com/openstack/cinder/blame/master/cinder/objects/volume.py#L97","commit_id":"b3f37476546d6170368138c6fe1e9bb176536ef8"}]}