Skip to content

Commit

Permalink
Merge branch 'master' into b-aws_lambda-update-qualified_arn-on-publish
Browse files Browse the repository at this point in the history
  • Loading branch information
gdavison committed Nov 4, 2020
2 parents 0b00088 + 08c9116 commit 97fa230
Show file tree
Hide file tree
Showing 6,246 changed files with 204,808 additions and 2,382,653 deletions.
The diff you're trying to view is too large. We only load the first 3000 changed files.
51 changes: 30 additions & 21 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,33 +1,42 @@
# Default owner for all pull requests
* @terraform-providers/aws-provider
* @hashicorp/terraform-aws

# Service specific owners
/aws/*apigatewayv2* @ewbankkit @terraform-providers/aws-provider
/website/**/apigatewayv2* @ewbankkit @terraform-providers/aws-provider
/aws/*apigatewayv2* @ewbankkit @hashicorp/terraform-aws
/website/**/apigatewayv2* @ewbankkit @hashicorp/terraform-aws

/aws/*appmesh* @ewbankkit @terraform-providers/aws-provider
/website/**/appmesh* @ewbankkit @terraform-providers/aws-provider
/aws/*appmesh* @ewbankkit @hashicorp/terraform-aws
/website/**/appmesh* @ewbankkit @hashicorp/terraform-aws

/aws/*backup* @ewbankkit @terraform-providers/aws-provider
/website/**/backup* @ewbankkit @terraform-providers/aws-provider
/aws/*backup* @ewbankkit @hashicorp/terraform-aws
/website/**/backup* @ewbankkit @hashicorp/terraform-aws

/aws/*_aws_codeartifact_*.go @DrFaust92 @terraform-providers/aws-provider
/website/**/codeartifact* @DrFaust92 @terraform-providers/aws-provider
/aws/*_aws_codeartifact_*.go @DrFaust92 @hashicorp/terraform-aws
/website/**/codeartifact* @DrFaust92 @hashicorp/terraform-aws

/aws/*_aws_fsx_*.go @DrFaust92 @terraform-providers/aws-provider
/website/**/fsx* @DrFaust92 @terraform-providers/aws-provider
/aws/*_aws_fsx_*.go @DrFaust92 @hashicorp/terraform-aws
/website/**/fsx* @DrFaust92 @hashicorp/terraform-aws

/aws/*globalaccelerator* @ewbankkit @terraform-providers/aws-provider
/website/**/globalaccelerator* @ewbankkit @terraform-providers/aws-provider
/aws/*globalaccelerator* @ewbankkit @hashicorp/terraform-aws
/website/**/globalaccelerator* @ewbankkit @hashicorp/terraform-aws

/aws/*_aws_glue_*.go @DrFaust92 @terraform-providers/aws-provider
/website/**/glue* @DrFaust92 @terraform-providers/aws-provider
/aws/*_aws_glue_*.go @DrFaust92 @hashicorp/terraform-aws
/website/**/glue* @DrFaust92 @hashicorp/terraform-aws

/aws/*route53_resolver* @ewbankkit @terraform-providers/aws-provider
/website/**/route53_resolver* @ewbankkit @terraform-providers/aws-provider
/aws/*kinesis_analytics* @ewbankkit @hashicorp/terraform-aws
/website/**/kinesis_analytics* @ewbankkit @hashicorp/terraform-aws

/aws/*_aws_storagegateway_*.go @DrFaust92 @terraform-providers/aws-provider
/website/**/storagegateway* @DrFaust92 @terraform-providers/aws-provider
/aws/*kinesisanalyticsv2* @ewbankkit @hashicorp/terraform-aws
/website/**/kinesisanalyticsv2* @ewbankkit @hashicorp/terraform-aws

/aws/*_aws_sagemaker_*.go @DrFaust92 @terraform-providers/aws-provider
/website/**/sagemaker* @DrFaust92 @terraform-providers/aws-provider
/aws/*route53_resolver* @ewbankkit @hashicorp/terraform-aws
/website/**/route53_resolver* @ewbankkit @hashicorp/terraform-aws

/aws/*_aws_storagegateway_*.go @DrFaust92 @hashicorp/terraform-aws
/website/**/storagegateway* @DrFaust92 @hashicorp/terraform-aws

/aws/*_aws_sagemaker_*.go @DrFaust92 @hashicorp/terraform-aws
/website/**/sagemaker* @DrFaust92 @hashicorp/terraform-aws

/aws/*_aws_workspaces_*.go @Tensho @hashicorp/terraform-aws
/website/**/workspaces* @Tensho @hashicorp/terraform-aws
5 changes: 1 addition & 4 deletions .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
blank_issues_enabled: false
contact_links:
- name: Terraform AWS Provider Roadmap
url: https://github.com/terraform-providers/terraform-provider-aws/blob/master/ROADMAP.md
url: https://github.com/hashicorp/terraform-provider-aws/blob/master/ROADMAP.md
about: View the Roadmap for the upcoming work planned on the AWS provider.
- name: Terraform AWS Provider Questions
url: https://discuss.hashicorp.com/c/terraform-providers/tf-aws
Expand All @@ -12,6 +12,3 @@ contact_links:
- name: Terraform Language or Workflow Questions
url: https://discuss.hashicorp.com/c/terraform-core
about: Please ask and answer language or workflow related questions through the Terraform Core Community Forum.
- name: Security Vulnerability
url: https://www.hashicorp.com/security.html
about: Please report security vulnerabilities responsibly.
4 changes: 2 additions & 2 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<!--- See what makes a good Pull Request at : https://github.com/terraform-providers/terraform-provider-aws/blob/master/docs/CONTRIBUTING.md --->
<!--- See what makes a good Pull Request at : https://github.com/hashicorp/terraform-provider-aws/blob/master/docs/CONTRIBUTING.md --->

<!--- Please keep this note for the community --->

Expand All @@ -12,7 +12,7 @@
<!--- If your PR fully resolves and should automatically close the linked issue, use Closes. Otherwise, use Relates --->
Relates OR Closes #0000

