-
-
Notifications
You must be signed in to change notification settings - Fork 214
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
Remove optional domain
attribute from aws_eip
resource
#173
Conversation
domain
attribute from aws_eip
resource to allow tf to compute the value
/terratest |
/terratest |
Thanks @mariot8 for creating this pull request! A maintainer will review your changes shortly. Please don't be discouraged if it takes a while. While you wait, make sure to review our contributor guidelines. Tip Need help or want to ask for a PR review to be expedited?Join us on Slack in the |
/terratest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @mariot8
Just a general note that the other option for domain was "standard" which is no longer valid due to EC2 Classic Networking being retired.
domain
attribute from aws_eip
resource to allow tf to compute the valuedomain
attribute from aws_eip
resource
what
aws_eip
optionaldomain
attributewhy
domain
attribute causes terraform to fail plan EC2 instancereferences
aws_eip
causes terraform fail #172