-
Notifications
You must be signed in to change notification settings - Fork 16
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
bug: module.lacework_cfg_iam_role should pin a specific version #86
Comments
This has now also affected us after your release of 2.0. Any plans to pin the version? |
Please fix it, it just broke my modules. |
@segfault79 @mgordziejewksi-clari we upgraded all modules to use lacework provider 2.0. Are you still running into any issues? Note that if you pin lacework/lacework to 1.0, please update it to 2.0. |
One thing is upgrading the provider in one of the versions of your module, another is keeping the versions pinned in versions of your module. Right now, if you use old version of your module, it is entirely broken if you depend on 1.0 version in any place. I think it's just good practice that old versions of your module don't break out of sudden. |
@mgordziejewksi-clari Definitely. 1.0 stopped working due to a breaking change in one of its dependencies. We had to force all customers to use 2.0 Sorry for the trouble. Here is a related thread lacework/terraform-aws-iam-role#60 |
Describe the bug
This module consumes the
lacework/iam-role/aws
module, but doesn't pin a specific version of that module. This means that whenever theiam-role
module ships a new version, users of theterraform-aws-config
module will inherit that new version ofiam-role
without really knowing it.Steps to reproduce
Take any terraform environment where you have used
terraform-aws-config
and you have already runterraform init
before lacework/terraform-aws-iam-role#46 was merged, runterraform plan
and see no diffs.Then run
rm -rf .terraform && terraform init && terraform plan
which will pull in the new version ofiam-role
and you'll see diffs.Expected behavior
You should never see diffs without explicitly bumping a version.
Screenshots
N/A
Please complete the following information):
The text was updated successfully, but these errors were encountered: