Skip to content

Commit 6cd4dc7

Browse files
authored
Updates to ChatOps - Automated commit (#13)
## What * Adds chatops commands - '/test all' - '/test bats' - '/test readme' - '/test terratest' * Drops codefresh * Drops slash-command-dispatch * Removes codefresh badge * Rebuilds README ## Why * Change over from codefresh to GH Actions * Facilitate testing of PRs from forks
1 parent 97b7fd3 commit 6cd4dc7

File tree

11 files changed

+227
-123
lines changed

11 files changed

+227
-123
lines changed

.github/CODEOWNERS

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
# Use this file to define individuals or teams that are responsible for code in a repository.
2+
# Read more: <https://help.github.com/articles/about-codeowners/>
3+
#
4+
# Order is important: the last matching pattern takes the most precedence
5+
6+
# These owners will be the default owners for everything
7+
* @cloudposse/engineering @cloudposse/contributors
8+
9+
# Cloud Posse must review any changes to Makefiles
10+
**/Makefile @cloudposse/engineering
11+
**/Makefile.* @cloudposse/engineering
12+
13+
# Cloud Posse must review any changes to GitHub actions
14+
.github/* @cloudposse/engineering

.github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
---
2+
name: Bug report
3+
about: Create a report to help us improve
4+
title: ''
5+
labels: 'bug'
6+
assignees: ''
7+
8+
---
9+
10+
Found a bug? Maybe our [Slack Community](https://slack.cloudposse.com) can help.
11+
12+
[![Slack Community](https://slack.cloudposse.com/badge.svg)](https://slack.cloudposse.com)
13+
14+
## Describe the Bug
15+
A clear and concise description of what the bug is.
16+
17+
## Expected Behavior
18+
A clear and concise description of what you expected to happen.
19+
20+
## Steps to Reproduce
21+
Steps to reproduce the behavior:
22+
1. Go to '...'
23+
2. Run '....'
24+
3. Enter '....'
25+
4. See error
26+
27+
## Screenshots
28+
If applicable, add screenshots or logs to help explain your problem.
29+
30+
## Environment (please complete the following information):
31+
32+
Anything that will help us triage the bug will help. Here are some ideas:
33+
- OS: [e.g. Linux, OSX, WSL, etc]
34+
- Version [e.g. 10.15]
35+
36+
## Additional Context
37+
Add any other context about the problem here.

.github/ISSUE_TEMPLATE/config.yml

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
blank_issues_enabled: false
2+
3+
contact_links:
4+
5+
- name: Community Slack Team
6+
url: https://cloudposse.com/slack/
7+
about: |-
8+
Please ask and answer questions here.
9+
10+
- name: Office Hours
11+
url: https://cloudposse.com/office-hours/
12+
about: |-
13+
Join us every Wednesday for FREE Office Hours (lunch & learn).
14+
15+
- name: DevOps Accelerator Program
16+
url: https://cloudposse.com/accelerate/
17+
about: |-
18+
Own your infrastructure in record time. We build it. You drive it.
Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
---
2+
name: Feature Request
3+
about: Suggest an idea for this project
4+
title: ''
5+
labels: 'feature request'
6+
assignees: ''
7+
8+
---
9+
10+
Have a question? Please checkout our [Slack Community](https://slack.cloudposse.com) or visit our [Slack Archive](https://archive.sweetops.com/).
11+
12+
[![Slack Community](https://slack.cloudposse.com/badge.svg)](https://slack.cloudposse.com)
13+
14+
## Describe the Feature
15+
16+
A clear and concise description of what the bug is.
17+
18+
## Expected Behavior
19+
20+
A clear and concise description of what you expected to happen.
21+
22+
## Use Case
23+
24+
Is your feature request related to a problem/challenge you are trying to solve? Please provide some additional context of why this feature or capability will be valuable.
25+
26+
## Describe Ideal Solution
27+
28+
A clear and concise description of what you want to happen. If you don't know, that's okay.
29+
30+
## Alternatives Considered
31+
32+
Explain what alternative solutions or features you've considered.
33+
34+
## Additional Context
35+
36+
Add any other context or screenshots about the feature request here.

.github/ISSUE_TEMPLATE/question.md

Whitespace-only changes.

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
## what
2+
* Describe high-level what changed as a result of these commits (i.e. in plain-english, what do these changes mean?)
3+
* Use bullet points to be concise and to the point.
4+
5+
## why
6+
* Provide the justifications for the changes (e.g. business case).
7+
* Describe why these changes were made (e.g. why do these commits fix the problem?)
8+
* Use bullet points to be concise and to the point.
9+
10+
## references
11+
* Link to any supporting github issues or helpful documentation to add some context (e.g. stackoverflow).
12+
* Use `closes #123`, if this PR closes a GitHub issue `#123`
13+

.github/workflows/chatops.yml

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
name: chatops
2+
on:
3+
issue_comment:
4+
types: [created]
5+
6+
jobs:
7+
default:
8+
runs-on: ubuntu-latest
9+
steps:
10+
- uses: actions/checkout@v2
11+
- name: "Handle common commands"
12+
uses: cloudposse/actions/github/[email protected]
13+
with:
14+
token: ${{ secrets.PUBLIC_REPO_ACCESS_TOKEN }}
15+
reaction-token: ${{ secrets.GITHUB_TOKEN }}
16+
repository: cloudposse/actions
17+
commands: rebuild-readme, terraform-fmt
18+
permission: none
19+
issue-type: pull-request
20+
21+
test:
22+
runs-on: ubuntu-latest
23+
steps:
24+
- name: "Checkout commit"
25+
uses: actions/checkout@v2
26+
- name: "Run tests"
27+
uses: cloudposse/actions/github/[email protected]
28+
with:
29+
token: ${{ secrets.PUBLIC_REPO_ACCESS_TOKEN }}
30+
reaction-token: ${{ secrets.GITHUB_TOKEN }}
31+
repository: cloudposse/actions
32+
commands: test
33+
permission: none
34+
issue-type: pull-request
35+
reactions: false
36+
37+

README.md

Lines changed: 39 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@
4242

4343
[![Cloud Posse][logo]](https://cpco.io/homepage)
4444

45-
# terraform-aws-kms-key [![Codefresh Build Status](https://g.codefresh.io/api/badges/pipeline/cloudposse/terraform-modules%2Fterraform-aws-kms-key?type=cf-1)](https://g.codefresh.io/public/accounts/cloudposse/pipelines/5d163820e38a0455f412f79f) [![Latest Release](https://img.shields.io/github/release/cloudposse/terraform-aws-kms-key.svg)](https://github.com/cloudposse/terraform-aws-kms-key/releases/latest) [![Slack Community](https://slack.cloudposse.com/badge.svg)](https://slack.cloudposse.com)
45+
# terraform-aws-kms-key [![Latest Release](https://img.shields.io/github/release/cloudposse/terraform-aws-kms-key.svg)](https://github.com/cloudposse/terraform-aws-kms-key/releases/latest) [![Slack Community](https://slack.cloudposse.com/badge.svg)](https://slack.cloudposse.com)
4646

4747

4848
Terraform module to provision a [KMS](https://aws.amazon.com/kms/) key with alias.
@@ -119,32 +119,47 @@ Available targets:
119119
lint Lint terraform code
120120
121121
```
122+
## Requirements
123+
124+
| Name | Version |
125+
|------|---------|
126+
| terraform | ~> 0.12.0 |
127+
| aws | ~> 2.0 |
128+
| local | ~> 1.2 |
129+
| null | ~> 2.0 |
130+
131+
## Providers
132+
133+
| Name | Version |
134+
|------|---------|
135+
| aws | ~> 2.0 |
136+
122137
## Inputs
123138

124139
| Name | Description | Type | Default | Required |
125-
|------|-------------|:----:|:-----:|:-----:|
126-
| alias | The display name of the alias. The name must start with the word `alias` followed by a forward slash | string | `` | no |
127-
| attributes | Additional attributes (e.g. `1`) | list(string) | `<list>` | no |
128-
| deletion_window_in_days | Duration in days after which the key is deleted after destruction of the resource | number | `10` | no |
129-
| delimiter | Delimiter to be used between `namespace`, `environment`, `stage`, `name` and `attributes` | string | `-` | no |
130-
| description | The description of the key as viewed in AWS console | string | `Parameter Store KMS master key` | no |
131-
| enable_key_rotation | Specifies whether key rotation is enabled | bool | `true` | no |
132-
| enabled | Set to false to prevent the module from creating any resources | bool | `true` | no |
133-
| environment | Environment, e.g. 'prod', 'staging', 'dev', 'pre-prod', 'UAT' | string | `` | no |
134-
| name | Solution name, e.g. 'app' or 'jenkins' | string | `` | no |
135-
| namespace | Namespace, which could be your organization name or abbreviation, e.g. 'eg' or 'cp' | string | `` | no |
136-
| policy | A valid KMS policy JSON document. Note that if the policy document is not specific enough (but still valid), Terraform may view the policy as constantly changing in a terraform plan. In this case, please make sure you use the verbose/specific version of the policy. | string | `` | no |
137-
| stage | Stage, e.g. 'prod', 'staging', 'dev', OR 'source', 'build', 'test', 'deploy', 'release' | string | `` | no |
138-
| tags | Additional tags (e.g. `map('BusinessUnit','XYZ')` | map(string) | `<map>` | no |
140+
|------|-------------|------|---------|:--------:|
141+
| alias | The display name of the alias. The name must start with the word `alias` followed by a forward slash | `string` | `""` | no |
142+
| attributes | Additional attributes (e.g. `1`) | `list(string)` | `[]` | no |
143+
| deletion\_window\_in\_days | Duration in days after which the key is deleted after destruction of the resource | `number` | `10` | no |
144+
| delimiter | Delimiter to be used between `namespace`, `environment`, `stage`, `name` and `attributes` | `string` | `"-"` | no |
145+
| description | The description of the key as viewed in AWS console | `string` | `"Parameter Store KMS master key"` | no |
146+
| enable\_key\_rotation | Specifies whether key rotation is enabled | `bool` | `true` | no |
147+
| enabled | Set to false to prevent the module from creating any resources | `bool` | `true` | no |
148+
| environment | Environment, e.g. 'prod', 'staging', 'dev', 'pre-prod', 'UAT' | `string` | `""` | no |
149+
| name | Solution name, e.g. 'app' or 'jenkins' | `string` | `""` | no |
150+
| namespace | Namespace, which could be your organization name or abbreviation, e.g. 'eg' or 'cp' | `string` | `""` | no |
151+
| policy | A valid KMS policy JSON document. Note that if the policy document is not specific enough (but still valid), Terraform may view the policy as constantly changing in a terraform plan. In this case, please make sure you use the verbose/specific version of the policy. | `string` | `""` | no |
152+
| stage | Stage, e.g. 'prod', 'staging', 'dev', OR 'source', 'build', 'test', 'deploy', 'release' | `string` | `""` | no |
153+
| tags | Additional tags (e.g. `map('BusinessUnit','XYZ')` | `map(string)` | `{}` | no |
139154

140155
## Outputs
141156

142157
| Name | Description |
143158
|------|-------------|
144-
| alias_arn | Alias ARN |
145-
| alias_name | Alias name |
146-
| key_arn | Key ARN |
147-
| key_id | Key ID |
159+
| alias\_arn | Alias ARN |
160+
| alias\_name | Alias name |
161+
| key\_arn | Key ARN |
162+
| key\_id | Key ID |
148163

149164

150165

@@ -200,6 +215,10 @@ We deliver 10x the value for a fraction of the cost of a full-time engineer. Our
200215

201216
Join our [Open Source Community][slack] on Slack. It's **FREE** for everyone! Our "SweetOps" community is where you get to talk with others who share a similar vision for how to rollout and manage infrastructure. This is the best place to talk shop, ask questions, solicit feedback, and work together as a community to build totally *sweet* infrastructure.
202217

218+
## Discourse Forums
219+
220+
Participate in our [Discourse Forums][discourse]. Here you'll find answers to commonly asked questions. Most questions will be related to the enormous number of projects we support on our GitHub. Come here to collaborate on answers, find solutions, and get ideas about the products and services we value. It only takes a minute to get started! Just sign in with SSO using your GitHub account.
221+
203222
## Newsletter
204223

205224
Sign up for [our newsletter][newsletter] that covers everything on our technology radar. Receive updates on what we're up to on GitHub as well as awesome new projects we discover.
@@ -313,6 +332,7 @@ Check out [our other projects][github], [follow us on twitter][twitter], [apply
313332
[testimonial]: https://cpco.io/leave-testimonial?utm_source=github&utm_medium=readme&utm_campaign=cloudposse/terraform-aws-kms-key&utm_content=testimonial
314333
[office_hours]: https://cloudposse.com/office-hours?utm_source=github&utm_medium=readme&utm_campaign=cloudposse/terraform-aws-kms-key&utm_content=office_hours
315334
[newsletter]: https://cpco.io/newsletter?utm_source=github&utm_medium=readme&utm_campaign=cloudposse/terraform-aws-kms-key&utm_content=newsletter
335+
[discourse]: https://ask.sweetops.com/?utm_source=github&utm_medium=readme&utm_campaign=cloudposse/terraform-aws-kms-key&utm_content=discourse
316336
[email]: https://cpco.io/email?utm_source=github&utm_medium=readme&utm_campaign=cloudposse/terraform-aws-kms-key&utm_content=email
317337
[commercial_support]: https://cpco.io/commercial-support?utm_source=github&utm_medium=readme&utm_campaign=cloudposse/terraform-aws-kms-key&utm_content=commercial_support
318338
[we_love_open_source]: https://cpco.io/we-love-open-source?utm_source=github&utm_medium=readme&utm_campaign=cloudposse/terraform-aws-kms-key&utm_content=we_love_open_source

README.yaml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,33 +1,25 @@
1-
---
21
#
32
# This is the canonical configuration for the `README.md`
43
# Run `make readme` to rebuild the `README.md`
54
#
65

76
# Name of this project
87
name: terraform-aws-kms-key
9-
108
# Logo for this project
119
#logo: docs/logo.png
1210

1311
# License of this project
1412
license: "APACHE2"
15-
1613
# Canonical GitHub repo
1714
github_repo: cloudposse/terraform-aws-kms-key
18-
1915
# Badges to display
2016
badges:
21-
- name: "Codefresh Build Status"
22-
image: "https://g.codefresh.io/api/badges/pipeline/cloudposse/terraform-modules%2Fterraform-aws-kms-key?type=cf-1"
23-
url: "https://g.codefresh.io/public/accounts/cloudposse/pipelines/5d163820e38a0455f412f79f"
2417
- name: "Latest Release"
2518
image: "https://img.shields.io/github/release/cloudposse/terraform-aws-kms-key.svg"
2619
url: "https://github.com/cloudposse/terraform-aws-kms-key/releases/latest"
2720
- name: "Slack Community"
2821
image: "https://slack.cloudposse.com/badge.svg"
2922
url: "https://slack.cloudposse.com"
30-
3123
related:
3224
- name: "terraform-aws-iam-chamber-user"
3325
description: "Terraform module to provision a basic IAM chamber user with access to SSM parameters and KMS key to decrypt secrets, suitable for CI/CD systems (e.g. TravisCI, CircleCI, CodeFresh) or systems which are external to AWS that cannot leverage AWS IAM Instance Profiles"
@@ -38,7 +30,6 @@ related:
3830
- name: "terraform-aws-ssm-iam-role"
3931
description: "Terraform module to provision an IAM role with configurable permissions to access SSM Parameter Store"
4032
url: "https://github.com/cloudposse/terraform-aws-ssm-iam-role"
41-
4233
# Short description of this project
4334
description: |-
4435
Terraform module to provision a [KMS](https://aws.amazon.com/kms/) key with alias.
@@ -47,7 +38,6 @@ description: |-
4738
4839
* https://aws.amazon.com/systems-manager/features
4940
* https://aws.amazon.com/blogs/mt/the-right-way-to-store-secrets-using-parameter-store
50-
5141
# How to use this project
5242
usage: |-
5343
```hcl
@@ -62,11 +52,9 @@ usage: |-
6252
alias = "alias/parameter_store_key"
6353
}
6454
```
65-
6655
include:
6756
- "docs/targets.md"
6857
- "docs/terraform.md"
69-
7058
# Contributors to this project
7159
contributors:
7260
- name: "Andriy Knysh"

codefresh/test.yml

Lines changed: 0 additions & 74 deletions
This file was deleted.

0 commit comments

Comments
 (0)