Skip to content

Commit

Permalink
temporarily disable ILB test for VMSS
Browse files Browse the repository at this point in the history
  • Loading branch information
devigned committed Jan 21, 2021
1 parent a25e517 commit c0d8e36
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
5 changes: 4 additions & 1 deletion test/e2e/azure_lb.go
Expand Up @@ -46,6 +46,7 @@ type AzureLBSpecInput struct {
SkipCleanup bool
IPv6 bool
Windows bool
IsVMSS bool
}

// AzureLBSpec implements a test that verifies Azure internal and external load balancers can
Expand Down Expand Up @@ -112,7 +113,9 @@ func AzureLBSpec(ctx context.Context, inputGetter func() AzureLBSpecInput) {

// TODO: fix and enable this. Internal LBs + IPv6 is currently in preview.
// https://docs.microsoft.com/en-us/azure/virtual-network/ipv6-dual-stack-standard-internal-load-balancer-powershell
if !input.IPv6 {
//
// TODO: fix and enable this for VMSS after NRP / CRP sync bug is resolved
if !input.IPv6 && !input.IsVMSS {
By("creating an internal Load Balancer service")

ilbService := webDeployment.GetService(ports, deploymentBuilder.InternalLoadbalancer)
Expand Down
3 changes: 3 additions & 0 deletions test/e2e/azure_test.go
Expand Up @@ -297,6 +297,7 @@ var _ = Describe("Workload cluster creation", func() {
Namespace: namespace,
ClusterName: clusterName,
SkipCleanup: skipCleanup,
IsVMSS: true,
}
})
})
Expand Down Expand Up @@ -485,6 +486,7 @@ var _ = Describe("Workload cluster creation", func() {
Namespace: namespace,
ClusterName: clusterName,
SkipCleanup: skipCleanup,
IsVMSS: true,
}
})
})
Expand All @@ -497,6 +499,7 @@ var _ = Describe("Workload cluster creation", func() {
ClusterName: clusterName,
SkipCleanup: skipCleanup,
Windows: true,
IsVMSS: true,
}
})
})
Expand Down

0 comments on commit c0d8e36

Please sign in to comment.