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: prepare release API 1.9.0/Core 1.25.0/Experimental 0.52.0 #4677

Merged

Conversation

pichlermarc
Copy link
Member

@pichlermarc pichlermarc commented May 3, 2024

1.9.0

🚀 (Enhancement)

  • feat(metrics): added synchronous gauge #4528 @clintonb
  • feat(api): allow adding span links after span creation #4536 @seemk
    • This change is non-breaking for end-users, but breaking for Trace SDK implmentations in accordance with the specification as new features need to be implemented.
  • feat: support node 22 #4666 @dyladan

1.25.0

🚀 (Enhancement)

📚 (Refine Doc)

  • refactor(examples): use new exported string constants for semconv in examples/esm-http-ts #4758 @Zen-cronic
  • refactor(examples): use new exported string constants for semconv in examples/basic-tracer-node #4759 @Zen-cronic
  • refactor(examples): use new exported string constants for semconv in examples/http #4750 @Zen-cronic
  • refactor(examples): use new exported string constants for semconv in examples/grpc-js #4760 @Zen-cronic
  • refactor(examples): use new exported string constants for semconv in examples/otlp-exporter-node #4762 @Zen-cronic
  • refactor(examples): use new exported string constants for semconv in examples/opentracing-shim #4761 @Zen-cronic

0.52.0

💥 Breaking Change

  • feat(exporter--otlp-)!: move serialization for Node.js exporters to @opentelemetry/otlp-transformer #4542 @pichlermarc
    • Breaking changes:
      • (user-facing) convert() now returns an empty object and will be removed in a follow-up
      • (internal) OTLPExporterNodeBase now has additional constructor parameters that are required
      • (internal) OTLPExporterNodeBase now has an additional ResponseType type parameter
  • feat(exporter--otlp-)!: move serialization for Node.js exporters to @opentelemetry/otlp-transformer #4581 @pichlermarc
    • Breaking changes:
      • (user-facing) convert() has been removed from all exporters
      • (internal) OTLPExporterBrowserBase: RequestType has been replaced by a ResponseType type-argument
      • (internal) OTLPExporterNodeBase: ServiceRequest has been replaced by a ServiceResponse type-argument
      • (internal) the @opentelemetry/otlp-exporter-proto-base package has been removed, and will from now on be deprecated in npm
  • feat(instrumentation): remove default value for config in base instrumentation constructor #4695: @blumamir
  • fix(instrumentation)!: remove unused supportedVersions from Instrumentation interface #4694 @blumamir
  • feat(instrumentation)!: simplify registerInstrumentations() API
    • Breaking changes:
      • removes InstrumentationOptions type
      • occurrences of InstrumentationOptions are now replaced by (Instrumentation | Instrumentation[])[]
        • migrate usages of registerInstrumentations({instrumentations: fooInstrumentation}) to registerInstrumentations({instrumentations: [fooInstrumentation]})
        • passing Instrumentation classes to registerInstrumentations() is now not possible anymore.
  • feat(sdk-node)!: simplify type of instrumentations option
    • Breaking changes:
      • replaces InstrumentationOptions with (Instrumentation | Instrumentation[])[]

🚀 (Enhancement)

  • feat(instrumentation): apply unwrap before wrap in base class #4692
  • feat(instrumentation): add util to execute span customization hook in base class #4663 @blumamir
  • feat(instrumentation): generic config type in instrumentation base #4659 @blumamir
  • feat: support node 22 #4666 @dyladan
  • feat(propagator-aws-xray-lambda): add AWS Xray Lambda propagator 4554
  • refactor(instrumentation-xml-http-request): use exported strings for semantic attributes. #4681

🐛 (Bug Fix)

📚 (Refine Doc)

  • docs(instrumentation): better docs for supportedVersions option #4693 @blumamir
  • docs: align all supported versions to a common format #4696 @blumamir
  • refactor(examples): use new exported string constants for semconv in experimental/examples/opencensus-shim #4763 @Zen-cronic

