Skip to content

Commit

Permalink
Merge pull request #1 from 3scale-ops/initial-release
Browse files Browse the repository at this point in the history
Initial release of tf-aws-cloudtrail
  • Loading branch information
3scale-robot authored Apr 22, 2021
2 parents 57574cd + c668089 commit c773502
Show file tree
Hide file tree
Showing 11 changed files with 554 additions and 1 deletion.
25 changes: 25 additions & 0 deletions .github/ISSUE_TEMPLATE/bug.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
---
name: Bug Report
about: Report a bug encountered while using the module
title: 'bug: '
labels: kind/bug

---

<!--
Please use this template while reporting a bug and provide as much info as possible.
Thanks!
-->

#### What happened:

#### What you expected to happen:

#### How to reproduce it (as minimally and precisely as possible):

#### Anything else we need to know?:

#### Environment:
- Terraform version:
- Module version:
- Others:
36 changes: 36 additions & 0 deletions .github/ISSUE_TEMPLATE/feature.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
---
name: Feature
about: Default feature template
title: 'feature: '
labels: needs-priority, kind/feature
assignees: ''

---

## Why?

> A clear and concise description of what the feature tries to achieve and its context. Consider coming back to a feature request months later and being able to understand where it came for just reading this.
## How?

> If this has been somehow defined in the grooming, specify here. Not always required.
## Requirements

> The list of requirements agreed on the grooming. This list should never change in the middle of a sprint, unless approved by all team members, and only exceptionally in that case.
- [ ] requirement1
- [ ] requirement2
- [ ] ...

## Acceptance Criteria

> Describe how to check if some of the requirements are met, when not obvious.
- [ ] check1
- [ ] check2
- [ ] ...

## Notes

> Optional seccion for notes related to the feature, links, ...
51 changes: 51 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,51 @@
<!-- Thanks for sending a pull request! -->

#### What type of PR is this?

<!--
Add one of the following kinds:
/kind bug
/kind cleanup
/kind documentation
/kind feature
/kind design
-->

#### What this PR does / why we need it:

#### Which issue(s) this PR fixes:
<!--
*Automatically closes linked issue when PR is merged.
Usage: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
-->

Fixes #

#### Special notes for your reviewer:

#### Does this PR introduce a user-facing change?

<!--
If no, just write "NONE" in the release-note block below.
If yes, a release note is required:
Enter your extended release note in the block below. If the PR requires additional action from users switching to the new release, include the string "action required".
-->

```release-note
```

#### Additional documentation e.g., usage docs, etc.:

<!--
This section can be blank if this pull request does not require a release note.
Please use the following format for linking documentation:
- [Usage]: <link>
- [Other doc]: <link>
-->

```docs
```
19 changes: 19 additions & 0 deletions .github/workflows/format-tests.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
name: format-tests

on:
pull_request:

jobs:

test:
runs-on: ubuntu-latest
container: quay.io/3scale/soyuz:v0.3.0-ci
steps:

- uses: actions/checkout@v2

- name: Run docs tests
run: make test-docs

- name: Run terraform tests
run: make test-terraform
32 changes: 32 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
# Local .terraform directories
**/.terraform/*

# .tfstate files
*.tfstate
*.tfstate.*

# Lock file
.terraform.lock.hcl

# Crash log files
crash.log

# Ignore any .tfvars files that are generated automatically for each Terraform run. Most
# .tfvars files are managed as part of configuration and so should be included in
# version control.
#
# example.tfvars

# Ignore override files as they are usually used to override resources locally and so
# are not checked in
override.tf
override.tf.json
*_override.tf
*_override.tf.json

# Include override files you do wish to add to version control using negated pattern
#
# !example_override.tf

# Include tfplan files to ignore the plan output of command: terraform plan -out=tfplan
# example: *tfplan*
39 changes: 39 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
TEST_DOCKER_IMAGE = quay.io/3scale/soyuz:v0.3.0-ci
TEST_DOCKER_RUN = docker run -ti --rm -w /src -v $(PWD):/src $(TEST_DOCKER_IMAGE)

help: ## Print this help
@awk 'BEGIN {FS = ":.*?## "} /^[a-zA-Z0-9_-]+:.*?## / {printf "\033[36m%-30s\033[0m %s\n", $$1, $$2}' $(MAKEFILE_LIST)

install-test-tools: ## Install test dependencies
GO111MODULE=on go get github.com/raviqqe/liche

update-test-tools: ## Update test dependencies
GO111MODULE=on go get -u github.com/raviqqe/liche

test: test-docs test-terraform ## Run all tests

docker-test: docker-test-docs docker-test-terraform ## Run all tests with docker

test-terraform: test-terraform-fmt ## Run all terraform tests

docker-test-terraform: docker-test-terraform-fmt ## Run all terraform tests with docker

TF_FMT_CHECK_CMD = terraform fmt -check -diff -recursive .

test-terraform-fmt: ## Run terraform format test
$(TF_FMT_CHECK_CMD)

docker-test-terraform-fmt: ## Run terraform format test with docker
$(TEST_DOCKER_RUN) $(TF_FMT_CHECK_CMD)

test-docs: test-docs-relative-links ## Run all documentation tests

docker-test-docs: docker-test-docs-relative-links ## Run all documentation tests with docker

DOCS_LICHE_CMD = liche -r . --exclude http.*

test-docs-relative-links: ## Run documentation relative links tests
$(DOCS_LICHE_CMD)

docker-test-docs-relative-links: ## Run documentation relative links tests with docker
$(TEST_DOCKER_RUN) $(DOCS_LICHE_CMD)
8 changes: 8 additions & 0 deletions OWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
approvers:
- raelga
- roivaz
- slopezz
reviewers:
- raelga
- roivaz
- slopezz
24 changes: 23 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,2 +1,24 @@
# tf-aws-cloudtrail
Terraform module to configure aws cloudtrail

[![format-tests](https://github.com/3scale-ops/tf-aws-cloudtrail/workflows/format-tests/badge.svg)](https://github.com/3scale-ops/tf-aws-cloudtrail/actions/workflows/format-tests.yaml?query=workflow%3Aformat-tests)
[![license](https://badgen.net/github/license/3scale-ops/tf-aws-cloudtrail)](https://github.com/3scale-ops/tf-aws-cloudtrail/blob/main/LICENSE)

This module tries to standarize the way we configure AWS Cloudtrail in aws accounts

## Outputs

| Output | Description |
| ------ | ----------------------------------------- |
| cloudtrail_bucket_name | Cloudtrail S3 bucket name |

## Contributing

You can contribute by:

* Raising any issues you find using the module
* Fixing issues by opening [Pull Requests](https://github.com/3scale-ops/tf-aws-cloudtrail/pulls)
* Submitting a patch or opening a PR
* Improving documentation
* Talking about the module

All bugs, tasks or enhancements are tracked as [GitHub issues](https://github.com/3scale-ops/tf-aws-cloudtrail/issues).
Loading

0 comments on commit c773502

Please sign in to comment.