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

upgrade 0.12 #27

Closed
wants to merge 12 commits into from
Closed

upgrade 0.12 #27

wants to merge 12 commits into from

Conversation

rverma-nikiai
Copy link

No description provided.

@osterman osterman added this to In progress in Terraform 0.12 Upgrades Jun 24, 2019
rverma-nikiai and others added 2 commits June 26, 2019 18:10
update terraform-aws-s3-log-storage source to 0.12 compatible cloudposse published upstream version (0.5.0)

update terraform-aws-route53-alias source to 0.12 compatible cloudposse published upstream version (0.3.0)
@geota
Copy link

geota commented Aug 14, 2019

I believe this commit finishes this work. I was able to get it to successfully run by fixing 0.12 upgrade bug from wrapping var.custom_error_response around wrapping it in a list.

I submitted a PR to the upstream fork:
rverma-nikiai#1

Not sure if it would be easier to just make a new PR, so I did that here:
#29

Happy to close it out if you prefer I work through this PR.

kfolinus-endava and others added 3 commits August 22, 2019 13:33
Fixing issue with list(string) for custom_error_response variable
dont wrap var.custom_error_response in a list - 0.12 upgrade fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants