Skip to content

Conversation

@adis-io
Copy link
Contributor

@adis-io adis-io commented Jan 29, 2025

Without this policy it won't be possible to use custom retention period

@adis-io adis-io requested review from a team as code owners January 29, 2025 10:17
@lecaros
Copy link
Contributor

lecaros commented Mar 17, 2025

@adis-io do you have a valid test case for this?
or a reference from the Cloudwatch's docs?

@adis-io
Copy link
Contributor Author

adis-io commented Mar 17, 2025

@adis-io do you have a valid test case for this? or a reference from the Cloudwatch's docs?

No, I don't have test case, I've just had this issue in my case

From AWS Docs:

When you create a log group, by default the log events in the log group do not expire. To set a retention policy so that events expire and are deleted after a specified time, use PutRetentionPolicy .

@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Jun 24, 2025
@adis-io
Copy link
Contributor Author

adis-io commented Jun 24, 2025

Anything is required from my side to get it merged?

@lecaros
Copy link
Contributor

lecaros commented Jun 24, 2025

hi @adis-io. We'd expect the conversations/reviews to be resolved before merging. If it is resolved, please mark it as such
cc @patrick-stephens

@github-actions github-actions bot removed the Stale label Jun 25, 2025
@github-actions
Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the Stale label Sep 24, 2025
@adis-io
Copy link
Contributor Author

adis-io commented Sep 24, 2025

hi @patrick-stephens, @lecaros

Please take a look

@patrick-stephens patrick-stephens merged commit 38016e3 into fluent:master Sep 24, 2025
7 checks passed
@adis-io adis-io deleted the patch-1 branch September 24, 2025 10:52
TomlinfreeGit pushed a commit to TomlinfreeGit/fluent-bit-docs that referenced this pull request Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants