Skip to content
This repository has been archived by the owner on Jul 12, 2023. It is now read-only.

Allow customizing log retention period #1902

Merged
merged 1 commit into from
Mar 2, 2021
Merged

Conversation

sethvargo
Copy link
Member

@sethvargo sethvargo commented Mar 2, 2021

Fixes #1901

Not marking auto-merge because I want to test manually.

Release Note

Allow customizing global log retention period for all services in the project. The default value is 14 days. **Note: this differs from the unconfigured value of 30 days!**. To retain the existing behavior, set `log_retention_period` to `30` in the Terraform configuration. However, we strongly recommend using a 14-day retention period instead.

@sethvargo sethvargo requested a review from a team as a code owner March 2, 2021 20:26
@google-cla google-cla bot added the cla: yes Auto: added by CLA bot when all committers have signed a CLA. label Mar 2, 2021
terraform/variables.tf Outdated Show resolved Hide resolved
@sethvargo
Copy link
Member Author

I've tested this and confirm it works as expected. Enabling auto-merge.

@sethvargo sethvargo enabled auto-merge (squash) March 2, 2021 21:18
@sethvargo sethvargo merged commit 8303b93 into main Mar 2, 2021
@sethvargo sethvargo deleted the sethvargo/log_retain branch March 2, 2021 21:20
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Auto: added by CLA bot when all committers have signed a CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set custom log retention thru terraform
2 participants