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: release SDK 1.17.1/Experimental 0.44.0 #4183

Conversation

pichlermarc
Copy link
Member

@pichlermarc pichlermarc commented Oct 3, 2023

1.17.1

🐛 (Bug Fix)

  • fix(sdk-trace-base): BatchSpanProcessor flushes when maxExportBatchSize is reached #3958 @nordfjord
  • fix(sdk-metrics): allow instrument names to contain '/' #4155
  • fix(sdk-metrics): prevent per-reader storages from keeping unreported accumulations in memory #4163 @pichlermarc
    • fixes a memory leak which occurred when two or more MetricReader instances are registered to a MeterProvider
  • fix(sdk-metrics): do not report empty scopes and metrics #4135 @pichlermarc
    • Instruments that were created, but did not have measurements will not be exported anymore
    • Meters (Scopes) that were created, but did not have any instruments with measurements under them will not be exported anymore.
  • fix(exporter-zipkin): round duration to the nearest int in annotations to be compliant with zipkin protocol #4167 @FelipeEmerim

📚 (Refine Doc)

  • docs(contributing): added guidelines for adding benchmark tests #4169

🏠 (Internal)

  • test: added a performance benchmark test for span creation #4105
  • test: added a workflow to run performance benchmark tests automatically #4144

Experimental 0.44.0

💥 Breaking Change

  • fix(exporter-logs-otlp-proto): change OTLPLogExporter to OTLPLogExporter #4140 @Vunovati
  • fix(sdk-node): remove explicit dependency on @opentelemetry/exporter-jaeger
    • '@opentelemetry/exporter-jaeger' is no longer be a dependency of this package. To continue using '@opentelemetry/exporter-jaeger', please install it manually.
      • NOTE: @opentelemetry/exporter-jaeger is deprecated, consider switching to one of the alternatives described here
  • fix(sdk-logs): hide internal methods with internal shared state #3865 @legendecas

🚀 (Enhancement)

🐛 (Bug Fix)

  • fix(otlp-exporter-base): replaced usage of window with _globalThis #4157 @cristianmadularu
  • fix(otlp-transformer): Avoid precision loss when converting from HrTime to unix nanoseconds. #4062

@pichlermarc pichlermarc requested a review from a team October 3, 2023 14:16
@weyert
Copy link
Contributor

weyert commented Oct 4, 2023

Don't we want to include this #4163 in this release?

@pichlermarc
Copy link
Member Author

pichlermarc commented Oct 5, 2023

Don't we want to include this #4163 in this release?

I'd like to, but it will depend on if there are enough reviews for it by the time this is approved. We can get 1.17.1/0.44.0 out and then release a patch version next week.

@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #4183 (6449099) into main (c320c98) will decrease coverage by 0.02%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

❗ Current head 6449099 differs from pull request most recent head 4c3dfa4. Consider uploading reports for the commit 4c3dfa4 to get more accurate results

@@            Coverage Diff             @@
##             main    #4183      +/-   ##
==========================================
- Coverage   92.26%   92.24%   -0.02%     
==========================================
  Files         331      331              
  Lines        9473     9465       -8     
  Branches     1999     1997       -2     
==========================================
- Hits         8740     8731       -9     
- Misses        733      734       +1     

see 2 files with indirect coverage changes

@legendecas
Copy link
Member

#4163 is approved.

@pichlermarc pichlermarc merged commit f8e187b into open-telemetry:main Oct 10, 2023
17 checks passed
@pichlermarc pichlermarc deleted the chore/release-1.17.1-0.44.0 branch October 10, 2023 13:46
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.

5 participants