)]}'
{"neutron/agent/linux/openvswitch_firewall/firewall.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"63e956547caa5fc0b98575ab2a6fd220dacccc16","unresolved":true,"context_lines":[{"line_number":900,"context_line":""},{"line_number":901,"context_line":"    def _initialize_egress_ipv6_icmp(self, port, allowed_pairs):"},{"line_number":902,"context_line":"        # NOTE(slaweq): should we include also fe80::/64 (link-local) subnet"},{"line_number":903,"context_line":"        # in the allowed pairs here?"},{"line_number":904,"context_line":"        for mac_addr, ip_addr in allowed_pairs:"},{"line_number":905,"context_line":"            for icmp_type in firewall.ICMPV6_ALLOWED_EGRESS_TYPES:"},{"line_number":906,"context_line":"                self._add_flow("}],"source_content_type":"text/x-python","patch_set":1,"id":"e1699994_53276793","line":903,"updated":"2021-02-19 15:00:51.000000000","message":"That\u0027s a good question. I would have thought that not having it would affect NS/NA from working but the tests passed.","commit_id":"4b5bcff64c4725b37f094dfd2613ec58f723d304"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"2e7e835cbee2771f0f7172c6b7c13eb5d5d665b6","unresolved":true,"context_lines":[{"line_number":900,"context_line":""},{"line_number":901,"context_line":"    def _initialize_egress_ipv6_icmp(self, port, allowed_pairs):"},{"line_number":902,"context_line":"        # NOTE(slaweq): should we include also fe80::/64 (link-local) subnet"},{"line_number":903,"context_line":"        # in the allowed pairs here?"},{"line_number":904,"context_line":"        for mac_addr, ip_addr in allowed_pairs:"},{"line_number":905,"context_line":"            for icmp_type in firewall.ICMPV6_ALLOWED_EGRESS_TYPES:"},{"line_number":906,"context_line":"                self._add_flow("}],"source_content_type":"text/x-python","patch_set":1,"id":"8d716242_6b0f9fcf","line":903,"in_reply_to":"c86ff5a6_3de042ad","updated":"2021-02-26 10:52:19.000000000","message":"+1","commit_id":"4b5bcff64c4725b37f094dfd2613ec58f723d304"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"92e5b2cace93d2f47d7f76060ef50c82e0ec2f5c","unresolved":true,"context_lines":[{"line_number":900,"context_line":""},{"line_number":901,"context_line":"    def _initialize_egress_ipv6_icmp(self, port, allowed_pairs):"},{"line_number":902,"context_line":"        # NOTE(slaweq): should we include also fe80::/64 (link-local) subnet"},{"line_number":903,"context_line":"        # in the allowed pairs here?"},{"line_number":904,"context_line":"        for mac_addr, ip_addr in allowed_pairs:"},{"line_number":905,"context_line":"            for icmp_type in firewall.ICMPV6_ALLOWED_EGRESS_TYPES:"},{"line_number":906,"context_line":"                self._add_flow("}],"source_content_type":"text/x-python","patch_set":1,"id":"c86ff5a6_3de042ad","line":903,"in_reply_to":"e1699994_53276793","updated":"2021-02-22 08:46:45.000000000","message":"yes, I also didn\u0027t noticed anything wrong locally so for now I would say, let\u0027s keep it like that and we can change if that will be needed someday. Wdyt?","commit_id":"4b5bcff64c4725b37f094dfd2613ec58f723d304"}]}