)]}'
{"neutron/agent/l3/router_info.py":[{"author":{"_account_id":8124,"name":"cbrandily","email":"zzelle@gmail.com","username":"cbrandily"},"change_message_id":"7a8064c5ece375fb02fa5ce8490746d2be4c74b3","unresolved":false,"context_lines":[{"line_number":539,"context_line":""},{"line_number":540,"context_line":"    def external_gateway_nat_rules(self, ex_gw_ip, interface_name):"},{"line_number":541,"context_line":"        dont_snat_internal_traffic_if_not_to_floating_ip \u003d ("},{"line_number":542,"context_line":"            \u0027POSTROUTING\u0027, \u0027! -i %(interface_name)s \u0027"},{"line_number":543,"context_line":"                           \u0027! -o %(interface_name)s -m conntrack ! \u0027"},{"line_number":544,"context_line":"                           \u0027--ctstate DNAT -j ACCEPT\u0027 %"},{"line_number":545,"context_line":"                           {\u0027interface_name\u0027: interface_name})"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a41bdd9_dea202fd","line":542,"updated":"2015-07-14 22:59:59.000000000","message":"-i option is not supported by POSTROUTING so the variable name seems incorrect as the rule also applies to external-\u003einternal snat traffic","commit_id":"55cddf5f1f35b2bd3b0a89139d6b448b3894c01f"},{"author":{"_account_id":7448,"name":"Carl Baldwin","email":"carl@ecbaldwin.net","username":"carl-baldwin"},"change_message_id":"649286269d26921a8645e69c39c44c24dca4df4c","unresolved":false,"context_lines":[{"line_number":539,"context_line":""},{"line_number":540,"context_line":"    def external_gateway_nat_rules(self, ex_gw_ip, interface_name):"},{"line_number":541,"context_line":"        dont_snat_internal_traffic_if_not_to_floating_ip \u003d ("},{"line_number":542,"context_line":"            \u0027POSTROUTING\u0027, \u0027! -i %(interface_name)s \u0027"},{"line_number":543,"context_line":"                           \u0027! -o %(interface_name)s -m conntrack ! \u0027"},{"line_number":544,"context_line":"                           \u0027--ctstate DNAT -j ACCEPT\u0027 %"},{"line_number":545,"context_line":"                           {\u0027interface_name\u0027: interface_name})"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a41bdd9_3eea0edc","line":542,"in_reply_to":"9a41bdd9_dea202fd","updated":"2015-07-14 23:07:30.000000000","message":"I know, I considered removing it in the rule but I decided that was out of the scope of the patch.  What do others think?\n\nI\u0027ll try to find a better variable name which I think is what you were suggesting anyway.","commit_id":"55cddf5f1f35b2bd3b0a89139d6b448b3894c01f"}]}