Skip to content

OSASINFRA-4375: Drop Legacy openshift-test entry point: release-4.21 - #306

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:release-4.21from
shiftstack:drop-legacy-openstack-test-4.21
Aug 4, 2026
Merged

OSASINFRA-4375: Drop Legacy openshift-test entry point: release-4.21#306
openshift-merge-bot[bot] merged 1 commit into
openshift:release-4.21from
shiftstack:drop-legacy-openstack-test-4.21

Conversation

@danchild

@danchilddanchild commented Jul 29, 2026•
edited
Loading

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

Dependencies:
openshift/release/pull/82506
openshift/release/pull/82674

@openshift-ciopenshift-ciBot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 29, 2026
@openshift-ci

openshift-ciBot commented Jul 29, 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

@danchild
Drop Legacy openshift-test entry point: release-4.21
224afe4
Following the migration of openstack-test to the
OTE framework, the entry point is no longer needed.
Changes:
- Remove `openstack-test` binary and related libs
in `pkg`
- 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>
@danchild
danchildforce-pushed the drop-legacy-openstack-test-4.21 branch from 3025e1c to 224afe4CompareJuly 29, 2026 14:53
@danchilddanchild changed the title Drop Legacy openshift-test entry point: release-4.21OSASINFRA-4375: Drop Legacy openshift-test entry point: release-4.21Jul 30, 2026
@openshift-ci-robot

openshift-ci-robot commented Jul 30, 2026•
edited by openshift-ci Bot
Loading

Copy link
Copy Markdown

@danchild: This pull request references OSASINFRA-4375 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 story to target the "4.21.z" version, but no target version was set.

Details

In response to this:

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: depends on openshift/release#82506; 4.21 release jobs need to drop callers of the legacy openstack-tests binary before this can be merged.

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-robotopenshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jul 30, 2026
@danchild
danchild marked this pull request as ready for review July 30, 2026 15:22
@openshift-ciopenshift-ciBot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 30, 2026
@openshift-ci
openshift-ciBot requested review from eshulman2 and gryfJuly 30, 2026 15:25
@ekuris-redhat

ekuris-redhat commented Jul 31, 2026

Copy link
Copy Markdown

/test test
/test build
/test images

1 similar comment
@danchild

danchild commented Aug 3, 2026

Copy link
Copy Markdown
MemberAuthor

/test test
/test build
/test images

@danchild

danchild commented Aug 3, 2026

Copy link
Copy Markdown
MemberAuthor

/verified by @danchild

@openshift-ci-robotopenshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Aug 3, 2026
@openshift-ci-robot

openshift-ci-robot commented Aug 3, 2026

Copy link
Copy Markdown

@danchild: This PR has been marked as verified by @danchild.

Details

In response to this:

/verified by @danchild

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

openshift-ciBot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

@danchild: all tests passed!

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.

ekuris-redhat
ekuris-redhat approved these changes Aug 4, 2026
View reviewed changes

@ekuris-redhatekuris-redhat left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@openshift-ciopenshift-ciBot added the lgtm Indicates that a PR is ready to be merged. label Aug 4, 2026
@ekuris-redhat

ekuris-redhat commented Aug 4, 2026

Copy link
Copy Markdown

/lgtm cancel

@openshift-ciopenshift-ciBot removed the lgtm Indicates that a PR is ready to be merged. label Aug 4, 2026
@gryf

gryf commented Aug 4, 2026

Copy link
Copy Markdown
Member

/lgtm
/approve

@openshift-ciopenshift-ciBot added the lgtm Indicates that a PR is ready to be merged. label Aug 4, 2026
@openshift-ci

openshift-ciBot commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ekuris-redhat, gryf

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

The pull request process is described 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

@openshift-ciopenshift-ciBot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 4, 2026
@openshift-merge-bot
openshift-merge-botBot merged commit a805651 into openshift:release-4.21Aug 4, 2026
5 checks passed
@openshift-merge-bot
openshift-merge-botBot deleted the drop-legacy-openstack-test-4.21 branch August 4, 2026 09:32
@danchilddanchild mentioned this pull request Aug 7, 2026
Merged
5 tasks
@danchilddanchild mentioned this pull request Aug 12, 2026
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eshulman2eshulman2Awaiting requested review from eshulman2
@gryfgryfAwaiting requested review from gryf
1 more reviewer
@ekuris-redhatekuris-redhatekuris-redhat approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@gryfgryf

@ekuris-redhatekuris-redhat

Labels

approvedIndicates a PR has been approved by an approver from all required OWNERS files.jira/valid-referenceIndicates that this PR references a valid Jira ticket of any type.lgtmIndicates that a PR is ready to be merged.verifiedSignifies that the PR passed pre-merge verification criteria

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@danchild@openshift-ci-robot@ekuris-redhat@gryf