Use the AWS API to retrieve instance network limits

lbernail commented 4 years ago

This PR has 2 commits:

Dynamically get instance network limits

Update the AWS sdk and dependencies
Get network limits using the DescribeInstanceTypes API (to avoid hard coding the instance network setup).
Cache the result 24h (we could probably do more) to avoid an API call for every pod
int -> int64 to match the answer from AWS
Add API mock for test and include test for caching logic

Propagate errors due to network limits When an instance was unknown, ENILimits used to return {0,0,0} which was a bit confusing in runtime log. The other commit makes an API call to AWS so we can face new errors. This commits propagate errors to ENILimits callers to make them explicit

theatrus commented 4 years ago

:excited!:

Let me give this a quick spin.

lbernail commented 4 years ago

@theatrus My tests went fine, let me know if you see any problem on your side

theatrus commented 4 years ago

Been a busy few weeks here, but this looks good to me. Merging and thank you!!

lyft / cni-ipvlan-vpc-k8s

Use the AWS API to retrieve instance network limits #78