terraform/terraform_remote_state: accept complex configs #18759
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
remote
backend config contains an attribute that is defined as a*schema.Set
, but currently onlystring
values are accepted as theconfig
attribute is defined as aschema.TypeMap
.Additionally the
b.Validate()
method wasn’t called to prevent a possible panic in case of unexpected configurations being passed tob.Configure()
.This commit is a bit of a hack to be able to support this in the 0.11 series. The 0.12 series will have proper support, so when merging 0.12 this should be reverted again.
The additional change to the
manta
backend was needed to (next to making it consistent as its the only attribute name that uses camelcase) because passing that attributeobjectName
ended up giving me this error:terraform/helper/schema/schema.go
Line 736 in 4d9a284