opentelemetry-go icon indicating copy to clipboard operation
opentelemetry-go copied to clipboard

Merge the new_sdk/main branch into main

Open MrAlias opened this issue 3 years ago • 1 comments

Blocked by all issues in https://github.com/open-telemetry/opentelemetry-go/milestone/22. This will be the last issue that closes the milestone.

MrAlias avatar Apr 19 '22 20:04 MrAlias

This needs to include deleting the new_sdk/example branch. It will no longer be needed.

MrAlias avatar Apr 19 '22 20:04 MrAlias

The PR number needs to be added to the changelog entries added in #3150

MrAlias avatar Sep 06 '22 21:09 MrAlias

Merging will orphan the following packages:

  • go.opentelemetry.io/otel/sdk/metric/aggregator/aggregatortest
  • go.opentelemetry.io/otel/sdk/metric/aggregator/histogram
  • go.opentelemetry.io/otel/sdk/metric/aggregator/lastvalue
  • go.opentelemetry.io/otel/sdk/metric/aggregator/sum
  • go.opentelemetry.io/otel/sdk/metric/aggregator
  • go.opentelemetry.io/otel/sdk/metric/controller/basic
  • go.opentelemetry.io/otel/sdk/metric/controller/controllertest
  • go.opentelemetry.io/otel/sdk/metric/controller/time
  • go.opentelemetry.io/otel/sdk/metric/export/aggregation
  • go.opentelemetry.io/otel/sdk/metric/export
  • go.opentelemetry.io/otel/sdk/metric/number
  • go.opentelemetry.io/otel/sdk/metric/processor/basic
  • go.opentelemetry.io/otel/sdk/metric/processor/processortest
  • go.opentelemetry.io/otel/sdk/metric/processor/reducer
  • go.opentelemetry.io/otel/sdk/metric/registry
  • go.opentelemetry.io/otel/sdk/metric/sdkapi
  • go.opentelemetry.io/otel/sdk/metric/selector/simple

Do we want to plan a deprecation strategy for these? (Related to https://github.com/open-telemetry/opentelemetry-go/issues/2328)

MrAlias avatar Sep 06 '22 22:09 MrAlias

Question about go.opentelemetry.io/otel/sdk/metric/aggregator/exponential/mapping (which exists today and should stay somewhere). I think it's OK if that package moves.

jmacd avatar Sep 08 '22 17:09 jmacd

Question about go.opentelemetry.io/otel/sdk/metric/aggregator/exponential/mapping (which exists today and should stay somewhere). I think it's OK if that package moves.

Indeed, this needs to be migrated to the new SDK. Please create an issue and add it to the beta milestone so we can track the work.

MrAlias avatar Sep 14 '22 18:09 MrAlias