Skip to content

Drop Legacy openshift-test entry point: releases 4.23+ - #303

Draft
danchild wants to merge 1 commit into
openshift:mainfrom
shiftstack:drop-legacy-openstack-test-main
Draft

Drop Legacy openshift-test entry point: releases 4.23+#303
danchild wants to merge 1 commit into
openshift:mainfrom
shiftstack:drop-legacy-openstack-test-main

Conversation

@danchild

@danchild danchild commented Jul 28, 2026

Copy link
Copy Markdown
Member

Following the migration of openstack-test to the
OTE framework, the entry point is no longer needed.

Changes:

  • Remove openshift-tests builder and entry point in
    the Containerfile
  • Remove stale openshift-tests references for make tooling
  • Update README.md with new instructions on how to manually
    invoke the new OTE binary
  • Update module and vendored dependencies

NOTE: openshift/release 4.23+ release jobs need to drop callers of the legacy openstack-tests binary before this can be merged.

Following the migration of openstack-test to the
OTE framework, the entry point is no longer needed.

Changes:
- Remove openshift-tests builder and entry point in
  the Containerfile
- Remove stale openshift-tests references for make tooling
- Update README.md with new instructions on how to manually
  invoke the new OTE binary
- update module and vendored dependencies

Signed-off-by: Dan Childers <dchilder@redhat.com>
@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 Jul 28, 2026
@openshift-ci

openshift-ci Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci

openshift-ci Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

[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 eshulman2 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

@danchild
danchild marked this pull request as ready for review July 28, 2026 17:40
@openshift-ci 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 Jul 28, 2026
@openshift-ci
openshift-ci Bot requested review from itzikb-redhat and rlobillo July 28, 2026 17:40
@danchild danchild changed the title Drop Legacy openshift-test entry point: release 4.23+ Drop Legacy openshift-test entry point: releases 4.23+ Jul 28, 2026
@openshift-ci

openshift-ci Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

@danchild: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/test f694ed3 link true /test test
ci/prow/build f694ed3 link true /test build
ci/prow/images f694ed3 link true /test images

Full PR test history. Your PR dashboard.

Details

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 kubernetes-sigs/prow repository. I understand the commands that are listed here.

@danchild
danchild marked this pull request as draft July 28, 2026 20:40
@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 Jul 28, 2026
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