)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"f16087a292a53d426e42199c507d98734da30569","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"31a105e8_bf96c70d","updated":"2021-10-27 07:10:21.000000000","message":"recheck","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"9046875c9ada8fa7bfaccf83635b55f0d8f43916","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"47391511_f39df7b3","updated":"2021-10-29 03:36:47.000000000","message":"recheck","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"9c30f0303327d6b0aa64533dd35ac237220b6109","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"70a5bbae_f5bb640f","updated":"2021-10-27 10:10:23.000000000","message":"recheck","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"1a2f80e65fe7b3df166b65f39ee3670c43971bb6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d568fccd_39dfc44d","updated":"2021-10-28 04:36:32.000000000","message":"recheck","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"},{"author":{"_account_id":5756,"name":"Terry Wilson","email":"twilson@redhat.com","username":"otherwiseguy"},"change_message_id":"e7f57b3f2e8dd438e044ba722f32136f2406cf0a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"09fabe13_b804ce29","updated":"2021-10-27 19:50:21.000000000","message":"recheck\n\n*shakes his fist at linuxbridge*","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"f05406ac9071bba133eb68a2c001453109f0fb94","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"12c55cd3_9ef58218","updated":"2021-10-26 15:26:23.000000000","message":"recheck - just to check once again if CI will be green 😊","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"}],"neutron/agent/common/ovs_lib.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"65826718cd3e083dd4cb4a5eb60df48b940611a5","unresolved":true,"context_lines":[{"line_number":265,"context_line":"        # TODO(ralonsoh): this is a workaround for LP#1948642. When a new"},{"line_number":266,"context_line":"        # controller is added, os-ken ``OfctlService`` needs to receive the"},{"line_number":267,"context_line":"        # update event and register this new datapath."},{"line_number":268,"context_line":"        time.sleep(1)"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    def del_controller(self):"},{"line_number":271,"context_line":"        self.ovsdb.del_controller(self.br_name).execute(check_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"52f74f1d_b85d2989","line":268,"range":{"start_line":268,"start_character":8,"end_line":268,"end_character":21},"updated":"2021-10-26 14:28:45.000000000","message":"won\u0027t it yield to another greenthread that can potentially execute another os_ken command?","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"136af9bd6a7617427271d5790e85b309dcda2d85","unresolved":true,"context_lines":[{"line_number":265,"context_line":"        # TODO(ralonsoh): this is a workaround for LP#1948642. When a new"},{"line_number":266,"context_line":"        # controller is added, os-ken ``OfctlService`` needs to receive the"},{"line_number":267,"context_line":"        # update event and register this new datapath."},{"line_number":268,"context_line":"        time.sleep(1)"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    def del_controller(self):"},{"line_number":271,"context_line":"        self.ovsdb.del_controller(self.br_name).execute(check_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"a2be4beb_8590c511","line":268,"range":{"start_line":268,"start_character":8,"end_line":268,"end_character":21},"in_reply_to":"4c08f158_15e367f6","updated":"2021-10-26 15:33:07.000000000","message":"Ok anyway it\u0027s still a workaround for now, and even if it\u0027s not working for all cases it shouldn\u0027t hurt.","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"15f6b9f5dac41590666fc01cee1f93311935a856","unresolved":true,"context_lines":[{"line_number":265,"context_line":"        # TODO(ralonsoh): this is a workaround for LP#1948642. When a new"},{"line_number":266,"context_line":"        # controller is added, os-ken ``OfctlService`` needs to receive the"},{"line_number":267,"context_line":"        # update event and register this new datapath."},{"line_number":268,"context_line":"        time.sleep(1)"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    def del_controller(self):"},{"line_number":271,"context_line":"        self.ovsdb.del_controller(self.br_name).execute(check_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"4c08f158_15e367f6","line":268,"range":{"start_line":268,"start_character":8,"end_line":268,"end_character":21},"in_reply_to":"52f74f1d_b85d2989","updated":"2021-10-26 15:23:39.000000000","message":"I don\u0027t think so as ovs agent is not using greenthreads at all","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ccd208067b0b2afc9e9cbfa44a7382dbcd726718","unresolved":true,"context_lines":[{"line_number":265,"context_line":"        # TODO(ralonsoh): this is a workaround for LP#1948642. When a new"},{"line_number":266,"context_line":"        # controller is added, os-ken ``OfctlService`` needs to receive the"},{"line_number":267,"context_line":"        # update event and register this new datapath."},{"line_number":268,"context_line":"        time.sleep(1)"},{"line_number":269,"context_line":""},{"line_number":270,"context_line":"    def del_controller(self):"},{"line_number":271,"context_line":"        self.ovsdb.del_controller(self.br_name).execute(check_error\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"61b9a8f3_4f9794eb","line":268,"range":{"start_line":268,"start_character":8,"end_line":268,"end_character":21},"in_reply_to":"52f74f1d_b85d2989","updated":"2021-10-26 15:23:38.000000000","message":"The OVS agent has one main execution thread. Other threads are attending RPC messages, OVSDB updates and OF updates (os-ken). But is the main thread the only one that has the os-ken instances.\n\nNOTE: when I say \"main\" thread I\u0027m not referring to the main thread when using threading library, just the one executing the ovs agent code.","commit_id":"0194856da19f0ce96b613968ab415821037f18d0"}]}