Bump @opentelemetry/core and applicationinsights - #2660
Conversation
|
@copilot, review and fix the whole PR. this PR is hvaing build issues. |
Addressed the build break in commit |
|
@copilot please update Changelog file as per changes. |
Updated in |
|
@jainakanksha-msft what else is need to get this merged in, please? |
| } | ||
|
|
||
| private static removeRoleInstance ( envelope: Contracts.EnvelopeTelemetry) : boolean { | ||
| private static removeRoleInstance(envelope: any): boolean { |
|
@md84419, thanks for checking in. |
|
@dependabot recreate |
7c64a44 to
f6d1d40
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 2 changed files in this pull request and generated no new comments.
Suppressed comments (1)
package.json:24
- If this repo commits a lockfile (e.g., package-lock.json / yarn.lock / pnpm-lock.yaml), it should be regenerated and included in the same PR when changing dependencies to keep installs reproducible and CI consistent.
"applicationinsights": "^3.15.1",
|
@dependabot recreate |
Bumps [@opentelemetry/core](https://github.com/open-telemetry/opentelemetry-js) to 2.10.0 and updates ancestor dependency [applicationinsights](https://github.com/microsoft/ApplicationInsights-node.js). These dependencies need to be updated together. Updates `@opentelemetry/core` from 1.30.1 to 2.10.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-js/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-js/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-js@v1.30.1...v2.10.0) Updates `applicationinsights` from 2.9.8 to 3.15.1 - [Release notes](https://github.com/microsoft/ApplicationInsights-node.js/releases) - [Changelog](https://github.com/microsoft/ApplicationInsights-node.js/blob/main/CHANGELOG.md) - [Commits](https://github.com/microsoft/ApplicationInsights-node.js/commits/3.15.1) --- updated-dependencies: - dependency-name: "@opentelemetry/core" dependency-version: 2.8.0 dependency-type: indirect - dependency-name: applicationinsights dependency-version: 3.15.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
f6d1d40 to
f317ad2
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 2 changed files in this pull request and generated no new comments.
Suppressed comments (1)
package.json:24
- This is a major-version upgrade (v2 → v3) of the telemetry SDK, which commonly includes breaking changes (e.g., initialization/config options, auto-collection defaults, or context propagation). Please confirm the app’s AI setup is compatible with v3 and that expected telemetry (requests/dependencies/exceptions) still flows correctly in your runtime environment; if there are required config/code changes, capture them in this PR (or link the migration notes) to avoid silently losing observability after deployment.
"applicationinsights": "^3.15.1",
|
Fixed in the latest commit. Here's a summary of what was done:
|
|
@copilot there is still one failing check: Azurite PR check / Azurite_Windows_Docker (pull_request) Failing after 1m.Run npm run docker:build-windows
skip failed to connect to the docker API at npipe:////./pipe/docker_engine; check if the path is correct and if the daemon is running: open //./pipe/docker_engine: The system cannot find the file specified.
|
Bumps @opentelemetry/core to 2.10.0 and updates ancestor dependency applicationinsights. These dependencies need to be updated together.
Updates
@opentelemetry/corefrom 1.30.1 to 2.10.0Release notes
Sourced from @opentelemetry/core's releases.
... (truncated)
Changelog
Sourced from @opentelemetry/core's changelog.
... (truncated)
Commits
76fa6b5chore: prepare next release (#6942)3787896chore(deps): update dependency webpack-cli to v7.2.1 (#6934)be5f757fix(deps): update dependency body-parser to v2.3.0 [security] (#6941)f6d8fbechore(deps): lock file maintenance (#6559)9612732chore: remove examples/dice from workspaces (#6937)7107906chore: start using min-release-age in .npmrc, disable minimumReleaseAge for r...a7e5d11chore(deps): update dependency webpack to v5.108.4 (#6933)af7a82dchore(deps): update dependency msw to v2.15.0 (#6831)b9f57c6chore(deps): update dependency@types/webpack-envto v1.18.8 (#6877)cbb4abfchore(deps): update ubuntu docker tag to v26 (#6635)Maintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for
@opentelemetry/coresince your current version.Updates
applicationinsightsfrom 2.9.8 to 3.15.1Release notes
Sourced from applicationinsights's releases.
... (truncated)
Changelog
Sourced from applicationinsights's changelog.
... (truncated)
Commits