Skip to content

Commit

Permalink
Merge pull request #21 from mackerelio-labs/fix-terraform-provider-aw…
Browse files Browse the repository at this point in the history
…s-v5.27.0

fix for terraform-provider-aws v5.27.0 or later.
  • Loading branch information
yseto authored Dec 8, 2023
2 parents 29a3028 + 5dda55c commit de9a148
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 4 deletions.
10 changes: 9 additions & 1 deletion cloudwatch-logs-aggregator/rule/main.tf
Original file line number Diff line number Diff line change
@@ -1,11 +1,19 @@
terraform {
required_providers {
aws = {
source = "hashicorp/aws"
version = ">= 5.27.0"
}
}
}
provider "aws" {
region = var.region
}

resource "aws_cloudwatch_event_rule" "this" {
name = var.rule_name
description = "created by mackerel-monitoring-modules"
is_enabled = var.is_enabled
state = var.state
schedule_expression = var.schedule_expression
}

Expand Down
6 changes: 3 additions & 3 deletions cloudwatch-logs-aggregator/rule/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,10 @@ variable "rule_name" {
type = string
}

variable "is_enabled" {
variable "state" {
description = "Whether the rules is enabled or not."
type = bool
default = true
type = string
default = "ENABLED"
}

variable "schedule_expression" {
Expand Down

0 comments on commit de9a148

Please sign in to comment.