[WIP]CNTRLPLANE-2273:Add operator health E2E tests #458

wangke19 · 2025-12-25T14:42:37Z

Summary

This PR adds E2E tests for the cluster-config-operator's ClusterOperator resource, focusing on operator-specific metadata and configuration verification. Tests are designed to run in parallel for optimal performance.

What's Added

Test Coverage

Four operator-specific verification tests running in parallel:

Version Information
- Validates operator version is reported
- Validates feature-gates version is reported
RelatedObjects Tracking
- Verifies operator.openshift.io/configs/cluster is tracked
- Verifies openshift-config namespace is tracked
- Verifies openshift-config-operator namespace is tracked
Required Namespaces
- Ensures openshift-config namespace exists
- Ensures openshift-config-managed namespace exists
- Ensures openshift-config-operator namespace exists
Operator Config CR
- Validates operator.openshift.io/configs/cluster CR exists

Files Added/Modified

New Files:

test/e2e/helpers.go - Shared utilities for test client setup
test/e2e/operator_health.go - ClusterOperator verification tests
cmd/cluster-config-operator-tests-ext/dependencymagnet.go - Ginkgo dependency tracking

Modified Files:

cmd/cluster-config-operator-tests-ext/main.go - Register test suite with OTE framework

Test Suite Configuration

Suite Name: openshift/cluster-config-operator/operator/parallel
Test Tags: [Operator][Parallel]
Execution Mode: Parallel (all tests are read-only and safe for concurrent execution)

Design Decisions

Parallel Execution

All tests perform read-only GET operations with no mutations or shared state, making them safe for parallel execution:

Performance: ~50-75% faster than serial execution
No race conditions: Each test creates its own client, no shared variables
Idempotent: Tests can run in any order

Focus on Operator-Specific Metadata

These tests complement (not duplicate) origin monitor tests:

Origin monitors check generic ClusterOperator status (Available/Degraded/Progressing) for ALL operators
These tests verify operator-specific metadata that monitors cannot validate:
- Version information format and presence
- RelatedObjects tracking correctness
- Required namespace relationships
- Operator Config CR structure

Testing

Run tests with:

make build
./cluster-config-operator-tests-ext run-suite openshift/cluster-config-operator/operator/parallel

coderabbitai · 2025-12-25T14:42:48Z

Walkthrough

Adds a Ginkgo/Gomega test extension and blank-import initializer, modifies test registry setup to build and register an operator/parallel suite, updates module dependencies, and introduces e2e test utilities plus a ClusterOperator health e2e test using Kubernetes/OpenShift and dynamic clients.

Changes

Cohort / File(s)	Summary
Test extension & initializer `cmd/cluster-config-operator-tests-ext/dependencymagnet.go`, `cmd/cluster-config-operator-tests-ext/main.go`	Adds a blank import file to force test package init; refactors registry setup to build Ginkgo specs, create an extension, add an `operator/parallel` suite filtered by name.contains("[Operator]") and name.contains("[Parallel]"), attach specs, and register the extension.
Module dependencies `go.mod`	Adds direct requires for `github.com/onsi/ginkgo/v2` v2.23.4 and `github.com/onsi/gomega` v1.38.0; adjusts indirect dependency entries (adds `github.com/go-task/slim-sprig/v3` and `golang.org/x/tools`, removes duplicate indirect ginkgo/gomega placeholders).
E2E helpers & ClusterOperator health test `test/e2e/helpers.go`, `test/e2e/operator_health.go`	Adds e2e helper utilities for REST/Kubernetes/config/dynamic clients, context and pointer helpers, constants, and a new e2e test that verifies ClusterOperator versions, relatedObjects, required namespaces, and presence of operator config CR.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

✨ Finishing touches

📝 Generate docstrings

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between 82e0623 and 28a1a59.

⛔ Files ignored due to path filters (259)

go.sum is excluded by !**/*.sum
vendor/github.com/go-task/slim-sprig/v3/.editorconfig is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/.gitattributes is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/.gitignore is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/CHANGELOG.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/LICENSE.txt is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/README.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/Taskfile.yml is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/crypto.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/date.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/defaults.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/dict.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/doc.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/functions.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/list.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/network.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/numeric.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/reflect.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/regex.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/strings.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/url.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/.gitignore is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/CHANGELOG.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/CONTRIBUTING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/LICENSE is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/Makefile is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/OWNERS is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/README.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/RELEASING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/config/deprecated.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/core_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/core_dsl_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/decorator_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/deprecated_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/formatter/colorable_others.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/formatter/colorable_windows.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/formatter/formatter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/build/build_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/command/abort.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/command/command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/command/program.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/boostrap_templates.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/bootstrap_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/generate_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/generate_templates.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/generators_common.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/compile.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/gocovmerge.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/profiles_and_reports.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/run.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/test_suite.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/utils.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/verify_version.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/labels/labels_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/main.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/ginkgo.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/import.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/outline.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/outline_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/run/run_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/unfocus/unfocus_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/delta.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/delta_tracker.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/dependencies.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/package_hash.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/package_hashes.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/suite.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/watch_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo_cli_dependencies.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo_t_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/counter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/failer.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/focus.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/global/init.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/group.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/interrupt_handler/interrupt_handler.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/interrupt_handler/sigquit_swallower_unix.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/interrupt_handler/sigquit_swallower_windows.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/node.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/ordering.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor_unix.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor_wasm.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor_win.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/client_server.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/http_client.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/http_server.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/rpc_client.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/rpc_server.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/server_handler.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_bsd.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_unix.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_wasm.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_win.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_reporter_manager.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/report_entry.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/spec.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/spec_context.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/spec_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/suite.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/suite_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/testingtproxy/testing_t_proxy.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/tree.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/writer.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/default_reporter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/deprecated_reporter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/json_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/junit_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/reporter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/teamcity_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporting_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/table_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/code_location.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/config.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/deprecated_types.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/deprecation_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/enum_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/errors.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/file_filter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/flags.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/label_filter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/report_entry.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/types.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/types_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/version.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/.gitignore is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/CHANGELOG.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/CONTRIBUTING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/LICENSE is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/README.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/RELEASING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/format/format.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/gomega_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/assertion.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/async_assertion.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/duration_bundle.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/gomega.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/gutil/post_ioutil.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/gutil/using_ioutil.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/polling_signal_error.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/vetoptdesc.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/and.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/assignable_to_type_of_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/attributes_slice.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_a_directory.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_a_regular_file.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_an_existing_file.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_closed_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_comparable_to_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_element_of_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_empty_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_equivalent_to_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_false_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_identical_to.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_key_of_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_nil_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_numerically_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_sent_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_temporally_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_true_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_zero_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/consist_of.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/contain_element_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/contain_elements_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/contain_substring_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/equal_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_cap_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_each_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_exact_elements.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_existing_field_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_field.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_http_body_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_http_header_with_value_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_http_status_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_key_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_key_with_value_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_len_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_occurred_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_prefix_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_suffix_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_value.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/internal/miter/type_support_iter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/internal/miter/type_support_noiter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_error_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_json_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_regexp_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_xml_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_yaml_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/not.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/or.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/panic_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/receive_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/satisfy_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/semi_structured_data_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/succeed_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/bipartitegraph/bipartitegraph.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/bipartitegraph/bipartitegraphmatching.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/edge/edge.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/node/node.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/util/util.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/type_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/with_transform.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/types/types.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/openshift-eng/openshift-tests-extension/pkg/ginkgo/logging.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/openshift-eng/openshift-tests-extension/pkg/ginkgo/parallel.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/openshift-eng/openshift-tests-extension/pkg/ginkgo/util.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/atom/atom.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/atom/table.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/charset/charset.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/const.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/doc.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/doctype.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/entity.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/escape.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/foreign.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/iter.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/node.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/parse.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/render.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/token.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/charmap/charmap.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/charmap/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/encoding.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/htmlindex/htmlindex.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/htmlindex/map.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/htmlindex/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/internal/identifier/identifier.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/internal/identifier/mib.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/internal/internal.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/all.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/eucjp.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/iso2022jp.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/shiftjis.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/korean/euckr.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/korean/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/all.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/gbk.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/hzgb2312.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/traditionalchinese/big5.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/traditionalchinese/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/unicode/override.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/unicode/unicode.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/internal/utf8internal/utf8internal.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/runes/cond.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/runes/runes.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/LICENSE is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/PATENTS is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/cover/profile.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/edge/edge.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/cursor.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/inspector.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/iter.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/typeof.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/walk.go is excluded by !vendor/**, !**/vendor/**
vendor/modules.txt is excluded by !vendor/**, !**/vendor/**

📒 Files selected for processing (5)

cmd/cluster-config-operator-tests-ext/dependencymagnet.go
cmd/cluster-config-operator-tests-ext/main.go
go.mod
test/e2e/helpers.go
test/e2e/operator_health.go

🚧 Files skipped from review as they are similar to previous changes (5)

cmd/cluster-config-operator-tests-ext/main.go
test/e2e/helpers.go
cmd/cluster-config-operator-tests-ext/dependencymagnet.go
go.mod
test/e2e/operator_health.go

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 1

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between b58d2c2 and 0b8a48c.

⛔ Files ignored due to path filters (259)

go.sum is excluded by !**/*.sum
vendor/github.com/go-task/slim-sprig/v3/.editorconfig is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/.gitattributes is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/.gitignore is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/CHANGELOG.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/LICENSE.txt is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/README.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/Taskfile.yml is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/crypto.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/date.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/defaults.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/dict.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/doc.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/functions.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/list.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/network.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/numeric.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/reflect.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/regex.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/strings.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/url.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/.gitignore is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/CHANGELOG.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/CONTRIBUTING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/LICENSE is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/Makefile is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/OWNERS is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/README.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/RELEASING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/config/deprecated.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/core_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/core_dsl_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/decorator_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/deprecated_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/formatter/colorable_others.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/formatter/colorable_windows.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/formatter/formatter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/build/build_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/command/abort.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/command/command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/command/program.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/boostrap_templates.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/bootstrap_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/generate_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/generate_templates.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/generators_common.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/compile.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/gocovmerge.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/profiles_and_reports.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/run.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/test_suite.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/utils.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/verify_version.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/labels/labels_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/main.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/ginkgo.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/import.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/outline.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/outline_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/run/run_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/unfocus/unfocus_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/delta.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/delta_tracker.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/dependencies.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/package_hash.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/package_hashes.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/suite.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/watch_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo_cli_dependencies.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo_t_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/counter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/failer.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/focus.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/global/init.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/group.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/interrupt_handler/interrupt_handler.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/interrupt_handler/sigquit_swallower_unix.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/interrupt_handler/sigquit_swallower_windows.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/node.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/ordering.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor_unix.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor_wasm.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor_win.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/client_server.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/http_client.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/http_server.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/rpc_client.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/rpc_server.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/server_handler.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_bsd.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_unix.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_wasm.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_win.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_reporter_manager.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/report_entry.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/spec.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/spec_context.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/spec_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/suite.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/suite_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/testingtproxy/testing_t_proxy.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/tree.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/writer.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/default_reporter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/deprecated_reporter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/json_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/junit_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/reporter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/teamcity_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporting_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/table_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/code_location.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/config.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/deprecated_types.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/deprecation_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/enum_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/errors.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/file_filter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/flags.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/label_filter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/report_entry.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/types.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/types_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/version.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/.gitignore is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/CHANGELOG.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/CONTRIBUTING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/LICENSE is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/README.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/RELEASING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/format/format.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/gomega_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/assertion.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/async_assertion.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/duration_bundle.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/gomega.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/gutil/post_ioutil.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/gutil/using_ioutil.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/polling_signal_error.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/vetoptdesc.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/and.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/assignable_to_type_of_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/attributes_slice.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_a_directory.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_a_regular_file.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_an_existing_file.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_closed_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_comparable_to_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_element_of_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_empty_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_equivalent_to_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_false_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_identical_to.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_key_of_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_nil_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_numerically_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_sent_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_temporally_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_true_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_zero_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/consist_of.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/contain_element_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/contain_elements_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/contain_substring_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/equal_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_cap_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_each_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_exact_elements.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_existing_field_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_field.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_http_body_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_http_header_with_value_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_http_status_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_key_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_key_with_value_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_len_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_occurred_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_prefix_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_suffix_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_value.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/internal/miter/type_support_iter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/internal/miter/type_support_noiter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_error_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_json_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_regexp_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_xml_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_yaml_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/not.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/or.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/panic_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/receive_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/satisfy_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/semi_structured_data_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/succeed_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/bipartitegraph/bipartitegraph.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/bipartitegraph/bipartitegraphmatching.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/edge/edge.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/node/node.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/util/util.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/type_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/with_transform.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/types/types.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/openshift-eng/openshift-tests-extension/pkg/ginkgo/logging.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/openshift-eng/openshift-tests-extension/pkg/ginkgo/parallel.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/openshift-eng/openshift-tests-extension/pkg/ginkgo/util.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/atom/atom.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/atom/table.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/charset/charset.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/const.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/doc.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/doctype.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/entity.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/escape.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/foreign.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/iter.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/node.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/parse.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/render.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/token.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/charmap/charmap.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/charmap/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/encoding.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/htmlindex/htmlindex.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/htmlindex/map.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/htmlindex/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/internal/identifier/identifier.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/internal/identifier/mib.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/internal/internal.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/all.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/eucjp.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/iso2022jp.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/shiftjis.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/korean/euckr.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/korean/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/all.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/gbk.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/hzgb2312.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/traditionalchinese/big5.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/traditionalchinese/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/unicode/override.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/unicode/unicode.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/internal/utf8internal/utf8internal.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/runes/cond.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/runes/runes.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/LICENSE is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/PATENTS is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/cover/profile.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/edge/edge.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/cursor.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/inspector.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/iter.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/typeof.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/walk.go is excluded by !vendor/**, !**/vendor/**
vendor/modules.txt is excluded by !vendor/**, !**/vendor/**

📒 Files selected for processing (5)

cmd/cluster-config-operator-tests-ext/dependencymagnet.go
cmd/cluster-config-operator-tests-ext/main.go
go.mod
test/e2e/helpers.go
test/e2e/operator_health.go

🧰 Additional context used

📓 Path-based instructions (1)

**

⚙️ CodeRabbit configuration file

-Focus on major issues impacting performance, readability, maintainability and security. Avoid nitpicks and avoid verbosity.

Files:

test/e2e/operator_health.go
cmd/cluster-config-operator-tests-ext/dependencymagnet.go
cmd/cluster-config-operator-tests-ext/main.go
test/e2e/helpers.go
go.mod

🔇 Additional comments (7)

cmd/cluster-config-operator-tests-ext/dependencymagnet.go (1)

1-8: LGTM!

Standard dependency magnet pattern to trigger test registration via blank import. The comment clearly explains the purpose.

go.mod (1)

10-11: LGTM!

Promoting Ginkgo/Gomega to direct dependencies aligns with the new e2e test infrastructure. The replace directive for the OCP fork is properly documented.

test/e2e/operator_health.go (1)

19-36: LGTM!

The deployment verification logic is sound - using Eventually with appropriate timeout for the deployment to become ready is the correct pattern.

test/e2e/helpers.go (2)

12-18: LGTM!

Constants are well-organized with reasonable polling defaults. Note that clusterOperatorName appears unused in the current tests—presumably reserved for future ClusterOperator status checks.

20-45: LGTM!

The helper functions follow standard client-go patterns for loading kubeconfig and creating clients. Clean and reusable utilities.

cmd/cluster-config-operator-tests-ext/main.go (2)

24-31: LGTM!

Good error handling pattern—failing early with klog.Fatal if command initialization fails prevents running with an incomplete registry.

67-90: Specs are automatically associated with the suite via implicit source-based filtering.

The OTE framework automatically associates specs with suites through the source field. When AddSpecs() is called, each spec's source is set to the component identifier (openshift:payload:cluster-config-operator). When AddSuite() is called without explicit qualifiers, it automatically adds a qualifier source == "openshift:payload:cluster-config-operator", which includes all specs from that extension. No explicit association is required; this is handled implicitly by the framework.

openshift-ci-robot · 2025-12-25T14:46:04Z

@wangke19: This pull request references CNTRLPLANE-2273 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Summary

This PR adds E2E tests for cluster-config-operator health verification using the OpenShift Tests Extension (OTE) framework with Ginkgo. This builds on PR #457 which refactored the test infrastructure.

Changes

Commit 1: Infrastructure Code

test/e2e/helpers.go (42 lines) - Shared test utilities

Kubernetes client setup with kubeconfig loading

Common constants (namespaces, timeouts)

Helper functions (testContext, int64Ptr)

test/e2e/operator_health.go (94 lines) - Operator health tests

[Operator][Serial] Operator Health Deployment Verification

Verifies deployment exists and has ready replicas

[Operator][Serial] Operator Health Pod Health

Verifies pods are running without fatal errors in logs

cmd/cluster-config-operator-tests-ext/main.go - OTE integration

Build Ginkgo test specs from test suite

Register test suite: openshift/cluster-config-operator/all

Improved error handling following oauth-apiserver PR Updating ose-cluster-config-operator builder & base images to be consistent with ART #162 pattern

cmd/cluster-config-operator-tests-ext/dependencymagnet.go (8 lines)

Blank import to register tests with OTE framework

Commit 2: Dependencies

Added Ginkgo v2 (OCP fork) for BDD testing

Added Gomega for assertions

Added OTE Ginkgo integration package

Vendored all dependencies

Test Coverage

✅ 2 tests added:

Deployment verification (ready replicas >= 1)

Pod health verification (running state + log checks for fatal errors)

Testing
# Build
make build

# List tests
./cluster-config-operator-tests-ext list tests --suite=openshift/cluster-config-operator/all

# Run tests (requires KUBECONFIG)
export KUBECONFIG=/path/to/kubeconfig
./cluster-config-operator-tests-ext run-suite openshift/cluster-config-operator/all
References

Based on: PR CNTRLPLANE-2272:Refactor tests-ext initialization to improve error handling #457 (tests-ext error handling refactor)

Follows pattern from: CNTRLPLANE-2260:cmd/oauth-apiserver-tests-ext: Update OTE framework integration oauth-apiserver#162

OTE example: https://github.com/openshift-eng/openshift-tests-extension/blob/main/cmd/example-tests/main.go

Notes

Tests use g and o import aliases for Ginkgo and Gomega (OpenShift convention)

Tests are labeled with [Operator][Serial] for proper categorization

Total new code: ~144 lines (infrastructure), well within 200-400 line target

Vendor directory: +260 files (standard for Ginkgo/Gomega dependencies)

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot · 2025-12-25T14:46:51Z

@wangke19: This pull request references CNTRLPLANE-2273 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Summary

This PR adds E2E tests for cluster-config-operator health verification using the OpenShift Tests Extension (OTE) framework with Ginkgo. This builds on PR #457 which refactored the test infrastructure.

Changes

Commit 1: Infrastructure Code

test/e2e/helpers.go (42 lines) - Shared test utilities

Kubernetes client setup with kubeconfig loading

Common constants (namespaces, timeouts)

Helper functions (testContext, int64Ptr)

test/e2e/operator_health.go (94 lines) - Operator health tests

[Operator][Serial] Operator Health Deployment Verification

Verifies deployment exists and has ready replicas

[Operator][Serial] Operator Health Pod Health

Verifies pods are running without fatal errors in logs

cmd/cluster-config-operator-tests-ext/main.go - OTE integration

Build Ginkgo test specs from test suite

Register test suite: openshift/cluster-config-operator/all

Improved error handling

cmd/cluster-config-operator-tests-ext/dependencymagnet.go (8 lines)

Blank import to register tests with OTE framework

Commit 2: Dependencies

Added Ginkgo v2 (OCP fork) for BDD testing

Added Gomega for assertions

Added OTE Ginkgo integration package

Vendored all dependencies

Test Coverage

✅ 2 tests added:

Deployment verification (ready replicas >= 1)

Pod health verification (running state + log checks for fatal errors)

Testing
# Build
make build

# List tests
./cluster-config-operator-tests-ext list tests --suite=openshift/cluster-config-operator/all

# Run tests (requires KUBECONFIG)
export KUBECONFIG=/path/to/kubeconfig
./cluster-config-operator-tests-ext run-suite openshift/cluster-config-operator/all
References

Based on: PR CNTRLPLANE-2272:Refactor tests-ext initialization to improve error handling #457 (tests-ext error handling refactor)

OTE example: https://github.com/openshift-eng/openshift-tests-extension/blob/main/cmd/example-tests/main.go

Notes

Tests use g and o import aliases for Ginkgo and Gomega (OpenShift convention)

Tests are labeled with [Operator][Serial] for proper categorization

Total new code: ~144 lines (infrastructure), well within 200-400 line target

Vendor directory: +260 files (standard for Ginkgo/Gomega dependencies)

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot · 2025-12-25T14:54:07Z

@wangke19: This pull request references CNTRLPLANE-2273 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Summary

This PR adds E2E tests for cluster-config-operator health verification using the OpenShift Tests Extension (OTE) framework with Ginkgo. This builds on PR #457 which refactored the test infrastructure.

Changes

Commit 1: Infrastructure Code

test/e2e/helpers.go (42 lines) - Shared test utilities

Kubernetes client setup with kubeconfig loading

Common constants (namespaces, timeouts)

Helper functions (testContext, int64Ptr)

test/e2e/operator_health.go (94 lines) - Operator health tests

[Operator][Serial] Operator Health Deployment Verification

Verifies deployment exists and has ready replicas

[Operator][Serial] Operator Health Pod Health

Verifies pods are running without fatal errors in logs

cmd/cluster-config-operator-tests-ext/main.go - OTE integration

Build Ginkgo test specs from test suite

Register test suite: openshift/cluster-config-operator/Operator/Serial

Improved error handling

cmd/cluster-config-operator-tests-ext/dependencymagnet.go (8 lines)

Blank import to register tests with OTE framework

Commit 2: Dependencies

Added Ginkgo v2 (OCP fork) for BDD testing

Added Gomega for assertions

Added OTE Ginkgo integration package

Vendored all dependencies

Test Coverage

✅ 2 tests added:

Deployment verification (ready replicas >= 1)

Pod health verification (running state + log checks for fatal errors)

Testing
# Build
make build

# List tests
./cluster-config-operator-tests-ext list tests --suite=openshift/cluster-config-operator/Operator/Serial

# Run tests (requires KUBECONFIG)
export KUBECONFIG=/path/to/kubeconfig
./cluster-config-operator-tests-ext run-suite openshift/cluster-config-operator/Operator/Serial
References

Based on: PR CNTRLPLANE-2272:Refactor tests-ext initialization to improve error handling #457 (tests-ext error handling refactor)

OTE example: https://github.com/openshift-eng/openshift-tests-extension/blob/main/cmd/example-tests/main.go

Notes

Tests use g and o import aliases for Ginkgo and Gomega (OpenShift convention)

Tests are labeled with [Operator][Serial] for proper categorization

Total new code: ~144 lines (infrastructure), well within 200-400 line target

Vendor directory: +260 files (standard for Ginkgo/Gomega dependencies)

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot · 2025-12-25T15:01:36Z

@wangke19: This pull request references CNTRLPLANE-2273 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Summary

This PR adds E2E tests for cluster-config-operator health verification using the OpenShift Tests Extension (OTE) framework with Ginkgo. This builds on PR #457 which refactored the test infrastructure.

Changes

Commit 1: Infrastructure Code

test/e2e/helpers.go (42 lines) - Shared test utilities

Kubernetes client setup with kubeconfig loading

Common constants (namespaces, timeouts)

Helper functions (testContext, int64Ptr)

test/e2e/operator_health.go (94 lines) - Operator health tests

[Operator][Serial] Operator Health Deployment Verification

Verifies deployment exists and has ready replicas

[Operator][Serial] Operator Health Pod Health

Verifies pods are running without fatal errors in logs

cmd/cluster-config-operator-tests-ext/main.go - OTE integration

Build Ginkgo test specs from test suite

Register test suite: openshift/cluster-config-operator/Operator/Serial

Improved error handling

cmd/cluster-config-operator-tests-ext/dependencymagnet.go (8 lines)

Blank import to register tests with OTE framework

Commit 2: Dependencies

Added Ginkgo v2 (OCP fork) for BDD testing

Added Gomega for assertions

Added OTE Ginkgo integration package

Vendored all dependencies

Test Coverage

✅ 2 tests added:

Deployment verification (ready replicas >= 1)

Pod health verification (running state + log checks for fatal errors)

Testing
# Build
make build

# List tests
./cluster-config-operator-tests-ext list tests --suite=openshift/cluster-config-operator/Operator/Serial

# Run tests (requires KUBECONFIG)
export KUBECONFIG=/path/to/kubeconfig
./cluster-config-operator-tests-ext run-suite openshift/cluster-config-operator/Operator/Serial
References

Based on: PR CNTRLPLANE-2272:Refactor tests-ext initialization to improve error handling #457 (tests-ext error handling refactor)

OTE example: https://github.com/openshift-eng/openshift-tests-extension/blob/main/cmd/example-tests/main.go

Notes

Tests are labeled with [Operator][Serial] for proper categorization

Total new code: ~144 lines (infrastructure), well within 200-400 line target

Vendor directory: +260 files (standard for Ginkgo/Gomega dependencies)

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot · 2025-12-26T07:25:24Z

@wangke19: This pull request references CNTRLPLANE-2273 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Summary

This PR adds E2E tests for cluster-config-operator health verification using the OpenShift Tests Extension (OTE) framework with Ginkgo. This builds on PR #457 which refactored the test infrastructure.

Changes

Commit 1: Infrastructure Code

test/e2e/helpers.go (42 lines) - Shared test utilities

Kubernetes client setup with kubeconfig loading

Common constants (namespaces, timeouts)

Helper functions (testContext, int64Ptr)

test/e2e/operator_health.go (94 lines) - Operator health tests

[Operator][Serial] Operator Health Deployment Verification

Verifies deployment exists and has ready replicas

[Operator][Serial] Operator Health Pod Health

Verifies pods are running without fatal errors in logs

cmd/cluster-config-operator-tests-ext/main.go - OTE integration

Build Ginkgo test specs from test suite

Register test suite: openshift/cluster-config-operator/Operator/Serial

Improved error handling

cmd/cluster-config-operator-tests-ext/dependencymagnet.go (8 lines)

Blank import to register tests with OTE framework

Commit 2: Dependencies

Added Ginkgo v2 (OCP fork) for BDD testing

Added Gomega for assertions

Added OTE Ginkgo integration package

Vendored all dependencies

Test Coverage

✅ 2 tests added:

Deployment verification (ready replicas >= 1)

Pod health verification (running state + log checks for fatal errors)

Testing
# Build
make build

# List tests
./cluster-config-operator-tests-ext list tests --suite=openshift/cluster-config-operator/Operator/Serial

# Run tests (requires KUBECONFIG)
export KUBECONFIG=/path/to/kubeconfig
./cluster-config-operator-tests-ext run-suite openshift/cluster-config-operator/Operator/Serial
References

Based on: PR CNTRLPLANE-2272:Refactor tests-ext initialization to improve error handling #457 (tests-ext error handling refactor)

OTE example: https://github.com/openshift-eng/openshift-tests-extension/blob/main/cmd/example-tests/main.go

##Related OTE CI job
openshift/release#72960

Notes

Tests are labeled with [Operator][Serial] for proper categorization

Total new code: ~144 lines (infrastructure), well within 200-400 line target

Vendor directory: +260 files (standard for Ginkgo/Gomega dependencies)

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot · 2025-12-26T07:25:40Z

@wangke19: This pull request references CNTRLPLANE-2273 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Summary

This PR adds E2E tests for cluster-config-operator health verification using the OpenShift Tests Extension (OTE) framework with Ginkgo. This builds on PR #457 which refactored the test infrastructure.

Changes

Commit 1: Infrastructure Code

test/e2e/helpers.go (42 lines) - Shared test utilities

Kubernetes client setup with kubeconfig loading

Common constants (namespaces, timeouts)

Helper functions (testContext, int64Ptr)

test/e2e/operator_health.go (94 lines) - Operator health tests

[Operator][Serial] Operator Health Deployment Verification

Verifies deployment exists and has ready replicas

[Operator][Serial] Operator Health Pod Health

Verifies pods are running without fatal errors in logs

cmd/cluster-config-operator-tests-ext/main.go - OTE integration

Build Ginkgo test specs from test suite

Register test suite: openshift/cluster-config-operator/Operator/Serial

Improved error handling

cmd/cluster-config-operator-tests-ext/dependencymagnet.go (8 lines)

Blank import to register tests with OTE framework

Commit 2: Dependencies

Added Ginkgo v2 (OCP fork) for BDD testing

Added Gomega for assertions

Added OTE Ginkgo integration package

Vendored all dependencies

Test Coverage

✅ 2 tests added:

Deployment verification (ready replicas >= 1)

Pod health verification (running state + log checks for fatal errors)

Testing
# Build
make build

# List tests
./cluster-config-operator-tests-ext list tests --suite=openshift/cluster-config-operator/Operator/Serial

# Run tests (requires KUBECONFIG)
export KUBECONFIG=/path/to/kubeconfig
./cluster-config-operator-tests-ext run-suite openshift/cluster-config-operator/Operator/Serial
References

Based on: PR CNTRLPLANE-2272:Refactor tests-ext initialization to improve error handling #457 (tests-ext error handling refactor)

OTE example: https://github.com/openshift-eng/openshift-tests-extension/blob/main/cmd/example-tests/main.go

Related OTE CI job

openshift/release#72960

Notes

Tests are labeled with [Operator][Serial] for proper categorization

Total new code: ~144 lines (infrastructure), well within 200-400 line target

Vendor directory: +260 files (standard for Ginkgo/Gomega dependencies)

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot · 2025-12-26T07:27:04Z

@wangke19: This pull request references CNTRLPLANE-2273 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Summary

This PR adds E2E tests for cluster-config-operator health verification using the OpenShift Tests Extension (OTE) framework with Ginkgo. This builds on PR #457 which refactored the test infrastructure.

Changes

Commit 1: Infrastructure Code

test/e2e/helpers.go (42 lines) - Shared test utilities

Kubernetes client setup with kubeconfig loading

Common constants (namespaces, timeouts)

Helper functions (testContext, int64Ptr)

test/e2e/operator_health.go (94 lines) - Operator health tests

[Operator][Serial] Operator Health Deployment Verification

Verifies deployment exists and has ready replicas

[Operator][Serial] Operator Health Pod Health

Verifies pods are running without fatal errors in logs

cmd/cluster-config-operator-tests-ext/main.go - OTE integration

Build Ginkgo test specs from test suite

Register test suite: openshift/cluster-config-operator/Operator/Serial

Improved error handling

cmd/cluster-config-operator-tests-ext/dependencymagnet.go (8 lines)

Blank import to register tests with OTE framework

Commit 2: Dependencies

Added Ginkgo v2 (OCP fork) for BDD testing

Added Gomega for assertions

Added OTE Ginkgo integration package

Vendored all dependencies

Test Coverage

✅ 2 tests added:

Deployment verification (ready replicas >= 1)

Pod health verification (running state + log checks for fatal errors)

Testing
# Build
make build

# List tests
./cluster-config-operator-tests-ext list tests --suite=openshift/cluster-config-operator/Operator/Serial

# Run tests (requires KUBECONFIG)
export KUBECONFIG=/path/to/kubeconfig
./cluster-config-operator-tests-ext run-suite openshift/cluster-config-operator/Operator/Serial
References

Based on: PR CNTRLPLANE-2272:Refactor tests-ext initialization to improve error handling #457 (tests-ext error handling refactor)

OTE example: https://github.com/openshift-eng/openshift-tests-extension/blob/main/cmd/example-tests/main.go

Related

Related OTE CI job: Add cluster-config-operator serial CI job for OTE release#72961

Notes

Tests are labeled with [Operator][Serial] for proper categorization

Total new code: ~144 lines (infrastructure), well within 200-400 line target

Vendor directory: +260 files (standard for Ginkgo/Gomega dependencies)

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

wangke19 · 2026-01-08T09:30:28Z

/retest

gangwgr · 2026-01-12T10:03:06Z

test/e2e/operator_health.go

+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+)
+
+var _ = g.Describe("[Operator][Serial] Operator Health", func() {


@wangke19 Do we need this case?

For this operator, we don't have any tests , so I think it is better to add basic some tests. hmmm, it's so simple, let me the following to replace it.
ClusterOperator Status Tests (cluster_operator.go)

✅ ClusterOperator resource exists

✅ Status conditions: Available=True, Degraded=False, Progressing=False

✅ Version information reported (operator + feature-gates)

✅ RelatedObjects properly tracked

✅ Required namespaces exist (openshift-config, openshift-config-managed, openshift-config-operator)

✅ Operator Config CR exists

gangwgr · 2026-01-12T10:18:27Z

/lgtm

coderabbitai

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @test/e2e/operator_health.go:
- Around line 76-84: After scanning podLogs with bufio.NewScanner into scanner
in operator_health.go, check scanner.Err() after the for scanner.Scan() loop and
report any error (e.g., call g.Fail or include it in the failure message) so
scanning errors (like token too long) are not silently ignored; update the block
that inspects lines for "fatal"/"panic" to perform this scanner.Err() check and
include pod.Name and the scanner error in the failure/reporting message.

🧹 Nitpick comments (1)

test/e2e/cluster_operator_status.go (1)

27-32: Consider using Eventually for resource existence checks.

Unlike the status condition tests that use Eventually, this existence check could be flaky during cluster initialization or reconciliation. For consistency and reliability, consider wrapping in Eventually.

Proposed fix

 	g.Context("ClusterOperator Resource", func() {
 		g.It("should exist", func() {
-			co, err := configClient.ConfigV1().ClusterOperators().Get(ctx, clusterOperatorName, metav1.GetOptions{})
-			o.Expect(err).NotTo(o.HaveOccurred(), "ClusterOperator resource should exist")
-			o.Expect(co).NotTo(o.BeNil())
-			o.Expect(co.Name).To(o.Equal(clusterOperatorName))
+			o.Eventually(func() error {
+				co, err := configClient.ConfigV1().ClusterOperators().Get(ctx, clusterOperatorName, metav1.GetOptions{})
+				if err != nil {
+					return fmt.Errorf("failed to get ClusterOperator: %w", err)
+				}
+				if co.Name != clusterOperatorName {
+					return fmt.Errorf("unexpected ClusterOperator name: %s", co.Name)
+				}
+				return nil
+			}, pollTimeout, pollInterval).Should(o.Succeed())
 		})
 	})

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between 0b8a48c and ae8920d.

⛔ Files ignored due to path filters (259)

go.sum is excluded by !**/*.sum
vendor/github.com/go-task/slim-sprig/v3/.editorconfig is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/.gitattributes is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/.gitignore is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/CHANGELOG.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/LICENSE.txt is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/README.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/Taskfile.yml is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/crypto.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/date.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/defaults.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/dict.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/doc.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/functions.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/list.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/network.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/numeric.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/reflect.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/regex.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/strings.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/go-task/slim-sprig/v3/url.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/.gitignore is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/CHANGELOG.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/CONTRIBUTING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/LICENSE is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/Makefile is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/OWNERS is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/README.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/RELEASING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/config/deprecated.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/core_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/core_dsl_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/decorator_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/deprecated_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/formatter/colorable_others.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/formatter/colorable_windows.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/formatter/formatter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/build/build_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/command/abort.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/command/command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/command/program.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/boostrap_templates.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/bootstrap_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/generate_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/generate_templates.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/generators/generators_common.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/compile.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/gocovmerge.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/profiles_and_reports.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/run.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/test_suite.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/utils.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/internal/verify_version.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/labels/labels_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/main.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/ginkgo.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/import.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/outline.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/outline/outline_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/run/run_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/unfocus/unfocus_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/delta.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/delta_tracker.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/dependencies.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/package_hash.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/package_hashes.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/suite.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo/watch/watch_command.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo_cli_dependencies.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/ginkgo_t_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/counter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/failer.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/focus.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/global/init.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/group.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/interrupt_handler/interrupt_handler.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/interrupt_handler/sigquit_swallower_unix.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/interrupt_handler/sigquit_swallower_windows.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/node.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/ordering.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor_unix.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor_wasm.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/output_interceptor_win.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/client_server.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/http_client.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/http_server.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/rpc_client.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/rpc_server.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/parallel_support/server_handler.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_bsd.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_unix.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_wasm.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_report_win.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/progress_reporter_manager.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/report_entry.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/spec.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/spec_context.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/spec_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/suite.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/suite_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/testingtproxy/testing_t_proxy.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/tree.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/internal/writer.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/default_reporter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/deprecated_reporter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/json_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/junit_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/reporter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporters/teamcity_report.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/reporting_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/table_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/code_location.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/config.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/deprecated_types.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/deprecation_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/enum_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/errors.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/file_filter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/flags.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/label_filter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/report_entry.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/types.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/types_patch.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/ginkgo/v2/types/version.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/.gitignore is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/CHANGELOG.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/CONTRIBUTING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/LICENSE is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/README.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/RELEASING.md is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/format/format.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/gomega_dsl.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/assertion.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/async_assertion.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/duration_bundle.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/gomega.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/gutil/post_ioutil.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/gutil/using_ioutil.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/polling_signal_error.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/internal/vetoptdesc.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/and.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/assignable_to_type_of_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/attributes_slice.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_a_directory.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_a_regular_file.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_an_existing_file.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_closed_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_comparable_to_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_element_of_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_empty_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_equivalent_to_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_false_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_identical_to.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_key_of_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_nil_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_numerically_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_sent_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_temporally_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_true_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/be_zero_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/consist_of.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/contain_element_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/contain_elements_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/contain_substring_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/equal_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_cap_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_each_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_exact_elements.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_existing_field_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_field.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_http_body_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_http_header_with_value_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_http_status_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_key_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_key_with_value_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_len_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_occurred_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_prefix_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_suffix_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/have_value.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/internal/miter/type_support_iter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/internal/miter/type_support_noiter.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_error_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_json_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_regexp_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_xml_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/match_yaml_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/not.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/or.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/panic_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/receive_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/satisfy_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/semi_structured_data_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/succeed_matcher.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/bipartitegraph/bipartitegraph.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/bipartitegraph/bipartitegraphmatching.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/edge/edge.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/node/node.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/support/goraph/util/util.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/type_support.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/matchers/with_transform.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/onsi/gomega/types/types.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/openshift-eng/openshift-tests-extension/pkg/ginkgo/logging.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/openshift-eng/openshift-tests-extension/pkg/ginkgo/parallel.go is excluded by !vendor/**, !**/vendor/**
vendor/github.com/openshift-eng/openshift-tests-extension/pkg/ginkgo/util.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/atom/atom.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/atom/table.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/charset/charset.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/const.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/doc.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/doctype.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/entity.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/escape.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/foreign.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/iter.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/node.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/parse.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/render.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/net/html/token.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/charmap/charmap.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/charmap/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/encoding.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/htmlindex/htmlindex.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/htmlindex/map.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/htmlindex/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/internal/identifier/identifier.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/internal/identifier/mib.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/internal/internal.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/all.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/eucjp.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/iso2022jp.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/shiftjis.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/japanese/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/korean/euckr.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/korean/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/all.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/gbk.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/hzgb2312.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/simplifiedchinese/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/traditionalchinese/big5.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/traditionalchinese/tables.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/unicode/override.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/encoding/unicode/unicode.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/internal/utf8internal/utf8internal.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/runes/cond.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/text/runes/runes.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/LICENSE is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/PATENTS is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/cover/profile.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/edge/edge.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/cursor.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/inspector.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/iter.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/typeof.go is excluded by !vendor/**, !**/vendor/**
vendor/golang.org/x/tools/go/ast/inspector/walk.go is excluded by !vendor/**, !**/vendor/**
vendor/modules.txt is excluded by !vendor/**, !**/vendor/**

📒 Files selected for processing (6)

cmd/cluster-config-operator-tests-ext/dependencymagnet.go
cmd/cluster-config-operator-tests-ext/main.go
go.mod
test/e2e/cluster_operator_status.go
test/e2e/helpers.go
test/e2e/operator_health.go

🚧 Files skipped from review as they are similar to previous changes (3)

cmd/cluster-config-operator-tests-ext/dependencymagnet.go
go.mod
test/e2e/helpers.go

🧰 Additional context used

📓 Path-based instructions (1)

**

⚙️ CodeRabbit configuration file

-Focus on major issues impacting performance, readability, maintainability and security. Avoid nitpicks and avoid verbosity.

Files:

cmd/cluster-config-operator-tests-ext/main.go
test/e2e/cluster_operator_status.go
test/e2e/operator_health.go

🔇 Additional comments (9)

test/e2e/operator_health.go (2)

14-37: LGTM!

The deployment verification test is well-structured with proper use of Eventually for retry logic and clear error messages.

39-66: LGTM!

The pod running verification with Eventually and label selector is appropriately implemented.

test/e2e/cluster_operator_status.go (6)

35-98: LGTM!

Status condition tests properly use Eventually with clear error messages including reason and message for debugging failures.

100-118: LGTM!

Version information checks are reasonable. The comment on line 116 appropriately notes that feature-gates version can be initially empty.

120-154: LGTM!

RelatedObjects verification provides good coverage for tracking expected resources.

156-183: LGTM!

Required namespaces checks validate essential infrastructure for the operator.

185-200: LGTM!

Operator Config CR verification using dynamic client is correctly implemented.

203-233: LGTM!

Helper functions are well-structured and use idiomatic Go patterns (returning pointer to slice element by index to avoid copying).

cmd/cluster-config-operator-tests-ext/main.go (1)

62-100: Implementation follows OTE patterns correctly.

The test registry setup with error handling and spec building is well-structured. The ClusterOperator/Serial suite is properly configured and will match tests tagged with [ClusterOperator][Serial] (e.g., in test/e2e/cluster_operator_status.go).

test/e2e/operator_health.go

coderabbitai

Actionable comments posted: 0

🧹 Nitpick comments (1)

test/e2e/operator_health.go (1)
156-183: Consider initializing k8sClient in BeforeEach for efficiency.

Each test in this context creates a new Kubernetes client via getKubernetesClient(). This could be moved to a BeforeEach block similar to how configClient is initialized, reducing repeated client creation overhead.
♻️ Suggested refactor
 	g.Context("Required Namespaces", func() {
+		var k8sClient kubernetes.Interface
+
+		g.BeforeEach(func() {
+			var err error
+			k8sClient, err = getKubernetesClient()
+			o.Expect(err).NotTo(o.HaveOccurred())
+		})
+
 		g.It("should have openshift-config namespace", func() {
-			k8sClient, err := getKubernetesClient()
-			o.Expect(err).NotTo(o.HaveOccurred())
-
 			ns, err := k8sClient.CoreV1().Namespaces().Get(ctx, "openshift-config", metav1.GetOptions{})
 			// ... rest of test
 		})
 		// Apply similar changes to other tests in this context
 	})

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to Reviews -> Disable Knowledge Base setting

📥 Commits

Reviewing files that changed from the base of the PR and between ae8920d and 19bcc3c.

📒 Files selected for processing (3)

cmd/cluster-config-operator-tests-ext/main.go
test/e2e/helpers.go
test/e2e/operator_health.go

🚧 Files skipped from review as they are similar to previous changes (1)

test/e2e/helpers.go

🧰 Additional context used

📓 Path-based instructions (1)

**

⚙️ CodeRabbit configuration file

-Focus on major issues impacting performance, readability, maintainability and security. Avoid nitpicks and avoid verbosity.

Files:

cmd/cluster-config-operator-tests-ext/main.go
test/e2e/operator_health.go

🧬 Code graph analysis (1)

test/e2e/operator_health.go (1)

pkg/version/version.go (1)

Get (29-41)

🔇 Additional comments (4)

cmd/cluster-config-operator-tests-ext/main.go (1)

67-93: LGTM!

The test registry setup follows proper OTE patterns: building Ginkgo specs, creating the extension, adding the suite with appropriate qualifiers matching the test tags [ClusterOperator][Serial], and registering with the registry. Error handling is correct with wrapped errors.

test/e2e/operator_health.go (3)

14-24: LGTM!

The test setup follows proper Ginkgo patterns with BeforeEach for client initialization and appropriate error handling using Gomega expectations.

35-97: LGTM!

Good use of Eventually for polling status conditions with timeout. The detailed error messages including condition reason and message will help debugging test failures. The three condition checks (Available=True, Degraded=False, Progressing=False) cover the standard operator health states.

203-233: LGTM!

The helper functions are well-structured and follow proper Go patterns for searching slices. Returning pointers to slice elements (in findCondition and findVersion) is safe since the slice lifetime encompasses the test execution.

gangwgr · 2026-01-12T13:40:10Z

/lgtm

wangke19 · 2026-01-12T15:20:36Z

/approve

openshift-ci · 2026-01-12T15:28:17Z

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: gangwgr, wangke19
Once this PR has been reviewed and has the lgtm label, please assign benluddy for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

p0lyn0mial · 2026-01-13T12:09:00Z

test/e2e/operator_health.go

+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+)
+
+var _ = g.Describe("[Operator][Serial] Operator Health", func() {


why these tests are serial and not parallel ?

I think parallel here.

p0lyn0mial · 2026-01-13T12:10:24Z

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


+	// Add test suites
+	extension.AddSuite(oteextension.Suite{
+		Name: "openshift/cluster-config-operator/Operator/Serial",


let's make operator/serial lowercase

p0lyn0mial · 2026-01-13T12:11:49Z

test/e2e/operator_health.go

+	)
+
+	g.Context("Deployment Verification", func() {
+		g.It("should have a running deployment with ready replicas", func() {


don't we have monitors for this ?
won't the operator go degraded in such a case ?

What Origin Monitor Tests Cover (Generic)

From legacycvomonitortests/operators.go and operatorstateanalyzer/:

✅ State Transitions - Track Available/Degraded/Progressing changes across ALL operators

✅ Duration Metrics - How long operators stay in bad states

✅ Known Exceptions - Platform-specific allowances (single-node, specific bugs)

✅ Regression Detection - Identify when ANY operator degrades

What they DON'T cover:

❌ Operator-specific resource creation (FeatureGate status population)

❌ Controller-specific logic (KubeCloudConfig for cloud platforms)

❌ RBAC setup verification

❌ Related objects tracking

❌ Operator Config CR structure

❌ Version reporting format

So I will remove redundant ClusterOperator status checks (Available/Degraded/Progressing) - covered by origin monitors

p0lyn0mial · 2026-01-13T12:12:34Z

test/e2e/operator_health.go

+	})
+
+	g.Context("Pod Health", func() {
+		g.It("should have running pods without fatal errors in logs", func() {


are you sure we don't have monitor test for this ?

You're absolutely correct.
Origin monitors DO have pod health checking including restart counts.

What Origin Monitors Actually Cover

Pod Health Monitoring (origin/pkg/monitortests/node/watchpods/collection.go):

✅ Container restart counts (RestartCount tracking)

✅ Container readiness (Ready/NotReady transitions)

✅ Pod phase changes (Pending, Running, etc.)

✅ Pod scheduling events

This means all pods including cluster-config-operator pods are automatically monitored for:

Excessive restarts

Readiness issues

Phase transitions

p0lyn0mial · 2026-01-14T12:17:30Z

test/e2e/operator_health.go

+	g.Context("ClusterOperator Resource", func() {
+		g.It("should exist", func() {
+			co, err := configClient.ConfigV1().ClusterOperators().Get(ctx, clusterOperatorName, metav1.GetOptions{})
+			o.Expect(err).NotTo(o.HaveOccurred(), "ClusterOperator resource should exist")


@wangke19

I am not sure if this is a good candidate for an e2e test.
If this resource does not exist, the cluster would not be created in the first place.

In my opinion e2e tests verify that a complete user flow works as expected across the entire system.
For example a test for checking encryption at rest would:

enable the encryption

check if data were encrypted in the db

Does it make sense ?

The test you added are more like "cluster monitor" tests.

Exactly! Good suggestion, I'm more concerned about how to test the current component.
What you mentioned is system testing or integration testing.

Let me re-plan. First, write the test cases, and then automate them.

openshift-ci · 2026-01-14T12:29:11Z

New changes are detected. LGTM label has been removed.

Add E2E tests for cluster-config-operator's ClusterOperator resource focusing on operator-specific metadata and configuration verification. Tests run in parallel for optimal performance. Test coverage: - Version information (operator + feature-gates versions) - RelatedObjects tracking verification - Required namespaces validation - Operator Config CR existence Test suite configuration: - Suite name: openshift/cluster-config-operator/operator/parallel - Test tags: [Operator][Parallel] - Execution mode: Parallel (read-only operations, no shared state) These tests complement origin monitors by verifying operator-specific metadata that generic monitors cannot validate. Origin monitors handle ClusterOperator status conditions (Available/Degraded/Progressing) for all operators. Files added: - test/e2e/helpers.go: Shared utilities for test client setup - test/e2e/operator_health.go: ClusterOperator verification tests - cmd/cluster-config-operator-tests-ext/dependencymagnet.go: Ginkgo dependency tracking Files modified: - cmd/cluster-config-operator-tests-ext/main.go: Register test suite with OTE framework

Add dependencies for Ginkgo v2 and Gomega testing frameworks required for ClusterOperator E2E tests. Dependencies added: - github.com/onsi/ginkgo/v2 v2.22.2: BDD testing framework - github.com/onsi/gomega v1.36.2: Assertion library - Supporting packages for HTML parsing, text encoding, and test coverage This commit contains only dependency updates (go.mod, go.sum, vendor/) to maintain clean separation from infrastructure code changes.

openshift-ci-robot · 2026-01-14T13:54:35Z

@wangke19: This pull request references CNTRLPLANE-2273 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Summary

This PR adds E2E tests for the cluster-config-operator's ClusterOperator resource, focusing on operator-specific metadata and configuration verification. Tests are designed to run in parallel for optimal performance.

What's Added

Test Coverage

Four operator-specific verification tests running in parallel:

Version Information

Validates operator version is reported

Validates feature-gates version is reported

RelatedObjects Tracking

Verifies operator.openshift.io/configs/cluster is tracked

Verifies openshift-config namespace is tracked

Verifies openshift-config-operator namespace is tracked

Required Namespaces

Ensures openshift-config namespace exists

Ensures openshift-config-managed namespace exists

Ensures openshift-config-operator namespace exists

Operator Config CR

Validates operator.openshift.io/configs/cluster CR exists

Files Added/Modified

New Files:

test/e2e/helpers.go - Shared utilities for test client setup

test/e2e/operator_health.go - ClusterOperator verification tests

cmd/cluster-config-operator-tests-ext/dependencymagnet.go - Ginkgo dependency tracking

Modified Files:

cmd/cluster-config-operator-tests-ext/main.go - Register test suite with OTE framework

Test Suite Configuration

Suite Name: openshift/cluster-config-operator/operator/parallel

Test Tags: [Operator][Parallel]

Execution Mode: Parallel (all tests are read-only and safe for concurrent execution)

Design Decisions

Parallel Execution

All tests perform read-only GET operations with no mutations or shared state, making them safe for parallel execution:

Performance: ~50-75% faster than serial execution

No race conditions: Each test creates its own client, no shared variables

Idempotent: Tests can run in any order

Focus on Operator-Specific Metadata

These tests complement (not duplicate) origin monitor tests:

Origin monitors check generic ClusterOperator status (Available/Degraded/Progressing) for ALL operators

These tests verify operator-specific metadata that monitors cannot validate:

Version information format and presence

RelatedObjects tracking correctness

Required namespace relationships

Operator Config CR structure

Testing

Run tests with:
make build
./cluster-config-operator-tests-ext run-suite openshift/cluster-config-operator/operator/parallel

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci bot requested review from benluddy and p0lyn0mial December 25, 2025 14:43

coderabbitai bot reviewed Dec 25, 2025

View reviewed changes

wangke19 changed the title ~~Add operator health E2E tests~~ [WIP]CNTRLPLANE-2273:Add operator health E2E tests Dec 25, 2025

openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 25, 2025

openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Dec 25, 2025

wangke19 force-pushed the add-operator-health-tests branch from 0b8a48c to ef7a677 Compare December 25, 2025 15:00

wangke19 force-pushed the add-operator-health-tests branch from ef7a677 to e96a577 Compare December 26, 2025 03:09

wangke19 force-pushed the add-operator-health-tests branch from e96a577 to 96a63c7 Compare January 9, 2026 13:05

wangke19 changed the title ~~[WIP]CNTRLPLANE-2273:Add operator health E2E tests~~ CNTRLPLANE-2273:Add operator health E2E tests Jan 12, 2026

openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 12, 2026

wangke19 mentioned this pull request Jan 12, 2026

Add e2e-aws-operator-serial-ote test for cluster-config-operator openshift/release#73356

Open

gangwgr reviewed Jan 12, 2026

View reviewed changes

openshift-ci bot assigned gangwgr Jan 12, 2026

openshift-ci bot added lgtm Indicates that a PR is ready to be merged. and removed lgtm Indicates that a PR is ready to be merged. labels Jan 12, 2026

coderabbitai bot reviewed Jan 12, 2026

View reviewed changes

test/e2e/operator_health.go Outdated Show resolved Hide resolved

wangke19 force-pushed the add-operator-health-tests branch from ae8920d to 19bcc3c Compare January 12, 2026 11:12

coderabbitai bot reviewed Jan 12, 2026

View reviewed changes

wangke19 changed the title ~~CNTRLPLANE-2273:Add operator health E2E tests~~ [WIP]CNTRLPLANE-2273:Add operator health E2E tests Jan 12, 2026

openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 12, 2026

wangke19 force-pushed the add-operator-health-tests branch from 19bcc3c to 545be61 Compare January 12, 2026 12:02

wangke19 changed the title ~~[WIP]CNTRLPLANE-2273:Add operator health E2E tests~~ CNTRLPLANE-2273:Add operator health E2E tests Jan 12, 2026

openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 12, 2026

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 12, 2026

p0lyn0mial reviewed Jan 13, 2026

View reviewed changes

p0lyn0mial reviewed Jan 14, 2026

View reviewed changes

openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jan 14, 2026

wangke19 force-pushed the add-operator-health-tests branch from 66e79cb to 82e0623 Compare January 14, 2026 13:13

wangke19 added 2 commits January 14, 2026 21:46

wangke19 force-pushed the add-operator-health-tests branch from 82e0623 to 28a1a59 Compare January 14, 2026 13:47

wangke19 changed the title ~~CNTRLPLANE-2273:Add operator health E2E tests~~ [WIP]CNTRLPLANE-2273:Add operator health E2E tests Jan 14, 2026

openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 14, 2026

wangke19 marked this pull request as draft January 14, 2026 14:03

[WIP]CNTRLPLANE-2273:Add operator health E2E tests #458

Are you sure you want to change the base?

[WIP]CNTRLPLANE-2273:Add operator health E2E tests #458

Uh oh!

Conversation

wangke19 commented Dec 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

What's Added

Test Coverage

Files Added/Modified

Test Suite Configuration

Design Decisions

Parallel Execution

Focus on Operator-Specific Metadata

Testing

Uh oh!

coderabbitai bot commented Dec 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

openshift-ci-robot commented Dec 25, 2025 • edited by openshift-ci bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Commit 1: Infrastructure Code

Commit 2: Dependencies

Test Coverage

Testing

References

Notes

Uh oh!

openshift-ci-robot commented Dec 25, 2025 • edited by openshift-ci bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Commit 1: Infrastructure Code

Commit 2: Dependencies

Test Coverage

Testing

References

Notes

Uh oh!

openshift-ci-robot commented Dec 25, 2025 • edited by openshift-ci bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Commit 1: Infrastructure Code

Commit 2: Dependencies

Test Coverage

Testing

References

Notes

Uh oh!

openshift-ci-robot commented Dec 25, 2025 • edited by openshift-ci bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Commit 1: Infrastructure Code

Commit 2: Dependencies

Test Coverage

Testing

References

Notes

Uh oh!

openshift-ci-robot commented Dec 26, 2025 • edited by openshift-ci bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Commit 1: Infrastructure Code

Commit 2: Dependencies

Test Coverage

Testing

References

Notes

wangke19 commented Dec 25, 2025 •

edited

Loading

coderabbitai bot commented Dec 25, 2025 •

edited

Loading

openshift-ci-robot commented Dec 25, 2025 •

edited by openshift-ci bot

Loading

openshift-ci-robot commented Dec 25, 2025 •

edited by openshift-ci bot

Loading

openshift-ci-robot commented Dec 25, 2025 •

edited by openshift-ci bot

Loading

openshift-ci-robot commented Dec 25, 2025 •

edited by openshift-ci bot

Loading

openshift-ci-robot commented Dec 26, 2025 •

edited by openshift-ci bot

Loading

openshift-ci-robot commented Dec 26, 2025 •

edited by openshift-ci bot

Loading

openshift-ci-robot commented Dec 26, 2025 •

edited by openshift-ci bot

Loading

wangke19 Jan 14, 2026 •

edited

Loading