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.

Sign up for GitHub

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

Add support for Terraform 0.14 #262

Merged
merged 15 commits into from
Dec 10, 2020
Merged

Add support for Terraform 0.14 #262

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
cf5087f
Add update for README, group target
Nuru Dec 7, 2020
f7d62a9
Fix typo in regex
Nuru Dec 7, 2020
ef4553e
bump-tf-12-min-version and rewrite-required-providers targets added t…
maximmi Dec 7, 2020
579c08b
readme update
maximmi Dec 7, 2020
75bc200
code cleaning
maximmi Dec 7, 2020
1e954ad
More Terraform updates
Nuru Dec 8, 2020
f1acd3d
removing apk cache for optimization and pass linter tests
maximmi Dec 8, 2020
12817ba
updating Dockerfile to pass Hadolint tests, Hadolint config added to …
maximmi Dec 8, 2020
2ad0adf
Allow you to run the build-harness docker image like Geodesic
Nuru Dec 9, 2020
1d2d5bf
Update to recipes for upgrading Terraform modules
Nuru Dec 9, 2020
b6362e0
CODEOWNERS, run docker, mergify, and other updates/fixes
Nuru Dec 10, 2020
167c78a
add validate-codeowners
Nuru Dec 10, 2020
4812b12
Pedantic linter fixes
Nuru Dec 10, 2020
51ce3b0
Make bulid-harness the authority on context.tf
Nuru Dec 10, 2020
4c05cbd
Always validate CODEOWNERS
Nuru Dec 10, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion modules/terraform/Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,19 @@ terraform/upgrade-modules:

## Rewrite the *.tf files to use registry notation for modules sources
terraform/rewrite-module-source:
@sed -i -E 's,"git::https://github.com/(.*?)/terraform-([^-]*?)-(.*?).git\?ref=tags/(.*?)","\1/\3/\2"\n version = "\4",g' $$(find . -type f -name '*.tf')
@sed -i -E 's,\s*source\s+=\s+"git::https://github.com/(.*?)/terraform-([^-]*?)-(.*?).git\?ref=(tags/)?(.*?)", source = "\1/\3/\2"\n version = "\5",g' $$(find . -type f -not -name context.tf -name '*.tf')
@$(TERRAFORM) fmt .
@$(TERRAFORM) fmt examples/complete

terraform/rewrite-readme-source:
@sed -i -E 's,^(\s*)source\s+=\s+"git::https://github.com/(.*?)/terraform-([^-]*?)-(.*?).git\?ref=(tags/)?master",\1source = "\2/\4/\2"\n\1# Cloud Posse recommends pinning module to a specific version\n\1# version = "x.x.x",g' README.yaml
Nuru marked this conversation as resolved.
Show resolved Hide resolved

## Rewrite versions.tf to remove upper bound for terraform core version constraint (like this ">= 0.12.0, < 0.14.0")
terraform/remove-upper-bound:
@sed -i -E 's,required_version\s*\=\s*\"(.*?)(\,\s*<.*)",required_version = "\1",g' $$(find . -type f -name 'versions.tf')
@$(TERRAFORM) fmt .
@$(TERRAFORM) fmt examples/complete

terraform/v14-rewrite: TERRAFORM = terraform-0.13
terraform/v14-rewrite: terraform/remove-upper-bound terraform/rewrite-module-source terraform/rewrite-readme-source
git diff --no-patch --exit-code README.yaml || $(MAKE) readme