-
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 sig-network
jobs to community cluster
#30761
Conversation
@@ -185,6 +186,13 @@ periodics: | |||
- --provider=gce | |||
- --test_args=--ginkgo.focus=\[Feature:Ingress\]|\[Feature:NEG\] | |||
- --timeout=320m | |||
resources: |
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.
Can we put some discussion of why these resources need to be added and where they are coming from in the commit message?
e.g. so git blame can be used to figure out why these exist and how they were obtained.
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.
Sure
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.
Updated.
This change moves several sig-network jobs from the Google owned GKE cluster to the community owned GKE cluster. All jobs running in community owned clusters must set resource quotas so that the autoscalers work correctly. These resources were chosen as a best guess based on related jobs in sig-networking and guidance from https://github.com/kubernetes/test-infra/blob/master/docs/eks-jobs-migration.md. Signed-off-by: rjsadow <[email protected]>
6f47ec0
to
ec30645
Compare
/retest |
/lgtm moving the ci jobs only |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, rjsadow 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 |
@rjsadow: 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 moves the sig-network jobs to the community owned cluster.
ref: #30277
/cc @aojea @bowei @cadmuxe @MrHohn @rramkumar1
/cc @ameukam @dims