-
-
Notifications
You must be signed in to change notification settings - Fork 161
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
Example error? #6
Comments
Probably example has to be updated, pull-request is welcome. |
I'm seeing the same error, and I'm not running one of the examples. I'm no expert, but the module source looks correct; I'm not sure why aws_redshift_cluster.this doesn't have a vpc_security_group_ids attribute. |
Only happens when no value is supplied for vpc_security_group_ids. I edited main.tf and hard-coded a dummy value (["abc"]) and no longer get the error. |
Think it may have to do with the count=0 behavior introduced in terraform 0.11: https://www.terraform.io/upgrade-guides/0-11.html I'm using 0.11. |
@csullens-work Thanks for opening this issue. Apparently, an example was not in a good shape.
Indeed `vpc_security_group_ids` is required.
I have updated the code in 4ba92b6 , tested it and released v1.4.0. |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Hi,
I'm very new to Terraform, and want to add my AWS Redshift to the mix. I'm trying to "plan" the very basic example and I'm hitting the error:
* module.redshift.output.this_redshift_cluster_vpc_security_group_ids: Resource 'aws_redshift_cluster.this' does not have attribute 'vpc_security_group_ids' for variable 'aws_redshift_cluster.this.vpc_security_group_ids'
Any ideas?
thanks,
Chris
The text was updated successfully, but these errors were encountered: