-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
migrate secrets-store-csi-driver
jobs to community cluster
#30667
Conversation
398b9e6
to
d1b103f
Compare
d1b103f
to
51f0620
Compare
51f0620
to
5fc934f
Compare
/retest |
5fc934f
to
a17923f
Compare
@@ -435,17 +452,21 @@ presubmits: | |||
privileged: true | |||
env: | |||
- name: KUBERNETES_VERSION | |||
value: "1.24.7" | |||
value: "1.25.13" |
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.
@rjsadow could you revert these changes? I would prefer doing these bumps in a separate PR so we can isolate any errors post merge.
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
resources: | ||
limits: | ||
cpu: 2 | ||
memory: 8Gi | ||
requests: | ||
cpu: 2 | ||
memory: 8Gi |
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 requests and limits changes aren't consistent across all jobs in the PR. Should we make it consistent?
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've updated all to 4cpu 6GB memory.
Signed-off-by: rjsadow <[email protected]>
a17923f
to
7a0803b
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: rjsadow 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 |
/retest |
@aramase can you PTAL again ? 🙏🏾 |
@@ -857,6 +890,7 @@ postsubmits: | |||
- aramase | |||
- ritazh | |||
- name: secrets-store-csi-driver-e2e-gcp-postsubmit | |||
cluster: eks-prow-build-cluster |
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 gcp jobs are configured to use workload identity with the prow cluster for accessing secrets manager used in CI. Moving it to the eks prow cluster would break this job?
cc @tam7t
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.
/hold
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.
PR needs rebase. 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 moves secrets-store-csi-driver jobs to the community owned EKS cluster. Additionally, updates k8s versions in jobs, removes 1.24 and adds 1.28 job.
ref: #29722
/cc @aramase @nilekhc @ritazh @tam7t