)]}'
{"neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"12c4fcc1f99bbfe00270013321cd6efd54d26bdf","unresolved":true,"context_lines":[{"line_number":312,"context_line":"    def should_post_fork_initialize(worker_class):"},{"line_number":313,"context_line":"        return worker_class in (wsgi.WorkerService,"},{"line_number":314,"context_line":"                                worker.MaintenanceWorker,"},{"line_number":315,"context_line":"                                worker.OvnPeriodicWorker,"},{"line_number":316,"context_line":"                                service.RpcWorker)"},{"line_number":317,"context_line":""},{"line_number":318,"context_line":"    def _setup_hash_ring(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"8cc2f7cc_7292698f","line":315,"range":{"start_line":315,"start_character":39,"end_line":315,"end_character":56},"updated":"2025-01-10 14:46:25.000000000","message":"We already have in ML2/OVN:\n* A maintenance worker, that runs in a separate process. E.g. [1]\n* A generic PeriodicWorker that will run in another separate process. E.g. [2]\n\nIs it really needed to create a 3rd worker type?\n\n[1]https://zuul.opendev.org/t/openstack/build/85963bfd59fa4316b7750ab003e9ca8f/log/controller/logs/screen-neutron-ovn-maintenance-worker.txt\n[2]https://zuul.opendev.org/t/openstack/build/85963bfd59fa4316b7750ab003e9ca8f/log/controller/logs/screen-neutron-periodic-workers.txt","commit_id":"563753cdea1ede5f67b30d81985722e590312f09"},{"author":{"_account_id":31976,"name":"Bodo Petermann","email":"b.petermann@syseleven.de","username":"bpetermann"},"change_message_id":"86df2b9ca9d9e2c08a3ae1d48bcf336378682e2d","unresolved":true,"context_lines":[{"line_number":312,"context_line":"    def should_post_fork_initialize(worker_class):"},{"line_number":313,"context_line":"        return worker_class in (wsgi.WorkerService,"},{"line_number":314,"context_line":"                                worker.MaintenanceWorker,"},{"line_number":315,"context_line":"                                worker.OvnPeriodicWorker,"},{"line_number":316,"context_line":"                                service.RpcWorker)"},{"line_number":317,"context_line":""},{"line_number":318,"context_line":"    def _setup_hash_ring(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"52ce1d52_0b5dce96","line":315,"range":{"start_line":315,"start_character":39,"end_line":315,"end_character":56},"in_reply_to":"8cc2f7cc_7292698f","updated":"2025-01-10 15:35:33.000000000","message":"Is it acceptable to enable the OVN IDL on all `neutron.worker.PeriodicWorker`?\nIf yes, it\u0027s not necessary to introduce OvnPeriodicWorker.\nI need a PeriodicWorker with access to OVN to periodically check the VPN agent list in neutron-vpnaas.","commit_id":"563753cdea1ede5f67b30d81985722e590312f09"}]}