Migrate resource size job to kops#36401
Conversation
k8s-ci-robot
added
size/L
serathius
force-pushed
the
resource-size-kops
branch
2 times, most recently
from
340f336 to
69da2b5
Compare
upodroid
left a comment
upodroid
left a comment
There was a problem hiding this comment.
Can you change the cron for this job to:
'1 17 1-31/1 * *' # Run on odd days at 9:01PST (17:01 UTC)
| value: "true" | ||
| - name: MASTER_SIZE | ||
| value: "n2-standard-64" # 64 vCPUs, 256GB memory | ||
| - name: ETCD_QUOTA_BACKEND_BYTES # 20GB |
There was a problem hiding this comment.
I need to amend this code https://github.com/kubernetes/kops/blob/master/tests/e2e/scenarios/scalability/run-test.sh#L86 to enable setting this.
/hold
| resources: | ||
| requests: | ||
| cpu: 6 | ||
| cpu: 7 |
There was a problem hiding this comment.
Set the memory requests/limits to 26GB please
There was a problem hiding this comment.
resource size job needed more memory, as migration to kops also increased memory I don't think decreasing here will work.
k8s-ci-robot
added
the
do-not-merge/hold
serathius
force-pushed
the
resource-size-kops
branch
from
69da2b5 to
5a4e427
Compare
|
kubernetes/kops#17929 was merged, please take another look. |
k8s-ci-robot
added
lgtm
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: serathius, upodroid The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
k8s-ci-robot
added
the
approved
|
@serathius: Updated the
DetailsIn 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-sigs/prow repository. |
3 participants
/cc @upodroid