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: proposal 1.6.0/experimental 0.32.0 #3192

Merged
merged 6 commits into from
Aug 24, 2022

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented Aug 23, 2022

1.6.0

🚀 (Enhancement)

  • perf(opentelemetry-core): improve hexToBase64 performance #3178
  • feat(sdk-trace-base): move Sampler declaration into sdk-trace-base #3088 @legendecas
  • fix(grpc-instrumentation): added grpc attributes in instrumentation #3127 @andrewzenkov
  • feat: support latest @opentelemetry/api #3177 @dyladan

🐛 (Bug Fix)

  • fix(context-async-hooks): Ensure listeners added using once can be removed using removeListener
    #3133

Experimental 0.32.0

💥 Breaking Change

  • Rename @opentelemetry/sdk-metrics-base package to @opentelemetry/sdk-metrics #3162 @hectorhdzg

🚀 (Enhancement)

  • fix(add-views-to-node-sdk): added the ability to define meter views in NodeSDK #3066 @weyert
  • feature(add-console-metrics-exporter): add ConsoleMetricExporter #3120 @weyert
  • feature(prometheus-serialiser): export the unit block when unit is set in metric descriptor #3066 @weyert
  • feat: support latest @opentelemetry/api #3177 @dyladan

🐛 (Bug Fix)

@dyladan dyladan requested a review from a team August 23, 2022 17:31
@codecov
Copy link

codecov bot commented Aug 23, 2022

Codecov Report

Merging #3192 (1bea328) into main (506f40b) will decrease coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 1bea328 differs from pull request most recent head af8c437. Consider uploading reports for the commit af8c437 to get more accurate results

@@            Coverage Diff             @@
##             main    #3192      +/-   ##
==========================================
- Coverage   92.17%   92.15%   -0.03%     
==========================================
  Files         191       87     -104     
  Lines        6055     2497    -3558     
  Branches     1292      547     -745     
==========================================
- Hits         5581     2301    -3280     
+ Misses        474      196     -278     
Impacted Files Coverage Δ
...s/opentelemetry-core/src/platform/node/sdk-info.ts 0.00% <0.00%> (-100.00%) ⬇️
...opentelemetry-core/src/platform/node/globalThis.ts 0.00% <0.00%> (-100.00%) ⬇️
...pentelemetry-core/src/platform/node/performance.ts 0.00% <0.00%> (-100.00%) ⬇️
...emetry-api-metrics/src/platform/node/globalThis.ts 0.00% <0.00%> (-100.00%) ⬇️
...lemetry-resources/src/detectors/ProcessDetector.ts 31.81% <0.00%> (-68.19%) ⬇️
...perimental/packages/otlp-exporter-base/src/util.ts 79.41% <0.00%> (-17.65%) ⬇️
...ckages/opentelemetry-exporter-zipkin/src/zipkin.ts 84.48% <0.00%> (-15.52%) ⬇️
...lemetry-resources/src/detectors/BrowserDetector.ts 93.33% <0.00%> (-6.67%) ⬇️
...es/opentelemetry-sdk-metrics/src/aggregator/Sum.ts
...l/packages/otlp-transformer/src/common/internal.ts
... and 113 more

Copy link
Member

@pichlermarc pichlermarc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!
Looks like https://github.com/open-telemetry/opentelemetry-js#compatibility-matrix needs to be updated with the new release (and 0.31.0), though. 🙂

@dyladan dyladan merged commit a5abee6 into open-telemetry:main Aug 24, 2022
@dyladan dyladan deleted the 1.6-0.32 branch August 24, 2022 17:19
@Flarna
Copy link
Member

Flarna commented Aug 31, 2022

I think @opentelemetry/sdk-metrics-base should be set to deprecated and referring to @opentelemetry/sdk-metrics

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.

6 participants