Go to completed phase after apply delta on warm import

Instead of going to resize phase, go directly to
completed phase for apply the delta snapshot. Resizing
has already happened when importing the initial snapshot.

This will cause validation of the size to not happen
every single delta. Since the resize steps validates
the available space on the target.

This was causing issues on filesystem volumes where
the space was being used and the available space is
less than the virtual size of the disk. We don't care
about the available space at this point since we
already verified there was enough space on the initial
snapshot import.

Signed-off-by: Alexander Wels <awels@redhat.com>

Author: awels

pkg/importer/vddk-datasource_amd64.go

@@ -1064,5 +1064,10 @@ func (vs *VDDKDataSource) TransferFile(fileName string) (ProcessingPhase, error)
 		}
 	}
 
+	if vs.PreviousSnapshot != "" {
+		// Don't resize when applying snapshot deltas as the resize has already happened
+		// when the first snapshot was imported.
+		return ProcessingPhaseComplete, nil
+	}
 	return ProcessingPhaseResize, nil
 }

pkg/importer/vddk-datasource_test.go

@@ -217,7 +217,7 @@ var _ = Describe("VDDK data source", func() {
 
 		phase, err = snap2.TransferFile(".")
 		Expect(err).ToNot(HaveOccurred())
-		Expect(phase).To(Equal(ProcessingPhaseResize))
+		Expect(phase).To(Equal(ProcessingPhaseComplete))
 
 		deltaSum := md5.Sum(mockSinkBuffer)
 		Expect(changedSourceSum).To(Equal(deltaSum))