-
Notifications
You must be signed in to change notification settings - Fork 646
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
github/workflows: remove arm64 jobs #844
Conversation
Signed-off-by: Ivan Valdes <[email protected]>
Skipping CI for Draft Pull Request. |
/test all |
/retest |
/test pull-bbolt-robustness-arm64 |
1 similar comment
/test pull-bbolt-robustness-arm64 |
@ivanvc: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. |
@jmhbnz and I have been discussing the migration of robustness tests. The current issue is that we can't install As we will need help from the infra team, James suggests reaching out to |
@ahrtr, do you know (or provide some guidance on) why the 4 CPU tests with race are getting stuck? I could add a timeout, but that would make the job fail. I'm running them in Prow using the same command we used in the GitHub workflow. |
It looks like the test infra's issue, the test process somehow terminated or being blocked. 1000 concurrent goroutines shouldn't cause too much resource pressure. Not sure whether we can request some help from test-infra, probably they have more insight.
|
Signed-off-by: Ivan Valdes <[email protected]>
Signed-off-by: Ivan Valdes <[email protected]>
1d7a65f
to
a1d8a84
Compare
/help test |
I'll mark this PR ready for review to check the triggered prow jobs. |
@ahrtr, other than the comment on the |
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.
Thanks
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ahrtr, ivanvc 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 |
Remove the ARM64 jobs from GitHub workflows. Add a comment regarding the use of sudo in robustness tests.
Link to #848