From 92884f373deca624055c299d82e312444048bac7 Mon Sep 17 00:00:00 2001 From: Maxim Mironenko Date: Sat, 23 Jan 2021 22:25:18 +0700 Subject: [PATCH] Terraform 0.14 upgrade (#54) * update to latest null-label for terraform 14.x support * add github_anonymous variable to pass tests * upgrading modules and workflows * terraform formatting * terraform-aws-ecs-container-definition doesn't support context * workflow updated, documentation fixed Co-authored-by: Dmitrij Nikitenko Co-authored-by: Dmitrij Nikitenko --- .github/CODEOWNERS | 13 +++- .github/auto-release.yml | 47 +++++++---- .github/mergify.yml | 58 ++++++++++++++ .github/renovate.json | 12 +++ .github/workflows/auto-context.yml | 57 ++++++++++++++ .github/workflows/auto-format.yml | 86 +++++++++++++++++++++ .github/workflows/auto-release.yml | 2 +- .github/workflows/chatops.yml | 4 +- .github/workflows/validate-codeowners.yml | 25 ++++++ README.md | 27 +++++-- README.yaml | 8 +- context.tf | 5 +- docs/terraform.md | 2 +- examples/complete/context.tf | 5 +- examples/complete/fixtures.us-east-2.tfvars | 2 + examples/complete/main.tf | 29 ++++--- examples/complete/variables.tf | 5 ++ examples/complete/versions.tf | 27 +++++-- main.tf | 62 +++++++++------ variables.tf | 1 - versions.tf | 27 +++++-- 21 files changed, 419 insertions(+), 85 deletions(-) create mode 100644 .github/mergify.yml create mode 100644 .github/renovate.json create mode 100644 .github/workflows/auto-context.yml create mode 100644 .github/workflows/auto-format.yml create mode 100644 .github/workflows/validate-codeowners.yml diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS index 07b38d2..ceb4644 100644 --- a/.github/CODEOWNERS +++ b/.github/CODEOWNERS @@ -1,7 +1,7 @@ # Use this file to define individuals or teams that are responsible for code in a repository. # Read more: # -# Order is important: the last matching pattern takes the most precedence +# Order is important: the last matching pattern has the highest precedence # These owners will be the default owners for everything * @cloudposse/engineering @cloudposse/contributors @@ -13,5 +13,12 @@ # Cloud Posse must review any changes to GitHub actions .github/* @cloudposse/engineering -# Cloud Posse must review any changes to standard context definition -**/context.tf @cloudposse/engineering +# Cloud Posse must review any changes to standard context definition, +# but some changes can be rubber-stamped. +**/context.tf @cloudposse/engineering @cloudposse/approvers +README.md @cloudposse/engineering @cloudposse/contributors @cloudposse/approvers +docs/*.md @cloudposse/engineering @cloudposse/contributors @cloudposse/approvers + +# Cloud Posse Admins must review all changes to CODEOWNERS or the mergify configuration +.github/mergify.yml @cloudposse/admins +.github/CODEOWNERS @cloudposse/admins diff --git a/.github/auto-release.yml b/.github/auto-release.yml index 2836185..c78a4d8 100644 --- a/.github/auto-release.yml +++ b/.github/auto-release.yml @@ -4,30 +4,35 @@ version-template: '$MAJOR.$MINOR.$PATCH' version-resolver: major: labels: - - 'major' + - 'major' minor: labels: - - 'minor' - - 'enhancement' + - 'minor' + - 'enhancement' patch: labels: - - 'patch' - - 'fix' - - 'bugfix' - - 'bug' - - 'hotfix' + - 'auto-update' + - 'patch' + - 'fix' + - 'bugfix' + - 'bug' + - 'hotfix' default: 'minor' categories: - - title: '🚀 Enhancements' - labels: - - 'enhancement' - - title: '🐛 Bug Fixes' - labels: - - 'fix' - - 'bugfix' - - 'bug' - - 'hotfix' +- title: '🚀 Enhancements' + labels: + - 'enhancement' + - 'patch' +- title: '🐛 Bug Fixes' + labels: + - 'fix' + - 'bugfix' + - 'bug' + - 'hotfix' +- title: '🤖 Automatic Updates' + labels: + - 'auto-update' change-template: |
@@ -38,3 +43,11 @@ change-template: | template: | $CHANGES + +replacers: +# Remove irrelevant information from Renovate bot +- search: '/---\s+^#.*Renovate configuration(?:.|\n)*?This PR has been generated .*/gm' + replace: '' +# Remove Renovate bot banner image +- search: '/\[!\[[^\]]*Renovate\][^\]]*\](\([^)]*\))?\s*\n+/gm' + replace: '' diff --git a/.github/mergify.yml b/.github/mergify.yml new file mode 100644 index 0000000..b010656 --- /dev/null +++ b/.github/mergify.yml @@ -0,0 +1,58 @@ +# https://docs.mergify.io/conditions.html +# https://docs.mergify.io/actions.html +pull_request_rules: +- name: "approve automated PRs that have passed checks" + conditions: + - "author~=^(cloudpossebot|renovate\\[bot\\])$" + - "base=master" + - "-closed" + - "head~=^(auto-update|renovate)/.*" + - "check-success=test/bats" + - "check-success=test/readme" + - "check-success=test/terratest" + - "check-success=validate-codeowners" + actions: + review: + type: "APPROVE" + bot_account: "cloudposse-mergebot" + message: "We've automatically approved this PR because the checks from the automated Pull Request have passed." + +- name: "merge automated PRs when approved and tests pass" + conditions: + - "author~=^(cloudpossebot|renovate\\[bot\\])$" + - "base=master" + - "-closed" + - "head~=^(auto-update|renovate)/.*" + - "check-success=test/bats" + - "check-success=test/readme" + - "check-success=test/terratest" + - "check-success=validate-codeowners" + - "#approved-reviews-by>=1" + - "#changes-requested-reviews-by=0" + - "#commented-reviews-by=0" + actions: + merge: + method: "squash" + +- name: "delete the head branch after merge" + conditions: + - "merged" + actions: + delete_head_branch: {} + +- name: "ask to resolve conflict" + conditions: + - "conflict" + - "-closed" + actions: + comment: + message: "This pull request is now in conflict. Could you fix it @{{author}}? 🙏" + +- name: "remove outdated reviews" + conditions: + - "base=master" + actions: + dismiss_reviews: + changes_requested: true + approved: true + message: "This Pull Request has been updated, so we're dismissing all reviews." diff --git a/.github/renovate.json b/.github/renovate.json new file mode 100644 index 0000000..ae4f0aa --- /dev/null +++ b/.github/renovate.json @@ -0,0 +1,12 @@ +{ + "extends": [ + "config:base", + ":preserveSemverRanges" + ], + "labels": ["auto-update"], + "enabledManagers": ["terraform"], + "terraform": { + "ignorePaths": ["**/context.tf", "examples/**"] + } +} + diff --git a/.github/workflows/auto-context.yml b/.github/workflows/auto-context.yml new file mode 100644 index 0000000..ab979e0 --- /dev/null +++ b/.github/workflows/auto-context.yml @@ -0,0 +1,57 @@ +name: "auto-context" +on: + schedule: + # Update context.tf nightly + - cron: '0 3 * * *' + +jobs: + update: + if: github.event_name == 'schedule' + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v2 + + - name: Update context.tf + shell: bash + id: update + env: + GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}" + run: | + if [[ -f context.tf ]]; then + echo "Discovered existing context.tf! Fetching most recent version to see if there is an update." + curl -o context.tf -fsSL https://raw.githubusercontent.com/cloudposse/terraform-null-label/master/exports/context.tf + if git diff --no-patch --exit-code context.tf; then + echo "No changes detected! Exiting the job..." + else + echo "context.tf file has changed. Update examples and rebuild README.md." + make init + make github/init/context.tf + make readme/build + echo "::set-output name=create_pull_request::true" + fi + else + echo "This module has not yet been updated to support the context.tf pattern! Please update in order to support automatic updates." + fi + + - name: Create Pull Request + if: steps.update.outputs.create_pull_request == 'true' + uses: cloudposse/actions/github/create-pull-request@0.22.0 + with: + token: ${{ secrets.PUBLIC_REPO_ACCESS_TOKEN }} + committer: 'cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>' + author: 'cloudpossebot <11232728+cloudpossebot@users.noreply.github.com>' + commit-message: Update context.tf from origin source + title: Update context.tf + body: |- + ## what + This is an auto-generated PR that updates the `context.tf` file to the latest version from `cloudposse/terraform-null-label` + + ## why + To support all the features of the `context` interface. + + branch: auto-update/context.tf + base: master + delete-branch: true + labels: | + auto-update + context diff --git a/.github/workflows/auto-format.yml b/.github/workflows/auto-format.yml new file mode 100644 index 0000000..990abed --- /dev/null +++ b/.github/workflows/auto-format.yml @@ -0,0 +1,86 @@ +name: Auto Format +on: + pull_request_target: + types: [opened, synchronize] + +jobs: + auto-format: + runs-on: ubuntu-latest + container: cloudposse/build-harness:slim-latest + steps: + # Checkout the pull request branch + # "An action in a workflow run can’t trigger a new workflow run. For example, if an action pushes code using + # the repository’s GITHUB_TOKEN, a new workflow will not run even when the repository contains + # a workflow configured to run when push events occur." + # However, using a personal access token will cause events to be triggered. + # We need that to ensure a status gets posted after the auto-format commit. + # We also want to trigger tests if the auto-format made no changes. + - uses: actions/checkout@v2 + if: github.event.pull_request.state == 'open' + name: Privileged Checkout + with: + token: ${{ secrets.PUBLIC_REPO_ACCESS_TOKEN }} + repository: ${{ github.event.pull_request.head.repo.full_name }} + # Check out the PR commit, not the merge commit + # Use `ref` instead of `sha` to enable pushing back to `ref` + ref: ${{ github.event.pull_request.head.ref }} + + # Do all the formatting stuff + - name: Auto Format + if: github.event.pull_request.state == 'open' + shell: bash + run: make BUILD_HARNESS_PATH=/build-harness PACKAGES_PREFER_HOST=true -f /build-harness/templates/Makefile.build-harness pr/auto-format/host + + # Commit changes (if any) to the PR branch + - name: Commit changes to the PR branch + if: github.event.pull_request.state == 'open' + shell: bash + id: commit + env: + SENDER: ${{ github.event.sender.login }} + run: | + set -x + output=$(git diff --name-only) + + if [ -n "$output" ]; then + echo "Changes detected. Pushing to the PR branch" + git config --global user.name 'cloudpossebot' + git config --global user.email '11232728+cloudpossebot@users.noreply.github.com' + git add -A + git commit -m "Auto Format" + # Prevent looping by not pushing changes in response to changes from cloudpossebot + [[ $SENDER == "cloudpossebot" ]] || git push + # Set status to fail, because the push should trigger another status check, + # and we use success to indicate the checks are finished. + printf "::set-output name=%s::%s\n" "changed" "true" + exit 1 + else + printf "::set-output name=%s::%s\n" "changed" "false" + echo "No changes detected" + fi + + - name: Auto Test + uses: cloudposse/actions/github/repository-dispatch@0.22.0 + # match users by ID because logins (user names) are inconsistent, + # for example in the REST API Renovate Bot is `renovate[bot]` but + # in GraphQL it is just `renovate`, plus there is a non-bot + # user `renovate` with ID 1832810. + # Mergify bot: 37929162 + # Renovate bot: 29139614 + # Cloudpossebot: 11232728 + # Need to use space separators to prevent "21" from matching "112144" + if: > + contains(' 37929162 29139614 11232728 ', format(' {0} ', github.event.pull_request.user.id)) + && steps.commit.outputs.changed == 'false' && github.event.pull_request.state == 'open' + with: + token: ${{ secrets.PUBLIC_REPO_ACCESS_TOKEN }} + repository: cloudposse/actions + event-type: test-command + client-payload: |- + { "slash_command":{"args": {"unnamed": {"all": "all", "arg1": "all"}}}, + "pull_request": ${{ toJSON(github.event.pull_request) }}, + "github":{"payload":{"repository": ${{ toJSON(github.event.repository) }}, + "comment": {"id": ""} + } + } + } diff --git a/.github/workflows/auto-release.yml b/.github/workflows/auto-release.yml index ccc27be..3f48017 100644 --- a/.github/workflows/auto-release.yml +++ b/.github/workflows/auto-release.yml @@ -6,7 +6,7 @@ on: - master jobs: - semver: + publish: runs-on: ubuntu-latest steps: # Drafts your next Release notes as Pull Requests are merged into "master" diff --git a/.github/workflows/chatops.yml b/.github/workflows/chatops.yml index 0d94310..4ddc067 100644 --- a/.github/workflows/chatops.yml +++ b/.github/workflows/chatops.yml @@ -9,7 +9,7 @@ jobs: steps: - uses: actions/checkout@v2 - name: "Handle common commands" - uses: cloudposse/actions/github/slash-command-dispatch@0.16.0 + uses: cloudposse/actions/github/slash-command-dispatch@0.22.0 with: token: ${{ secrets.PUBLIC_REPO_ACCESS_TOKEN }} reaction-token: ${{ secrets.GITHUB_TOKEN }} @@ -24,7 +24,7 @@ jobs: - name: "Checkout commit" uses: actions/checkout@v2 - name: "Run tests" - uses: cloudposse/actions/github/slash-command-dispatch@0.16.0 + uses: cloudposse/actions/github/slash-command-dispatch@0.22.0 with: token: ${{ secrets.PUBLIC_REPO_ACCESS_TOKEN }} reaction-token: ${{ secrets.GITHUB_TOKEN }} diff --git a/.github/workflows/validate-codeowners.yml b/.github/workflows/validate-codeowners.yml new file mode 100644 index 0000000..386eb28 --- /dev/null +++ b/.github/workflows/validate-codeowners.yml @@ -0,0 +1,25 @@ +name: Validate Codeowners +on: + pull_request: + +jobs: + validate-codeowners: + runs-on: ubuntu-latest + steps: + - name: "Checkout source code at current commit" + uses: actions/checkout@v2 + - uses: mszostok/codeowners-validator@v0.5.0 + if: github.event.pull_request.head.repo.full_name == github.repository + name: "Full check of CODEOWNERS" + with: + # For now, remove "files" check to allow CODEOWNERS to specify non-existent + # files so we can use the same CODEOWNERS file for Terraform and non-Terraform repos + # checks: "files,syntax,owners,duppatterns" + checks: "syntax,owners,duppatterns" + # GitHub access token is required only if the `owners` check is enabled + github_access_token: "${{ secrets.PUBLIC_REPO_ACCESS_TOKEN }}" + - uses: mszostok/codeowners-validator@v0.5.0 + if: github.event.pull_request.head.repo.full_name != github.repository + name: "Syntax check of CODEOWNERS" + with: + checks: "syntax,duppatterns" diff --git a/README.md b/README.md index b1720da..e02376f 100644 --- a/README.md +++ b/README.md @@ -1,4 +1,6 @@ + # terraform-aws-ecs-codepipeline [![GitHub Action Tests](https://github.com/cloudposse/terraform-aws-ecs-codepipeline/workflows/test/badge.svg?branch=master)](https://github.com/cloudposse/terraform-aws-ecs-codepipeline/actions) [![Latest Release](https://img.shields.io/github/release/cloudposse/terraform-aws-ecs-codepipeline.svg)](https://github.com/cloudposse/terraform-aws-ecs-codepipeline/releases/latest) [![Slack Community](https://slack.cloudposse.com/badge.svg)](https://slack.cloudposse.com) + [![README Header][readme_header_img]][readme_header_link] @@ -62,8 +64,15 @@ We literally have [*hundreds of terraform modules*][terraform_modules] that are ## Usage -**IMPORTANT:** The `master` branch is used in `source` just as an example. In your code, do not pin to `master` because there may be breaking changes between releases. -Instead pin to the release tag (e.g. `?ref=tags/x.y.z`) of one of our [latest releases](https://github.com/cloudposse/terraform-aws-ecs-codepipeline/releases). +**IMPORTANT:** We do not pin modules to versions in our examples because of the +difficulty of keeping the versions in the documentation in sync with the latest released versions. +We highly recommend that in your code you pin the version to the exact version you are +using so that your infrastructure remains stable, and update versions in a +systematic way so that they do not catch you by surprise. + +Also, because of a bug in the Terraform registry ([hashicorp/terraform#21417](https://github.com/hashicorp/terraform/issues/21417)), +the registry shows many of our inputs as required when in fact they are optional. +The table below correctly indicates which inputs are required. @@ -76,7 +85,9 @@ For automated tests of the complete example using `bats` and `Terratest`, see [t In this example, we'll trigger the pipeline anytime the `master` branch is updated. ```hcl module "ecs_push_pipeline" { - source = "git::https://github.com/cloudposse/terraform-aws-ecs-codepipeline.git?ref=master" + source = "cloudposse/ecs-codepipeline/aws" + # Cloud Posse recommends pinning every module to a specific version + # version = "x.x.x" name = "app" namespace = "eg" stage = "staging" @@ -97,7 +108,9 @@ In this example, we'll trigger anytime a new GitHub release is cut by setting th ```hcl module "ecs_release_pipeline" { - source = "git::https://github.com/cloudposse/terraform-aws-ecs-codepipeline.git?ref=master" + source = "cloudposse/ecs-codepipeline/aws" + # Cloud Posse recommends pinning every module to a specific version + # version = "x.x.x" name = "app" namespace = "eg" stage = "staging" @@ -176,7 +189,7 @@ Available targets: | Name | Version | |------|---------| -| terraform | >= 0.12.0 | +| terraform | >= 0.12.26 | | aws | >= 2.0 | | local | >= 1.2 | | null | >= 2.0 | @@ -359,7 +372,7 @@ In general, PRs are welcome. We follow the typical "fork-and-pull" Git workflow. ## Copyright -Copyright © 2017-2020 [Cloud Posse, LLC](https://cpco.io/copyright) +Copyright © 2017-2021 [Cloud Posse, LLC](https://cpco.io/copyright) @@ -416,8 +429,10 @@ Check out [our other projects][github], [follow us on twitter][twitter], [apply ### Contributors + | [![Erik Osterman][osterman_avatar]][osterman_homepage]
[Erik Osterman][osterman_homepage] | [![Igor Rodionov][goruha_avatar]][goruha_homepage]
[Igor Rodionov][goruha_homepage] | [![Andriy Knysh][aknysh_avatar]][aknysh_homepage]
[Andriy Knysh][aknysh_homepage] | [![Sarkis Varozian][sarkis_avatar]][sarkis_homepage]
[Sarkis Varozian][sarkis_homepage] | [![Joe Niland][joe-niland_avatar]][joe-niland_homepage]
[Joe Niland][joe-niland_homepage] | |---|---|---|---|---| + [osterman_homepage]: https://github.com/osterman [osterman_avatar]: https://img.cloudposse.com/150x150/https://github.com/osterman.png diff --git a/README.yaml b/README.yaml index 8cff681..8d7513a 100644 --- a/README.yaml +++ b/README.yaml @@ -67,7 +67,9 @@ usage: |- In this example, we'll trigger the pipeline anytime the `master` branch is updated. ```hcl module "ecs_push_pipeline" { - source = "git::https://github.com/cloudposse/terraform-aws-ecs-codepipeline.git?ref=master" + source = "cloudposse/ecs-codepipeline/aws" + # Cloud Posse recommends pinning every module to a specific version + # version = "x.x.x" name = "app" namespace = "eg" stage = "staging" @@ -88,7 +90,9 @@ usage: |- ```hcl module "ecs_release_pipeline" { - source = "git::https://github.com/cloudposse/terraform-aws-ecs-codepipeline.git?ref=master" + source = "cloudposse/ecs-codepipeline/aws" + # Cloud Posse recommends pinning every module to a specific version + # version = "x.x.x" name = "app" namespace = "eg" stage = "staging" diff --git a/context.tf b/context.tf index 64630e4..f5f2797 100644 --- a/context.tf +++ b/context.tf @@ -19,7 +19,8 @@ # module "this" { - source = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.21.0" + source = "cloudposse/label/null" + version = "0.22.1" // requires Terraform >= 0.12.26 enabled = var.enabled namespace = var.namespace @@ -164,4 +165,4 @@ variable "id_length_limit" { EOT } -#### End of copy of cloudposse/terraform-null-label/variables.tf \ No newline at end of file +#### End of copy of cloudposse/terraform-null-label/variables.tf diff --git a/docs/terraform.md b/docs/terraform.md index b1e1167..8c09b5a 100644 --- a/docs/terraform.md +++ b/docs/terraform.md @@ -3,7 +3,7 @@ | Name | Version | |------|---------| -| terraform | >= 0.12.0 | +| terraform | >= 0.12.26 | | aws | >= 2.0 | | local | >= 1.2 | | null | >= 2.0 | diff --git a/examples/complete/context.tf b/examples/complete/context.tf index 64630e4..f5f2797 100644 --- a/examples/complete/context.tf +++ b/examples/complete/context.tf @@ -19,7 +19,8 @@ # module "this" { - source = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.21.0" + source = "cloudposse/label/null" + version = "0.22.1" // requires Terraform >= 0.12.26 enabled = var.enabled namespace = var.namespace @@ -164,4 +165,4 @@ variable "id_length_limit" { EOT } -#### End of copy of cloudposse/terraform-null-label/variables.tf \ No newline at end of file +#### End of copy of cloudposse/terraform-null-label/variables.tf diff --git a/examples/complete/fixtures.us-east-2.tfvars b/examples/complete/fixtures.us-east-2.tfvars index 608533e..b0a1a7d 100644 --- a/examples/complete/fixtures.us-east-2.tfvars +++ b/examples/complete/fixtures.us-east-2.tfvars @@ -80,6 +80,8 @@ container_port_mappings = [ github_oauth_token = "test" +github_anonymous = true + repo_owner = "cloudposse" repo_name = "terraform-aws-ecs-codepipeline" diff --git a/examples/complete/main.tf b/examples/complete/main.tf index e89f833..0a16b96 100644 --- a/examples/complete/main.tf +++ b/examples/complete/main.tf @@ -3,20 +3,24 @@ provider "aws" { } module "vpc" { - source = "git::https://github.com/cloudposse/terraform-aws-vpc.git?ref=tags/0.18.0" + source = "cloudposse/vpc/aws" + version = "0.18.1" cidr_block = var.vpc_cidr_block - context = module.this.context + + context = module.this.context } module "subnets" { - source = "git::https://github.com/cloudposse/terraform-aws-dynamic-subnets.git?ref=tags/0.31.0" + source = "cloudposse/dynamic-subnets/aws" + version = "0.33.0" availability_zones = var.availability_zones vpc_id = module.vpc.vpc_id igw_id = module.vpc.igw_id cidr_block = module.vpc.vpc_cidr_block nat_gateway_enabled = true nat_instance_enabled = false - context = module.this.context + + context = module.this.context } resource "aws_ecs_cluster" "default" { @@ -25,7 +29,8 @@ resource "aws_ecs_cluster" "default" { } module "container_definition" { - source = "git::https://github.com/cloudposse/terraform-aws-ecs-container-definition.git?ref=tags/0.41.0" + source = "cloudposse/ecs-container-definition/aws" + version = "0.46.1" container_name = var.container_name container_image = var.container_image container_memory = var.container_memory @@ -38,12 +43,8 @@ module "container_definition" { } module "ecs_alb_service_task" { - source = "git::https://github.com/cloudposse/terraform-aws-ecs-alb-service-task.git?ref=tags/0.40.2" - namespace = module.this.namespace - stage = module.this.stage - name = module.this.name - attributes = module.this.attributes - delimiter = module.this.delimiter + source = "cloudposse/ecs-alb-service-task/aws" + version = "0.42.3" alb_security_group = module.vpc.vpc_default_security_group_id container_definition_json = module.container_definition.json_map_encoded_list ecs_cluster_arn = aws_ecs_cluster.default.arn @@ -62,12 +63,15 @@ module "ecs_alb_service_task" { desired_count = var.desired_count task_memory = var.task_memory task_cpu = var.task_cpu + + context = module.this.context } module "ecs_codepipeline" { source = "../../" region = var.region github_oauth_token = var.github_oauth_token + github_anonymous = var.github_anonymous repo_owner = var.repo_owner repo_name = var.repo_name branch = var.branch @@ -83,5 +87,6 @@ module "ecs_codepipeline" { environment_variables = var.environment_variables ecs_cluster_name = aws_ecs_cluster.default.name service_name = module.ecs_alb_service_task.service_name - context = module.this.context + + context = module.this.context } diff --git a/examples/complete/variables.tf b/examples/complete/variables.tf index 1354fb8..8e39a47 100644 --- a/examples/complete/variables.tf +++ b/examples/complete/variables.tf @@ -126,6 +126,11 @@ variable "github_oauth_token" { description = "GitHub OAuth Token with permissions to access private repositories" } +variable "github_anonymous" { + type = bool + description = "Github Anonymous API (if `true`, token must not be set as GITHUB_TOKEN or `github_token`)" +} + variable "repo_owner" { type = string description = "GitHub Organization or Username" diff --git a/examples/complete/versions.tf b/examples/complete/versions.tf index 231179c..5240dea 100644 --- a/examples/complete/versions.tf +++ b/examples/complete/versions.tf @@ -1,11 +1,26 @@ terraform { - required_version = ">= 0.12.0" + required_version = ">= 0.12.26" required_providers { - aws = ">= 2.0" - template = ">= 2.0" - local = ">= 1.2" - random = ">= 2.1" - null = ">= 2.0" + aws = { + source = "hashicorp/aws" + version = ">= 2.0" + } + template = { + source = "hashicorp/template" + version = ">= 2.0" + } + local = { + source = "hashicorp/local" + version = ">= 1.2" + } + random = { + source = "hashicorp/random" + version = ">= 2.1" + } + null = { + source = "hashicorp/null" + version = ">= 2.0" + } } } diff --git a/main.tf b/main.tf index 69e1b9f..5f2a391 100644 --- a/main.tf +++ b/main.tf @@ -1,7 +1,9 @@ module "codepipeline_label" { - source = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.21.0" - attributes = compact(concat(var.attributes, ["codepipeline"])) - context = module.this.context + source = "cloudposse/label/null" + version = "0.22.1" + attributes = ["codepipeline"] + + context = module.this.context } resource "aws_s3_bucket" "default" { @@ -13,9 +15,11 @@ resource "aws_s3_bucket" "default" { } module "codepipeline_assume_role_label" { - source = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.21.0" - context = module.this.context - attributes = compact(concat(var.attributes, ["codepipeline", "assume"])) + source = "cloudposse/label/null" + version = "0.22.1" + attributes = ["codepipeline", "assume"] + + context = module.this.context } resource "aws_iam_role" "default" { @@ -83,9 +87,11 @@ resource "aws_iam_role_policy_attachment" "s3" { } module "codepipeline_s3_policy_label" { - source = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.21.0" - attributes = compact(concat(var.attributes, ["codepipeline", "s3"])) - context = module.this.context + source = "cloudposse/label/null" + version = "0.22.1" + attributes = ["codepipeline", "s3"] + + context = module.this.context } resource "aws_iam_policy" "s3" { @@ -123,9 +129,11 @@ resource "aws_iam_role_policy_attachment" "codebuild" { } module "codebuild_label" { - source = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.21.0" - attributes = compact(concat(var.attributes, ["codebuild"])) - context = module.this.context + source = "cloudposse/label/null" + version = "0.22.1" + attributes = ["codebuild"] + + context = module.this.context } resource "aws_iam_policy" "codebuild" { @@ -155,10 +163,12 @@ resource "aws_iam_role_policy_attachment" "codestar" { } module "codestar_label" { - source = "git::https://github.com/cloudposse/terraform-null-label.git?ref=tags/0.17.0" + source = "cloudposse/label/null" + version = "0.22.1" enabled = module.this.enabled && var.codestar_connection_arn != "" - attributes = compact(concat(var.attributes, ["codestar"])) - context = module.this.context + attributes = ["codestar"] + + context = module.this.context } resource "aws_iam_policy" "codestar" { @@ -197,18 +207,14 @@ data "aws_region" "default" { } module "codebuild" { - source = "git::https://github.com/cloudposse/terraform-aws-codebuild.git?ref=tags/0.25.0" - enabled = module.this.enabled - namespace = module.this.namespace - name = module.this.name - stage = module.this.stage + source = "cloudposse/codebuild/aws" + version = "0.27.0" build_image = var.build_image build_compute_type = var.build_compute_type build_timeout = var.build_timeout buildspec = var.buildspec delimiter = module.this.delimiter - attributes = concat(module.this.attributes, ["build"]) - tags = module.this.tags + attributes = ["build"] privileged_mode = var.privileged_mode aws_region = var.region != "" ? var.region : data.aws_region.default.name aws_account_id = var.aws_account_id != "" ? var.aws_account_id : data.aws_caller_identity.default.account_id @@ -219,6 +225,8 @@ module "codebuild" { badge_enabled = var.badge_enabled cache_type = var.cache_type local_cache_modes = var.local_cache_modes + + context = module.this.context } resource "aws_iam_role_policy_attachment" "codebuild_s3" { @@ -303,7 +311,7 @@ resource "aws_codepipeline" "default" { } lifecycle { - # prevent github OAuthToken from causing updates, since it's removed from state file + # prevent github OAuthToken from causing updates, since it's removed from state file ignore_changes = [stage[0].action[0].configuration] } } @@ -416,7 +424,10 @@ resource "aws_codepipeline_webhook" "webhook" { } module "github_webhooks" { - source = "git::https://github.com/cloudposse/terraform-github-repository-webhooks.git?ref=tags/0.10.0" + source = "cloudposse/repository-webhooks/github" + version = "0.10.0" + # TODO: update version after release of TF 0.14 and context.tf support + # version = "0.11.0" enabled = module.this.enabled && var.webhook_enabled ? true : false github_anonymous = var.github_anonymous github_organization = var.repo_owner @@ -426,4 +437,7 @@ module "github_webhooks" { webhook_secret = local.webhook_secret webhook_content_type = "json" events = var.github_webhook_events + + # TODO: uncomment after release + # context = module.this.context } diff --git a/variables.tf b/variables.tf index 32b1e79..397b08b 100644 --- a/variables.tf +++ b/variables.tf @@ -177,4 +177,3 @@ variable "local_cache_modes" { default = [] description = "Specifies settings that AWS CodeBuild uses to store and reuse build dependencies. Valid values: LOCAL_SOURCE_CACHE, LOCAL_DOCKER_LAYER_CACHE, and LOCAL_CUSTOM_CACHE" } - diff --git a/versions.tf b/versions.tf index 231179c..5240dea 100644 --- a/versions.tf +++ b/versions.tf @@ -1,11 +1,26 @@ terraform { - required_version = ">= 0.12.0" + required_version = ">= 0.12.26" required_providers { - aws = ">= 2.0" - template = ">= 2.0" - local = ">= 1.2" - random = ">= 2.1" - null = ">= 2.0" + aws = { + source = "hashicorp/aws" + version = ">= 2.0" + } + template = { + source = "hashicorp/template" + version = ">= 2.0" + } + local = { + source = "hashicorp/local" + version = ">= 1.2" + } + random = { + source = "hashicorp/random" + version = ">= 2.1" + } + null = { + source = "hashicorp/null" + version = ">= 2.0" + } } }