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
Prepare Cluster API periodics and presubmit for v1alpha4 #19424
Conversation
94dfdb1
to
050288a
Compare
/test pull-test-infra-bazel |
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 for adding this @vincepri i did not have time :/
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cpanato, vincepri 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 |
/hold for the assigned persons to review |
/test pull-test-infra-bazel |
Not sure why the test above is failing, seems maybe unrelated 🤔 ? |
yep, looks like, because afaik this job runs the go tests, but I can be completely wrong on that |
config/jobs/kubernetes-sigs/cluster-api/cluster-api-periodics-main.yaml
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,160 @@ | |||
presubmits: | |||
kubernetes-sigs/cluster-api: | |||
- name: pull-cluster-api-build |
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.
Do you want all these to have the -main suffix?
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.
Maybe? No strong opinions here, up to you :)
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.
I think all the other jobs either have -main or -release-0-3 on them?
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.
Done!
@@ -59,15 +61,16 @@ presubmits: | |||
image: gcr.io/k8s-testimages/kubekins-e2e:v20200929-82b41a1-1.18 | |||
annotations: | |||
testgrid-dashboards: sig-cluster-lifecycle-cluster-api | |||
testgrid-tab-name: pr-apidiff | |||
- name: pull-cluster-api-verify | |||
testgrid-tab-name: capi-pr-apidiff-release-0-3 |
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.
I see you're renaming the PR tabs to be prefixed with capi-. Any particular reason/need for this change?
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.
Mostly for consistency, and if we ever also add the providers here, it'll be already separated
050288a
to
f253009
Compare
Signed-off-by: Vince Prignano <vincepri@vmware.com>
f253009
to
df79d57
Compare
/lgtm |
@@ -0,0 +1,83 @@ | |||
periodics: | |||
- name: periodic-cluster-api-test-release-0-3 | |||
interval: 1h |
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.
I think we can increase the frequency for v0.3 jobs (applies to all)
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.
Let's do it in another PR?
/hold cancel |
@vincepri: Updated the
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. |
This PR splits all periodics and presubmit between release / main branches, and changes the images to the proper version we'll use next.
/assign @ncdc @detiber