Skip to content

Commit

Permalink
fix config for globally
Browse files Browse the repository at this point in the history
  • Loading branch information
anmolnagpal committed Nov 10, 2021
1 parent e49da55 commit 9e917ae
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 16 deletions.
1 change: 0 additions & 1 deletion main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ module "config" {
managedby = var.managedby
enabled = var.enabled && var.config_enabled
only_config_enabled = var.only_config_enabled
s3_key_prefix = var.s3_key_prefix
config_role_arn = var.config_role_arn
config_s3_bucket_name = var.config_s3_bucket_name
target_config_bucket = var.target_config_bucket
Expand Down
1 change: 0 additions & 1 deletion modules/config/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -277,7 +277,6 @@ resource "aws_config_delivery_channel" "bucket" {

name = format("%s-delivery-channel", module.labels.id)
s3_bucket_name = var.config_s3_bucket_name
s3_key_prefix = var.s3_key_prefix
sns_topic_arn = module.sns.topic-arn

snapshot_delivery_properties {
Expand Down
10 changes: 2 additions & 8 deletions modules/config/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -1624,12 +1624,6 @@ variable "only_config_enabled" {

variable "config_role_arn" {
type = string
default = ""
description = "config role arn"
}

variable "s3_key_prefix" {
type = string
default = "AWSLogs/442186832995/Config"
description = "s3 key prefix"
default = "config role arn"
description = "enable when need only config without alerts"
}
6 changes: 0 additions & 6 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -1829,9 +1829,3 @@ variable "config_role_arn" {
default = ""
description = "config role arn"
}

variable "s3_key_prefix" {
type = string
default = "AWSLogs/442186832995/Config"
description = "s3 key prefix"
}

0 comments on commit 9e917ae

Please sign in to comment.