)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"c424570a99ccbe942deb8348163f990b4479b1a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"303c4ca9_db188129","updated":"2021-11-30 16:23:34.000000000","message":"That\u0027s a cryptic message. ;) But I agree, this is not the way forward and tagging doesn\u0027t seem to be so much of an issue if we\u0027re not bashing Neutron with concurrent bulk port requests.","commit_id":"842f35f5473b331b21b921a55ea9ce0707eef8bf"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"392f8e01ca3ead724930a7a0cd8f6dcdfa96546f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"37047d25_575f8b68","updated":"2021-11-24 16:36:50.000000000","message":"This is a bit risky, as adding the ports to the trunk may take some time (as well as fail), delaying the tagging could lead to leaks. I think adding the ports to the trunks is actually the most expensive call (as it requires installation of flows). Any kuryr-controller crash in between those actions (creation of ports and complete attachment to the trunk) would lead to orphan resources","commit_id":"842f35f5473b331b21b921a55ea9ce0707eef8bf"},{"author":{"_account_id":27032,"name":"Maysa de Macedo Souza","email":"maysa.macedo95@gmail.com","username":"maysa"},"change_message_id":"7b23dd21df9bd911063932315b90fa1dcf1eaebb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"613a338f_29004803","in_reply_to":"37047d25_575f8b68","updated":"2021-11-24 17:02:23.000000000","message":"If addition to the trunk fail, the ports are deleted right after. For some reason adding tags seems to be taking more time than adding to the trunk some times, it needs to go over all Ports. Good point on the controller restart while adding to the trunk, I will re-evaluate","commit_id":"842f35f5473b331b21b921a55ea9ce0707eef8bf"}]}