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

Update opentelemetry-go to v1.28.0 and refactor references to semantic conventions #5698

Merged

Conversation

renovate-bot
Copy link
Contributor

@renovate-bot renovate-bot commented Jul 2, 2024

In addition to dependency upgrade:

  • some fixes to Makefile to install tools
  • move all references to versioned semconv to a helper package otelsemconv so that we don't have to keep semver in sync in many places (but we still have the linter for that).
  • fix otel_tag adjuster to look for the new telemetry.sdk.* attributes, not the deprecated otel.library.*
  • change db instrumentation to use new db.query.text attrs instead of deprecated db.statement

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
go.opentelemetry.io/otel v1.27.0 -> v1.28.0 age adoption passing confidence
go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.27.0 -> v1.28.0 age adoption passing confidence
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.27.0 -> v1.28.0 age adoption passing confidence
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracehttp v1.27.0 -> v1.28.0 age adoption passing confidence
go.opentelemetry.io/otel/exporters/prometheus v0.49.0 -> v0.50.0 age adoption passing confidence
go.opentelemetry.io/otel/exporters/stdout/stdouttrace v1.27.0 -> v1.28.0 age adoption passing confidence
go.opentelemetry.io/otel/metric v1.27.0 -> v1.28.0 age adoption passing confidence
go.opentelemetry.io/otel/sdk v1.27.0 -> v1.28.0 age adoption passing confidence
go.opentelemetry.io/otel/sdk/metric v1.27.0 -> v1.28.0 age adoption passing confidence
go.opentelemetry.io/otel/trace v1.27.0 -> v1.28.0 age adoption passing confidence

Release Notes

open-telemetry/opentelemetry-go (go.opentelemetry.io/otel)

v1.28.0: Releases v1.28.0/v0.50.0/v0.4.0

Compare Source

Overview

Added
  • The IsEmpty method is added to the Instrument type in go.opentelemetry.io/otel/sdk/metric.
    This method is used to check if an Instrument instance is a zero-value. (#​5431)
  • Store and provide the emitted context.Context in ScopeRecords of go.opentelemetry.io/otel/sdk/log/logtest. (#​5468)
  • The go.opentelemetry.io/otel/semconv/v1.26.0 package.
    The package contains semantic conventions from the v1.26.0 version of the OpenTelemetry Semantic Conventions. (#​5476)
  • The AssertRecordEqual method to go.opentelemetry.io/otel/log/logtest to allow comparison of two log records in tests. (#​5499)
  • The WithHeaders option to go.opentelemetry.io/otel/exporters/zipkin to allow configuring custom http headers while exporting spans. (#​5530)
Changed
  • Tracer.Start in go.opentelemetry.io/otel/trace/noop no longer allocates a span for empty span context. (#​5457)
  • Upgrade go.opentelemetry.io/otel/semconv/v1.25.0 to go.opentelemetry.io/otel/semconv/v1.26.0 in go.opentelemetry.io/otel/example/otel-collector. (#​5490)
  • Upgrade go.opentelemetry.io/otel/semconv/v1.25.0 to go.opentelemetry.io/otel/semconv/v1.26.0 in go.opentelemetry.io/otel/example/zipkin. (#​5490)
  • Upgrade go.opentelemetry.io/otel/semconv/v1.25.0 to go.opentelemetry.io/otel/semconv/v1.26.0 in go.opentelemetry.io/otel/exporters/zipkin. (#​5490)
    • The exporter no longer exports the deprecated "otel.library.name" or "otel.library.version" attributes.
  • Upgrade go.opentelemetry.io/otel/semconv/v1.25.0 to go.opentelemetry.io/otel/semconv/v1.26.0 in go.opentelemetry.io/otel/sdk/resource. (#​5490)
  • Upgrade go.opentelemetry.io/otel/semconv/v1.25.0 to go.opentelemetry.io/otel/semconv/v1.26.0 in go.opentelemetry.io/otel/sdk/trace. (#​5490)
  • SimpleProcessor.OnEmit in go.opentelemetry.io/otel/sdk/log no longer allocates a slice which makes it possible to have a zero-allocation log processing using SimpleProcessor. (#​5493)
  • Use non-generic functions in the Start method of "go.opentelemetry.io/otel/sdk/trace".Trace to reduce memory allocation. (#​5497)
  • service.instance.id is populated for a Resource created with "go.opentelemetry.io/otel/sdk/resource".Default with a default value when OTEL_GO_X_RESOURCE is set. (#​5520)
  • Improve performance of metric instruments in go.opentelemetry.io/otel/sdk/metric by removing unnecessary calls to time.Now. (#​5545)
Fixed
  • Log a warning to the OpenTelemetry internal logger when a Record in go.opentelemetry.io/otel/sdk/log drops an attribute due to a limit being reached. (#​5376)
  • Identify the Tracer returned from the global TracerProvider in go.opentelemetry.io/otel/global with its schema URL. (#​5426)
  • Identify the Meter returned from the global MeterProvider in go.opentelemetry.io/otel/global with its schema URL. (#​5426)
  • Log a warning to the OpenTelemetry internal logger when a Span in go.opentelemetry.io/otel/sdk/trace drops an attribute, event, or link due to a limit being reached. (#​5434)
  • Document instrument name requirements in go.opentelemetry.io/otel/metric. (#​5435)
  • Prevent random number generation data-race for experimental rand exemplars in go.opentelemetry.io/otel/sdk/metric. (#​5456)
  • Fix counting number of dropped attributes of Record in go.opentelemetry.io/otel/sdk/log. (#​5464)
  • Fix panic in baggage creation when a member contains 0x80 char in key or value. (#​5494)
  • Correct comments for the priority of the WithEndpoint and WithEndpointURL options and their corresponding environment variables in go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc. (#​5508)
  • Retry trace and span ID generation if it generated an invalid one in go.opentelemetry.io/otel/sdk/trace. (#​5514)
  • Fix stale timestamps reported by the last-value aggregation. (#​5517)
  • Indicate the Exporter in go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp must be created by the New method. (#​5521)
  • Improved performance in all {Bool,Int64,Float64,String}SliceValue functions of go.opentelemetry.io/attributes by reducing the number of allocations. (#​5549)

What's Changed

New Contributors

Full Changelog: open-telemetry/opentelemetry-go@v1.27.0...v1.28.0


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 has been generated by Mend Renovate. View repository job log here.

Signed-off-by: Mend Renovate <bot@renovateapp.com>
@renovate-bot renovate-bot requested a review from a team as a code owner July 2, 2024 22:02
@forking-renovate forking-renovate bot added the changelog:dependencies Update to dependencies label Jul 2, 2024
Copy link

ℹ 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):

  • 4 additional dependencies were updated

Details:

Package Change
github.com/go-logr/logr v1.4.1 -> v1.4.2
go.opentelemetry.io/proto/otlp v1.2.0 -> v1.3.1
google.golang.org/genproto/googleapis/api v0.0.0-20240528184218-531527333157 -> v0.0.0-20240701130421-f6361c86f094
google.golang.org/genproto/googleapis/rpc v0.0.0-20240528184218-531527333157 -> v0.0.0-20240701130421-f6361c86f094

yurishkuro and others added 3 commits July 2, 2024 23:29
Signed-off-by: Yuri Shkuro <github@ysh.us>
Signed-off-by: Yuri Shkuro <github@ysh.us>
@yurishkuro yurishkuro changed the title Update opentelemetry-go monorepo Update opentelemetry-go to v1.28.0 and refactor references to semantic conventions Jul 3, 2024
@yurishkuro yurishkuro removed the changelog:dependencies Update to dependencies label Jul 3, 2024
Signed-off-by: Yuri Shkuro <github@ysh.us>
@yurishkuro yurishkuro added changelog:breaking-change Change that is breaking public APIs or established behavior and removed changelog:bugfix-or-minor-feature labels Jul 3, 2024
@yurishkuro yurishkuro enabled auto-merge (squash) July 3, 2024 03:44
Copy link

codecov bot commented Jul 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.40%. Comparing base (1b4f355) to head (d46cf6e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5698      +/-   ##
==========================================
- Coverage   96.42%   96.40%   -0.03%     
==========================================
  Files         334      334              
  Lines       16144    16144              
==========================================
- Hits        15567    15563       -4     
- Misses        402      404       +2     
- Partials      175      177       +2     
Flag Coverage Δ
badger_v1 8.06% <0.00%> (ø)
badger_v2 1.90% <0.00%> (ø)
cassandra-3.x-v1 16.63% <100.00%> (ø)
cassandra-3.x-v2 1.82% <0.00%> (ø)
cassandra-4.x-v1 16.63% <100.00%> (ø)
cassandra-4.x-v2 1.82% <0.00%> (ø)
elasticsearch-7.x-v1 18.86% <0.00%> (ø)
elasticsearch-8.x-v1 19.04% <0.00%> (ø)
elasticsearch-8.x-v2 19.04% <0.00%> (-0.02%) ⬇️
grpc_v1 9.46% <0.00%> (ø)
grpc_v2 ?
kafka 9.75% <0.00%> (ø)
opensearch-1.x-v1 18.90% <0.00%> (ø)
opensearch-2.x-v1 18.90% <0.00%> (ø)
opensearch-2.x-v2 18.90% <0.00%> (ø)
unittests 94.19% <100.00%> (-0.03%) ⬇️

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.

@yurishkuro yurishkuro merged commit bbd863f into jaegertracing:main Jul 3, 2024
43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/otel changelog:breaking-change Change that is breaking public APIs or established behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants