-
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
Move root CI lanes to 1.31 #3841
base: main
Are you sure you want to change the base?
Conversation
@dhiller, does this look right? |
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.
Hi @rthallisey - thanks for the PR.
We won't require the changes to the release branches as those will have older kubevirtci versions so these changes can be dropped
Just need to focus on jobs on the main branch so the changes in github/ci/prow-deploy/files/jobs/kubevirt/kubevirt/kubevirt-presubmits.yaml look good.
There are some periodic lanes that run against main which will need to be bumped -
project-infra/github/ci/prow-deploy/files/jobs/kubevirt/kubevirt/kubevirt-periodics.yaml
Line 1437 in 7037fe3
name: periodic-kubevirt-e2e-k8s-1.29-sig-storage-root |
/rehearse ? |
Rehearsal is available for the following jobs in 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-k8s-1.31-sig-compute-root |
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 |
The failure we see on the sig-storage-root lane is known - we see the same on the periodic. |
@rthallisey sorry, forgot to mention that the periodic root lanes need the same bumping:
|
Signed-off-by: Ryan Hallisey <[email protected]>
c2d5986
to
53950a6
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 |
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.
/approve
/hold to be removed if the rehearsals only show the usual behavior.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dhiller 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 |
/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 |
/rehearse |
@acardace can help run the rehearsal. If that doesn't work, ping someone from this list:
|
/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 |
@rthallisey: 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. |
What this PR does / why we need it:
Move Root CI lanes to 1.31 k8s provider
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:
Checklist
Release note: