-
Notifications
You must be signed in to change notification settings - Fork 66
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
Added AWS Credentials to main.tf #9
base: master
Are you sure you want to change the base?
Conversation
…h credentials was found On branch master Changes: modified: AUTHORS.md - added myself :) modified: main.tf - added aws credentials to kubernetes_deployment.spec.container.env modified: variables.tf - added vars aws access key and secret modified: versions.tf - terraform 0.13update
On branch master modified: main.tf
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.
required_version = ">= 0.13"
why would you want to stop this module from using with TF .12?
Bug fixes, better formated tf files. |
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.
First of all: Thank you very much for the PR!
Credentials should be optional, though. Example: In case of EKS and IAM based permissions, no AWS access key is required.
Same goes for setups that utilize kube2iam or kiam.
I see the use-case your PR tries to fit, though. I'll gladly merge it, once the access key variables are purely optional.
@@ -407,6 +408,19 @@ resource "kubernetes_deployment" "this" { | |||
"--aws-max-retries=10", | |||
] | |||
|
|||
env { |
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.
A dynamic block with optional environment variables would be a cleaner solution, I think.
Any chance that this is possible?
I had the issue like kubernetes-sigs/aws-load-balancer-controller#601
And it was ok to add AWS credits to the env block of the container.
Now it works for me.