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

feat(deps): update deps matching '@opentelemetry/*' #2582

Merged

Conversation

pichlermarc
Copy link
Member

@pichlermarc pichlermarc commented Dec 4, 2024

Description

This PR updates all OpenTelemetry dependencies from the core repo to the latest versions.
Partially created using workflow from #2536 (it failed on PR creation, so it's me not @opentelemetrybot opening the PR 😞 )

Changes

0.55.0 -> 0.56.0 @opentelemetry/api-logs (range-bump)
0.55.0 -> 0.56.0 @opentelemetry/instrumentation (range-bump)
0.55.0 -> 0.56.0 @opentelemetry/instrumentation-fetch (range-bump)
0.55.0 -> 0.56.0 @opentelemetry/instrumentation-grpc (range-bump)
0.55.0 -> 0.56.0 @opentelemetry/instrumentation-http (range-bump)
0.55.0 -> 0.56.0 @opentelemetry/instrumentation-xml-http-request (range-bump)
0.55.0 -> 0.56.0 @opentelemetry/otlp-transformer (range-bump)
0.55.0 -> 0.56.0 @opentelemetry/sdk-logs (range-bump)
0.55.0 -> 0.56.0 @opentelemetry/sdk-node (range-bump)
1.27.0 -> 1.28.0 @opentelemetry/semantic-conventions
1.28.0 -> 1.29.0 @opentelemetry/context-async-hooks
1.28.0 -> 1.29.0 @opentelemetry/context-zone-peer-dep
1.28.0 -> 1.29.0 @opentelemetry/core
1.28.0 -> 1.29.0 @opentelemetry/exporter-jaeger
1.28.0 -> 1.29.0 @opentelemetry/propagator-b3
1.28.0 -> 1.29.0 @opentelemetry/propagator-jaeger
1.28.0 -> 1.29.0 @opentelemetry/resources
1.28.0 -> 1.29.0 @opentelemetry/sdk-metrics
1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-base
1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-node
1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-web

Copy link

codecov bot commented Dec 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.75%. Comparing base (f611353) to head (8c37156).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2582   +/-   ##
=======================================
  Coverage   90.75%   90.75%           
=======================================
  Files         169      169           
  Lines        8026     8026           
  Branches     1635     1635           
=======================================
  Hits         7284     7284           
  Misses        742      742           

@trentm
Copy link
Contributor

trentm commented Dec 4, 2024

This PR updates all OpenTelemetry dependencies from the core repo to the latest versions.

"except the @opentelemetry/semantic-conventions dependency"
... because update-otel-deps.js cannot update that dep currently because it is currently pinned in instrumentation-pg's package.json.

@trentm
Copy link
Contributor

trentm commented Dec 4, 2024

