)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"6710fb14f332d907ed4bcc11294aa30c505bf053","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c7b5fd7d_88066dde","updated":"2026-02-24 11:42:27.000000000","message":"recheck - new patch set in the depends-on change","commit_id":"1f201f3d2f17ad9cf11135d915fd5f330dbd100f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"bf29de549fbcccc7871e280a701eeea7e079a187","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"296aeb37_fe63ad68","updated":"2026-02-27 12:08:59.000000000","message":"Apart from the inline comments, I would like to ask why OVN cannot be registered in the loggin driver. We can log the calls without using RPC","commit_id":"bb3f558603619462b578e540027066efebe91c31"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9ad1769f2e1e816d7feee4a5a98472ae718ae5f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cecb75e5_bc7b418d","updated":"2026-02-24 13:03:13.000000000","message":"I tested it in the DNM patch https://review.opendev.org/c/openstack/neutron-fwaas/+/977776/ which depends on the n-t-p patch to enable logging in the fwaas-openvswitch job","commit_id":"bb3f558603619462b578e540027066efebe91c31"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2ad0ca88da05d1aeae12f853f8463d741c9cca46","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b5e9c8e8_4fdc1dba","in_reply_to":"296aeb37_fe63ad68","updated":"2026-03-04 10:07:44.000000000","message":"what do you mean by \"cannot be registered\"? If there will be logging driver for ovn backend, it may be registered, currently there is simply no need for that","commit_id":"bb3f558603619462b578e540027066efebe91c31"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"524d66eaf6c3f226a34f05ae08f32cf9602df514","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"04677f82_bd3d6b69","in_reply_to":"b5e9c8e8_4fdc1dba","updated":"2026-03-04 14:06:41.000000000","message":"Thanks for the online explanation. The OVN driver does not have logapi implementation, only iptables [1]. I\u0027ll create a LP bug to add this parity gap.\n\n[1]github.com/openstack/neutron-fwaas/tree/master/neutron_fwaas/services/logapi/agents/drivers/iptables","commit_id":"bb3f558603619462b578e540027066efebe91c31"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"bf29de549fbcccc7871e280a701eeea7e079a187","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d9094224_ec5941dc","in_reply_to":"cecb75e5_bc7b418d","updated":"2026-02-27 12:08:59.000000000","message":"Hi! I don\u0027t see the dependency on the testing upper patch. Where is the n-t-p to enable logging [1] referred?\n\n[1]https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/977780","commit_id":"bb3f558603619462b578e540027066efebe91c31"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2ad0ca88da05d1aeae12f853f8463d741c9cca46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5a27c366_e42a76d7","in_reply_to":"d9094224_ec5941dc","updated":"2026-03-04 10:07:44.000000000","message":"I forgot to add this dependency. Now it should run correctly. Sorry","commit_id":"bb3f558603619462b578e540027066efebe91c31"}],"neutron_fwaas/services/firewall/service_drivers/driver_api.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"bf29de549fbcccc7871e280a701eeea7e079a187","unresolved":true,"context_lines":[{"line_number":55,"context_line":"    def register_logging_driver(self):"},{"line_number":56,"context_line":"        LOG.debug(\"No logging driver supported by %s firewall driver\","},{"line_number":57,"context_line":"                  self.__class__.__name__)"},{"line_number":58,"context_line":"        return"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    # Firewall Group"},{"line_number":61,"context_line":"    @abc.abstractmethod"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa6eb214_3622e5f6","line":58,"range":{"start_line":58,"start_character":8,"end_line":58,"end_character":14},"updated":"2026-02-27 12:08:59.000000000","message":"nit: not needed","commit_id":"bb3f558603619462b578e540027066efebe91c31"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"2ad0ca88da05d1aeae12f853f8463d741c9cca46","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    def register_logging_driver(self):"},{"line_number":56,"context_line":"        LOG.debug(\"No logging driver supported by %s firewall driver\","},{"line_number":57,"context_line":"                  self.__class__.__name__)"},{"line_number":58,"context_line":"        return"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    # Firewall Group"},{"line_number":61,"context_line":"    @abc.abstractmethod"}],"source_content_type":"text/x-python","patch_set":3,"id":"4cf71641_8f707356","line":58,"range":{"start_line":58,"start_character":8,"end_line":58,"end_character":14},"in_reply_to":"fa6eb214_3622e5f6","updated":"2026-03-04 10:07:44.000000000","message":"Done","commit_id":"bb3f558603619462b578e540027066efebe91c31"}]}