Skip to content

Commit fc33f68

Browse files
Jacob Woffendenbryantbiggs
andauthored
feat!: Add support for kms_key_arn; bump MSV of Terraform and AWS provider to 1.3 and 5.32 respectively to support (#16)
Co-authored-by: Bryant Biggs <bryantbiggs@gmail.com>
1 parent ce8c537 commit fc33f68

File tree

7 files changed

+21
-13
lines changed

7 files changed

+21
-13
lines changed

.pre-commit-config.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
repos:
22
- repo: https://github.com/antonbabenko/pre-commit-terraform
3-
rev: v1.88.0
3+
rev: v1.89.1
44
hooks:
55
- id: terraform_fmt
66
- id: terraform_docs
@@ -24,7 +24,7 @@ repos:
2424
- '--args=--only=terraform_unused_required_providers'
2525
- id: terraform_validate
2626
- repo: https://github.com/pre-commit/pre-commit-hooks
27-
rev: v4.5.0
27+
rev: v4.6.0
2828
hooks:
2929
- id: check-merge-conflict
3030
- id: end-of-file-fixer

README.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,14 +56,14 @@ Examples codified under the [`examples`](https://github.com/terraform-aws-module
5656

5757
| Name | Version |
5858
|------|---------|
59-
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 0.13.1 |
60-
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 4.35 |
59+
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.3 |
60+
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.32 |
6161

6262
## Providers
6363

6464
| Name | Version |
6565
|------|---------|
66-
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 4.35 |
66+
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.32 |
6767

6868
## Modules
6969

@@ -84,6 +84,7 @@ No modules.
8484
| <a name="input_alert_manager_definition"></a> [alert\_manager\_definition](#input\_alert\_manager\_definition) | The alert manager definition that you want to be applied. See more in the [AWS Docs](https://docs.aws.amazon.com/prometheus/latest/userguide/AMP-alert-manager.html) | `string` | `"alertmanager_config: |\n route:\n receiver: 'default'\n receivers:\n - name: 'default'\n"` | no |
8585
| <a name="input_create"></a> [create](#input\_create) | Determines whether a resources will be created | `bool` | `true` | no |
8686
| <a name="input_create_workspace"></a> [create\_workspace](#input\_create\_workspace) | Determines whether a workspace will be created or to use an existing workspace | `bool` | `true` | no |
87+
| <a name="input_kms_key_arn"></a> [kms\_key\_arn](#input\_kms\_key\_arn) | The ARN of the KMS Key to for encryption at rest | `string` | `null` | no |
8788
| <a name="input_logging_configuration"></a> [logging\_configuration](#input\_logging\_configuration) | The logging configuration of the prometheus workspace. | `map(string)` | `{}` | no |
8889
| <a name="input_rule_group_namespaces"></a> [rule\_group\_namespaces](#input\_rule\_group\_namespaces) | A map of one or more rule group namespace definitions | `map(any)` | `{}` | no |
8990
| <a name="input_tags"></a> [tags](#input\_tags) | A map of tags to add to all resources | `map(string)` | `{}` | no |

examples/complete/README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,14 +23,14 @@ Note that this example may create resources which will incur monetary charges on
2323

2424
| Name | Version |
2525
|------|---------|
26-
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 0.13.1 |
27-
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 4.35 |
26+
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.3 |
27+
| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 5.32 |
2828

2929
## Providers
3030

3131
| Name | Version |
3232
|------|---------|
33-
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 4.35 |
33+
| <a name="provider_aws"></a> [aws](#provider\_aws) | >= 5.32 |
3434

3535
## Modules
3636

examples/complete/versions.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
terraform {
2-
required_version = ">= 0.13.1"
2+
required_version = ">= 1.3"
33

44
required_providers {
55
aws = {
66
source = "hashicorp/aws"
7-
version = ">= 4.35"
7+
version = ">= 5.32"
88
}
99
}
1010
}

main.tf

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,8 @@ locals {
99
resource "aws_prometheus_workspace" "this" {
1010
count = var.create && var.create_workspace ? 1 : 0
1111

12-
alias = var.workspace_alias
12+
alias = var.workspace_alias
13+
kms_key_arn = var.kms_key_arn
1314

1415
dynamic "logging_configuration" {
1516
for_each = length(var.logging_configuration) > 0 ? [var.logging_configuration] : []

variables.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,12 @@ variable "logging_configuration" {
3838
default = {}
3939
}
4040

41+
variable "kms_key_arn" {
42+
description = "The ARN of the KMS Key to for encryption at rest"
43+
type = string
44+
default = null
45+
}
46+
4147
################################################################################
4248
# Alert Manager Definition
4349
################################################################################

versions.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
terraform {
2-
required_version = ">= 0.13.1"
2+
required_version = ">= 1.3"
33

44
required_providers {
55
aws = {
66
source = "hashicorp/aws"
7-
version = ">= 4.35"
7+
version = ">= 5.32"
88
}
99
}
1010
}

0 commit comments

Comments
 (0)