Skip to content

[NR-475488] enable retention period for log groups#46

Open
ahegdeNR wants to merge 3 commits intomainfrom
change-log-groups-retention-period
Open

[NR-475488] enable retention period for log groups#46
ahegdeNR wants to merge 3 commits intomainfrom
change-log-groups-retention-period

Conversation

@ahegdeNR
Copy link
Copy Markdown
Contributor

Inline lambdas create log groups to post errors during cloud formation template deployment.

  1. Added retention period of 14 days.
  2. Created deletion policy so log groups get deleted when stack is deleted.

@sanathramesh sanathramesh self-requested a review October 23, 2025 09:14
Copy link
Copy Markdown
Contributor

@hrai-nr hrai-nr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change the instrumentation version as well in version.go and e2e test case related file.
Also change the property SemanticVersion, in all the template files, else it will fail.

Comment thread lambda-template.yaml
Type: AWS::Logs::LogGroup
DeletionPolicy: Delete
Properties:
LogGroupName: !Sub "/aws/lambda/${NewRelicLogsServerlessLogForwarder}"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing property RetentionInDays: 14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants