)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"fb15e9a53b975d673f4ba1a363ca76da2bcfc5ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1130bbb4_724cc169","updated":"2023-03-17 07:17:43.000000000","message":"Adding -1 just to ensure this is not missed: what happens with the subports already created? \nAlso, was that added in the first place as a side effect (meaning it was not needed for anything)? Asking just in case we need to update the requested chassis as part of the migration (as in the main trunk port). This most probably is not needed at all so, dismiss it if not needed, as this solution is simpler","commit_id":"8f4cccd26a21519b234caab60c99b67d9d20565e"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"71b7ce3fb59f944648225fa92526dded165736ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"059855cd_f74d796a","updated":"2023-03-17 08:52:05.000000000","message":"sorry, I mixed live-migration of VMs with migrating from OVS to OVN","commit_id":"8f4cccd26a21519b234caab60c99b67d9d20565e"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"df81afa0514ec0352dfec0bfbc07f8bbab533298","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"df2ce1f6_2edd404a","updated":"2023-04-19 14:40:08.000000000","message":"Thanks for the patch. I think we should first investigate why the problem happens only after the OVS -\u003e OVN switch. Sounds like db-sync is not creating the subports/trunks correctly.","commit_id":"651cab17c1772c01c8124165665919b304815e80"},{"author":{"_account_id":32586,"name":"Elvira García Ruiz","display_name":"Elvira","email":"egarciar@redhat.com","username":"elvira"},"change_message_id":"fdcf47a957de64b5304e98da45116183ac5214a2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9a4d9ec3_547bb07a","updated":"2023-03-24 16:55:19.000000000","message":"lgtm!","commit_id":"651cab17c1772c01c8124165665919b304815e80"}],"neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"1f12c77615d27a5f6c3ac69f02d2b82bdd73d9db","unresolved":true,"context_lines":[{"line_number":430,"context_line":"                        options[\u0027activation-strategy\u0027] \u003d \u0027rarp\u0027"},{"line_number":431,"context_line":"            # Do not add the LSP_OPTIONS_REQUESTED_CHASSIS_KEY if the port"},{"line_number":432,"context_line":"            # type is trunk_subport since this will mess up the live-migration"},{"line_number":433,"context_line":"            # This happens only while the environment is migrated from OVS to OVN"},{"line_number":434,"context_line":"            if port_type !\u003d trunk_const.TRUNK_SUBPORT_OWNER:"},{"line_number":435,"context_line":"                options[ovn_const.LSP_OPTIONS_REQUESTED_CHASSIS_KEY] \u003d chassis"},{"line_number":436,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ee3bcdc0_b5728979","line":433,"in_reply_to":"227766e8_4a5dcd08","updated":"2023-03-16 09:02:15.000000000","message":"\u003e pep8: E501 line too long (81 \u003e 79 characters)\n\nEasy one, but please fix to make Zuul happy","commit_id":"8f4cccd26a21519b234caab60c99b67d9d20565e"}]}