-
Notifications
You must be signed in to change notification settings - Fork 519
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
aws-iam-authenticator: Upgrade to v0.6.8 #2956
Conversation
Fixes building on the tip of `develop` when using GOPROXY="direct". v0.6.2 of aws-iam-authenticator has a Go code dependency on kubernetes 1.22. There are privated GitHub repositories that can not be pulled down through the "direct" method. Signed-off-by: John McBride <[email protected]>
Also related: kubernetes-sigs/aws-iam-authenticator#586 |
TestingBuilt a kubernetes variant off this branch. Registered the AMI and used that AMI in an ASG (which currently has nodes scaled down to 0) related to one of my clusters. Using that AMI and scaling the ASG to 1, the node joins the cluster and I'm able to access the cluster via my local IAM role:
I'm not sure what deeper validation we need here and I'm not seeing alot of notes on testing this package specifically. Any thoughts? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the testing is sufficient. If I remember correctly, the package is mostly called as part of the AWS cloud provider code. If you want to be extra sure, you can try deploying the AWS EBS CSI driver and the one of the examples.
Also deployed the sample EBS CSI driver application, and looks good - merging!
|
Issue number:
N/a
Description of changes:
Example of the broken build using
GOPROXY="direct"
:And a quick and dirty way to reproduce with the broken GitHub repo:
Testing done:
Coming soon. Able to build locally now.
Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.