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

Incorrect AWS provider version requirement #181

Open
cflanny opened this issue Jan 5, 2024 · 1 comment
Open

Incorrect AWS provider version requirement #181

cflanny opened this issue Jan 5, 2024 · 1 comment
Labels
bug 🐛 An issue with the system

Comments

@cflanny
Copy link

cflanny commented Jan 5, 2024

Describe the Bug

The AWS provider requirement is still listed as >=2.x, but creating an instance with this module with an AWS provider version <5.x will result in the error Can't configure a value for "domain": its value will be decided automatically based on the result of applying this configuration. due to attempting to set the "domain" value on main.tf:181

Expected Behavior

No errors about setting derived attributes that can't be set or overridden in any way without back-versioning the module to a version where this line doesn't exist

Steps to Reproduce

Attempt to create an instance using this module, with AWS provider <5.x

Screenshots

╷
│ Error: Value for unconfigurable attribute
│ 
│   with module.ec2_zscaler.aws_eip.additional,
│   on .terraform/modules/ec2_zscaler/eni.tf line 29, in resource "aws_eip" "additional":
│   29:   domain            = "vpc"
│ 
│ Can't configure a value for "domain": its value will be decided
│ automatically based on the result of applying this configuration.
╵
╷
│ Error: Value for unconfigurable attribute
│ 
│   with module.ec2_zscaler.aws_eip.default,
│   on .terraform/modules/ec2_zscaler/main.tf line 181, in resource "aws_eip" "default":
│  181:   domain   = "vpc"
│ 
│ Can't configure a value for "domain": its value will be decided
│ automatically based on the result of applying this configuration.
╵

Environment

No response

Additional Context

No response

@cflanny cflanny added the bug 🐛 An issue with the system label Jan 5, 2024
@irl
Copy link
Contributor

irl commented Nov 25, 2024

The requirement was updated to >= 4.7.0 in a8a3214. This issue can likely be closed.

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