Skip to content

NO-ISSUE: Remove test case 62738 from ginkgo tests#5785

Merged
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
agullon:patch-30
Nov 23, 2025
Merged

NO-ISSUE: Remove test case 62738 from ginkgo tests#5785
openshift-merge-bot[bot] merged 1 commit intoopenshift:mainfrom
agullon:patch-30

Conversation

@agullon
Copy link
Contributor

@agullon agullon commented Nov 21, 2025 •
edited
Loading

@agullon
Remove test case 62738 from ginkgo tests
4eaedb3 
Remove test case 62738 from the selected cases.
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Nov 21, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 21, 2025

@agullon: This pull request explicitly references no jira issue.

Details

In response to this:

Remove test case 62738 from the selected cases.

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-ci bot requested review from eslutsky and jogeo November 21, 2025 14:29
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 21, 2025
@agullon
Copy link
Contributor Author

agullon commented Nov 21, 2025

/test e2e-aws-tests-bootc-release-arm e2e-aws-tests-bootc-release e2e-aws-tests-release e2e-aws-tests-release-arm

pacevedom
pacevedom approved these changes Nov 21, 2025
View reviewed changes
Copy link
Contributor

@pacevedom pacevedom left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 21, 2025
@jogeo
Copy link
Contributor

jogeo commented Nov 21, 2025

/lgtm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 21, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: agullon, jogeo, pacevedom

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:
  • OWNERS [agullon,jogeo,pacevedom]

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

@agullon
Copy link
Contributor Author

agullon commented Nov 21, 2025

/retest

1 similar comment
@agullon
Copy link
Contributor Author

agullon commented Nov 21, 2025

/retest

@pacevedom
Copy link
Contributor

pacevedom commented Nov 21, 2025

/verified by CI

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Nov 21, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 21, 2025

@pacevedom: This PR has been marked as verified by CI.

Details

In response to this:

/verified by CI

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
Copy link

openshift-ci-robot commented Nov 22, 2025

/retest-required

Remaining retests: 0 against base HEAD 5dcdcb5 and 2 for PR HEAD 4eaedb3 in total

@ggiguash
Copy link
Contributor

ggiguash commented Nov 22, 2025

@agullon , we need to document this removal:

  • Add a comment in the code on why we are doing it
  • Potentially create a JIRA issue with the problem description (especially if we plan to backport the change)

/hold

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 22, 2025
@agullon
Copy link
Contributor Author

agullon commented Nov 22, 2025

/retest

4 similar comments
@agullon
Copy link
Contributor Author

agullon commented Nov 22, 2025

/retest

@agullon
Copy link
Contributor Author

agullon commented Nov 22, 2025

/retest

@agullon
Copy link
Contributor Author

agullon commented Nov 22, 2025

/retest

@agullon
Copy link
Contributor Author

agullon commented Nov 22, 2025

/retest

@pacevedom
Copy link
Contributor

pacevedom commented Nov 22, 2025

/hold cancel

@openshift-ci openshift-ci bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Nov 22, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Nov 23, 2025

/retest-required

Remaining retests: 0 against base HEAD 5f1a885 and 1 for PR HEAD 4eaedb3 in total

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 23, 2025

@agullon: 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.

@openshift-merge-bot openshift-merge-bot bot merged commit 728a9b4 into openshift:main Nov 23, 2025
17 checks passed
@agullon
Copy link
Contributor Author

agullon commented Nov 24, 2025

@agullon , we need to document this removal:

  • Add a comment in the code on why we are doing it
  • Potentially create a JIRA issue with the problem description (especially if we plan to backport the change)

/hold

Jira ticket created to remove this change and addressed properly: https://issues.redhat.com/browse/USHIFT-6340

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

Reviewers

@eslutsky eslutsky Awaiting requested review from eslutsky

@jogeo jogeo Awaiting requested review from jogeo

1 more reviewer

@pacevedom pacevedom pacevedom approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jogeo jogeo

@pacevedom pacevedom

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@agullon @openshift-ci-robot @jogeo @pacevedom @ggiguash