-
Notifications
You must be signed in to change notification settings - Fork 192
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
feat(auth): support auth through iam-authenticator, add example, Closes #306 #332
Conversation
// | ||
// Create an API client for an AWS EKS cluster using IAM authentication | ||
// Note: In order for this to work, you must set the environment | ||
// variable AWS_PROFILE |
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.
AWS_PROFILE
is used here: https://github.com/godaddy/kubernetes-client/pull/332/files#diff-7b7c8857ce22c1739c6ad1ea186ca9ddR22
Should this comment be in that file?
examples/eks-auth.js
Outdated
@@ -0,0 +1,51 @@ | |||
/* eslint no-console:0 */ | |||
// | |||
// Create an API client for an AWS EKS cluster |
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.
What's the difference between this example and the one below?
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.
Sorry, there should only be one. Versioning error.
@@ -0,0 +1,53 @@ | |||
/* eslint no-console:0 */ |
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.
Can you add a link to this example from the README.md?
README.md
Outdated
@@ -155,6 +155,8 @@ const client = new Client({ config: config.fromKubeconfig() }); | |||
log an error: [canary-controller.js](./examples/canary-controller.js) | |||
* Create a `client` using basic-auth: | |||
[basic-auth.js](./examples/basic-auth.js) | |||
* Create a `client` using IAM authenticator and cmd auth: |
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.
Maybe:
Create a `client` using IAM authenticator and cmd auth (works with EKS):
I'm trying to get EKS in there :)
No description provided.