Note to self: I guess this summary of updates from the failed workflow (https://github.com/open-telemetry/opentelemetry-js-contrib/actions/runs/12164311207/job/33926373050) is relevant:

Summary of changes (possible commit message):
--
chore(deps): update deps matching "@opentelemetry/*"

    0.55.0 -> 0.56.0 @opentelemetry/api-logs (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-fetch (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-grpc (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-http (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-xml-http-request (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/otlp-transformer (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/sdk-logs (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/sdk-node (range-bump)
    1.27.0 -> 1.28.0 @opentelemetry/semantic-conventions
    1.28.0 -> 1.29.0 @opentelemetry/context-async-hooks
    1.28.0 -> 1.29.0 @opentelemetry/context-zone-peer-dep
    1.28.0 -> 1.29.0 @opentelemetry/core
    1.28.0 -> 1.29.0 @opentelemetry/exporter-jaeger
    1.28.0 -> 1.29.0 @opentelemetry/propagator-b3
    1.28.0 -> 1.29.0 @opentelemetry/propagator-jaeger
    1.28.0 -> 1.29.0 @opentelemetry/resources
    1.28.0 -> 1.29.0 @opentelemetry/sdk-metrics
    1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-base
    1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-node
    1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-web
--

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.

LGTM, assuming the tests pass (still one TAV test running).

@pichlermarc pichlermarc marked this pull request as ready for review December 4, 2024 18:16
@pichlermarc pichlermarc requested a review from a team as a code owner December 4, 2024 18:16
@pichlermarc
Copy link
Member Author

Note to self: I guess this summary of updates from the failed workflow (https://github.com/open-telemetry/opentelemetry-js-contrib/actions/runs/12164311207/job/33926373050) is relevant:

Summary of changes (possible commit message):
--
chore(deps): update deps matching "@opentelemetry/*"

    0.55.0 -> 0.56.0 @opentelemetry/api-logs (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-fetch (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-grpc (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-http (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/instrumentation-xml-http-request (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/otlp-transformer (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/sdk-logs (range-bump)
    0.55.0 -> 0.56.0 @opentelemetry/sdk-node (range-bump)
    1.27.0 -> 1.28.0 @opentelemetry/semantic-conventions
    1.28.0 -> 1.29.0 @opentelemetry/context-async-hooks
    1.28.0 -> 1.29.0 @opentelemetry/context-zone-peer-dep
    1.28.0 -> 1.29.0 @opentelemetry/core
    1.28.0 -> 1.29.0 @opentelemetry/exporter-jaeger
    1.28.0 -> 1.29.0 @opentelemetry/propagator-b3
    1.28.0 -> 1.29.0 @opentelemetry/propagator-jaeger
    1.28.0 -> 1.29.0 @opentelemetry/resources
    1.28.0 -> 1.29.0 @opentelemetry/sdk-metrics
    1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-base
    1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-node
    1.28.0 -> 1.29.0 @opentelemetry/sdk-trace-web
--

I'll put it in the description. Thanks! 🙂

@pichlermarc
Copy link
Member Author

I also opened #2583 to fix the workflow so that we can actually use it all the way to PR creation next time :)

@pichlermarc
Copy link
Member Author

Hmm, TAV for tedious on Node.js v22 failed - maybe flaky? I'll re-run to see if it passes on the second try.

@pichlermarc
Copy link
Member Author

all green now :)

@pichlermarc pichlermarc enabled auto-merge (squash) December 4, 2024 21:33
@pichlermarc pichlermarc disabled auto-merge December 4, 2024 21:33
@pichlermarc
Copy link
Member Author

pichlermarc commented Dec 4, 2024

I just realized that I cannot rebase this because I opened it from @opentelemetrybot's branch - so I'm running the workflow again.

Edit: link https://github.com/open-telemetry/opentelemetry-js-contrib/actions/runs/12168721340/job/33940083076

@pichlermarc pichlermarc mentioned this pull request Dec 4, 2024
@pichlermarc pichlermarc merged commit 5df02cb into open-telemetry:main Dec 4, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg:auto-instrumentations-node pkg:auto-instrumentations-web pkg:instrumentation-amqplib pkg:instrumentation-aws-lambda pkg:instrumentation-aws-sdk pkg:instrumentation-bunyan pkg:instrumentation-cassandra-driver pkg:instrumentation-connect pkg:instrumentation-cucumber pkg:instrumentation-dataloader pkg:instrumentation-dns pkg:instrumentation-document-load pkg:instrumentation-express pkg:instrumentation-fastify pkg:instrumentation-fs pkg:instrumentation-generic-pool pkg:instrumentation-graphql pkg:instrumentation-hapi pkg:instrumentation-ioredis pkg:instrumentation-knex pkg:instrumentation-koa pkg:instrumentation-long-task pkg:instrumentation-lru-memoizer pkg:instrumentation-memcached pkg:instrumentation-mongodb pkg:instrumentation-mongoose pkg:instrumentation-mysql pkg:instrumentation-mysql2 pkg:instrumentation-nestjs-core pkg:instrumentation-net pkg:instrumentation-pg pkg:instrumentation-pino pkg:instrumentation-redis pkg:instrumentation-redis-4 pkg:instrumentation-restify pkg:instrumentation-router pkg:instrumentation-runtime-node pkg:instrumentation-socket.io pkg:instrumentation-tedious pkg:instrumentation-undici pkg:instrumentation-user-interaction pkg:instrumentation-winston pkg:propagation-utils pkg:resource-detector-alibaba-cloud pkg:resource-detector-aws pkg:resource-detector-azure pkg:resource-detector-container pkg:resource-detector-gcp pkg:resource-detector-instana pkg:sampler-aws-xray pkg:test-utils pkg-status:unmaintained This package is unmaintained. Only bugfixes may be acceped until a new owner has been found.
Projects
None yet
Development

Successfully merging this pull request may close these issues.