-
Notifications
You must be signed in to change notification settings - Fork 139
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
arm64: Update arm64 lanes to use the newer kind-1.31 cluster provider #3827
base: main
Are you sure you want to change the base?
Conversation
Skipping CI for Draft Pull Request. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/rehearse |
Rehearsal jobs created for this PR:
You can trigger rehearsal for all jobs by commenting either For a specific PR you can comment For a list of jobs that you can rehearse you can comment |
Hi Brian, I will try to fix kubevirt/kubevirt#13501 first, then verify the PR. |
|
Hi @brianmcarey , it works fine on my local environment. I have list the tests I done in the previous comment. |
Signed-off-by: Brian Carey <[email protected]>
502e878
to
34f89a0
Compare
/rehearse |
Rehearsal jobs created for this PR:
You can trigger rehearsal for all jobs by commenting either For a specific PR you can comment For a list of jobs that you can rehearse you can comment |
@0xFelix @lyarwood I am trying to bump the kind clusters used for the arm builds in common-instancetypes - it looks like it fails on a cleanup from common-instancetypes? |
Not sure that'll make a difference but can you rebase on |
/rehearse |
Rehearsal jobs created for this PR:
You can trigger rehearsal for all jobs by commenting either For a specific PR you can comment For a list of jobs that you can rehearse you can comment |
@brianmcarey: The following tests failed, say
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. |
/rehearse rehearsal-pull-kubevirt-e2e-arm64 |
@brianmcarey this looks like a failure within kubevirtci tbh, I'd like to see |
Rehearsal jobs created for this PR:
You can trigger rehearsal for all jobs by commenting either For a specific PR you can comment For a list of jobs that you can rehearse you can comment |
/rehearse pull-kubevirt-e2e-arm64 The reporting is currently not turned on for the arm64 e2e lanes as there are a couple of known test failures but heres the link to the rehearsal. |
Rehearsal jobs created for this PR:
You can trigger rehearsal for all jobs by commenting either For a specific PR you can comment For a list of jobs that you can rehearse you can comment |
Interesting, that deploys at least. I'll look more in the morning, thanks! |
The fix for one of the failures (the softboot test) is still under discussion. Other failures stem from new patches and have only appeared since yesterday. I need to investigate these issues further. You can compare the rehearsal test reports between December 18 and December 19 |
Hi, @brianmcarey @lyarwood, I have submitted a PR to use the decorator to mark arm64 related tests. We may wait for that PR get merged. kubevirt/kubevirt#13578 |
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
/cc @dhiller @zhlhahaha @lyarwood
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.
Release note: