Skip to content

Commit 16070ef

Browse files
Rohit Patilropatil010
andcommitted
Migrate test/e2e-oidc to OpenShift Tests Extension framework
This commit migrates the OIDC e2e tests from the old test framework to the OpenShift Tests Extension (OTE) framework, ensuring compliance with cluster monitoring and image registry policies. Key changes: - Migrate test/e2e-oidc tests to OTE framework structure - Add ImageStream support to satisfy known-image-checker monitoring requirements - Deploy IDP pods (GitLab, Keycloak) using internal registry images - Set ClusterStability to Disruptive for serial test suite to handle temporary degraded states - Remove [Disruptive] tags from individual OIDC tests - Refactor cleanup logic for proper ImageStream and pod resource management - Add vendor dependencies for OpenShift image client Test improvements: - Tests now comply with OpenShift cluster monitoring policies - Proper cleanup order prevents resource leakage - Suite-level stability configuration handles authentication operator degraded states during cleanup - Fixed namespace lifecycle management Co-Authored-By: Rohit Patil <ropatil@redhat.com>
1 parent e35c554 commit 16070ef

42 files changed

Lines changed: 4926 additions & 1196 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

cmd/cluster-authentication-operator-tests-ext/main.go

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import (
1515
_ "github.com/openshift/cluster-authentication-operator/test/e2e"
1616
_ "github.com/openshift/cluster-authentication-operator/test/e2e-encryption-kms"
1717
_ "github.com/openshift/cluster-authentication-operator/test/e2e-encryption-perf"
18+
_ "github.com/openshift/cluster-authentication-operator/test/e2e-oidc"
1819

1920
"k8s.io/klog/v2"
2021
)
@@ -75,11 +76,15 @@ func prepareOperatorTestsRegistry() (*oteextension.Registry, error) {
7576
// The following suite runs tests that must execute serially (one at a time)
7677
// because they modify cluster-wide resources like OAuth configuration.
7778
// Tests tagged with [Serial] and any of [Operator], [OIDC], [Templates], [Tokens] are included in this suite.
79+
// Excludes [Disruptive] tests which run in the disruptive suite instead.
80+
// ClusterStability set to Disruptive: OIDC tests temporarily put authentication operator
81+
// in Degraded state during cleanup when IDP configuration is rolled back.
7882
extension.AddSuite(oteextension.Suite{
79-
Name: "openshift/cluster-authentication-operator/operator/serial",
80-
Parallelism: 1,
83+
Name: "openshift/cluster-authentication-operator/operator/serial",
84+
Parallelism: 1,
85+
ClusterStability: oteextension.ClusterStabilityDisruptive,
8186
Qualifiers: []string{
82-
`name.contains("[Serial]") && (name.contains("[Operator]") || name.contains("[OIDC]") || name.contains("[Templates]") || name.contains("[Tokens]"))`,
87+
`name.contains("[Serial]") && !name.contains("[Disruptive]") && (name.contains("[Operator]") || name.contains("[OIDC]") || name.contains("[Templates]") || name.contains("[Tokens]"))`,
8388
},
8489
})
8590

0 commit comments

Comments
 (0)