@pichlermarc pichlermarc marked this pull request as ready for review May 3, 2024 16:48
@pichlermarc pichlermarc requested a review from a team as a code owner May 3, 2024 16:48
@pichlermarc pichlermarc changed the title chore: prepare release 1.25.0/0.52.0 chore: prepare release API 1.10.0/Core 1.25.0/Experimental 0.52.0 May 6, 2024
@dyladan
Copy link
Member

dyladan commented May 7, 2024

I reached out to Datadog about this through @Qard. As the PR currently stands it would break dd-trace which I would like to avoid. I think we should delay this for a short time in order to allow them to respond, but I don't want to delay it forever.

@Qard
Copy link

Qard commented May 7, 2024

I've forwarded it to the relevant person internally so hopefully we'll come up with some solution soon...

@hamin
Copy link

hamin commented May 10, 2024

Any chance of #4536 enhancement having a published release soon? I realize there's a breaking change with a dd lib but was hoping to have the enhancement in a release sooner than that's resolved?

Appreciate everyone's efforts here.

@hamin
Copy link

hamin commented May 14, 2024

Any chance of #4536 enhancement having a published release soon? I realize there's a breaking change with a dd lib but was hoping to have the enhancement in a release sooner than that's resolved?

Appreciate everyone's efforts here.

@Qard @dyladan Any chance the dd-trace blockers don't hold back API 1.9.0 enhancements? Is it possible to have a release for API without waiting on dd-trace resolution? Thank you in advance!

@dyladan
Copy link
Member

dyladan commented May 14, 2024

It's possible but it's not ideal. If we release in the current state we will definitely break dd-trace and it reflects poorly on the project. We're doing our best to work with them. Sorry it's delaying the release but please be patient and we'll have a release soon.

@dyladan dyladan changed the title chore: prepare release API 1.10.0/Core 1.25.0/Experimental 0.52.0 chore: prepare release API 1.9.0/Core 1.25.0/Experimental 0.52.0 May 15, 2024
@dyladan
Copy link
Member

dyladan commented May 15, 2024

We're going to move forward with this change next week. It looks to me like dd-trace hasn't done anything yet to address the issue, but we have been very clear for months. @pichlermarc please update this PR and we can get it merged when I get back.

Copy link

codecov bot commented May 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.81%. Comparing base (ecc88a3) to head (6554b25).
Report is 21 commits behind head on main.

Current head 6554b25 differs from pull request most recent head 8da8fd4

Please upload reports for the commit 8da8fd4 to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4677      +/-   ##
==========================================
- Coverage   91.04%   89.81%   -1.24%     
==========================================
  Files          89      147      +58     
  Lines        1954     3220    +1266     
  Branches      416      699     +283     
==========================================
+ Hits         1779     2892    +1113     
- Misses        175      328     +153     

see 98 files with indirect coverage changes

@pichlermarc
Copy link
Member Author

@open-telemetry/javascript-approvers this release PR is now unblocked:

@pichlermarc
Copy link
Member Author

There's an unreleased bug on main (missing dependency), fixed by #4732.
Other than that there's nothing blocking this release anymore.

@trentm
Copy link
Contributor

trentm commented May 31, 2024

  • feat(propagator-aws-xray-lambda): add AWS Xray Lambda propagator 4554

@pichlermarc Is there a need/reason to avoid publishing this propagator from the core repo? I suppose it is fine. It can be moved and published from contrib repo later.

Copy link
Contributor

@trentm trentm left a comment

Choose a reason for hiding this comment

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

Merge conflicts to sort out, and I asked a Q above about the new AWS Xray Lambda propagator package.

@pichlermarc
Copy link
Member Author

  • feat(propagator-aws-xray-lambda): add AWS Xray Lambda propagator 4554

@pichlermarc Is there a need/reason to avoid publishing this propagator from the core repo? I suppose it is fine. It can be moved and published from contrib repo later.

It's not spec compliant but I think it's fine for now. We can move it to contrib at any time and be spec compliant again.

@pichlermarc pichlermarc added this pull request to the merge queue Jun 5, 2024
Merged via the queue into open-telemetry:main with commit 26974c4 Jun 5, 2024
19 checks passed
@pichlermarc pichlermarc deleted the release/1.25.0-0.52.0 branch June 5, 2024 14:52
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.

None yet

5 participants