Skip to content

Commit ba45918

Browse files
wangke19claude
authored andcommitted
Enable TLS tests on IPv6 clusters
Remove the IPv4-only restriction from TLS tests to support IPv6 clusters. The tests use port-forwarding to localhost, which works for both IPv4 and IPv6 environments since localhost resolves appropriately in both cases. Changes: - Removed IPv4-only check in BeforeEach that skipped tests on IPv6 clusters - Removed unused networking import This allows the TLS configuration tests to run on: - IPv4-only clusters - IPv6-only clusters - Dual-stack (IPv4+IPv6) clusters 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
1 parent d0be47d commit ba45918

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

test/extended/apiserver/tls.go

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import (
1717

1818
configv1 "github.com/openshift/api/config/v1"
1919
"github.com/openshift/library-go/pkg/crypto"
20-
"github.com/openshift/origin/test/extended/networking"
2120
exutil "github.com/openshift/origin/test/extended/util"
2221
)
2322

@@ -46,12 +45,6 @@ var _ = g.Describe("[sig-api-machinery][Feature:APIServer]", func() {
4645
if isMicroShift || isHyperShift {
4746
g.Skip("TLS configuration for the apiserver resource is not applicable to MicroShift or HyperShift clusters - skipping")
4847
}
49-
50-
hasIPv4, _, err := networking.GetIPAddressFamily(oc)
51-
o.Expect(err).NotTo(o.HaveOccurred())
52-
if !hasIPv4 {
53-
g.Skip("TLS configuration is only tested on IPv4 clusters, skipping")
54-
}
5548
})
5649

5750
g.It("TestTLSMinimumVersions", func() {

0 commit comments

Comments
 (0)