)]}'
{"neutron/db/securitygroups_db.py":[{"author":{"_account_id":17120,"name":"Manjeet Singh Bhatia","email":"manjeet.s.bhatia@intel.com","username":"manjeets"},"change_message_id":"cf181c0515602582faf33d7f2a681bc11c21278c","unresolved":false,"context_lines":[{"line_number":122,"context_line":"            # Expunge it to ensure the following get_object doesn\u0027t"},{"line_number":123,"context_line":"            # use the instance."},{"line_number":124,"context_line":"            context.session.expunge_all()"},{"line_number":125,"context_line":"            sg \u003d sg_obj.SecurityGroup.get_object(context, id\u003dsg.id)"},{"line_number":126,"context_line":"            secgroup_dict \u003d self._make_security_group_dict(sg)"},{"line_number":127,"context_line":"            kwargs[\u0027security_group\u0027] \u003d secgroup_dict"},{"line_number":128,"context_line":"            self._registry_notify(resources.SECURITY_GROUP,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_ccf5b6cc","line":125,"range":{"start_line":125,"start_character":12,"end_line":125,"end_character":67},"updated":"2017-06-19 23:14:21.000000000","message":"this is basically done to load rules with sg.db_obj, I think sg will still have rules as attribute, so this fetch may not be needed, testing it on https://review.openstack.org/#/c/475468/ looks like there no issue, unless any decomposed plugin is broken.","commit_id":"08b29f02db00fd2bf0a45087fe449211801cbc1c"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"8b8e701f8c13dd830be98d5a2c2ea262242a5531","unresolved":false,"context_lines":[{"line_number":122,"context_line":"            # Expunge it to ensure the following get_object doesn\u0027t"},{"line_number":123,"context_line":"            # use the instance."},{"line_number":124,"context_line":"            context.session.expunge_all()"},{"line_number":125,"context_line":"            sg \u003d sg_obj.SecurityGroup.get_object(context, id\u003dsg.id)"},{"line_number":126,"context_line":"            secgroup_dict \u003d self._make_security_group_dict(sg)"},{"line_number":127,"context_line":"            kwargs[\u0027security_group\u0027] \u003d secgroup_dict"},{"line_number":128,"context_line":"            self._registry_notify(resources.SECURITY_GROUP,"}],"source_content_type":"text/x-python","patch_set":1,"id":"7f231b9d_11e29b02","line":125,"range":{"start_line":125,"start_character":12,"end_line":125,"end_character":67},"in_reply_to":"7f231b9d_ccf5b6cc","updated":"2017-06-20 02:31:09.000000000","message":"it might be. your patch doesn\u0027t seem to have any tests though.","commit_id":"08b29f02db00fd2bf0a45087fe449211801cbc1c"}]}