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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: proposal 1.7.0/0.33.0 #3259

Merged
merged 4 commits into from
Sep 16, 2022

Conversation

dyladan
Copy link
Member

@dyladan dyladan commented Sep 15, 2022

1.7.0

馃悰 (Bug Fix)

  • fix(sdk-trace-base): make span start times resistant to hrtime clock drift
    #3129

馃摎 (Refine Doc)

  • docs(metrics): add missing metrics packages to SDK reference documentation #3239 @dyladan

馃彔 (Internal)

Experimental/0.33.0

馃挜 Breaking Change

  • Add resourceDetectors option to NodeSDK #3210

    • NodeSDK.detectResources() function is no longer able to receive config as a parameter.
      Instead, the detectors are passed to the constructor.
  • chore(metrics-sdk): clean up exports #3197 @pichlermarc

    • removes export for:
      • AccumulationRecord
      • Aggregator
      • AggregatorKind
      • Accumulation
      • createInstrumentDescriptor
      • createInstrumentDescriptorWithView
      • isDescriptorCompatibleWith
  • chore(api-metrics): clean up exports #3198 @pichlermarc

    • removes export for:
      • NOOP_COUNTER_METRIC
      • NOOP_HISTOGRAM_METRIC
      • NOOP_METER_PROVIDER
      • NOOP_OBSERVABLE_COUNTER_METRIC
      • NOOP_OBSERVABLE_GAUGE_METRIC
      • NOOP_OBSERVABLE_UP_DOWN_COUNTER_METRIC
      • NOOP_UP_DOWN_COUNTER_METRIC
      • NoopCounterMetric
      • NoopHistogramMetric
      • NoopMeter
      • NoopMeterProvider
      • NoopMetric
      • NoopObservableCounterMetric
      • NoopObservableGaugeMetric
      • NoopObservableMetric
      • NoopObservableUpDownCounterMetric
      • NoopUpDownCounterMetric
  • feat(sdk-metrics): align MetricReader with specification and other language implementations #3225 @pichlermarc

  • chore(sdk-metrics): remove accidental export of the SDK Meter class #3243 @pichlermarc

馃殌 (Enhancement)

馃摎 (Refine Doc)

馃彔 (Internal)

@dyladan dyladan requested a review from a team as a code owner September 15, 2022 13:03
@codecov
Copy link

codecov bot commented Sep 15, 2022

Codecov Report

Merging #3259 (004835e) into main (4422d8c) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3259   +/-   ##
=======================================
  Coverage   93.29%   93.29%           
=======================================
  Files         203      203           
  Lines        6606     6606           
  Branches     1389     1389           
=======================================
  Hits         6163     6163           
  Misses        443      443           

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! 馃檪
We'll need to update the README.md too. 馃檪

Copy link
Member

@legendecas legendecas left a comment

Choose a reason for hiding this comment

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

馃帀 thank you for making the release!

@dyladan dyladan linked an issue Sep 16, 2022 that may be closed by this pull request
@dyladan dyladan merged commit ad88c3d into open-telemetry:main Sep 16, 2022
@dyladan dyladan deleted the candidate-1.7.0-0.33.0 branch September 16, 2022 11:55
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.

Sleeping computer causes span timings to be incorrect in browser
4 participants