Skip to content

[release-4.20] Add timeout to metrics query and cleanup job failure logs in e2e test #3389

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: release-4.20
Choose a base branch
from
Open

[release-4.20] Add timeout to metrics query and cleanup job failure logs in e2e test #3389

wants to merge 2 commits into from

Conversation

@openshift-cherrypick-robot
Copy link

@openshift-cherrypick-robot openshift-cherrypick-robot commented Oct 1, 2025

This is an automated cherry-pick of #3378

/assign mansikulkarni96

sebsoto added 2 commits October 1, 2025 17:15
@sebsoto
Add timeout to metrics query
369f054 
Prevents client.Do from blocking indefinitely.
@sebsoto
Cleanup job failure logs
67af946 
When a job fails a wall of text is dumped in the test job logs,
including the full job spec as well as all events associated with it.
This hasn't proven to be useful for debugging, and makes the errors
harder to read.

All of this information can be found elsewhere in the artifact
directory.
@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Oct 1, 2025
Merged
@mansikulkarni96
Copy link
Member

mansikulkarni96 commented Oct 1, 2025

/approve
/lgtm

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 1, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mansikulkarni96, openshift-cherrypick-robot

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

The pull request process is described here

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-ci openshift-ci bot added approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. labels Oct 1, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 1, 2025

/retest-required

Remaining retests: 0 against base HEAD a3765cc and 2 for PR HEAD 67af946 in total

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 2, 2025

/retest-required

Remaining retests: 0 against base HEAD ce80213 and 1 for PR HEAD 67af946 in total

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 2, 2025

/retest-required

Remaining retests: 0 against base HEAD aa95c72 and 0 for PR HEAD 67af946 in total

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 2, 2025

/hold

Revision 67af946 was retested 3 times: holding

@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 Oct 2, 2025
@sebsoto
Copy link
Contributor

sebsoto commented Oct 9, 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 Oct 9, 2025
@sebsoto
Copy link
Contributor

sebsoto commented Oct 9, 2025

/override ci/prow/azure-e2e-upgrade ci/prow/nutanix-e2e-operator

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 9, 2025

@sebsoto: Overrode contexts on behalf of sebsoto: ci/prow/azure-e2e-upgrade, ci/prow/nutanix-e2e-operator

In response to this:

/override ci/prow/azure-e2e-upgrade ci/prow/nutanix-e2e-operator

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.

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 9, 2025

/retest-required

Remaining retests: 0 against base HEAD 2c7a080 and 2 for PR HEAD 67af946 in total

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 9, 2025

/retest-required

Remaining retests: 0 against base HEAD f664abd and 1 for PR HEAD 67af946 in total

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 16, 2025

/retest-required

Remaining retests: 0 against base HEAD 0215b71 and 0 for PR HEAD 67af946 in total

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 16, 2025

/hold

Revision 67af946 was retested 3 times: holding

@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 Oct 16, 2025
@jrvaldes
Copy link
Contributor

jrvaldes commented Oct 23, 2025

/override ci/prow/azure-e2e-upgrade

@jrvaldes
Copy link
Contributor

jrvaldes commented Oct 23, 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 Oct 23, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 23, 2025

@jrvaldes: Overrode contexts on behalf of jrvaldes: ci/prow/azure-e2e-upgrade

In response to this:

/override ci/prow/azure-e2e-upgrade

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.

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 23, 2025

/retest-required

Remaining retests: 0 against base HEAD 3556d8e and 2 for PR HEAD 67af946 in total

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 25, 2025

/retest-required

Remaining retests: 0 against base HEAD 96c0fa3 and 1 for PR HEAD 67af946 in total

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 25, 2025

/retest-required

Remaining retests: 0 against base HEAD f9274eb and 0 for PR HEAD 67af946 in total

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Oct 25, 2025

@openshift-cherrypick-robot: The following test 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/nutanix-e2e-operator 67af946 link true /test nutanix-e2e-operator

Full PR test history. Your PR dashboard.

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

openshift-ci-robot commented Oct 25, 2025

/hold

Revision 67af946 was retested 3 times: holding

@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 Oct 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jrvaldes jrvaldes Awaiting requested review from jrvaldes

@mansikulkarni96 mansikulkarni96 Awaiting requested review from mansikulkarni96

Assignees

@mansikulkarni96 mansikulkarni96

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@openshift-cherrypick-robot @mansikulkarni96 @openshift-ci-robot @sebsoto @jrvaldes