Open mattock opened 1 year ago
The solution is to provide at least one of the context null label inputs such as name
A better fix for people who do not want to use name would be to use a join(" ", compact([module.this.id, "Transit Gateway"]))
. The description
should probably be its own exposed input with the same default value Transit Gateway
as it is today
Thanks @mattock for creating this pull request!
A maintainer will review your changes shortly. Please don't be discouraged if it takes a while.
While you wait, make sure to review our contributor guidelines.
[!TIP]
Need help or want to ask for a PR review to be expedited?
Join us on Slack in the
#pr-reviews
channel.
what
This PR fixes flickering on every Terraform run when var.transit_gateway_description is unset.
The problem is caused by the leading space in the description: something that AWS APIs accept without erroring out or warning, but which gets removed or ignored. Hence Terraform tries to "fix" the situation on every run.
why
Without this change this happens on every Terraform run if var.transit_gateway_description is not defined:
With this change applied things work as expected: