Skip to content

CAB-4856: Apply master-branch kno2-discharge-summary Terraform and check for drift #38

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
irsanchez merged 2 commits into from
Oct 2, 2025
Merged

CAB-4856: Apply master-branch kno2-discharge-summary Terraform and check for drift #38

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
878bc93
New vars/logic to allow conditional mgmt of log group resource
irsanchez Oct 2, 2025
dfc19e2
Reverted subscription count logic
irsanchez Oct 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion lambda_function/lambda_function.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ locals {
}

resource "aws_cloudwatch_log_group" "lambda_log_group" {
count = var.manage_log_group ? 1 : 0
name = local.log_group_name
retention_in_days = var.log_retention_in_days
tags = var.log_tags == {} ? var.tags : var.log_tags
Expand All @@ -12,7 +13,7 @@ resource "aws_cloudwatch_log_group" "lambda_log_group" {
resource "aws_cloudwatch_log_subscription_filter" "example_subscription_filter" {
count = var.ship_logs_to_sumo ? 1 : 0
name = coalesce(var.subscription_filter_name, "${var.function_name}_subscription_filter")
log_group_name = aws_cloudwatch_log_group.lambda_log_group.name
log_group_name = var.manage_log_group ? aws_cloudwatch_log_group.lambda_log_group[0].name : local.log_group_name
destination_arn = "arn:aws:lambda:${data.aws_region.current.name}:${data.aws_caller_identity.current.account_id}:function:SumoCWLogsLambda"
filter_pattern = ""
}
Expand Down
5 changes: 5 additions & 0 deletions lambda_function/variables.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -201,3 +201,8 @@ variable "log_tags" {
default = {}
description = "Tags to apply to the log group. Defaults to the same tags as the lambda function if nothing is passed in."
}

variable "manage_log_group" {
type = bool
default = true
}