)]}'
{"dragonflow/controller/common/utils.py":[{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"6fd5b988c1df011a033616a01baee9946147e457","unresolved":false,"context_lines":[{"line_number":97,"context_line":"        LOG.exception(_LE(\"Failed execute conntrack command %s\"), cmd)"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"def get_port_match_list_from_port_range(port_range_min, port_range_max):"},{"line_number":101,"context_line":"    port_range \u003d netaddr.IPRange(port_range_min, port_range_max)"},{"line_number":102,"context_line":"    ports_match_list \u003d []"},{"line_number":103,"context_line":"    for cidr in port_range.cidrs():"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a30ddce_683c2d30","line":100,"range":{"start_line":100,"start_character":4,"end_line":100,"end_character":39},"updated":"2017-03-07 09:51:09.000000000","message":"This function is part of a group of functions in SG that belong in a specific library file. Maybe move all of them to a \u0027RangeToMasked\u0027 library, including testing?\n\nDoing it method by method is also fine, but then there needs to be a bug to track the effort. And this patch should move the function to its final place.","commit_id":"980e22868740c0b25a9ffdc2a6775f7e40631eb0"}],"dragonflow/tests/unit/test_common.py":[{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"4a8d9cb2bc8dbda10bc3ff13fd34ed241c2da6a2","unresolved":false,"context_lines":[{"line_number":97,"context_line":"        expected_port_match_list \u003d [(20, 0xfffc), (24, 0xfffc), (28, 0xfffe),"},{"line_number":98,"context_line":"                                    (30, 0xffff)]"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        self.assertItemsEqual(port_match_list, expected_port_match_list)"}],"source_content_type":"text/x-python","patch_set":11,"id":"dfeb2761_25cecdb3","line":100,"range":{"start_line":100,"start_character":30,"end_line":100,"end_character":71},"updated":"2017-04-05 11:14:01.000000000","message":"nit: I think the order of arguments should be reversed: First expected, then actual.","commit_id":"b45304101957e6dd79c53f5ee0003843de7d502a"}]}
