migration should be set to completed even for failed migrations

Signed-off-by: Vladik Romanovsky <[email protected]>
portalgun-io · Mar 1, 2019 · cfe16de · cfe16de
1 parent 69a22e9
commit cfe16de
Showing 1 changed file with 1 addition and 2 deletions.
diff --git 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)