Skip to content

Commit c8c62ae

Browse files
test: temporary disable VirtualMachineOperationRestore (#2324)
Description Temporarily skip VirtualMachineOperationRestore on sds-replicated-volume. Why do we need it, and what problem does it solve? The test is currently unstable on the existing replicated storage implementation because of snapshot-related issues. We are waiting for the new replicated implementation, so the test is skipped for this storage class for now. --------------- Signed-off-by: Nikita Korolev <nikita.korolev@flant.com>
1 parent e26b037 commit c8c62ae

1 file changed

Lines changed: 2 additions & 7 deletions

File tree

test/e2e/vmop/restore.go

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ var _ = Describe("VirtualMachineOperationRestore", label.Slow(), Label(precheck.
8484

8585
t := newRestoreTest(f)
8686
if !t.IsStorageClassAvailableForTest(t.VM) {
87-
Skip("Storage class is not available for test")
87+
Skip("Temporary skip on sds-replicated-volume until snapshot functionality is fixed")
8888
}
8989

9090
By("Environment preparation", func() {
@@ -536,10 +536,5 @@ func (t *restoreModeTest) IsStorageClassAvailableForTest(vm *v1alpha2.VirtualMac
536536
sc, err := legacy.GetDefaultStorageClass()
537537
Expect(err).NotTo(HaveOccurred())
538538

539-
if sc.Provisioner != "replicated.csi.storage.deckhouse.io" {
540-
return true
541-
}
542-
543-
placementCount, ok := sc.Parameters["replicated.csi.storage.deckhouse.io/placementCount"]
544-
return ok && placementCount == "1"
539+
return sc.Provisioner != framework.SDSReplicatedVolume
545540
}

0 commit comments

Comments
 (0)