Skip to content
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

AWS provider v4 breaks module #83

Open
Dogers opened this issue Feb 11, 2022 · 2 comments
Open

AWS provider v4 breaks module #83

Dogers opened this issue Feb 11, 2022 · 2 comments
Labels
bug 🐛 An issue with the system

Comments

@Dogers
Copy link

Dogers commented Feb 11, 2022

Describe the Bug

When using the v4.0.0 AWS provider, the module no longer functions correctly:

Error: Unsupported attribute

  on .terraform/modules/logs/main.tf line 30, in resource "aws_s3_bucket" "default":
  30:         for_each = var.enable_glacier_transition ? [1] : []

This object does not have an attribute named "enable_glacier_transition".

Error: Unsupported attribute

  on .terraform/modules/logs/main.tf line 44, in resource "aws_s3_bucket" "default":
  44:         for_each = var.enable_glacier_transition ? [1] : []

This object does not have an attribute named "enable_glacier_transition".
time=2022-02-11T12:18:13Z level=error msg=1 error occurred:
	* exit status 1

Expected Behavior

Module should work!

Steps to Reproduce

Run module as per examples with the v4.0.0 AWS provider

Additional Context

Issue appears to lie with the s3-log-storage module. I have pushed a PR to that module - PR 64 but this module will also need to be updated with new version produced there.

@Dogers Dogers added the bug 🐛 An issue with the system label Feb 11, 2022
@cristobalcl
Copy link

I'm stuck by this issue as well.
Any progress?

@Dogers
Copy link
Author

Dogers commented Feb 24, 2022

Having just seen their update to the S3 storage module, I don't believe they're going to be able to fix this nicely now, so I think we're going to have to either migrate to a new module or fork and maintain our own version :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue with the system
Projects
None yet
Development

No branches or pull requests

2 participants