-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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.
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
feat(aws_cloudwatch_logs): Add retention policy for cloudwatch logs sink #15955
feat(aws_cloudwatch_logs): Add retention policy for cloudwatch logs sink #15955
Conversation
✅ Deploy Preview for vector-project ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for vrl-playground canceled.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Add tests
- Add validation for
days
field - Add documentation examples
pub enabled: bool, | ||
|
||
#[configurable(derived)] | ||
pub days: i32, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add validation as this parameter supports only limited values set https://docs.aws.amazon.com/AmazonCloudWatchLogs/latest/APIReference/API_PutRetentionPolicy.html#API_PutRetentionPolicy_RequestSyntax
Regression Test Results
Run ID: b17b2389-fd35-4020-9473-06c5e2a60f35 Explanation
A regression test is an integrated performance test for
The table below, if present, lists those experiments that have experienced a
statistically significant change in their Changes in
Fine details of change detection per experiment.
|
Regression Test Results
Run ID: f6fc201e-2e04-4ee3-adc4-97faa658dca8 Explanation
A regression test is an integrated performance test for
The table below, if present, lists those experiments that have experienced a
statistically significant change in their No interesting changes in Fine details of change detection per experiment.
|
@dizlv just confirming you're working on this and didn't need any review or feedback yet? |
@spencergilbert yeah, got burden with regular stuff previous week, most likely will get back on this tomorrow |
No rush, just wanted to make sure I wasn't blocking you on something 😅 |
hello @dizlv ! Do plan to continue the development of this feature or maybe you need help? |
Closed by #18865 |
Solves #12038