)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b3b01f51348cea28e5cf85bf9830183ac7c3ccd3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a4c5dc91_bef5a36d","updated":"2026-09-02 08:20:02.000000000","message":"being tested by https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1003026","commit_id":"9897160e43ca9558f3b033015104d79bd065d666"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"488e41db4a6a57dedcf27540c986e37b927c4266","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e5828a7f_5b6e06b0","updated":"2026-09-03 11:45:11.000000000","message":"https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/1003026 is now passing and no errors observed there.","commit_id":"c2dcba91b6117f6ff1029c05bada81345f4ac633"}],"neutron_fwaas/privileged/netfilter_log/libnetfilter_log.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ae28e7095a0623d3f362c6620820efe0052dfdce","unresolved":true,"context_lines":[{"line_number":323,"context_line":"    def start(self):"},{"line_number":324,"context_line":"        with self.lock:"},{"line_number":325,"context_line":"            def loop():"},{"line_number":326,"context_line":"                while not self.shutdown_flag.is_set():"},{"line_number":327,"context_line":"                    if self.callback:"},{"line_number":328,"context_line":"                        ctx \u003d zmq.Context(1)"},{"line_number":329,"context_line":"                        sub \u003d ctx.socket(zmq.XREQ)"}],"source_content_type":"text/x-python","patch_set":1,"id":"d56390ee_b7a04f7e","line":326,"range":{"start_line":326,"start_character":31,"end_line":326,"end_character":44},"updated":"2026-09-02 07:44:28.000000000","message":"Shouldn\u0027t this be `shutdown_event`","commit_id":"46944240e3fd1a87466b6a5f799bcbc0308ac773"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b3b01f51348cea28e5cf85bf9830183ac7c3ccd3","unresolved":false,"context_lines":[{"line_number":323,"context_line":"    def start(self):"},{"line_number":324,"context_line":"        with self.lock:"},{"line_number":325,"context_line":"            def loop():"},{"line_number":326,"context_line":"                while not self.shutdown_flag.is_set():"},{"line_number":327,"context_line":"                    if self.callback:"},{"line_number":328,"context_line":"                        ctx \u003d zmq.Context(1)"},{"line_number":329,"context_line":"                        sub \u003d ctx.socket(zmq.XREQ)"}],"source_content_type":"text/x-python","patch_set":1,"id":"2aa6a3ba_e2c05152","line":326,"range":{"start_line":326,"start_character":31,"end_line":326,"end_character":44},"in_reply_to":"d56390ee_b7a04f7e","updated":"2026-09-02 08:20:02.000000000","message":"Done","commit_id":"46944240e3fd1a87466b6a5f799bcbc0308ac773"}],"neutron_fwaas/services/logapi/agents/drivers/iptables/log.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"74c9f22b8b9ea6f419def8ae74ac36d8dc9f034a","unresolved":false,"context_lines":[{"line_number":226,"context_line":"    def start_logging(self, context, **kwargs):"},{"line_number":227,"context_line":"        LOG.debug(\"Start logging: %s\", str(kwargs))"},{"line_number":228,"context_line":"        if self.log_app:"},{"line_number":229,"context_line":"            self.log_app.start()"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"        for resource_type in self.SUPPORTED_LOGGING_TYPES:"},{"line_number":232,"context_line":"            router_info \u003d kwargs.get(\u0027router_info\u0027)"}],"source_content_type":"text/x-python","patch_set":5,"id":"402e217f_9e9564ab","line":229,"range":{"start_line":229,"start_character":25,"end_line":229,"end_character":30},"updated":"2026-09-04 15:55:47.000000000","message":"I noticed we should not start/stop the thread according to start/stop_logging because these are called per resources. We could probably implement a logic to use a shared thread and start only when it\u0027s not running but that may not be worth implementing urgently.","commit_id":"c2dcba91b6117f6ff1029c05bada81345f4ac633"}]}
