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

fix(deps): update all otel collector packages #6670

Conversation

renovate-bot
Copy link
Contributor

@renovate-bot renovate-bot commented Feb 4, 2025

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
go.opentelemetry.io/collector/confmap/provider/fileprovider v1.24.0 -> v1.25.0 age adoption passing confidence
go.opentelemetry.io/collector/confmap/provider/httpprovider v1.24.0 -> v1.25.0 age adoption passing confidence
go.opentelemetry.io/collector/exporter/debugexporter v0.118.0 -> v0.119.0 age adoption passing confidence
go.opentelemetry.io/collector/exporter/otlphttpexporter v0.118.0 -> v0.119.0 age adoption passing confidence
go.opentelemetry.io/collector/extension/zpagesextension v0.118.0 -> v0.119.0 age adoption passing confidence
go.opentelemetry.io/collector/otelcol v0.118.0 -> v0.119.0 age adoption passing confidence
go.opentelemetry.io/collector/processor/batchprocessor v0.118.0 -> v0.119.0 age adoption passing confidence
go.opentelemetry.io/collector/processor/memorylimiterprocessor v0.118.0 -> v0.119.0 age adoption passing confidence

Release Notes

open-telemetry/opentelemetry-collector (go.opentelemetry.io/collector/confmap/provider/fileprovider)

v1.25.0

