openshift
[release-5.9] LOG-5156: Add rotated files to exclude_paths globs
Description
Simple change, but need to make sure it's complete and my logic is sound. I rewrote this one final time and simply added the exclude pattern for rotated logs.
Vector config exclude glob patterns all end with *.log. This results in rotated infra log files being sent to the app index in certain scenarios. This change adds .log.* to the existing excluded extensions to avoid any rotated files being collected. the .gz and .tmp extensions are already added to exclude in all cases.
/cc @Clee2691 @vparfonov /assign @jcantrill
/cherry-pick master
Links
- JIRA: https://issues.redhat.com/browse/LOG-5156
@cahartma: This pull request references LOG-5156 which is a valid jira issue.
In response to this:
Description
Simple change, but need to make sure it's complete and my logic is sound. I rewrote this one final time and simply added the exclude pattern for rotated logs.
Vector config exclude glob patterns all end with
*.log. This results in rotated infra log files being sent to the app index in certain scenarios. This change adds.log.*to the existing excluded extensions to avoid any rotated files being collected. the.gzand.tmpextensions are already added to exclude in all cases./cc @Clee2691 @vparfonov /assign @jcantrill
/cherry-pick master
Links
- JIRA: https://issues.redhat.com/browse/LOG-5156
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cahartma
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [cahartma]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
![openshift-ci[bot] avatar](https://avatars.githubusercontent.com/in/91280?v=4 "Published by a pub.dev verified publisher"){kind=small}
@cahartma: all tests passed!
Full PR test history. Your PR dashboard.
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.