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

LogDNA example: Invalid value subdomain must consist of lower case alphanumeric characters #1950

Open
jeffhoek opened this issue Oct 7, 2020 · 0 comments

Comments

@jeffhoek
Copy link
Contributor

jeffhoek commented Oct 7, 2020

Terraform Version

* provider.ibm: version = "~> 1.12"
* provider.kubernetes: version = "~> 1.13"

Affected Resource(s)

Please list the resources as a list, for example:
LogDNA example:
https://github.com/IBM-Cloud/terraform-provider-ibm/blob/master/examples/ibm-logdna-cluster-integration/main.tf#L8

If this issue appears to affect multiple resources, it may be an issue with Terraform's core, so please mention this.

Terraform Configuration Files

variable "chart_name" {
  default = "logDNA_chart"
}

Expected Behavior

Helm chart should successfully apply.

Actual Behavior

Terraform logs show:

Error: create: failed to create: Secret "sh.helm.release.v1.logDNA-chart-1db1b7ff.v1" is invalid: metadata.name: Invalid value: "sh.helm.release.v1.lo
gDNA-chart-1db1b7ff.v1": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanume
ric 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])?)*')

Steps to Reproduce

Please list the steps required to reproduce the issue, for example:
0. pull the helm chart:

helm repo add stable https://kubernetes-charts.storage.googleapis.com
"stable" has been added to your repositories
  1. terraform apply

Important Factoids

Are there anything atypical about your accounts that we should know? For example: Running in EC2 Classic? Custom version of OpenStack? Tight ACLs?

References

Are there any other GitHub issues (open or closed) or Pull Requests that should be linked here? For example:

Comments

The fix is straightforward, the chart_name value in variables.tf needs to be adjusted to something like:

variable "chart_name" {
  default = "logdna-chart"
}

I will post a pull request 🔜 .

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

No branches or pull requests

1 participant