)]}'
{"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"fe42a10edded1d4f1f3b68ff3f60f57e0f92a1d1","unresolved":true,"context_lines":[{"line_number":115,"context_line":""},{"line_number":116,"context_line":"    # TODO(ihrachys) remove when min OVN version \u003e\u003d 20.03"},{"line_number":117,"context_line":"    def is_stateless_nat_supported(self):"},{"line_number":118,"context_line":"        return self._nb_idl.is_col_present(\u0027NAT\u0027, \u0027options\u0027)"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    def _get_allowed_addresses_from_port(self, port):"},{"line_number":121,"context_line":"        if not port.get(psec.PORTSECURITY):"}],"source_content_type":"text/x-python","patch_set":4,"id":"b4bb0fff_d3b48b1a","line":118,"updated":"2021-09-09 19:59:05.000000000","message":"are we going to backport it to stable branches too? If not I don\u0027t think we need that check here. According to https://governance.openstack.org/tc/reference/project-testing-interface.html we should be testing (supporting) latest LTS version of Ubuntu (which is 20.04 now) and latest Centos (which is 8).\nIn Ubuntu shipped ovn is 20.03 - https://packages.ubuntu.com/focal-updates/ovn-central\nIn Centos IIUC, ovn is provided by RDO and it also should be at least 20.03 I guess.","commit_id":"278f66ce448fd39b764ceee948c44057b41ae1d4"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"ae13b0a134d55456a8ffa993ffbc25bea2c85e55","unresolved":false,"context_lines":[{"line_number":115,"context_line":""},{"line_number":116,"context_line":"    # TODO(ihrachys) remove when min OVN version \u003e\u003d 20.03"},{"line_number":117,"context_line":"    def is_stateless_nat_supported(self):"},{"line_number":118,"context_line":"        return self._nb_idl.is_col_present(\u0027NAT\u0027, \u0027options\u0027)"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    def _get_allowed_addresses_from_port(self, port):"},{"line_number":121,"context_line":"        if not port.get(psec.PORTSECURITY):"}],"source_content_type":"text/x-python","patch_set":4,"id":"c1264a10_70a9cda7","line":118,"in_reply_to":"b4bb0fff_d3b48b1a","updated":"2021-09-10 00:23:11.000000000","message":"I may miss something but I find \"testing environment definition\" not equals \"supported environments\". Even if we test under environments that all satisfy the minimal version requirements, unless we explicitly define that Neutron won\u0027t work with some specific older OVN versions, we may have to continue adding these runtime checks.\n\nThe last time I checked, there was no explicit claim made anywhere about the minimal OVN version required (nor OVS). Perhaps I missed it; or perhaps the project should define one. But if it\u0027s not defined, then, as I said, runtime checks should proliferate. (IMHO. Let me know if the official project stance is different.)","commit_id":"278f66ce448fd39b764ceee948c44057b41ae1d4"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"f87eb44c5714c33ff6a9067de6a52a90622f6260","unresolved":false,"context_lines":[{"line_number":115,"context_line":""},{"line_number":116,"context_line":"    # TODO(ihrachys) remove when min OVN version \u003e\u003d 20.03"},{"line_number":117,"context_line":"    def is_stateless_nat_supported(self):"},{"line_number":118,"context_line":"        return self._nb_idl.is_col_present(\u0027NAT\u0027, \u0027options\u0027)"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    def _get_allowed_addresses_from_port(self, port):"},{"line_number":121,"context_line":"        if not port.get(psec.PORTSECURITY):"}],"source_content_type":"text/x-python","patch_set":4,"id":"63e9e68e_99dc0ea9","line":118,"in_reply_to":"c1264a10_70a9cda7","updated":"2021-09-10 17:24:50.000000000","message":"You\u0027re right. We never really defined strictly versions with which Neutron will work or not work. So let\u0027s keep it like that at least for now.","commit_id":"278f66ce448fd39b764ceee948c44057b41ae1d4"}]}