)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"0b36adfda19f8de9c243f404acff5abaf38a9ba0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b4a402af_ae5ff9d3","updated":"2022-02-11 16:53:03.000000000","message":"Are you going to implement the event handler for the deleted/added ports?","commit_id":"140bb63665223d7cd2a7fee8c1d1494ebd2a802f"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c6461cc7763b98ae000611c5b421100ee32de21f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c185f737_c46abe9f","updated":"2022-02-11 17:29:42.000000000","message":"Nice series.","commit_id":"140bb63665223d7cd2a7fee8c1d1494ebd2a802f"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"2f5a3f4f6dcd2070b8e5067ddf8135d17b177d47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e7ad0251_54802af0","updated":"2022-02-03 16:10:45.000000000","message":"recheck","commit_id":"140bb63665223d7cd2a7fee8c1d1494ebd2a802f"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"af42274d96d1f1a9941db40ae5f18cd5cf0077e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f88d3b93_a2213318","in_reply_to":"b4a402af_ae5ff9d3","updated":"2022-02-11 17:35:30.000000000","message":"that would be a good followup, or checking the original comments suggested way with ordered event handling (https://opendev.org/openstack/neutron/src/branch/master/neutron/services/trunk/drivers/openvswitch/agent/ovsdb_handler.py#L212-L220 )","commit_id":"140bb63665223d7cd2a7fee8c1d1494ebd2a802f"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"38062e27b96f93799a2f4a83d4b20e92bcb3a976","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"561353d2_a1293edb","in_reply_to":"f88d3b93_a2213318","updated":"2022-02-11 18:31:43.000000000","message":"Right. This sleep is just a placebo. We had examples, in customers, were the port was added to the bridge just before the trunk services deletes the bridge.\n\nThere are two approaches, both necessary:\n- Make the change in os-vif: this is the safest way. In the \"_plug_vhostuser\" method, create a single OVS DB transaction to ensure the bridge is created and create the port. That ensures nobody can delete the bridge when the port is being added.\n- Here:\n1) remove the sleep\n2) monitor the ovsdb and check when the port has been added\n3) just before deleting the trunk bridge, check if the port is present again. If so, exit the method","commit_id":"140bb63665223d7cd2a7fee8c1d1494ebd2a802f"}]}