Skip to content

run make generate-limits#2835

Merged
orsenthil merged 2 commits intoaws:masterfrom
jaydeokar:master
Mar 14, 2024
Merged

run make generate-limits#2835
orsenthil merged 2 commits intoaws:masterfrom
jaydeokar:master

Conversation

@jaydeokar
Copy link
Copy Markdown
Collaborator

What type of PR is this?
cleanup

Which issue does this PR fix?:

N/A

What does this PR do / Why do we need it?:
This updates the instance types and the limits

Testing done on this change:
N/A

go run  scripts/gen_vpc_ip_limits.go

Adding "bmn-sf1.metal": {15 50 0 [] "unknown" true} since it is missing from the API
Adding "cr1.8xlarge": {8 30 0 [] "unknown" false} since it is missing from the API
Adding "u-12tb1.metal": {5 30 0 [] "unknown" true} since it is missing from the API
Adding "u-24tb1.metal": {15 50 0 [] "unknown" true} since it is missing from the API
Adding "u-6tb1.metal": {5 30 0 [] "unknown" true} since it is missing from the API
Adding "u-9tb1.metal": {5 30 0 [] "unknown" true} since it is missing from the API
Replacing API value {15 50 0 [{15 0 }] "unknown" true} with override {15 50 0 [] "nitro" true} for "c7g.metal"
Adding "hs1.8xlarge": {8 30 0 [] "unknown" false} since it is missing from the API
Adding "u-18tb1.metal": {15 50 0 [] "unknown" true} since it is missing from the API
Adding "c5a.metal": {15 50 0 [] "unknown" true} since it is missing from the API
Adding "c5ad.metal": {15 50 0 [] "unknown" true} since it is missing from the API
Adding "p4de.24xlarge": {15 50 0 [] "nitro" false} since it is missing from the API
{"level":"info","ts":"2024-03-08T14:48:23.133-0800","caller":"/opt/homebrew/Cellar/go/1.21.4/libexec/src/runtime/proc.go:267","msg":"Generated pkg/vpc/vpc_ip_resource_limit.go"}
{"level":"info","ts":"2024-03-08T14:48:23.137-0800","caller":"/opt/homebrew/Cellar/go/1.21.4/libexec/src/runtime/proc.go:267","msg":"Generated misc/eni-max-pods.txt"}

Will this PR introduce any new dependencies?:
N/A

Will this break upgrades or downgrades? Has updating a running cluster been tested?:
N/A

Does this change require updates to the CNI daemonset config files to work?:

N/A
Does this PR introduce any user-facing change?:
N/A

Update instance type networking limits.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jaydeokar jaydeokar requested a review from a team as a code owner March 8, 2024 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants