)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"810706197531e066bd91cde74e7897abef905114","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Swaminathan Vasudevan \u003cswaminathan.vasudevan@hp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-19 16:48:28 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP: Debug patch to test the gate failures on SSHTimeOut"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is a test patch to check the gate failures on SSHTimeOut."},{"line_number":10,"context_line":"The intent of this patch is to add some sort of self test with"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba8a016a_a52f06a0","line":7,"updated":"2015-11-24 01:07:09.000000000","message":"can you take the WIP out?","commit_id":"2dadf91fe6cb27070daa170f7cce983499a60a3c"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"255807e342188a64faa645892605b1ccf89cf7ce","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Swaminathan Vasudevan \u003cswaminathan.vasudevan@hp.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-19 16:48:28 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP: Debug patch to test the gate failures on SSHTimeOut"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is a test patch to check the gate failures on SSHTimeOut."},{"line_number":10,"context_line":"The intent of this patch is to add some sort of self test with"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a8ffd7b_eb13f55a","line":7,"in_reply_to":"ba8a016a_a52f06a0","updated":"2015-11-24 19:11:40.000000000","message":"yes I will remove the WiP out.","commit_id":"2dadf91fe6cb27070daa170f7cce983499a60a3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"810706197531e066bd91cde74e7897abef905114","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This is a test patch to check the gate failures on SSHTimeOut."},{"line_number":10,"context_line":"The intent of this patch is to add some sort of self test with"},{"line_number":11,"context_line":"in the agent to make sure that the path between the fip and"},{"line_number":12,"context_line":"router namespace is connected and reachable."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1ed5c815fde66845f84d92382b6f07a9f443b393"},{"line_number":15,"context_line":"Related-Bug: #1515360"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ba8a016a_e50c2e44","line":12,"updated":"2015-11-24 01:07:09.000000000","message":"I\u0027d say let\u0027s merge this as is, so that we can collect more data point. We should then issue a revert before the release.\n\nAnother less invasive approach would be to have this config driven and then have the dvr job set the config by default, but the impact caused by the ping is minimal and it doesn\u0027t seem justified the extra pain to involve changes to d-g.","commit_id":"2dadf91fe6cb27070daa170f7cce983499a60a3c"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"255807e342188a64faa645892605b1ccf89cf7ce","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This is a test patch to check the gate failures on SSHTimeOut."},{"line_number":10,"context_line":"The intent of this patch is to add some sort of self test with"},{"line_number":11,"context_line":"in the agent to make sure that the path between the fip and"},{"line_number":12,"context_line":"router namespace is connected and reachable."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1ed5c815fde66845f84d92382b6f07a9f443b393"},{"line_number":15,"context_line":"Related-Bug: #1515360"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a8ffd7b_4b5c8125","line":12,"in_reply_to":"ba8a016a_e50c2e44","updated":"2015-11-24 19:11:40.000000000","message":"I still see some issues with the \"ping\" not happening with the existing code as such.\nI am in the process of debugging it. \nAny help on that area would be appreciated.","commit_id":"2dadf91fe6cb27070daa170f7cce983499a60a3c"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"953aa24ce98c53805612b07ba9e6f6120995dc7a","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This is a debug patch to check the gate failures on SSHTimeOut."},{"line_number":10,"context_line":"The intent of this patch is to add some sort of self test with"},{"line_number":11,"context_line":"in the agent to make sure that the path between the fip and"},{"line_number":12,"context_line":"router namespace is connected and reachable."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1ed5c815fde66845f84d92382b6f07a9f443b393"},{"line_number":15,"context_line":"Related-Bug: #1515360"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a8ffd7b_60a15294","line":12,"updated":"2015-11-25 03:46:41.000000000","message":"I\u0027d state that the intention here is to remove this probe once code instability is addressed.","commit_id":"be8157586b25930db684e0962ae7d03e210fe613"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"953aa24ce98c53805612b07ba9e6f6120995dc7a","unresolved":false,"context_lines":[{"line_number":12,"context_line":"router namespace is connected and reachable."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I1ed5c815fde66845f84d92382b6f07a9f443b393"},{"line_number":15,"context_line":"Related-Bug: #1515360"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a8ffd7b_e0946238","line":15,"updated":"2015-11-25 03:46:41.000000000","message":"sorry, I forgot to post this comment on PS2, the bug is invalid IMO, so you might as well get rid of this.","commit_id":"be8157586b25930db684e0962ae7d03e210fe613"}],"etc/neutron/rootwrap.d/l3.filters":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fd9454375b0874915f10ab0347a6ab34b7b00619","unresolved":false,"context_lines":[{"line_number":29,"context_line":"find: RegExpFilter, find, root, find, /sys/class/net, -maxdepth, 1, -type, l, -printf, %.*"},{"line_number":30,"context_line":"ip_exec: IpNetnsExecFilter, ip, root"},{"line_number":31,"context_line":"arp: RegExpFilter, arp, root, arp, -a"},{"line_number":32,"context_line":"ping: RegExpFilter, ping, root, ping, -w, \\d+, -c, \\d+, [0-9\\.]+"},{"line_number":33,"context_line":"ping_alt: RegExpFilter, ping, root, ping, -c, \\d+, -w, \\d+, [0-9\\.]+"},{"line_number":34,"context_line":"ping6: RegExpFilter, ping6, root, ping6, -w, \\d+, -c, \\d+, [0-9A-Fa-f:]+"},{"line_number":35,"context_line":"ping6_alt: RegExpFilter, ping6, root, ping6, -c, \\d+, -w, \\d+, [0-9A-Fa-f:]+"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"9a8ffd7b_114d0d99","line":32,"updated":"2015-12-01 20:18:55.000000000","message":"Change this to CommandFilter like L16 so you don\u0027t need the regex, or am I missing something?","commit_id":"055fafb1c0afabee31edaaed7de072201745695a"}],"neutron/agent/l3/dvr_local_router.py":[{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"a07e44cfc4985399d6927488d7861cd79ccfee60","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        # sure that the FIP is reachable. This is a just for debug"},{"line_number":112,"context_line":"        # purpose and can be removed later."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        # Check if we can ping the private ip from the router namespace"},{"line_number":115,"context_line":"        try:"},{"line_number":116,"context_line":"            self._fip_path_validate(self.ns_name, fixed_ip, timeout\u003d2, count\u003d4)"},{"line_number":117,"context_line":"        except Exception:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_7d3387f6","line":114,"updated":"2015-11-19 20:06:01.000000000","message":"Do we need to setup any security group rules before attempting to do this?","commit_id":"b4dfafd3c3754f04c5cadb6029d6b6dae8151ede"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"5f07d773667c4d6a65d1da361f41a81b02a30b03","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        # sure that the FIP is reachable. This is a just for debug"},{"line_number":112,"context_line":"        # purpose and can be removed later."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        # Check if we can ping the private ip from the router namespace"},{"line_number":115,"context_line":"        try:"},{"line_number":116,"context_line":"            self._fip_path_validate(self.ns_name, fixed_ip, timeout\u003d2, count\u003d4)"},{"line_number":117,"context_line":"        except Exception:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_9b44304a","line":114,"in_reply_to":"ba8a016a_0eaa4385","updated":"2015-11-19 22:05:57.000000000","message":"Yes I think the gate jobs are already setting up the right credentials.","commit_id":"b4dfafd3c3754f04c5cadb6029d6b6dae8151ede"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"976a1d49a747ecb012dc99435da1ac343d158143","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        # sure that the FIP is reachable. This is a just for debug"},{"line_number":112,"context_line":"        # purpose and can be removed later."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        # Check if we can ping the private ip from the router namespace"},{"line_number":115,"context_line":"        try:"},{"line_number":116,"context_line":"            self._fip_path_validate(self.ns_name, fixed_ip, timeout\u003d2, count\u003d4)"},{"line_number":117,"context_line":"        except Exception:"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_0eaa4385","line":114,"in_reply_to":"ba8a016a_7d3387f6","updated":"2015-11-19 21:29:23.000000000","message":"Probably, unless the gate jobs are already doing that.","commit_id":"b4dfafd3c3754f04c5cadb6029d6b6dae8151ede"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"976a1d49a747ecb012dc99435da1ac343d158143","unresolved":false,"context_lines":[{"line_number":118,"context_line":"            LOG.debug(\"Private IP %(private_ip)s not reachable from \""},{"line_number":119,"context_line":"                      \"router namespace %(namespace)s\","},{"line_number":120,"context_line":"                      {\u0027private_ip\u0027: fixed_ip,"},{"line_number":121,"context_line":"                       \u0027namespace\u0027: self.ns_name})"},{"line_number":122,"context_line":"        # Check if we can ping the floating ip from the fip namespace"},{"line_number":123,"context_line":"        try:"},{"line_number":124,"context_line":"            self._fip_path_validate("}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_8edab315","line":121,"updated":"2015-11-19 21:29:23.000000000","message":"Is there anything else we want to gather here or on L127 like a snapshot of the namespace interfaces or routes?","commit_id":"b4dfafd3c3754f04c5cadb6029d6b6dae8151ede"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"79848977025b3074de842119ea015b70dc199f77","unresolved":false,"context_lines":[{"line_number":118,"context_line":"            LOG.debug(\"Private IP %(private_ip)s not reachable from \""},{"line_number":119,"context_line":"                      \"router namespace %(namespace)s\","},{"line_number":120,"context_line":"                      {\u0027private_ip\u0027: fixed_ip,"},{"line_number":121,"context_line":"                       \u0027namespace\u0027: self.ns_name})"},{"line_number":122,"context_line":"        # Check if we can ping the floating ip from the fip namespace"},{"line_number":123,"context_line":"        try:"},{"line_number":124,"context_line":"            self._fip_path_validate("}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_85528933","line":121,"in_reply_to":"ba8a016a_8edab315","updated":"2015-11-20 00:52:30.000000000","message":"How do I dump the namespace data in here?","commit_id":"b4dfafd3c3754f04c5cadb6029d6b6dae8151ede"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"5f07d773667c4d6a65d1da361f41a81b02a30b03","unresolved":false,"context_lines":[{"line_number":118,"context_line":"            LOG.debug(\"Private IP %(private_ip)s not reachable from \""},{"line_number":119,"context_line":"                      \"router namespace %(namespace)s\","},{"line_number":120,"context_line":"                      {\u0027private_ip\u0027: fixed_ip,"},{"line_number":121,"context_line":"                       \u0027namespace\u0027: self.ns_name})"},{"line_number":122,"context_line":"        # Check if we can ping the floating ip from the fip namespace"},{"line_number":123,"context_line":"        try:"},{"line_number":124,"context_line":"            self._fip_path_validate("}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_5b61e8d1","line":121,"in_reply_to":"ba8a016a_8edab315","updated":"2015-11-19 22:05:57.000000000","message":"If the ping succeeds we don\u0027t think we need anything.\nMay be if the ping fails, we can try and dump the namespace data in here.","commit_id":"b4dfafd3c3754f04c5cadb6029d6b6dae8151ede"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"976a1d49a747ecb012dc99435da1ac343d158143","unresolved":false,"context_lines":[{"line_number":133,"context_line":"        ipversion \u003d netaddr.IPAddress(dest_ip).version"},{"line_number":134,"context_line":"        ping_command \u003d \u0027ping\u0027 if ipversion \u003d\u003d 4 else \u0027ping6\u0027"},{"line_number":135,"context_line":"        ns_ip_wrapper \u003d ip_lib.IPWrapper(namespace)"},{"line_number":136,"context_line":"        ns_ip_wrapper.netns.execute([ping_command, \u0027-c\u0027, count, \u0027-W\u0027, timeout,"},{"line_number":137,"context_line":"                                     dest_ip])"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    def floating_ip_removed_dist(self, fip_cidr):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_0e81e3fa","line":136,"updated":"2015-11-19 21:29:23.000000000","message":"add -n too do not involve DNS","commit_id":"b4dfafd3c3754f04c5cadb6029d6b6dae8151ede"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"5f07d773667c4d6a65d1da361f41a81b02a30b03","unresolved":false,"context_lines":[{"line_number":133,"context_line":"        ipversion \u003d netaddr.IPAddress(dest_ip).version"},{"line_number":134,"context_line":"        ping_command \u003d \u0027ping\u0027 if ipversion \u003d\u003d 4 else \u0027ping6\u0027"},{"line_number":135,"context_line":"        ns_ip_wrapper \u003d ip_lib.IPWrapper(namespace)"},{"line_number":136,"context_line":"        ns_ip_wrapper.netns.execute([ping_command, \u0027-c\u0027, count, \u0027-W\u0027, timeout,"},{"line_number":137,"context_line":"                                     dest_ip])"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    def floating_ip_removed_dist(self, fip_cidr):"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba8a016a_3ba184f8","line":136,"in_reply_to":"ba8a016a_0e81e3fa","updated":"2015-11-19 22:05:57.000000000","message":"will add it.","commit_id":"b4dfafd3c3754f04c5cadb6029d6b6dae8151ede"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"a706cae26ee6416b64f074dec4c1c3389372c221","unresolved":false,"context_lines":[{"line_number":124,"context_line":"            self._fip_path_validate("},{"line_number":125,"context_line":"                fip_ns_name, floating_ip, timeout\u003d2, count\u003d4)"},{"line_number":126,"context_line":"        except Exception:"},{"line_number":127,"context_line":"            LOG.debug(\"Floating IP %(fip_ip)s not reachable from \""},{"line_number":128,"context_line":"                      \"fip namespace %(namespace)s\","},{"line_number":129,"context_line":"                      {\u0027fip_ip\u0027: floating_ip,"},{"line_number":130,"context_line":"                       \u0027namespace\u0027: fip_ns_name})"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba8a016a_f0af0ae1","line":127,"updated":"2015-11-24 01:11:40.000000000","message":"don\u0027t you want to dump the content of the namespace to see what happened? Otherwise perhaps this is not enough.","commit_id":"2dadf91fe6cb27070daa170f7cce983499a60a3c"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"255807e342188a64faa645892605b1ccf89cf7ce","unresolved":false,"context_lines":[{"line_number":124,"context_line":"            self._fip_path_validate("},{"line_number":125,"context_line":"                fip_ns_name, floating_ip, timeout\u003d2, count\u003d4)"},{"line_number":126,"context_line":"        except Exception:"},{"line_number":127,"context_line":"            LOG.debug(\"Floating IP %(fip_ip)s not reachable from \""},{"line_number":128,"context_line":"                      \"fip namespace %(namespace)s\","},{"line_number":129,"context_line":"                      {\u0027fip_ip\u0027: floating_ip,"},{"line_number":130,"context_line":"                       \u0027namespace\u0027: fip_ns_name})"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a8ffd7b_6b74e59b","line":127,"in_reply_to":"ba8a016a_f0af0ae1","updated":"2015-11-24 19:11:40.000000000","message":"Yes you are right, I will try to add in the namespace dump and arp dump as well.","commit_id":"2dadf91fe6cb27070daa170f7cce983499a60a3c"},{"author":{"_account_id":15070,"name":"Ruijing","email":"ruijing.guo@intel.com","username":"ruijing"},"change_message_id":"c2057aaf57b1e610d4f5aa27c88d60a2e678917c","unresolved":false,"context_lines":[{"line_number":104,"context_line":"                  \"sent from fip namespace: %s\", floating_ip)"},{"line_number":105,"context_line":"        # update internal structures"},{"line_number":106,"context_line":"        self.dist_fip_count \u003d self.dist_fip_count + 1"},{"line_number":107,"context_line":"        # NOTE: Let us let us try to do some self-test to make"},{"line_number":108,"context_line":"        # sure that the FIP is reachable. This is a just for debug"},{"line_number":109,"context_line":"        # purpose and can be removed later."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_a65e6de7","line":107,"updated":"2015-11-25 08:14:06.000000000","message":"We need to dynamic enable/disable debug code so that we don\u0027t need to remove this block","commit_id":"48c1e216f7e04ecd69a4a4d9ae49bdfe53e963e4"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"004e728b8ff13e77725941ee9876d4a2395dac29","unresolved":false,"context_lines":[{"line_number":104,"context_line":"                  \"sent from fip namespace: %s\", floating_ip)"},{"line_number":105,"context_line":"        # update internal structures"},{"line_number":106,"context_line":"        self.dist_fip_count \u003d self.dist_fip_count + 1"},{"line_number":107,"context_line":"        # NOTE: Let us let us try to do some self-test to make"},{"line_number":108,"context_line":"        # sure that the FIP is reachable. This is a just for debug"},{"line_number":109,"context_line":"        # purpose and can be removed later."},{"line_number":110,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9a8ffd7b_b912f507","line":107,"in_reply_to":"9a8ffd7b_a65e6de7","updated":"2015-11-25 14:40:14.000000000","message":"Yes we will include the conf option as mentioned by armando, once we figure out the real problem in the gate.","commit_id":"48c1e216f7e04ecd69a4a4d9ae49bdfe53e963e4"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fd9454375b0874915f10ab0347a6ab34b7b00619","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        ping_command \u003d \u0027ping\u0027 if ipversion \u003d\u003d 4 else \u0027ping6\u0027"},{"line_number":132,"context_line":"        ns_ip_wrapper \u003d ip_lib.IPWrapper(namespace)"},{"line_number":133,"context_line":"        ping_response \u003d ns_ip_wrapper.netns.execute("},{"line_number":134,"context_line":"            [ping_command, \u0027-w\u0027, timeout, \u0027-c\u0027, count, dest_ip],"},{"line_number":135,"context_line":"            check_exit_code\u003dFalse)"},{"line_number":136,"context_line":"        if (\u0027100% packet loss\u0027 in ping_response):"},{"line_number":137,"context_line":"            LOG.debug(\"DVR: Ping Response: %s\", ping_response)"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a8ffd7b_b1e761ab","line":134,"updated":"2015-12-01 20:18:55.000000000","message":"Should add a \u0027-n\u0027 here to not involve DNS.","commit_id":"055fafb1c0afabee31edaaed7de072201745695a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fd9454375b0874915f10ab0347a6ab34b7b00619","unresolved":false,"context_lines":[{"line_number":139,"context_line":""},{"line_number":140,"context_line":"    def _dump_namespace(self, namespace):"},{"line_number":141,"context_line":"        ns_ip_wrapper \u003d ip_lib.IPWrapper(namespace)"},{"line_number":142,"context_line":"        arp_table \u003d ns_ip_wrapper.netns.execute([\u0027arp\u0027, \u0027-a\u0027])"},{"line_number":143,"context_line":"        LOG.debug(\"DVR: Arp entry: %s\", arp_table)"},{"line_number":144,"context_line":"        iptable_rules \u003d ns_ip_wrapper.netns.execute([\u0027iptables-save\u0027])"},{"line_number":145,"context_line":"        LOG.debug(\"DVR: IP Tables entry: %s\", iptable_rules)"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a8ffd7b_7126d917","line":142,"updated":"2015-12-01 20:18:55.000000000","message":"Should add a \u0027-n\u0027 here to not involve DNS, or maybe just use the neigh code in ip_lib?  I\u0027ll look into that today.","commit_id":"055fafb1c0afabee31edaaed7de072201745695a"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"fd9454375b0874915f10ab0347a6ab34b7b00619","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        ns_ip_wrapper \u003d ip_lib.IPWrapper(namespace)"},{"line_number":142,"context_line":"        arp_table \u003d ns_ip_wrapper.netns.execute([\u0027arp\u0027, \u0027-a\u0027])"},{"line_number":143,"context_line":"        LOG.debug(\"DVR: Arp entry: %s\", arp_table)"},{"line_number":144,"context_line":"        iptable_rules \u003d ns_ip_wrapper.netns.execute([\u0027iptables-save\u0027])"},{"line_number":145,"context_line":"        LOG.debug(\"DVR: IP Tables entry: %s\", iptable_rules)"},{"line_number":146,"context_line":"        ip_route \u003d ns_ip_wrapper.netns.execute([\u0027route\u0027])"},{"line_number":147,"context_line":"        LOG.debug(\"DVR: Route entry: %s\", ip_route)"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a8ffd7b_112e6dfa","line":144,"updated":"2015-12-01 20:18:55.000000000","message":"Should add a \u0027-c\u0027 here to print stats on the chains/rules","commit_id":"055fafb1c0afabee31edaaed7de072201745695a"}]}