)]}'
{"neutron/agent/linux/ip_conntrack.py":[{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"1534aa40a6ccbb8b1838b19e6dd610b5d89036c3","unresolved":false,"context_lines":[{"line_number":48,"context_line":"                 unfiltered_ports, execute\u003dNone, namespace\u003dNone,"},{"line_number":49,"context_line":"                 zone_per_port\u003dFalse):"},{"line_number":50,"context_line":"        self.get_rules_for_table_func \u003d get_rules_for_table_func"},{"line_number":51,"context_line":"        self._executor \u003d self._load_conntrack_driver("},{"line_number":52,"context_line":"            cfg.CONF.SECURITYGROUP.conntrack_driver)"},{"line_number":53,"context_line":"        self.execute \u003d execute or self._executor"},{"line_number":54,"context_line":"        self.namespace \u003d namespace"},{"line_number":55,"context_line":"        self.filtered_ports \u003d filtered_ports"},{"line_number":56,"context_line":"        self.unfiltered_ports \u003d unfiltered_ports"}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_ea4a85aa","line":53,"range":{"start_line":51,"start_character":0,"end_line":53,"end_character":48},"updated":"2017-06-07 04:08:20.000000000","message":"IMO, It look not so good to me. How about:\n\ndef ConntrackUtils(object):\n    def delete_conntrack_state():\n        pass\n\ndef ConntrackCMD(ConntrackUtils):\n    def delete_conntrack_state():\n        pass\n\ndef ConntrackNetlink(ConntrackUtils):  \n    def delete_conntrack_state()\n        pass\n\n?","commit_id":"420f3e3d583575e92d2871e55be9a6f64290564f"}],"neutron/conf/agent/securitygroups_rpc.py":[{"author":{"_account_id":15471,"name":"Nguyen Phuong An","email":"annp.cs51@gmail.com","username":"An-FVL"},"change_message_id":"ed0ee51488f8e80548c18eeb7618fe25ebc1b5c3","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        help\u003d_(\u0027Use ipset to speed-up the iptables based security groups. \u0027"},{"line_number":38,"context_line":"               \u0027Enabling ipset support requires that ipset is installed on L2 \u0027"},{"line_number":39,"context_line":"               \u0027agent node.\u0027)),"},{"line_number":40,"context_line":"    cfg.StrOpt("},{"line_number":41,"context_line":"        \u0027conntrack_driver\u0027,"},{"line_number":42,"context_line":"        default\u003d\u0027conntrack_cmd\u0027,"},{"line_number":43,"context_line":"        help\u003d_(\u0027Driver for conntrack to manage conntrack entries\u0027))"},{"line_number":44,"context_line":"]"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_6ae0b5fb","line":43,"range":{"start_line":40,"start_character":0,"end_line":43,"end_character":67},"updated":"2017-06-07 03:47:58.000000000","message":"IMO, It should be in IpConntrackManager\u0027s configuration!","commit_id":"420f3e3d583575e92d2871e55be9a6f64290564f"}]}