-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
allow for top level terraform flag assignment #558
base: main
Are you sure you want to change the base?
allow for top level terraform flag assignment #558
Conversation
43962b9
to
9921f57
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a bit confused by this PR. Is it missing files? The changes seem to affect how Terraform sets -var
flags rather than -backend-config
flags?
9921f57
to
5847837
Compare
@brikis98 terratest/modules/terraform/format.go Lines 71 to 75 in 719caf2
|
oh I feel silly. That's the wrong test. I'll get back to you. |
5847837
to
f11c061
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, OK, this makes more sense now!
Could you add comments to terraform.Options
to indicate that terraform.KeyOnly
can be used as a value in the BackendConfig
map to indicate only the key should be passed to Terraform?
Co-authored-by: Yevgeniy Brikman <brikis98@users.noreply.github.com>
Bumping this one up too :) |
@ThisGuyCodes lmk if you could use a hand on this, could deff use this feature. |
bump? (4 years later) |
fixes #517
also adds tests for
FormatTerraformBackendConfigAsArgs
This change is