[PATCH] Fixed missing VM vport when batch start or migration partially failed

Daniel Henrique Barboza danielhb413 at gmail.com
Mon Jun 15 18:04:24 UTC 2020



On 6/12/20 3:18 AM, gongwei at smartx.com wrote:
> From: gongwei <gongwei at smartx.com>
> 
> start to failed will not remove the openvswitch port,
> the port recycling in this case lets openvswitch handle it by itself
> 
> Signed-off-by: gongwei <gongwei at smartx.com>
> ---

Can you please elaborate on the commit message? By the commit title and
the code, I'm assuming that you're saying that we shouldn't remove the
openvswitch port if the QEMU process failed to start, for any other
reason aside from SHUTOFF_FAILED.


The code itself looks ok.



>   src/qemu/qemu_process.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index d36088ba98..439bd5b396 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -7482,7 +7482,8 @@ void qemuProcessStop(virQEMUDriverPtr driver,
>           if (vport) {
>               if (vport->virtPortType == VIR_NETDEV_VPORT_PROFILE_MIDONET) {
>                   ignore_value(virNetDevMidonetUnbindPort(vport));
> -            } else if (vport->virtPortType == VIR_NETDEV_VPORT_PROFILE_OPENVSWITCH) {
> +            } else if (vport->virtPortType == VIR_NETDEV_VPORT_PROFILE_OPENVSWITCH &&
> +                       reason != VIR_DOMAIN_SHUTOFF_FAILED) {
>                   ignore_value(virNetDevOpenvswitchRemovePort(
>                                    virDomainNetGetActualBridgeName(net),
>                                    net->ifname));
> 




More information about the libvir-list mailing list