-
Notifications
You must be signed in to change notification settings - Fork 540
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
Version 1.1.3 aws_auth_backend_role breaking changes #177
Comments
See the conversation here: #153 (comment) |
The breaking change PR was reverted and a patch version released! This should be fixed on the latest patch version. I've re-added the original PR and I'm currently working on some compatibility code. Will cut a minor release when that's in. |
From our perspective, this was fixed by #185 🎉 |
@idubinskiy @dhild could you two take a look at #189 and comment upon whether that's what you had in mind? |
Terraform Version
Affected Resource(s)
(There may be others)
Terraform Configuration Files
Debug Output
The issue appears to be related to versioning, I'm not sure debug output is necessary here.
Panic Output
There was no panic.
Expected Behavior
Terraform plan & apply should still work
This issue appears where we have existing resources written to use the
String
type, where the new version of the plugin expects these to be aList
type. We would expect that the old configurations still work when using the~> 1.0
version specification for thevault
provider.Actual Behavior
Steps to Reproduce
With the terraform listed above:
terraform init
terraform plan
Important Factoids
Nothing comes to mind.
References
It looks like the issue appeared from this commit: 768c03b#diff-a97ccbd83928090a41b6c44716cd9f18L45
The text was updated successfully, but these errors were encountered: