Skip to content

Commit 602c026

Browse files
authored
Add ginkgo test changes
Add ginkgo test changes
2 parents 467589e + d755596 commit 602c026

6 files changed

Lines changed: 24 additions & 14 deletions

test/openshift/e2e/ginkgo/parallel/1-005_validate_route_tls_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ var _ = Describe("GitOps Operator Parallel E2E Tests", func() {
9595
Expect(serverRoute.Spec.TLS.InsecureEdgeTerminationPolicy).To(Equal(routev1.InsecureEdgeTerminationPolicyRedirect))
9696
Expect(serverRoute.Spec.TLS.Termination).To(Equal(routev1.TLSTerminationReencrypt))
9797

98-
webhookRoute := &routev1.Route{ObjectMeta: metav1.ObjectMeta{Name: "example-applicationset-controller-webhook", Namespace: ns.Name}}
98+
webhookRoute := &routev1.Route{ObjectMeta: metav1.ObjectMeta{Name: "example-appset-webhook", Namespace: ns.Name}}
9999
Eventually(webhookRoute).Should(k8sFixture.ExistByName())
100100
Expect(webhookRoute.Spec.To.Kind).To(Equal("Service"))
101101
Expect(webhookRoute.Spec.To.Name).To(Equal("example-applicationset-controller"))

test/openshift/e2e/ginkgo/parallel/1-019_validate_volume_mounts_test.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -248,8 +248,11 @@ var _ = Describe("GitOps Operator Parallel E2E Tests", func() {
248248
{
249249
Name: "argocd-cmd-params-cm", VolumeSource: corev1.VolumeSource{
250250
ConfigMap: &corev1.ConfigMapVolumeSource{
251-
DefaultMode: ptr.To(int32(420)),
252-
Items: []corev1.KeyToPath{{Key: "controller.profile.enabled", Path: "profiler.enabled"}},
251+
DefaultMode: ptr.To(int32(420)),
252+
Items: []corev1.KeyToPath{
253+
{Key: "controller.profile.enabled", Path: "profiler.enabled"},
254+
{Key: "controller.resource.health.persist", Path: "controller.resource.health.persist"},
255+
},
253256
LocalObjectReference: corev1.LocalObjectReference{Name: "argocd-cmd-params-cm"},
254257
Optional: ptr.To(true)},
255258
},

test/openshift/e2e/ginkgo/parallel/1-031_validate_toolchain_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -85,9 +85,9 @@ var _ = Describe("GitOps Operator Parallel E2E Tests", func() {
8585
It("verifies that toolchain versions have the expected values", func() {
8686

8787
// These variables need to be maintained according to the component matrix: https://spaces.redhat.com/display/GITOPS/GitOps+Component+Matrix
88-
expected_kustomizeVersion := "v5.4.3"
89-
expected_helmVersion := "v3.16.3"
90-
expected_argocdVersion := "v2.14.4"
88+
expected_kustomizeVersion := "v5.6.0"
89+
expected_helmVersion := "v3.17.1"
90+
expected_argocdVersion := "v3.0.11"
9191

9292
var expected_dexVersion string
9393
var expected_redisVersion string
@@ -99,8 +99,8 @@ var _ = Describe("GitOps Operator Parallel E2E Tests", func() {
9999

100100
} else {
101101
// when running against RC/ released version of gitops
102-
expected_dexVersion = "v2.35.1"
103-
expected_redisVersion = "6.2.7"
102+
expected_dexVersion = "v2.41.1"
103+
expected_redisVersion = "7.2.7"
104104
}
105105

106106
By("locating pods containing toolchain in openshift-gitops")

test/openshift/e2e/ginkgo/parallel/1-067_validate_redis_secure_comm_no_autotls_ha_test.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,14 @@ var _ = Describe("GitOps Operator Parallel E2E Tests", func() {
8181

8282
deploymentsShouldExist := []string{"argocd-redis-ha-haproxy", "argocd-server", "argocd-repo-server"}
8383
for _, depl := range deploymentsShouldExist {
84+
replicas := 1
85+
if depl == "argocd-redis-ha-haproxy" {
86+
replicas = 3
87+
}
88+
8489
depl := &appsv1.Deployment{ObjectMeta: metav1.ObjectMeta{Name: depl, Namespace: ns.Name}}
8590
Eventually(depl).Should(k8sFixture.ExistByName())
86-
Eventually(depl).Should(deplFixture.HaveReplicas(1))
87-
Eventually(depl).Should(deplFixture.HaveReadyReplicas(1))
91+
Eventually(depl).Should(deplFixture.HaveReadyReplicas(replicas))
8892
}
8993

9094
statefulsetsShouldExist := []string{"argocd-redis-ha-server", "argocd-application-controller"}

test/openshift/e2e/ginkgo/parallel/1-069_validate_redis_secure_comm_autotls_ha_test.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,10 +79,13 @@ var _ = Describe("GitOps Operator Parallel E2E Tests", func() {
7979

8080
deploymentsShouldExist := []string{"argocd-redis-ha-haproxy", "argocd-server", "argocd-repo-server"}
8181
for _, depl := range deploymentsShouldExist {
82+
replicas := 1
83+
if depl == "argocd-redis-ha-haproxy" {
84+
replicas = 3
85+
}
8286
depl := &appsv1.Deployment{ObjectMeta: metav1.ObjectMeta{Name: depl, Namespace: ns.Name}}
8387
Eventually(depl).Should(k8sFixture.ExistByName())
84-
Eventually(depl).Should(deplFixture.HaveReplicas(1))
85-
Eventually(depl).Should(deplFixture.HaveReadyReplicas(1))
88+
Eventually(depl).Should(deplFixture.HaveReadyReplicas(replicas))
8689
}
8790

8891
statefulsetsShouldExist := []string{"argocd-redis-ha-server", "argocd-application-controller"}

test/openshift/e2e/ginkgo/sequential/1-020_validate_redis_ha_nonha_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ var _ = Describe("GitOps Operator Sequential E2E Tests", func() {
101101
},
102102
}))
103103

104-
Eventually(&appsv1.Deployment{ObjectMeta: metav1.ObjectMeta{Name: "openshift-gitops-redis-ha-haproxy", Namespace: "openshift-gitops"}}, "60s", "5s").Should(deploymentFixture.HaveReadyReplicas(1))
104+
Eventually(&appsv1.Deployment{ObjectMeta: metav1.ObjectMeta{Name: "openshift-gitops-redis-ha-haproxy", Namespace: "openshift-gitops"}}, "60s", "5s").Should(deploymentFixture.HaveReadyReplicas(3))
105105

106106
By("verifying non-HA resources no longer exist, since HA is enabled")
107107

@@ -132,7 +132,7 @@ var _ = Describe("GitOps Operator Sequential E2E Tests", func() {
132132
By("verifying Deployment and StatefulSet have expected resources that we set in previous step")
133133

134134
depl = &appsv1.Deployment{ObjectMeta: metav1.ObjectMeta{Name: "openshift-gitops-redis-ha-haproxy", Namespace: "openshift-gitops"}}
135-
Eventually(depl, "2m", "5s").Should(deploymentFixture.HaveReadyReplicas(1))
135+
Eventually(depl, "2m", "5s").Should(deploymentFixture.HaveReadyReplicas(3))
136136

137137
haProxyContainer := deploymentFixture.GetTemplateSpecContainerByName("haproxy", *depl)
138138

0 commit comments

Comments
 (0)