)]}'
{"nova/objects/instance.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"2b5125d456edf5cfd73bee5e75780964885301ba","unresolved":false,"context_lines":[{"line_number":230,"context_line":""},{"line_number":231,"context_line":"    def _save_security_groups(self, context):"},{"line_number":232,"context_line":"        for secgroup in self.security_groups:"},{"line_number":233,"context_line":"            secgroup.save(context)"},{"line_number":234,"context_line":""},{"line_number":235,"context_line":"    @base.remotable"},{"line_number":236,"context_line":"    def save(self, context, expected_task_state\u003dNone):"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAPn%2F%2FqVk%3D","line":233,"updated":"2013-06-25 18:50:35.000000000","message":"as discussed:  save_all() on lists... and call that here.","commit_id":"eec6ddcfaf2af215a0e041ab1b7bbcce8ebb5aad"},{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"2b5125d456edf5cfd73bee5e75780964885301ba","unresolved":false,"context_lines":[{"line_number":249,"context_line":"        for field in self.fields:"},{"line_number":250,"context_line":"            if (hasattr(self, base.get_attrname(field)) and"},{"line_number":251,"context_line":"                isinstance(self[field], base.NovaObject)):"},{"line_number":252,"context_line":"                getattr(self, \u0027_save_%s\u0027 % field)(context)"},{"line_number":253,"context_line":"            elif field in changes:"},{"line_number":254,"context_line":"                updates[field] \u003d self[field]"},{"line_number":255,"context_line":"        if expected_task_state is not None:"}],"source_content_type":"text/x-python","patch_set":10,"id":"AAAAPn%2F%2FqVc%3D","line":252,"updated":"2013-06-25 18:50:35.000000000","message":"Just to put my comments in the review vs IRC...  I kinda wish we didn\u0027t need the special _save_* stuff here..  which we could avoid if we gave special meaning to *List.save().  But I also understand the reasons for NOT doing that.","commit_id":"eec6ddcfaf2af215a0e041ab1b7bbcce8ebb5aad"}]}