search

dbt-labs / terraform-provider-dbtcloud

dbt Cloud Terraform Provider
https://registry.terraform.io/providers/dbt-labs/dbtcloud
MIT License
82 stars 19 forks source link

Update dbt_cloud_environment_variable.md #109

Closed subrasub closed 1 year ago

subrasub commented 1 year ago

Add a note to title-case environment names. If we don't title-casing the environment names, it can throw a 500 error when we run terraform apply. For eg: if we are setting the environment variables for the Production env, the environment name should also be Production.

Related - https://github.com/GtheSheep/terraform-provider-dbt-cloud/issues/108#issue-1586440179

GtheSheep commented 1 year ago

Hey @subrasub - thanks for this! But these descriptions are set in the resource object, the markdown is then generated from them using make doc, could you update? thanks!

GtheSheep commented 1 year ago

Also, reading your comment, I'm not sure this is clear? Are you just saying the environment names are case sensitive and need to exactly match how the environment is configured? i.e. Environment name Production can't have env vars under production?

GtheSheep commented 1 year ago

Just tested this out and it works for non title-cased envs too, i.e. staging