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

Release v1.23.0/v0.45.1 #4885

Merged
merged 5 commits into from
Feb 6, 2024
Merged

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Feb 5, 2024

This release contains the first stable, v1, release of the following modules:

  • go.opentelemetry.io/otel/bridge/opencensus
  • go.opentelemetry.io/otel/bridge/opencensus/test
  • go.opentelemetry.io/otel/example/opencensus
  • go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetricgrpc
  • go.opentelemetry.io/otel/exporters/otlp/otlpmetric/otlpmetrichttp
  • go.opentelemetry.io/otel/exporters/stdout/stdoutmetric

See our versioning policy for more information about these stability guarantees.

Added

Changed

  • The Merge and New functions in go.opentelemetry.io/otel/sdk/resource now returns a partial result if there is a schema URL merge conflict. Instead of returning nil when two Resources with different (non-empty) schema URLs are merged the merged Resource, along with the new ErrSchemaURLConflict error, is returned. It is up to the user to decide if they want to use the returned Resource or not. It may have desired attributes overwritten or include stale semantic conventions. (Return merged Resource on schema conflict #4876)

Fixed

@MrAlias MrAlias changed the title Release v1.23.0 Release v1.23.0/v0.45.1 Feb 5, 2024
@MrAlias MrAlias added this to the v1.23.0 milestone Feb 5, 2024
@MrAlias
Copy link
Contributor Author

MrAlias commented Feb 5, 2024

The experimental-metric modset has a patch version bump to capture the dependency bump. No minor release given the lack of significant changes there.

Copy link

codecov bot commented Feb 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2c7761d) 82.6% compared to head (694d468) 82.6%.
Report is 1 commits behind head on main.

❗ Current head 694d468 differs from pull request most recent head c0460a9. Consider uploading reports for the commit c0460a9 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #4885   +/-   ##
=====================================
  Coverage   82.6%   82.6%           
=====================================
  Files        232     232           
  Lines      18862   18868    +6     
=====================================
+ Hits       15595   15603    +8     
+ Misses      2977    2975    -2     
  Partials     290     290           
Files Coverage Δ
bridge/opencensus/version.go 100.0% <100.0%> (ø)
...xporters/otlp/otlpmetric/otlpmetricgrpc/version.go 100.0% <100.0%> (ø)
...xporters/otlp/otlpmetric/otlpmetrichttp/version.go 100.0% <100.0%> (ø)
exporters/otlp/otlptrace/version.go 100.0% <100.0%> (ø)
sdk/metric/version.go 100.0% <100.0%> (ø)
sdk/resource/auto.go 91.6% <100.0%> (+0.9%) ⬆️
sdk/resource/resource.go 76.6% <100.0%> (+0.1%) ⬆️
sdk/version.go 100.0% <100.0%> (ø)
version.go 100.0% <100.0%> (ø)

... and 1 file with indirect coverage changes

@pellared
Copy link
Member

pellared commented Feb 6, 2024

Consider merging/waiting for #4887 before the release (nice to have, not a blocker).

@dashpole

This comment was marked as resolved.

@MrAlias
Copy link
Contributor Author

MrAlias commented Feb 6, 2024

Consider merging/waiting for #4887 before the release (nice to have, not a blocker).

I'm going to move ahead with merging this today. Blocking on that PR means this release is delayed another 24 hours at least.

CHANGELOG.md Outdated Show resolved Hide resolved
@MrAlias MrAlias merged commit e3eb3f7 into open-telemetry:main Feb 6, 2024
22 of 23 checks passed
@MrAlias MrAlias deleted the release-v1.23.0 branch February 6, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants