Create separate Go module for test extension

This commit isolates test dependencies from production code by creating a separate
Go module for the test extension at test/extended/tests-extension/.

Key changes:
- Move test files to test/extended/tests-extension/ directory structure
- Create separate go.mod for test dependencies (ginkgo, gomega, openshift-tests-extension)
- Remove test dependencies from root go.mod (~XXX lines removed from vendor)
- Update root Makefile to use delegation pattern for test extension targets
- Update Dockerfile.rhel7 paths to reference new binary location
- Update .gitignore to ignore test/extended/tests-extension/bin/

Benefits:
- Smaller production images (test dependencies not included in production builds)
- Faster builds (production builds don't need to vendor test dependencies)
- Cleaner dependency management (test dependencies isolated)
- Better CI performance (smaller images, faster pulls, less storage)

Related: Similar to openshift/openshift-apiserver#571

Author: gangwgr

.gitignore

@@ -1,7 +1,6 @@
 /cluster-kube-controller-manager-operator
-/cluster-kube-controller-manager-operator-tests-ext
 .idea/
 _output
 telepresence.log
-.openshift-tests-extension/openshift_payload_*.json
+/test/extended/tests-extension/bin/
 

Dockerfile.rhel7

@@ -1,16 +1,16 @@
 FROM registry.ci.openshift.org/ocp/builder:rhel-9-golang-1.24-openshift-4.21 AS builder
 WORKDIR /go/src/github.com/openshift/cluster-kube-controller-manager-operator
 COPY . .
-RUN make build --warn-undefined-variables
-RUN make tests-ext-build --warn-undefined-variables \
-    && gzip cluster-kube-controller-manager-operator-tests-ext
+RUN make build --warn-undefined-variables \
+    && make tests-ext-build \
+    && gzip test/extended/tests-extension/bin/cluster-kube-controller-manager-operator-tests-ext
 
 FROM registry.ci.openshift.org/ocp/4.21:base-rhel9
 RUN mkdir -p /usr/share/bootkube/manifests/bootstrap-manifests/ /usr/share/bootkube/manifests/config/ /usr/share/bootkube/manifests/manifests/
 COPY --from=builder /go/src/github.com/openshift/cluster-kube-controller-manager-operator/bindata/bootkube/bootstrap-manifests /usr/share/bootkube/manifests/bootstrap-manifests/
 COPY --from=builder /go/src/github.com/openshift/cluster-kube-controller-manager-operator/bindata/bootkube/config /usr/share/bootkube/manifests/config/
 COPY --from=builder /go/src/github.com/openshift/cluster-kube-controller-manager-operator/bindata/bootkube/manifests /usr/share/bootkube/manifests/manifests/
 COPY --from=builder /go/src/github.com/openshift/cluster-kube-controller-manager-operator/cluster-kube-controller-manager-operator /usr/bin/
-COPY --from=builder /go/src/github.com/openshift/cluster-kube-controller-manager-operator/cluster-kube-controller-manager-operator-tests-ext.gz /usr/bin/
+COPY --from=builder /go/src/github.com/openshift/cluster-kube-controller-manager-operator/test/extended/tests-extension/bin/cluster-kube-controller-manager-operator-tests-ext.gz /usr/bin/
 COPY manifests /manifests
 LABEL io.openshift.release.operator true

test/extended/README.md

@@ -0,0 +1,83 @@
+# Get the directory where this Makefile is, so we can use it below for including
+DIR := $(strip $(shell dirname $(realpath $(lastword $(MAKEFILE_LIST)))))
+
+# Definitions for the extended tests
+GO_PKG_NAME := github.com/openshift-eng/openshift-tests-extension
+
+GIT_COMMIT := $(shell git rev-parse --short HEAD 2>/dev/null || echo 'unknown')
+BUILD_DATE := $(shell date -u +'%Y-%m-%dT%H:%M:%SZ')
+GIT_TREE_STATE := $(shell if git rev-parse --git-dir > /dev/null 2>&1; then if git diff --quiet; then echo clean; else echo dirty; fi; else echo unknown; fi)
+
+LDFLAGS := -X '$(GO_PKG_NAME)/pkg/version.CommitFromGit=$(GIT_COMMIT)' \
+           -X '$(GO_PKG_NAME)/pkg/version.BuildDate=$(BUILD_DATE)' \
+           -X '$(GO_PKG_NAME)/pkg/version.GitTreeState=$(GIT_TREE_STATE)'
+
+METADATA := $(shell pwd)/.openshift-tests-extension
+
+TOOLS_BIN_DIR := $(CURDIR)/bin
+BINARY_NAME := cluster-kube-controller-manager-operator-tests-ext
+
+.PHONY: help
+help: #HELP Display essential help.
+	@awk 'BEGIN {FS = ":[^#]*#HELP"; printf "\nUsage:\n  make \033[36m<target>\033[0m\n\n"} /^[a-zA-Z_0-9-]+:.*#HELP / { printf "  \033[36m%-17s\033[0m %s\n", $$1, $$2 } ' $(MAKEFILE_LIST)
+
+#SECTION Development
+.PHONY: verify #HELP To verify the code
+verify: tidy fmt vet
+
+.PHONY: tidy #HELP To tidy the go.mod and go.sum files
+tidy:
+	go mod tidy
+
+.PHONY: fmt #HELP To run go fmt
+fmt:
+	go fmt ./...
+
+.PHONY: vet #HELP To run go vet
+vet:
+	go vet ./...
+
+#SECTION Build
+.PHONY: build
+build: #HELP Build the extended tests binary
+	@mkdir -p $(TOOLS_BIN_DIR)
+	GO_COMPLIANCE_POLICY="exempt_all" CGO_ENABLED=0 go build -mod=mod -ldflags "$(LDFLAGS)" -o $(TOOLS_BIN_DIR)/$(BINARY_NAME) ./cmd/...
+
+.PHONY: update-metadata
+update-metadata: build #HELP Build and run 'update-metadata' to generate test metadata
+	$(TOOLS_BIN_DIR)/$(BINARY_NAME) update --output-dir=$(METADATA)
+
+.PHONY: build-update
+build-update: build #HELP Build and update test metadata (strips machine-specific codeLocations)
+	$(TOOLS_BIN_DIR)/$(BINARY_NAME) update --output-dir=$(METADATA)
+	for f in $(METADATA)/*.json; do \
+		jq 'map(del(.codeLocations))' "$$f" > tmpp && mv tmpp "$$f"; \
+	done
+
+.PHONY: clean
+clean: #HELP Remove the bin directory
+	rm -rf $(TOOLS_BIN_DIR)
+
+.PHONY: run-suite
+run-suite: build #HELP Run a test suite (usage: make run-suite SUITE=<suite-name> [JUNIT_DIR=<dir>])
+	@if [ -z "$(SUITE)" ]; then \
+		echo "Error: SUITE variable is required. Usage: make run-suite SUITE=<suite-name> [JUNIT_DIR=<dir>]"; \
+		exit 1; \
+	fi
+	@JUNIT_ARG=""; \
+	if [ -n "$(JUNIT_DIR)" ]; then \
+		mkdir -p $(JUNIT_DIR); \
+		JUNIT_ARG="--junit-path=$(JUNIT_DIR)/junit.xml"; \
+	fi; \
+	$(TOOLS_BIN_DIR)/$(BINARY_NAME) run-suite $(SUITE) $$JUNIT_ARG
+
+.PHONY: list-test-names
+list-test-names: build #HELP List all test names
+	$(TOOLS_BIN_DIR)/$(BINARY_NAME) list
+
+.PHONY: verify-metadata
+verify-metadata: build #HELP Verify metadata is up to date
+	@echo "Verifying metadata is up to date..."
+	@$(TOOLS_BIN_DIR)/$(BINARY_NAME) update --output-dir=$(METADATA) --dry-run || \
+		(echo "Error: metadata is not up to date. Run 'make build-update' to update it."; exit 1)
+	@echo "Metadata is up to date."