chore: release SDK 1.8.0 / Experimental 0.34.0 with metrics GA

[dyladan]

This graduates the metrics API and SDK to GA. The exporters are still not yet moved. If there are other packages which should be moved comment below with your reasoning. If there are PRs or issues which should block this release, comment below and I will add them to this PR description.

Blocking issues / PRs:

• fix(resources): strict OTEL_RESOURCE_ATTRIBUTES decoding #3341
• Revert #3134 - remove anchored clock #3359
• feat(api): merge api-metrics into api #3374

---

SDK 1.8.0

• @opentelemetry/sdk-metrics has been promoted to stable

🚀 (Enhancement)

• feat(sdk-trace): re-export sdk-trace-base in sdk-trace-node and web #3319 @legendecas
• feat: enable tree shaking #3329 @pkanal

🐛 (Bug Fix)

• fix(sdk-trace): enforce consistent span durations
  #3327 @dyladan
• fix(resources): fix EnvDetector throwing errors when attribute values contain spaces
  #3295
• fix(trace): fix an issue which caused negative span durations in web based spans
  #3359 @dyladan
• fix(resources): strict OTEL_RESOURCE_ATTRIBUTES baggage octet decoding
  #3341 @legendecas

🏠 (Internal)

• ci: run browser tests without circle #3328 @dyladan

Experimental 0.34.0

• @opentelemetry/sdk-metrics moved to packages/sdk-metrics
• @opentelemetry/api-metrics deprecated and merged into api

🚀 (Enhancement)

• feat(metrics-sdk): Add tracing suppresing for Metrics Export #3332 @hectorhdzg
• feat(instrumentation): implement require-in-the-middle singleton #3161 @mhassan1
• feat(sdk-node): configure trace exporter with environment variables #3143 @svetlanabrennan
• feat: enable tree shaking #3329 @pkanal
• feat(prometheus): serialize resource as target_info gauge #3300 @pichlermarc
• feat(detectors): add browser detector module #3292 @Abinet18
• deps: remove unused proto-loader dependencies and update grpc-js and proto-loader versions #3337 @seemk
• feat(metrics-exporters): configure temporality via environment variable #3305 @pichlermarc

🐛 (Bug Fix)

• fix(node-sdk): move @opentelemetry/semantic-conventions to dependencies #3283 @mhassan1
• fix(exporters): do not append trailing '/' when URL contains path #3274 @pichlermarc
• fix(instrumentation): debug log on no modules defined for instrumentation #3308 @legendecas

📚 (Refine Doc)

• docs(metrics-exporters): fix wrong exporter const name in example #3270 @pichlermarc

🏠 (Internal)

• ci(instrumentation-http): remove got devDependency
  #3347 @dyladan
• deps(instrumentation-http): move sdk-metrics to dev dependencies #3380 @pichlermarc

[codecov]

Codecov Report

Merging #3340 (e3054d2) into main (baf0fee) will decrease coverage by 0.51%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3340      +/-   ##
==========================================
- Coverage   93.77%   93.25%   -0.52%     
==========================================
  Files          45      247     +202     
  Lines         643     7344    +6701     
  Branches      102     1511    +1409     
==========================================
+ Hits          603     6849    +6246     
- Misses         40      495     +455     

Impacted Files	Coverage Δ
packages/sdk-metrics/src/InstrumentDescriptor.ts	100.00% <ø> (ø)
packages/sdk-metrics/src/Instruments.ts	94.59% <ø> (ø)
packages/sdk-metrics/src/Meter.ts	100.00% <ø> (ø)
packages/sdk-metrics/src/MeterProvider.ts	100.00% <ø> (ø)
packages/sdk-metrics/src/ObservableResult.ts	95.83% <ø> (ø)
packages/sdk-metrics/src/aggregator/Drop.ts	100.00% <ø> (ø)
packages/sdk-metrics/src/aggregator/Histogram.ts	92.00% <ø> (ø)
packages/sdk-metrics/src/aggregator/LastValue.ts	100.00% <ø> (ø)
packages/sdk-metrics/src/aggregator/Sum.ts	100.00% <ø> (ø)
packages/sdk-metrics/src/aggregator/index.ts	100.00% <ø> (ø)
... and 239 more

[dyladan]

/easycla

[weyert]

Awesome, super excited about Metrics going v1 :))

[dyladan]

Converting to draft as it seems my approach must have had some fundamental flaw somewhere and I want to have confidence that the version change is good before making too many people look at this.

[anthony-telljohann]

I propose we postpone this release until #3341 is merged. I think it makes sense to wait until EnvDetector correctly handles decoding the OTEL_RESOURCE_ATTRIBUTES environment variable. If we were to release without it, I think it is fair to say EnvDetector is in a better state than before but it is still not working as intended.

[weyert]

Note: This is an experimental package under active development. New releases may include breaking changes.

Do we want to remove the references to this in some Metrics packages's README files?

[dyladan]

Note: This is an experimental package under active development. New releases may include breaking changes.

Do we want to remove the references to this in some Metrics packages's README files?

Yes we definitely do. I'm holding this right now while we resolve #3368

[pkanal]

Does this release include the tree shaking change? It's definitely in main but I don't see it in the release description in this PR.

[dyladan]

Yes I just haven't updated the description. I'll update it

[dyladan]

@open-telemetry/javascript-approvers this now contains an API minor version bump with metrics. Please review it so we can release metrics.

[dyladan]

All I am going to let this go for the weekend at least to let people review it and try to release monday if there are a few approvals. Enjoy your weekends.

[legendecas]

Worth noting that we've got comments on the GA interface in the TC review on metric API/SDK: open-telemetry/community#1204

[dyladan]

Converting back to draft until the API questions are resolved. I'll open another release PR which releases the other packages with important fixes.

[dyladan]

Going to release this in the morning

[dyladan]

I'm not sure what is causing the bootstrap to fail but we obviously can't merge like this