-
Notifications
You must be signed in to change notification settings - Fork 476
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(loki/src/k8s): not restart tailers in loki.source.kubernetes
component by above-average time deltas if K8s version is > 1.29.0
#6263
Conversation
loki.source.kubernetes
component by above-average time deltas if K8s version is >= 1.29.1loki.source.kubernetes
component by above-average time deltas if K8s version is >= 1.29.0
loki.source.kubernetes
component by above-average time deltas if K8s version is >= 1.29.0loki.source.kubernetes
component by above-average time deltas if K8s version is > 1.29.0
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.
Sorry this one got lost! LGTM, if you have time to fix the merge conflicts let me know, otherwise I'll push a commit to fix them once I get a chance.
…omponent by above-average time deltas if K8s version is >= 1.29.1 Signed-off-by: hainenber <dotronghai96@gmail.com>
…g verbosity when fail to restart tailer Signed-off-by: hainenber <dotronghai96@gmail.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
50fc778
to
6abb1c3
Compare
I've resolved the merge conflict :D |
Signed-off-by: Paschalis Tsilias <paschalist0@gmail.com>
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.
Sorry that this one slipped through the cracks @hainenber! I've taken the liberty to fix merge conflicts so we can merge and cherry-pick over to the Alloy repo.
Again, thank you!
PR Description
Detect K8s version to decide whether to kick off goroutine to restart tailers by above-average deltas. Helps decreasing extraneous logs and might be a tiny bit of performance.
Which issue(s) this PR fixes
Fixes grafana/alloy#264
Notes to the Reviewer
PR Checklist