Skip to content

Commit

Permalink
Merge pull request #1999 from andyzhangx/skip-anti-affinity-hpc
Browse files Browse the repository at this point in the history
test: skip anti-affinity in hpc deployment test
  • Loading branch information
andyzhangx committed Oct 9, 2023
2 parents a4ae3c1 + f0b248e commit 2b57287
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions test/e2e/dynamic_provisioning_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -747,6 +747,7 @@ func (t *dynamicProvisioningTestSuite) defineTests(isMultiZone bool) {
CSIDriver: testDriver,
Pod: pod,
ShouldOverwrite: false,
IsWindowsHPCDeployment: isWindowsHPCDeployment,
PodWithSnapshot: podWithSnapshot,
StorageClassParameters: map[string]string{"skuName": "StandardSSD_LRS"},
SnapshotStorageClassParameters: map[string]string{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ type DynamicallyProvisionedVolumeSnapshotTest struct {
PodWithSnapshot PodDetails
StorageClassParameters map[string]string
SnapshotStorageClassParameters map[string]string
IsWindowsHPCDeployment bool
}

func (t *DynamicallyProvisionedVolumeSnapshotTest) Run(ctx context.Context, client clientset.Interface, restclient restclientset.Interface, namespace *v1.Namespace) {
Expand Down Expand Up @@ -132,7 +133,7 @@ func (t *DynamicallyProvisionedVolumeSnapshotTest) Run(ctx context.Context, clie
defer tPodWithSnapshotCleanup[i](ctx)
}

if t.ShouldOverwrite {
if t.ShouldOverwrite && !t.IsWindowsHPCDeployment {
// TODO: add test case which will schedule the original disk and the copied disk on the same node once the conflicting UUID issue is fixed.
ginkgo.By("Set pod anti-affinity to make sure two pods are scheduled on different nodes")
tPodWithSnapshot.SetAffinity(&TestPodAntiAffinity)
Expand Down

0 comments on commit 2b57287

Please sign in to comment.