)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"cb5edb66a6111c7009e52aaed1ee29c6a4fc609e","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Map OpenStack SG stateful\u003dFalse to OVN ACL allow-stateless action verb."},{"line_number":10,"context_line":"The verb is added in the latest OVN release, 21.06. Inspect db schema to"},{"line_number":11,"context_line":"determine if the new action is supported by OVN before trying to create"},{"line_number":12,"context_line":"it. Fall back to allow-related when it\u0027s not supported yet."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Also-Needs: I7343fb609fab91c20490842378747f7265241e82"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7ed1db73_955f2bf7","line":12,"updated":"2021-06-02 22:28:16.000000000","message":"we could probably also fall back to \u0027allow\u0027 which in some scenarios is also stateless (when no other allow-related ACLs defined on the network), but that\u0027s beyond the scope here and probably could be implemented independently.\n\nanother idea that could be explored is making an attempt to create a stateless SG with no OVN support to fail instead of silently swallowing it and falling back to allow-related. The swallowing is existing behavior and is also out of scope here.","commit_id":"bf94b32ca4750d984665d640976c9fb78c3c351b"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py":[{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"5f29655f6b75ac420eeb8f71af3420962b87e013","unresolved":true,"context_lines":[{"line_number":83,"context_line":"            self.segments_plugin \u003d ("},{"line_number":84,"context_line":"                manager.NeutronManager.load_class_for_provider("},{"line_number":85,"context_line":"                    \u0027neutron.service_plugins\u0027, \u0027segments\u0027)())"},{"line_number":86,"context_line":"        self._stateless_supported \u003d ("},{"line_number":87,"context_line":"            self._ovn_client.is_allow_stateless_supported()"},{"line_number":88,"context_line":"        )"},{"line_number":89,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"028348b8_926bbc80","line":86,"updated":"2021-06-09 18:12:27.000000000","message":"Is it possible that we could be updating OVN w/o restarting neutron? If so, then caching this would break on upgrade.","commit_id":"1dcc2105c216afb7fc9f1f07bd87f80cb5a9258b"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"cd8f92040fb5f655cb644bbcfbf42f7c50e1b4f6","unresolved":false,"context_lines":[{"line_number":83,"context_line":"            self.segments_plugin \u003d ("},{"line_number":84,"context_line":"                manager.NeutronManager.load_class_for_provider("},{"line_number":85,"context_line":"                    \u0027neutron.service_plugins\u0027, \u0027segments\u0027)())"},{"line_number":86,"context_line":"        self._stateless_supported \u003d ("},{"line_number":87,"context_line":"            self._ovn_client.is_allow_stateless_supported()"},{"line_number":88,"context_line":"        )"},{"line_number":89,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"e73af4c9_abe7e259","line":86,"in_reply_to":"028348b8_926bbc80","updated":"2021-06-14 21:05:53.000000000","message":"you are right. I didn\u0027t realize that this module is used beyond the sync CLI util (in which case caching would be fine).","commit_id":"1dcc2105c216afb7fc9f1f07bd87f80cb5a9258b"}]}