)]}'
{"nova/api/openstack/compute/security_groups.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d7fdd962c778a9787b6da73253b09c0dcc5d0d58","unresolved":false,"context_lines":[{"line_number":527,"context_line":"    def server_create(self, server_dict, create_kwargs, body_deprecated_param):"},{"line_number":528,"context_line":"        security_groups \u003d server_dict.get(ATTRIBUTE_NAME)"},{"line_number":529,"context_line":"        if security_groups is not None:"},{"line_number":530,"context_line":"            create_kwargs[\u0027security_groups\u0027] \u003d ["},{"line_number":531,"context_line":"                sg[\u0027name\u0027] for sg in security_groups if sg.get(\u0027name\u0027)]"},{"line_number":532,"context_line":"            create_kwargs[\u0027security_groups\u0027] \u003d list("},{"line_number":533,"context_line":"                set(create_kwargs[\u0027security_groups\u0027]))"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_9f384e3f","line":530,"updated":"2016-11-09 20:25:52.000000000","message":"I\u0027m not totally sure how kosher this is. In newton we killed API extensions, so nothing in the request chain should be relying on this, anyone with out of tree extensions is going to have to deal with the pain. So I\u0027d think this is OK as it\u0027s all internal plumbing for nova, but not really sure.","commit_id":"d70baac537623f2b7424700572c028ad00b22101"}]}
