Skip to content
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

Stop sending metrics reported by batch processor #4273

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

dmitryax
Copy link
Contributor

@dmitryax dmitryax commented Feb 7, 2024

The batch processor metrics are not being used in any default dashboards but signicantly contribute towards metric timeseries created on the backend.

@dmitryax dmitryax requested review from a team as code owners February 7, 2024 21:33
@dmitryax dmitryax force-pushed the disable-excessive-batch-metrics branch from eb378ce to d1d6701 Compare February 7, 2024 21:34
Copy link
Contributor

@crobert-1 crobert-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to drop these in the logs_config_linux.yaml config as well?

CHANGELOG.md Outdated Show resolved Hide resolved
@dmitryax
Copy link
Contributor Author

dmitryax commented Feb 7, 2024

Do we want to drop these in the logs_config_linux.yaml config as well?

There is no internal metrics pipeline there

@dmitryax dmitryax force-pushed the disable-excessive-batch-metrics branch from d1d6701 to a7197f0 Compare February 7, 2024 21:51
The batch processor metrics are not being used in any default dashboards but signicantly contribute towards metric timeseries created on the backend.
- otelcol_processor_batch_batch_send_size
- otelcol_processor_batch_timeout_trigger_send
@dmitryax dmitryax force-pushed the disable-excessive-batch-metrics branch from a7197f0 to d4aa7d5 Compare February 7, 2024 22:19
@dmitryax dmitryax merged commit 61f9230 into signalfx:main Feb 7, 2024
128 of 131 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 7, 2024
@dmitryax dmitryax deleted the disable-excessive-batch-metrics branch February 8, 2024 21:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants