Skip to content

Drop legacy entry point#261

Draft
mandre wants to merge 3 commits into
openshift:mainfrom
shiftstack:drop-legacy-entry-point
Draft

Drop legacy entry point#261
mandre wants to merge 3 commits into
openshift:mainfrom
shiftstack:drop-legacy-entry-point

Conversation

@mandre
Copy link
Copy Markdown
Member

@mandre mandre commented May 21, 2026

Depends on #260 and openshift/release#79487.

mandre added 3 commits May 21, 2026 10:48
Add cmd/extension/main.go as a new OTE-based test extension binary
alongside the existing openshift-tests entry point. This registers
all openstack tests with the OTE framework, adds [OTP] annotations
to all Describe blocks, and integrates the extension build into the
Makefile and Containerfile.

The OTE binary discovers 51 tests and does not pull in origin's
monitor test framework, avoiding the duplicate test results seen
with the old entry point.
The old cmd/openshift-tests/ entry point used origin's test runner
framework which unconditionally loaded 40+ monitor tests (disruption
analyzers, termination message policy checks, etc.), producing
duplicate/unwanted JUnit results in CI alongside the actual openstack
tests.

Remove the old entry point and all its supporting code:
- cmd/openshift-tests/
- pkg/cmd/openshift-tests/ and pkg/testsuites/
- test/extended/util/annotate/ (annotation generator)
- test/extended/include.go

This shrinks the vendored origin footprint from 29MB/796 files to
2.5MB/58 files, as only test/extended/util (CLI helpers) is still
needed.
Bump openshift/origin from 140c6726a606 (Jun 2025) to
4486518a1248 (May 2026), upgrading k8s.io/kubernetes from
v1.32.5 to v1.35.1 and all related dependencies.
@mandre mandre marked this pull request as draft May 21, 2026 09:48
@openshift-ci 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 May 21, 2026
@openshift-ci openshift-ci Bot requested review from gryf and itzikb-redhat May 21, 2026 09:49
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented May 21, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign mandre 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:

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant