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

[chore] Update collector image to 0.119.0 #1671

Merged
merged 4 commits into from
Feb 21, 2025
Merged

Conversation

dmitryax
Copy link
Contributor

@dmitryax dmitryax commented Feb 21, 2025

Applied additional changes to adopt changes in the 0.119.0 image:

  1. Update internal metrics prometheus reader to default values to replicate default values in the new reader fields as set in https://github.com/open-telemetry/opentelemetry-collector/blob/main/service/telemetry/config.go#L92-L94
  2. Update default exclusion rules for internal metrics to avoid scraping redundant metrics as done in [chore] Update default metric_relabel_configs for internal metrics splunk-otel-collector#5918
  3. Remove bumped number of batch exporter workers in noDropLogsPipeline. The number of workers was initially increased to overcome [exporterhelper] Awkwardness due to API between queue sender and batch sender open-telemetry/opentelemetry-collector#10368. The new pulling batcher resolving the issue was enabled by default in 0.119.0. So increasing the number of workers isn't needed anymore.

@dmitryax dmitryax requested review from a team as code owners February 21, 2025 01:40
@dmitryax dmitryax force-pushed the update-collector-image branch from 7fb7611 to 7d66c98 Compare February 21, 2025 04:27
@jvoravong jvoravong merged commit 0160e16 into main Feb 21, 2025
71 checks passed
@jvoravong jvoravong deleted the update-collector-image branch February 21, 2025 16:05
@github-actions github-actions bot locked and limited conversation to collaborators Feb 21, 2025
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.

2 participants