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

Update: Move from deprecated list used in dynamic-subnet/aws #120

Closed
wants to merge 3 commits into from

Conversation

savannahar68
Copy link

@savannahar68 savannahar68 commented Jul 15, 2021

While running complete example terraform validate (version 0.15+) will throw error because cloudposse/dynamic-subnets/aws" 0.38.0 version uses list datatype which is deprecated

output "nat_ips" {
  description = "IP Addresses in use for NAT"
  value       = coalescelist(aws_eip.default.*.public_ip, aws_eip.nat_instance.*.public_ip, data.aws_eip.nat_ips.*.public_ip, list(""))
}

https://github.com/cloudposse/terraform-aws-dynamic-subnets/blob/0.38.0/outputs.tf

Using the latest version 0.39+ will resolve this error

what

  • This will resolve the issue of terraform throwing error for using deprecated list post using 0.15 version

why

  • Using latest version or version post 0.39+ uses tolist datatype which is valid post terraform 0.15+

…/aws" 0.38.0 to latest version

While running complete example terraform validate (version 0.15+) will throw error because  cloudposse/dynamic-subnets/aws" 0.38.0 version uses list datatype which is deprecated

```
output "nat_ips" {
  description = "IP Addresses in use for NAT"
  value       = coalescelist(aws_eip.default.*.public_ip, aws_eip.nat_instance.*.public_ip, data.aws_eip.nat_ips.*.public_ip, list(""))
}
```

https://github.com/cloudposse/terraform-aws-dynamic-subnets/blob/0.38.0/outputs.tf

Using the latest version 0.39+ will resolve this error
@savannahar68 savannahar68 requested review from a team as code owners July 15, 2021 18:15
@Nuru Nuru requested a review from a team as a code owner July 17, 2021 03:57
@Nuru Nuru added terraform/0.15 no-release Do not create a new release (wait for additional code changes) labels Jul 17, 2021
@Nuru
Copy link
Sponsor Contributor

Nuru commented Jul 17, 2021

/test all

@Nuru
Copy link
Sponsor Contributor

Nuru commented Jul 17, 2021

@savannahar68 Thank you for this contribution.

This PR is currently blocked due to failing tests. However, the test failures are not your fault, the problem is that some of our tests are not compatible with Terraform 0.15. That means you will not be able to fix the failures, so don't even try. We will update the tests and re-run them.

@savannahar68
Copy link
Author

savannahar68 commented Jul 17, 2021

@Nuru thanks for the update 👍🏻

Looking forward to PR getting merged.

@mergify mergify bot closed this Jul 17, 2021
@mergify
Copy link

mergify bot commented Jul 17, 2021

This pull request has been automatically closed by Mergify because there are no longer any changes.

@Nuru
Copy link
Sponsor Contributor

Nuru commented Jul 17, 2021

@savannahar68 Your change was included in v0.42.1 so you would not have to wait for our testing to catch up. Thank you again for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-release Do not create a new release (wait for additional code changes)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants