Skip to content

eni: update limits list#11793

Merged
borkmann merged 1 commit intocilium:masterfrom
DataDog:bp/update-limits
Jun 3, 2020
Merged

eni: update limits list#11793
borkmann merged 1 commit intocilium:masterfrom
DataDog:bp/update-limits

Conversation

@bpineau
Copy link
Copy Markdown
Contributor

@bpineau bpineau commented May 30, 2020

Re-sync the AWS ENI limits list with
https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-eni.html?shortFooter=true#AvailableIpPerENI

For those not using --update-ec2-apdater-limit-via-api CLI option.

Update ENI limits list

Re-sync the AWS ENI limits list with
https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-eni.html?shortFooter=true#AvailableIpPerENI

For those not using `--update-ec2-apdater-limit-via-api`.

Signed-off-by: Benjamin Pineau <benjamin.pineau@datadoghq.com>
@bpineau bpineau requested a review from a team as a code owner May 30, 2020 13:53
@maintainer-s-little-helper
Copy link
Copy Markdown

Please set the appropriate release note label.

@coveralls
Copy link
Copy Markdown

coveralls commented May 30, 2020

Coverage Status

Coverage decreased (-0.02%) to 36.897% when pulling e04b4f2 on DataDog:bp/update-limits into b5ed91d on cilium:master.

@ungureanuvladvictor ungureanuvladvictor added the release-note/misc This PR makes changes that have no direct user impact. label May 31, 2020
@ungureanuvladvictor ungureanuvladvictor added the area/eni Impacts ENI based IPAM. label May 31, 2020
@ungureanuvladvictor
Copy link
Copy Markdown
Member

test-me-please

@christarazi
Copy link
Copy Markdown
Member

retest-runtime

@christarazi
Copy link
Copy Markdown
Member

retest-4.9

@bpineau
Copy link
Copy Markdown
Contributor Author

bpineau commented Jun 2, 2020

Thanks for the reviews.
Would it be possible to cherry-pick this to the upcoming 1.8?

@christarazi
Copy link
Copy Markdown
Member

retest-runtime

@christarazi
Copy link
Copy Markdown
Member

retest-runtime

Thanks for the reviews.
Would it be possible to cherry-pick this to the upcoming 1.8?

If you mean that you'd like this in 1.8, we can backport.

@maintainer-s-little-helper maintainer-s-little-helper Bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 3, 2020
@borkmann borkmann merged commit b7c7fec into cilium:master Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/eni Impacts ENI based IPAM. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants