)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"c3eb7db5e84939d044caed450dcd4d0dfd96764a","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2021-05-06 16:26:31 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"When neutron ACL synchronization is added, the code of the OVN northbound database is mistakenly deleted"},{"line_number":8,"context_line":"Closes-Bug: #1927293"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I06f0572dd8058bc4aa59ebacda3bf3927a4277b1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"b17d6304_823d69c2","line":8,"updated":"2021-05-06 09:44:39.000000000","message":"The commit message needs to be edit according to the guidelines: https://wiki.openstack.org/wiki/GitCommitMessages#Git_Commit_Good_Practice","commit_id":"d16bf2aacb18b529a122fab3fa26efcf8c30e2f3"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"c3eb7db5e84939d044caed450dcd4d0dfd96764a","unresolved":true,"context_lines":[{"line_number":224,"context_line":""},{"line_number":225,"context_line":"    def _get_acls_from_port_groups(self, pg_name):"},{"line_number":226,"context_line":"        ovn_acls \u003d []"},{"line_number":227,"context_line":"        port_groups \u003d self.ovn_api.db_find_rows(\u0027Port_Group\u0027, (\u0027name\u0027, \u0027\u003d\u0027, pg_name)).execute()"},{"line_number":228,"context_line":"        for pg in port_groups:"},{"line_number":229,"context_line":"            acls \u003d getattr(pg, \u0027acls\u0027, [])"},{"line_number":230,"context_line":"            for acl in acls:"}],"source_content_type":"text/x-python","patch_set":2,"id":"8b233d34_ea22b1cb","line":227,"updated":"2021-05-06 09:44:39.000000000","message":"I would like to have a conversation with the team prior to merging something like this because as Rodolfo stated previously right now we assume that the OVN Northbound DB is only updated by Neutron itself and that\u0027s what this tool (OVN DB sync) does.\n\nIf you have other systems editing the OVN NB DB it\u0027s tough for us to guarantee it\u0027s consistency, also, the use of this script is not mandatory you can disable it [0] / not run it.\n\nCan you explain better the use case here and why do you need both things (other systems modifying the OVN NB DB and this sync script) ?\n\n[0] https://github.com/openstack/neutron/blob/603951809a45797697135e03eb4870c094371480/neutron/conf/plugins/ml2/drivers/ovn/ovn_conf.py#L90","commit_id":"d16bf2aacb18b529a122fab3fa26efcf8c30e2f3"}]}