🛑 Breaking changes 🛑
  • exporterhelper: Rename exporter span signal specific attributes (e.g. "sent_spans" / "send_failed_span") to "items.sent" / "items.failed". (#​12165)
  • cmd/mdatagen: Remove dead field telemetry::level (#​12144)
  • exporterhelper: Change exporter ID to be a Span level attribute instead on each event. (#​12164)
    This does not have an impact on the level of information emitted, but on the structure of the Span.
  • cmd/mdatagen: Remove level field from metrics definition (#​12145)
    This mechanism will be added back once a new views mechanism is implemented.
💡 Enhancements 💡
  • configtls: Allow users to mention their preferred curve types for ECDHE handshake (#​12174)
  • service: remove custom code and instead use config package to instantiate meter provider. (#​11611)
  • otelcol: Adds support for listing config providers in components command's output (#​11570)
  • general: Reduce memory allocations when loading configuration and parsing component names (#​11964)
🧰 Bug fixes 🧰
  • exporterhelper: Fix bug that the exporter with new batcher may have been marked as non mutation. (#​12239)
    Only affects users that manually turned on exporter.UsePullingBasedExporterQueueBatcher featuregate.

  • service: Preserve URL normalization logic that was present before. (#​12254)

  • confighttp: confighttp.ToServer now sets ErrorLog with a default logger backed by Zap (#​11820)

    This change ensures that the http.Server's ErrorLog is correctly set using Zap's logger at the error level, addressing the issue of error logs being printed using a different logger.

  • exporterhelper: Fix context propagation for DisabledBatcher (#​12231)

  • mdatagen: apply fieldalignment to generated code (#​12125)

  • mdatagen: Fix bug where Histograms were marked as not supporting temporaly aggregation (#​12168)

  • exporterhelper: Fix MergeSplit issue that ignores the initial message size. (#​12257)

  • service: Include validation errors from telemetry.Config when validating the service config (#​12100)
    Previously validation errors were only printed to the console

  • service-telemetry: pass the missing async error channel into service telemetry settings (#​11417)


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate-bot renovate-bot requested a review from a team as a code owner February 4, 2025 05:59
@forking-renovate forking-renovate bot added the changelog:dependencies Update to dependencies label Feb 4, 2025
Copy link

forking-renovate bot commented Feb 4, 2025

ℹ Artifact update notice

File name: go.mod

In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

  • 13 additional dependencies were updated

Details:

Package Change
go.opentelemetry.io/collector/internal/memorylimiter v0.118.0 -> v0.119.0
go.opentelemetry.io/collector/service v0.118.0 -> v0.119.0
go.opentelemetry.io/contrib/bridges/otelzap v0.6.0 -> v0.9.0
go.opentelemetry.io/contrib/config v0.10.0 -> v0.14.0
go.opentelemetry.io/contrib/propagators/b3 v1.31.0 -> v1.34.0
go.opentelemetry.io/contrib/zpages v0.56.0 -> v0.59.0
go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp v0.7.0 -> v0.10.0
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc v1.32.0 -> v1.34.0
go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp v1.32.0 -> v1.34.0
go.opentelemetry.io/otel/exporters/stdout/stdoutlog v0.7.0 -> v0.10.0
go.opentelemetry.io/otel/exporters/stdout/stdoutmetric v1.32.0 -> v1.34.0
go.opentelemetry.io/otel/log v0.8.0 -> v0.10.0
go.opentelemetry.io/otel/sdk/log v0.7.0 -> v0.10.0

Copy link

codecov bot commented Feb 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.98%. Comparing base (57ce1b4) to head (b752bd0).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6670      +/-   ##
==========================================
+ Coverage   95.96%   95.98%   +0.01%     
==========================================
  Files         363      363              
  Lines       20601    20601              
==========================================
+ Hits        19770    19773       +3     
+ Misses        634      632       -2     
+ Partials      197      196       -1     
Flag Coverage Δ
badger_v1 9.82% <ø> (ø)
badger_v2 1.82% <ø> (ø)
cassandra-4.x-v1-manual 14.79% <ø> (ø)
cassandra-4.x-v2-auto 1.81% <ø> (ø)
cassandra-4.x-v2-manual 1.81% <ø> (ø)
cassandra-5.x-v1-manual 14.79% <ø> (ø)
cassandra-5.x-v2-auto 1.81% <ø> (ø)
cassandra-5.x-v2-manual 1.81% <ø> (ø)
elasticsearch-6.x-v1 19.12% <ø> (ø)
elasticsearch-7.x-v1 19.20% <ø> (ø)
elasticsearch-8.x-v1 19.37% <ø> (ø)
elasticsearch-8.x-v2 1.82% <ø> (ø)
grpc_v1 10.80% <ø> (ø)
grpc_v2 7.81% <ø> (ø)
kafka-3.x-v1 10.12% <ø> (ø)
kafka-3.x-v2 1.82% <ø> (ø)
memory_v2 1.82% <ø> (ø)
opensearch-1.x-v1 19.25% <ø> (ø)
opensearch-2.x-v1 19.25% <ø> (ø)
opensearch-2.x-v2 1.82% <ø> (ø)
tailsampling-processor 0.48% <ø> (ø)
unittests 94.86% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@renovate-bot renovate-bot force-pushed the renovate/all-otel-collector-packages branch from b5c8d8c to f896a87 Compare February 4, 2025 06:23
@yurishkuro yurishkuro enabled auto-merge (squash) February 4, 2025 07:29
@yurishkuro
Copy link
Member

Kafka ingester failing with:

 ⏩⏩⏩ Start jaeger-v2-ingester error logs:
  2025/02/04 07:36:03 application version: git-commit=, git-version=, build-date=
  Error: failed to create SDK: binding address 0.0.0.0:8888 for Prometheus exporter: listen tcp 0.0.0.0:8888: bind: address already in use
  2025/02/04 07:36:03 failed to create SDK: binding address 0.0.0.0:8888 for Prometheus exporter: listen tcp 0.0.0.0:8888: bind: address already in use

cc @mahadzaryab1

auto-merge was automatically disabled February 5, 2025 04:10

Head branch was pushed to by a user without write access

@renovate-bot renovate-bot force-pushed the renovate/all-otel-collector-packages branch from f896a87 to 7ce0b0e Compare February 5, 2025 04:10
@yurishkuro
Copy link
Member

we have a bug in cmd/jaeger/config-kafka-ingester.yaml
we already use different port for healthcheck

  healthcheckv2:
    use_v2: true
    http:
      # use different port to avoid conflict with collector
      endpoint: 0.0.0.0:14133 

but we didn't do the same for the metrics port

    metrics:
      level: detailed
      address: 0.0.0.0:8888

I am not sure what changed in the collector that this is now causing a port conflict when it didn't before, but we did have port conflicts on the healthchecks, so we need to change our config

@renovate-bot renovate-bot force-pushed the renovate/all-otel-collector-packages branch from 7ce0b0e to b752bd0 Compare February 5, 2025 13:19
@mahadzaryab1 mahadzaryab1 enabled auto-merge (squash) February 5, 2025 13:20
@mahadzaryab1 mahadzaryab1 merged commit 8afbf86 into jaegertracing:main Feb 5, 2025
55 checks passed
@renovate-bot renovate-bot deleted the renovate/all-otel-collector-packages branch February 5, 2025 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants