-
Notifications
You must be signed in to change notification settings - Fork 190
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.
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
USHIFT-1948: Fail fast for the greenboot checks in scenario.sh #2660
USHIFT-1948: Fail fast for the greenboot checks in scenario.sh #2660
Conversation
ggiguash
commented
Nov 28, 2023
•
edited
edited
- When greenboot reports "failed" status, stop the wait loop
- Add functionality to reopen junit files
- Implement a w/a for USHIFT-1952 by deleting TopoLVM pods and retrying greenboot checks
Skipping CI for Draft Pull Request. |
/test microshift-metal-test |
@ggiguash: The specified target(s) for
The following commands are available to trigger optional jobs:
Use
In response to this:
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/test-infra repository. |
/test microshift-metal-tests |
dbaa37a
to
3bc072a
Compare
@ggiguash: This pull request references USHIFT-1948 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.15.0" version, but no target version was set. In response to this: 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/test-infra repository. |
6a87c09
to
e1cbc2e
Compare
@ggiguash: This pull request references USHIFT-1948 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.15.0" version, but no target version was set. In response to this:
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/test-infra repository. |
1 similar comment
@ggiguash: This pull request references USHIFT-1948 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.15.0" version, but no target version was set. In response to this:
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/test-infra repository. |
e1cbc2e
to
bffe3b6
Compare
@ggiguash: This pull request references USHIFT-1948 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.15.0" version, but no target version was set. In response to this:
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/test-infra repository. |
92e6406
to
590f6ac
Compare
/test microshift-metal-tests metal-periodic-test |
/test microshift-metal-tests metal-periodic-test |
1 similar comment
/test microshift-metal-tests metal-periodic-test |
/test microshift-metal-tests metal-periodic-test |
/test microshift-metal-tests metal-periodic-test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The retry logic looks good. I've done some different work that will conflict with the junit changes but solves the problem in a more robust way, so maybe we can agree on the order to merge.
a667749
to
be8e555
Compare
be8e555
to
74426a8
Compare
/lgtm |
/test metal-periodic-test microshift-metal-tests |
/lgtm |
/test metal-periodic-test microshift-metal-tests |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ggiguash, pmtk 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 |
/test metal-periodic-test |
/unhold |
@ggiguash: all tests passed! 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/test-infra repository. I understand the commands that are listed here. |