Release note for [CHANGELOG](https://github.com/terraform-providers/terraform-provider-aws/blob/master/CHANGELOG.md):
Release note for [CHANGELOG](https://github.com/hashicorp/terraform-provider-aws/blob/master/CHANGELOG.md):
<!--
If change is not user facing, just write "NONE" in the release-note block below.
-->
Expand Down
25 changes: 25 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,28 @@ updates:
directory: "/"
schedule:
interval: "daily"
- package-ecosystem: "gomod"
directory: "/"
ignore:
- dependency-name: "golang.org/x/tools"
- dependency-name: "google.golang.org/grpc"
schedule:
interval: "daily"
- package-ecosystem: "gomod"
directory: "/awsproviderlint"
ignore:
- dependency-name: "golang.org/x/tools"
- dependency-name: "google.golang.org/grpc"
schedule:
interval: "daily"
- package-ecosystem: "gomod"
directory: "/tools"
ignore:
- dependency-name: "golang.org/x/tools"
- dependency-name: "google.golang.org/grpc"
schedule:
interval: "daily"
- package-ecosystem: "terraform"
directory: "/infrastructure/repository"
schedule:
interval: "daily"
4 changes: 4 additions & 0 deletions .github/labeler-needs-triage.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
needs-triage:
- '**'
- '.*'
- '.*/**'
75 changes: 75 additions & 0 deletions .github/workflows/acctest-terraform-lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,75 @@
name: Acceptance Test Linting
on:
push:
branches:
- master
- "release/**"
pull_request:
paths:
- .github/workflows/acctest-terraform-lint.yml
- aws/*_test.go
- scripts/validate-terraform.sh
- tools/go.mod

env:
GO_VERSION: "1.15"
GO111MODULE: on

jobs:
terrafmt:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-go@v2
with:
go-version: ${{ env.GO_VERSION }}
- uses: actions/cache@v2
continue-on-error: true
timeout-minutes: 2
with:
path: ~/go/pkg/mod
key: ${{ runner.os }}-go-pkg-mod-${{ hashFiles('go.sum') }}
- run: cd tools && go install github.com/katbyte/terrafmt
# - run: terrafmt diff ./aws --check --pattern '*_test.go' --quiet --fmtcompat
- run: |
find ./aws -type f -name '*_test.go' \
| sort -u \
| grep -v resource_aws_apigatewayv2_domain_name_test.go \
| grep -v resource_aws_ecs_capacity_provider_test.go \
| grep -v resource_aws_efs_file_system_test.go \
| grep -v resource_aws_kinesis_stream_test.go \
| grep -v resource_aws_kms_grant_test.go \
| grep -v resource_aws_quicksight_user_test.go \
| grep -v resource_aws_s3_bucket_object_test.go \
| grep -v resource_aws_sns_platform_application_test.go \
| xargs -I {} terrafmt diff --check --quiet --fmtcompat {}
validate-terraform:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-go@v2
with:
go-version: ${{ env.GO_VERSION }}
- uses: actions/cache@v2
continue-on-error: true
timeout-minutes: 2
with:
path: ~/go/pkg/mod
key: ${{ runner.os }}-go-pkg-mod-${{ hashFiles('go.sum') }}
- run: cd tools && go install github.com/katbyte/terrafmt
- run: cd tools && go install github.com/terraform-linters/tflint
- run: |
find ./aws -type f -name '*_test.go' \
| sort -u \
| grep -v resource_aws_apigatewayv2_domain_name_test.go \
| grep -v resource_aws_ecs_capacity_provider_test.go \
| grep -v resource_aws_efs_file_system_test.go \
| grep -v resource_aws_elasticache_cluster_test.go \
| grep -v resource_aws_kinesis_stream_test.go \
| grep -v resource_aws_kms_grant_test.go \
| grep -v resource_aws_lambda_permission_test.go \
| grep -v resource_aws_quicksight_user_test.go \
| grep -v resource_aws_s3_bucket_object_test.go \
| grep -v resource_aws_sns_platform_application_test.go \
| ./scripts/validate-terraform.sh
37 changes: 30 additions & 7 deletions .github/workflows/changelog.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,30 +7,53 @@ on:
pull_request:
paths:
- CHANGELOG.md
pull_request_target:

env:
GO_VERSION: "1.14"
GO111MODULE: on

jobs:
changes:
if: github.event_name == 'pull_request_target' && !contains(fromJSON('["anGie44", "bflad", "breathingdust", "gdavison", "maryelizbeth", "YakDriver"]'), github.actor)
name: Filter Changes
runs-on: ubuntu-latest
outputs:
changed: ${{ steps.filter.outputs.changed }}
steps:
- uses: dorny/paths-filter@v2
id: filter
with:
filters: |
changed:
- CHANGELOG.md
comment:
if: github.event_name == 'pull_request' && !contains(fromJSON('["bflad", "breathingdust", "ewbankkit", "gdavison", "maryelizbeth"]'), github.actor)
needs: changes
if: ${{ needs.changes.outputs.changed == 'true' }}
name: Comment
runs-on: ubuntu-latest
steps:
- name: Find Existing PR Comment
id: prc
uses: peter-evans/find-comment@v1
with:
issue-number: ${{ github.event.pull_request.number }}
comment-author: "github-actions[bot]"
body-includes: "Please note that the `CHANGELOG.md` file contents are handled by the maintainers during merge"
- run: echo ${{ steps.prc.outputs.comment-id }}
- name: PR Comment
uses: unsplash/comment-on-pr@v1.2.0
if: ${{ steps.prc.outputs.comment-id == '' }}
uses: peter-evans/create-or-update-comment@v1
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
msg: |-
issue-number: ${{ github.event.pull_request.number }}
body: |-
Thank you for your contribution! :rocket:
Please note that the `CHANGELOG.md` file contents are handled by the maintainers during merge. This is to prevent pull request merge conflicts, especially for contributions which may not be merged immediately. Please see the [Contributing Guide](https://github.com/terraform-providers/terraform-provider-aws/blob/master/.github/CONTRIBUTING.md) for additional pull request review items.
Please note that the `CHANGELOG.md` file contents are handled by the maintainers during merge. This is to prevent pull request merge conflicts, especially for contributions which may not be merged immediately. Please see the [Contributing Guide](https://github.com/hashicorp/terraform-provider-aws/blob/master/docs/CONTRIBUTING.md) for additional pull request review items.
Remove any changes to the `CHANGELOG.md` file and commit them in this pull request to prevent delays with reviewing and potentially merging this pull request.
- name: Fail the check
run: exit 1
misspell:
runs-on: ubuntu-latest
steps:
Expand All @@ -44,5 +67,5 @@ jobs:
with:
path: ~/go/pkg/mod
key: ${{ runner.os }}-go-pkg-mod-${{ hashFiles('go.sum') }}
- run: go install github.com/client9/misspell/cmd/misspell
- run: cd tools && go install github.com/client9/misspell/cmd/misspell
- run: misspell -error -source text CHANGELOG.md
52 changes: 34 additions & 18 deletions .github/workflows/dependencies.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,39 +5,59 @@ on:
branches:
- master
- 'release/**'
pull_request:
paths:
- awsproviderlint/** # go/analysis/analysistest requires vendoring
- go.mod
- go.sum
- vendor/**
pull_request_target:

env:
GO_VERSION: "1.14"

jobs:
changes:
if: github.event_name == 'pull_request_target' && !contains(fromJSON('["anGie44", "bflad", "breathingdust", "dependabot[bot]", "DrFaust92", "ewbankkit", "gdavison", "maryelizbeth", "YakDriver"]'), github.actor)
name: Filter Changes
runs-on: ubuntu-latest
outputs:
changed: ${{ steps.filter.outputs.changed }}
steps:
- uses: dorny/paths-filter@v2
id: filter
with:
filters: |
changed:
- awsproviderlint/**
- go.mod
- go.sum
comment:
if: github.event_name == 'pull_request' && !contains(fromJSON('["anGie44", "bflad", "breathingdust", "DrFaust92", "ewbankkit", "gdavison", "maryelizbeth", "YakDriver", "renovate[bot]"]'), github.actor)
needs: changes
if: ${{ needs.changes.outputs.changed == 'true' }}
name: Comment
runs-on: ubuntu-latest
steps:
- name: Find Existing PR Comment
id: prc
uses: peter-evans/find-comment@v1
with:
issue-number: ${{ github.event.pull_request.number }}
comment-author: "github-actions[bot]"
body-includes: "Please note that typically Go dependency changes"
- run: echo ${{ steps.prc.outputs.comment-id }}
- name: PR Comment
uses: unsplash/comment-on-pr@v1.2.0
if: ${{ steps.prc.outputs.comment-id == '' }}
uses: peter-evans/create-or-update-comment@v1
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
check_duplicate_msg: true
msg: |-
issue-number: ${{ github.event.pull_request.number }}
body: |-
Thank you for your contribution! :rocket:
Please note that typically Go dependency changes are handled in this repository by Renovate Bot or the maintainers. This is to prevent pull request merge conflicts and further delay reviews of contributions. Remove any changes to the `go.mod`, `go.sum`, and `vendor/` files and commit them into this pull request.
Please note that typically Go dependency changes are handled in this repository by dependabot or the maintainers. This is to prevent pull request merge conflicts and further delay reviews of contributions. Remove any changes to the `go.mod` or `go.sum` files and commit them into this pull request.
Additional details:
* Check [open pull requests with the `dependencies` label](https://github.com/terraform-providers/terraform-provider-aws/pulls?q=is%3Aopen+is%3Apr+label%3Adependencies) to view other dependency updates.
* If this pull request includes an update the AWS Go SDK (or any other dependency) version, only updates submitted via Renovate Bot will be merged. This pull request will need to remove these changes and will need to be rebased after the existing dependency update via Renovate Bot has been merged for this pull request to be reviewed.
* Check [open pull requests with the `dependencies` label](https://github.com/hashicorp/terraform-provider-aws/pulls?q=is%3Aopen+is%3Apr+label%3Adependencies) to view other dependency updates.
* If this pull request includes an update the AWS Go SDK (or any other dependency) version, only updates submitted via dependabot will be merged. This pull request will need to remove these changes and will need to be rebased after the existing dependency update via dependabot has been merged for this pull request to be reviewed.
* If this pull request is for supporting a new AWS service:
* Ensure the new AWS service changes are following the [Contributing Guide section on new services](https://github.com/terraform-providers/terraform-provider-aws/blob/master/docs/contributing/contribution-checklists.md#new-service), in particular that the dependency addition and initial provider support are in a separate pull request from other changes (e.g. new resources). Contributions not following this item will not be reviewed until the changes are split.
* Ensure the new AWS service changes are following the [Contributing Guide section on new services](https://github.com/hashicorp/terraform-provider-aws/blob/master/docs/contributing/contribution-checklists.md#new-service), in particular that the dependency addition and initial provider support are in a separate pull request from other changes (e.g. new resources). Contributions not following this item will not be reviewed until the changes are split.
* If this pull request is already a separate pull request from the above item, you can ignore this message.
go_mod:
name: go mod
Expand All @@ -53,7 +73,3 @@ jobs:
go mod tidy
git diff --exit-code -- go.mod go.sum || \
(echo; echo "Unexpected difference in go.mod/go.sum files. Run 'go mod tidy' command or revert any go.mod/go.sum changes and commit."; exit 1)
echo "==> Checking source code with go mod vendor..."
go mod vendor
git diff --compact-summary --exit-code -- vendor || \
(echo; echo "Unexpected difference in vendor/ directory. Run 'go mod vendor' command or revert any go.mod/go.sum/vendor changes and commit."; exit 1)
2 changes: 1 addition & 1 deletion .github/workflows/documentation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,5 +47,5 @@ jobs:
with:
path: ~/go/pkg/mod
key: ${{ runner.os }}-go-pkg-mod-${{ hashFiles('go.sum') }}
- run: go install github.com/client9/misspell/cmd/misspell
- run: cd tools && go install github.com/client9/misspell/cmd/misspell
- run: misspell -error -source text docs/
Loading

0 comments on commit 97fa230

Please sign in to comment.