Skip to content
This repository has been archived by the owner on Apr 14, 2024. It is now read-only.

Bug/metadata lower chars #14

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

markoskandylis
Copy link

Is the EKS cluster has Upper chars secret was not created with error:

 Error: metadata.0.name a lowercase RFC 1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
│
│   with module.gitops_bridge_bootstrap.kubernetes_secret_v1.cluster[0],
│   on .terraform/modules/gitops_bridge_bootstrap/main.tf line 127, in resource "kubernetes_secret_v1" "cluster":
│  127:     name        = local.argocd.metadata.name

Fixing the issue by using the lower function in the clusters name

…cluster has upper case

Signed-off-by: markos kandylis <markos_kandylis@hotmail.com>
…cluster has upper case

Signed-off-by: markos kandylis <markos_kandylis@hotmail.com>
…cluster has upper case

Signed-off-by: markos kandylis <markos_kandylis@hotmail.com>
Copy link
Member

@csantanapr csantanapr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@csantanapr
Copy link
Member

We need to do same fix in the terraform module that is used to publish https://github.com/gitops-bridge-dev/terraform-helm-gitops-bridge
This repo is deprecated

@csantanapr
Copy link
Member

Can you clone this PR and open a new PR in the new github repo https://github.com/gitops-bridge-dev/terraform-helm-gitops-bridge

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants