Skip to content
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

bug: aws-auth-config-fixed #68

Open
wants to merge 48 commits into
base: master
Choose a base branch
from
Open

bug: aws-auth-config-fixed #68

wants to merge 48 commits into from

Conversation

amanverma678
Copy link

@amanverma678 amanverma678 commented Oct 25, 2024

What

Using apply_config_map_aws_auth = false throws error mentioned below.

│ 
│   on ../../aws_auth.tf line 91, in provider "kubernetes":
│   91:   token                  = data.aws_eks_cluster_auth.eks[0].token
│     ├────────────────
│     │ data.aws_eks_cluster_auth.eks is empty tuple
│ 
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│ 
│   on ../../aws_auth.tf line 92, in provider "kubernetes":
│   92:   host                   = data.aws_eks_cluster.eks[0].endpoint
│     ├────────────────
│     │ data.aws_eks_cluster.eks is empty tuple
│ 
│ The given key does not identify an element in this collection value: the collection has no elements.
╵
╷
│ Error: Invalid index
│ 
│   on ../../aws_auth.tf line 93, in provider "kubernetes":
│   93:   cluster_ca_certificate = base64decode(data.aws_eks_cluster.eks[0].certificate_authority.0.data)
│     ├────────────────
│     │ data.aws_eks_cluster.eks is empty tuple
│ 
│ The given key does not identify an element in this collection value: the collection has no elements.

Why

It should be optional to user if they want to manage aws-auth confimap manually or via terraform.

@clouddrove-ci
Copy link
Member

Terraform Security Scan Failed

Show Output

rishitz
rishitz previously approved these changes Oct 25, 2024
@h1manshu98
Copy link
Member

Hello @amanverma678, thanks for the enhancements 👏
Can you please update your terraform code to remove TF-Lint warnings

image

@amanverma678
Copy link
Author

Hello @amanverma678, thanks for the enhancements 👏 Can you please update your terraform code to remove TF-Lint warnings

image

All warnings are fixed now please review

@clouddrove-ci
Copy link
Member

Terraform Security Scan Failed

Show Output

@clouddrove-ci
Copy link
Member

Terraform Security Scan Failed

Show Output

@clouddrove-ci
Copy link
Member

Terraform Security Scan Failed

Show Output

@amanverma678
Copy link
Author

Hello @amanverma678, thanks for the enhancements 👏 Can you please update your terraform code to remove TF-Lint warnings
image

All warnings are fixed

variables.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants