Skip to content

Commit acdcf9f

Browse files
tobiasgiesek8s-infra-cherrypick-robot
authored andcommitted
Revert "Fix ginkgov2 conformance tests"
This reverts commit 32a9fc9. Signed-off-by: Tobias Giese <[email protected]>
1 parent e6ff3b9 commit acdcf9f

File tree

6 files changed

+3
-33
lines changed

6 files changed

+3
-33
lines changed

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ module sigs.k8s.io/cluster-api-provider-openstack
33
go 1.19
44

55
require (
6-
github.com/blang/semver v3.5.1+incompatible
76
github.com/davecgh/go-spew v1.1.1
87
github.com/go-logr/logr v1.2.3
98
github.com/golang/mock v1.6.0
@@ -43,6 +42,7 @@ require (
4342
github.com/antlr/antlr4/runtime/Go/antlr v0.0.0-20220418222510-f25a4f6275ed // indirect
4443
github.com/asaskevich/govalidator v0.0.0-20190424111038-f61b66f89f4a // indirect
4544
github.com/beorn7/perks v1.0.1 // indirect
45+
github.com/blang/semver v3.5.1+incompatible // indirect
4646
github.com/blang/semver/v4 v4.0.0 // indirect
4747
github.com/cespare/xxhash/v2 v2.1.2 // indirect
4848
github.com/coredns/caddy v1.1.0 // indirect

test/e2e/data/kubetest/conformance-fast-ginkgo-v2.yaml

Lines changed: 0 additions & 9 deletions
This file was deleted.

test/e2e/data/kubetest/conformance-fast.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,5 @@ ginkgo.progress: true
55
ginkgo.slowSpecThreshold: 120.0
66
ginkgo.flakeAttempts: 3
77
ginkgo.trace: true
8+
ginkgo.timeout: 3h
89
ginkgo.v: true

test/e2e/data/kubetest/conformance-ginkgo-v2.yaml

Lines changed: 0 additions & 8 deletions
This file was deleted.

test/e2e/data/kubetest/conformance.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,5 @@ ginkgo.progress: true
44
ginkgo.slowSpecThreshold: 120.0
55
ginkgo.flakeAttempts: 3
66
ginkgo.trace: true
7+
ginkgo.timeout: 3h
78
ginkgo.v: true

test/e2e/suites/conformance/conformance_test.go

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,7 @@ import (
2424
"fmt"
2525
"path/filepath"
2626
"strconv"
27-
"strings"
2827

29-
"github.com/blang/semver"
3028
. "github.com/onsi/ginkgo"
3129
. "github.com/onsi/gomega"
3230
corev1 "k8s.io/api/core/v1"
@@ -66,15 +64,6 @@ var _ = Describe("conformance tests", func() {
6664
Expect(err).NotTo(HaveOccurred())
6765
}
6866

69-
// Starting with Kubernetes v1.25, the kubetest config file needs to be compatible with Ginkgo V2.
70-
v125 := semver.MustParse("1.25.0-alpha.0.0")
71-
v, err := semver.ParseTolerant(kubernetesVersion)
72-
Expect(err).NotTo(HaveOccurred())
73-
if v.GTE(v125) {
74-
// Use the Ginkgo V2 config file.
75-
e2eCtx.Settings.KubetestConfigFilePath = getGinkgoV2ConfigFilePath(e2eCtx.Settings.KubetestConfigFilePath)
76-
}
77-
7867
workerMachineCount, err := strconv.ParseInt(e2eCtx.E2EConfig.GetVariable("CONFORMANCE_WORKER_MACHINE_COUNT"), 10, 64)
7968
Expect(err).NotTo(HaveOccurred())
8069
controlPlaneMachineCount, err := strconv.ParseInt(e2eCtx.E2EConfig.GetVariable("CONFORMANCE_CONTROL_PLANE_MACHINE_COUNT"), 10, 64)
@@ -122,7 +111,3 @@ var _ = Describe("conformance tests", func() {
122111
shared.DumpSpecResourcesAndCleanup(ctx, specName, namespace, e2eCtx)
123112
})
124113
})
125-
126-
func getGinkgoV2ConfigFilePath(kubetestConfigFilePath string) string {
127-
return strings.Replace(kubetestConfigFilePath, ".yaml", "-ginkgo-v2.yaml", 1)
128-
}

0 commit comments

Comments
 (0)