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 to terraform 0.13 #3

Merged
merged 2 commits into from
Aug 31, 2020
Merged

Upgrade to terraform 0.13 #3

merged 2 commits into from
Aug 31, 2020

Conversation

dheles
Copy link
Contributor

@dheles dheles commented Aug 28, 2020

No description provided.

@dheles dheles requested a review from jonathangreen August 28, 2020 19:25
@bbranan
Copy link

bbranan commented Aug 28, 2020

Just a note that when we tried updating to Terraform 0.13 last week we ran into this issue: hashicorp/terraform#25752 which is now fixed in 0.13.1. I'm just calling it out in case you want to go ahead and bump to 0.13.1, just to avoid the potential of a problem.

@dheles
Copy link
Contributor Author

dheles commented Aug 31, 2020

Thanks, @bbranan .

It isn't obvious from this PR, but I also ran onto the issue with 0.13.0 and was able to succeed with 0.13.1. The terraform 0.13upgrade tool made these changes without specifying a point version, so I left things as-is, relying on agreement to not try to use 0.13.0. In my testing, using it didn't break anything, it just didn't work.

In your repo, did you hand-edit to specify a point version (I can't seem to find it to check)?

@bbranan
Copy link

bbranan commented Aug 31, 2020

When we ran into the issue 0.13.1 hadn't been released yet, so we just backed out the upgrade to continue with 0.12.29 until the problem was resolved. We haven't yet made it back to test with 0.13.1.

@dheles
Copy link
Contributor Author

dheles commented Aug 31, 2020

Hand-edited to require 0.13.1 & re-tested enough to confirm that terraform will to roll with & enforce the change.

Copy link
Member

@jonathangreen jonathangreen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@jonathangreen jonathangreen merged commit 9d6e87e into master Aug 31, 2020
@jonathangreen jonathangreen deleted the v1.3 branch August 31, 2020 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants