aws / aws-application-networking-k8s

A Kubernetes controller for Amazon VPC Lattice
https://www.gateway-api-controller.eks.aws.dev/
Apache License 2.0
175 stars 50 forks source link

chore: Bump aws-sdk-go for EKS Pod Identity support #594

Closed askulkarni2 closed 10 months ago

askulkarni2 commented 10 months ago

What type of PR is this? feature

Which issue does this PR fix:

593

What does this PR do / Why do we need it: Bumps up aws-sdk-go to v1.50.3 to support EKS Pod Identity

If an issue # is not available please add repro steps and logs from aws-gateway-controller showing the issue:

Testing done on this change:

make presubmit                                                                                                                                                                      

W0125 15:28:50.573879   67633 parse.go:863] Making unsupported type entry "T" for: &types.TypeParam{check:(*types.Checker)(nil), id:0x1, obj:(*types.TypeName)(0x1401ab6a870), index:0, bound:(*types.Interface)(0x14000096af0)}
Running golangci-lint
ok      github.com/aws/aws-application-networking-k8s/pkg/apis/applicationnetworking/v1alpha1   0.719s  coverage: 1.4% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/aws   1.125s  coverage: 29.5% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/aws/services  1.518s  coverage: 6.4% of statements
?       github.com/aws/aws-application-networking-k8s/pkg/k8s   [no test files]
ok      github.com/aws/aws-application-networking-k8s/pkg/config    1.683s  coverage: 33.3% of statements
?       github.com/aws/aws-application-networking-k8s/pkg/model/lattice [no test files]
?       github.com/aws/aws-application-networking-k8s/pkg/runtime   [no test files]
ok      github.com/aws/aws-application-networking-k8s/pkg/controllers   1.969s  coverage: 15.2% of statements
?       github.com/aws/aws-application-networking-k8s/pkg/utils/gwlog   [no test files]
?       github.com/aws/aws-application-networking-k8s/pkg/utils/retry   [no test files]
?       github.com/aws/aws-application-networking-k8s/pkg/utils/ttime   [no test files]
ok      github.com/aws/aws-application-networking-k8s/pkg/controllers/eventhandlers 2.409s  coverage: 52.7% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/deploy    3.519s  coverage: 19.5% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/deploy/externaldns    3.593s  coverage: 75.6% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/deploy/lattice    4.239s  coverage: 76.6% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/gateway   4.268s  coverage: 72.1% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/k8s/policyhelper  4.288s  coverage: 15.9% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/model/core    4.407s  coverage: 60.6% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/model/core/graph  3.952s  coverage: 17.2% of statements
ok      github.com/aws/aws-application-networking-k8s/pkg/utils 4.310s  coverage: 23.2% of statements

Automation added to e2e:

Will this PR introduce any new dependencies?:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:

Does this PR introduce any user-facing change?:

No

Bump up aws-sdk-go to v1.50.3 to support EKS Pod Identity

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