From cfe16de74baa8af677bb2ba1b6460142a935bf35 Mon Sep 17 00:00:00 2001 From: Vladik Romanovsky Date: Wed, 27 Feb 2019 15:16:35 -0500 Subject: [PATCH] migration should be set to completed even for failed migrations Signed-off-by: Vladik Romanovsky --- pkg/virt-launcher/virtwrap/manager.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pkg/virt-launcher/virtwrap/manager.go b/pkg/virt-launcher/virtwrap/manager.go index efdaab0f3210..a97862d931ef 100644 --- a/pkg/virt-launcher/virtwrap/manager.go +++ b/pkg/virt-launcher/virtwrap/manager.go @@ -199,9 +199,8 @@ func (l *LibvirtDomainManager) setMigrationResultHelper(vmi *v1.VirtualMachineIn if failed { domainSpec.Metadata.KubeVirt.Migration.Failed = true domainSpec.Metadata.KubeVirt.Migration.FailureReason = reason - } else { - domainSpec.Metadata.KubeVirt.Migration.Completed = true } + domainSpec.Metadata.KubeVirt.Migration.Completed = true domainSpec.Metadata.KubeVirt.Migration.EndTimestamp = &now _, err = l.setDomainSpecWithHooks(vmi, domainSpec)