-
Notifications
You must be signed in to change notification settings - Fork 829
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
[SDK 2.0] merge changes from next
to main
#5284
Merged
pichlermarc
merged 64 commits into
open-telemetry:main
from
dynatrace-oss-contrib:next-main-2024-12-16
Dec 18, 2024
Merged
Changes from all commits
Commits
Show all changes
64 commits
Select commit
Hold shift + click to select a range
0f8aa79
Update `next` branch (#4294)
dyladan e0efd7a
Merge remote-tracking branch 'origin/main' into next-main-11-16
dyladan ba4de42
Execute binaries from root node modules (#4304)
dyladan 6031bb4
Merge branch 'next' into next-main-11-16
dyladan ab52a84
[next] deprecate 14,16 and support 20
dyladan 9d62673
Fix prom test for node 20
dyladan 7cf46c7
Name exports to fix ESM tests
dyladan d229b3a
Temporarily disable node 20
dyladan 343ca78
lint
dyladan bf30779
Do not export node-specific names
dyladan 4cbb635
Move node module file to root
dyladan 21da78f
Export node module file
dyladan 26fcb88
Merge pull request #4308 from dyladan/next-main-11-16
dyladan 5be08f4
Merge branch 'next' into deprecate-old-runtimes
dyladan dc10b81
Bump to 18 on other workflows
dyladan 3b8f0af
Merge branch 'deprecate-old-runtimes' of github.com:dyladan/opentelem…
dyladan 543f0b4
Benchmark a tested version
dyladan 35be022
Merge pull request #4295 from dyladan/deprecate-old-runtimes
dyladan 09bdeba
chore: require changelog entries for next branch (#4342)
dyladan ea8bfa2
`[next]` merge changes from `main` (#4363)
dyladan ba00147
chore: remove checks for unsupported versions (#4341)
dyladan fac6d14
[next] remove README and add warning (#4226)
dyladan 7f613e0
chore(otel-core): replace deprecated SpanAttributes (#4408)
JamieDanielson 9727686
chore(shim-opentracing): replace deprecated spanAttributes (#4430)
JamieDanielson 0aba75c
chore(otel-resources): replace deprecated spanAttributes (#4428)
JamieDanielson a9af247
feat(sdk-metrics)!: remove MeterProvider.addMetricReader() in favor o…
pichlermarc b7c5382
Merge branch 'main' into next-main-02-09
pichlermarc 19b433e
chore: fix conflicts
pichlermarc 83becc7
Merge pull request #4471 from dynatrace-oss-contrib/next-main-02-09
pichlermarc 12dec9b
Merge branch 'main' into next-main-2024-02-23
pichlermarc c379326
Merge pull request #4503 from dynatrace-oss-contrib/next-main-2024-02-23
pichlermarc 6804066
[next] merge changes from main
pichlermarc a30989f
Merge pull request #4531 from dynatrace-oss-contrib/next-main-2024-03-07
pichlermarc 55f15e9
feat(sdk-metrics)!: replace attributeKeys with an option to add custo…
pichlermarc 154155f
Merge branch 'main' into next-main-2024-07-30
pichlermarc eee78fa
chore: clean up merge
pichlermarc bb9caaf
Merge pull request #4887 from dynatrace-oss-contrib/next-main-2024-07-30
pichlermarc 00683ff
Merge branch 'main' into next-main-2024-08-21
pichlermarc f7dccbf
Merge pull request #4934 from dynatrace-oss-contrib/next-main-2024-08-21
pichlermarc f4d0dcf
Merge branch 'main' into next-main-2024-09-23
pichlermarc 32564ad
[next] merge changes from main
pichlermarc 1804925
Merge pull request #5010 from dynatrace-oss-contrib/next-main-2024-09-23
pichlermarc e15d5b3
feat(sdk-trace-base): replace SpanAttributes with Attributes (#5009)
david-luna 91c67ba
refactor(resources)!: replace ResourceAttributes with Attributes (#5016)
david-luna 98143af
fix(sdk-trace): make activeSpanProcessor private (#4965)
legalimpurity 8fed1b1
feat(sdk-metrics)!: drop View and Aggregation for options (#4931)
pichlermarc b057c93
[next] merge changes from main
pichlermarc cc8c54a
Merge pull request #5052 from dynatrace-oss-contrib/next-main-2024-10-08
pichlermarc dcc1a87
docs: [Browser] Define the support browser runtimes (#5059)
MSNev 508946c
refactor(sdk-trace-base)!: remove Span class from exports (#5048)
david-luna 4a394cc
Merge branch 'main' into next-main-
pichlermarc a833d9c
Merge pull request #5144 from dynatrace-oss-contrib/next-main-
pichlermarc b7343ef
feat(sdk-trace-base)!: remove `addSpanProcessor` API (#5152)
david-luna f3a6310
refactor(sdk-trace-base): remove `_registeredSpanProcessors` from Bas…
david-luna c28abac
refactor(sdk-trace-base): make resource private and remove getActiveS…
david-luna 810b618
refactor(sdk-trace-base): rename `activeSpanProcessor` private proper…
david-luna a8b4cd2
Merge branch 'main' into next-main-2024-12-04
pichlermarc 47212bb
fix: remove accidentally re-added empty files
pichlermarc eb81e28
Merge pull request #5236 from dynatrace-oss-contrib/next-main-2024-12-04
pichlermarc 6d7a73a
refactor(sdk-metrics) Swap workaround types for @otel/api types (#5254)
chancancode a8befa6
Merge branch 'main' into next-main-2024-12-16
pichlermarc 4179ddb
Merge branch 'main' into next-main-2024-12-16
pichlermarc b8d7966
Merge pull request #5275 from dynatrace-oss-contrib/next-main-2024-12-16
pichlermarc 2610521
docs: merge changelog and readme
pichlermarc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,6 +11,20 @@ For semantic convention package changes, see the [semconv CHANGELOG](packages/se | |
|
||
### :boom: Breaking Change | ||
|
||
* feat(sdk-metrics)!: bump minimum version of `@opentelemetry/api` peer dependency to 1.9.0 [#5254](https://github.com/open-telemetry/opentelemetry-js/pull/5254) @chancancode | ||
* chore(shim-opentracing): replace deprecated SpanAttributes [#4430](https://github.com/open-telemetry/opentelemetry-js/pull/4430) @JamieDanielson | ||
* chore(otel-core): replace deprecated SpanAttributes [#4408](https://github.com/open-telemetry/opentelemetry-js/pull/4408) @JamieDanielson | ||
* feat(sdk-metrics)!: remove MeterProvider.addMetricReader() in favor of constructor option [#4419](https://github.com/open-telemetry/opentelemetry-js/pull/4419) @pichlermarc | ||
* chore(otel-resources): replace deprecated SpanAttributes [#4428](https://github.com/open-telemetry/opentelemetry-js/pull/4428) @JamieDanielson | ||
* feat(sdk-metrics)!: remove MeterProvider.addMetricReader() in favor of constructor option [#4419](https://github.com/open-telemetry/opentelemetry-js/pull/4419) @pichlermarc | ||
* feat(sdk-metrics)!: replace attributeKeys with custom processors option [#4532](https://github.com/open-telemetry/opentelemetry-js/pull/4532) @pichlermarc | ||
* refactor(sdk-trace-base)!: replace `SpanAttributes` with `Attributes` [#5009](https://github.com/open-telemetry/opentelemetry-js/pull/5009) @david-luna | ||
* refactor(resources)!: replace `ResourceAttributes` with `Attributes` [#5016](https://github.com/open-telemetry/opentelemetry-js/pull/5016) @david-luna | ||
* feat(sdk-metrics)!: drop `View` and `Aggregation` in favor of `ViewOptions` and `AggregationOption` [#4931](https://github.com/open-telemetry/opentelemetry-js/pull/4931) @pichlermarc | ||
* refactor(sdk-trace-base)!: remove `new Span` constructor in favor of `Tracer.startSpan` API [#5048](https://github.com/open-telemetry/opentelemetry-js/pull/5048) @david-luna | ||
* refactor(sdk-trace-base)!: remove `BasicTracerProvider.addSpanProcessor` API in favor of constructor options. [#5134](https://github.com/open-telemetry/opentelemetry-js/pull/5134) @david-luna | ||
* refactor(sdk-trace-base)!: make `resource` property private in `BasicTracerProvider` and remove `getActiveSpanProcessor` API. [#5192](https://github.com/open-telemetry/opentelemetry-js/pull/5192) @david-luna | ||
|
||
Comment on lines
+14
to
+27
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. copied from |
||
### :rocket: (Enhancement) | ||
|
||
### :bug: (Bug Fix) | ||
|
@@ -19,6 +33,11 @@ For semantic convention package changes, see the [semconv CHANGELOG](packages/se | |
|
||
### :house: (Internal) | ||
|
||
* refactor(sdk-metrics): remove `Gauge` and `MetricAdvice` workaround types in favor of the upstream `@opentelemetry/api` types [#5254](https://github.com/open-telemetry/opentelemetry-js/pull/5254) @chancancode | ||
* chore: remove checks for unsupported node versions [#4341](https://github.com/open-telemetry/opentelemetry-js/pull/4341) @dyladan | ||
* refactor(sdk-trace-base): remove `BasicTracerProvider._registeredSpanProcessors` private property. [#5134](https://github.com/open-telemetry/opentelemetry-js/pull/5134) @david-luna | ||
* refactor(sdk-trace-base): rename `BasicTracerProvider.activeSpanProcessor` private property. [#5211](https://github.com/open-telemetry/opentelemetry-js/pull/5211) @david-luna | ||
Comment on lines
+36
to
+39
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. copied from |
||
|
||
## 1.30.0 | ||
|
||
### :rocket: (Enhancement) | ||
|
@@ -69,6 +88,8 @@ For semantic convention package changes, see the [semconv CHANGELOG](packages/se | |
* `@opentelemetry/propagator-aws-xray` is now located in [open-telemetry/opentelemetry-js-contrib](https://github.com/open-telemetry/opentelemetry-js-contrib) | ||
* `@opentelemetry/propagator-aws-xray-lambda` is now located in [open-telemetry/opentelemetry-js-contrib](https://github.com/open-telemetry/opentelemetry-js-contrib) | ||
|
||
* docs: [Browser] Define the supported browser runtimes [Issue #4168](https://github.com/open-telemetry/opentelemetry-js/issues/4168) PR:[#5059](https://github.com/open-telemetry/opentelemetry-js/pull/5059) @MSNev | ||
|
||
### :house: (Internal) | ||
|
||
* deps: set `@opentelemetry/api` dependency min version to 1.3.0 in `examples`, `experimental/packages`, `integration-tests` and `selenium-tests` | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,6 +20,6 @@ | |
"author": "OpenTelemetry Authors", | ||
"license": "Apache-2.0", | ||
"engines": { | ||
"node": ">=14" | ||
"node": ">=18" | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,6 +20,6 @@ | |
"author": "OpenTelemetry Authors", | ||
"license": "Apache-2.0", | ||
"engines": { | ||
"node": ">=14" | ||
"node": ">=18" | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eventually we'll be able to drop this change, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes - I'll follow up with a PR to clean this up :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we likely could drop it now. The
next
branch will go awayThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pichlermarc when you get around to this, PRs to the v1.x maintenance branch no longer triggers this workflow. As the two branches diverge over time, there would probably be some PRs that needs to target that branch once in a while. I would fix it, but I can never remember whether you are supposed to change the workflow file on main or the one on the branch :)