-
Notifications
You must be signed in to change notification settings - Fork 3
fix(deps): update opentelemetry-js monorepo #392
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
base: main
Are you sure you want to change the base?
Conversation
|
View your CI Pipeline Execution ↗ for commit eab42c7
☁️ Nx Cloud last updated this comment at |
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project status has failed because the head coverage (18.75%) is below the target coverage (40.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #392 +/- ##
=======================================
Coverage 18.75% 18.75%
=======================================
Files 138 138
Lines 27368 27368
Branches 951 951
=======================================
Hits 5132 5132
Misses 22236 22236 🚀 New features to boost your workflow:
|
Deployed 87e2aa2 to https://ForgeRock.github.io/ping-javascript-sdk/pr-392/87e2aa24e33f10b98697324d6cf92d9df76333c0 branch gh-pages in ForgeRock/ping-javascript-sdk |
📦 Bundle Size Analysis📦 Bundle Size Analysis🆕 New Packages🆕 @forgerock/device-client - 9.2 KB (new) 13 packages analyzed • Baseline from latest Legend🆕 New package ℹ️ How bundle sizes are calculated
🔄 Updated automatically on each push to this PR |
6b13f3a
to
fa2b6a0
Compare
fa2b6a0
to
e54c889
Compare
e54c889
to
bc4bba9
Compare
82b9936
to
e5c4e23
Compare
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
e5c4e23
to
39677c9
Compare
9ceaf51
to
6402937
Compare
6402937
to
2c41e8e
Compare
2c41e8e
to
1873e07
Compare
1873e07
to
4f5a7e4
Compare
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.
Nx Cloud has identified a possible root cause for your failed CI:
The failure is classified as 'flaky_task' and no code changes are warranted.
The PR updates OpenTelemetry SDK packages (telemetry/observability libraries) in the mock API service. These packages are used for tracking metrics, traces, and logs, and have no impact on the authentication flow or browser navigation behavior being tested.
The failing test "PingOne tokens › login and get tokens" times out while waiting for a page navigation to complete after clicking a "Sign On" button. This is a classic flaky E2E test scenario with the following characteristics:
-
External Service Dependency: The test relies on PingOne authentication service responding within the timeout period, which can be unreliable in E2E environments.
-
Timing-Sensitive Operation: The 30-second timeout waiting for network idle is inherently prone to intermittent failures based on network conditions, service load, or infrastructure performance.
-
No Causal Relationship: OpenTelemetry SDK version updates (from 2.0.1 to 2.1.0 for trace packages, 0.202.0 to 0.205.0 for logs) do not modify authentication flows, HTTP request handling, or page navigation logic. These are passive observability libraries that instrument code for monitoring purposes.
-
Isolated to Test Environment: The failure occurs in an E2E test that exercises real authentication flows, which are susceptible to environmental variability that dependency updates cannot influence.
The OpenTelemetry dependency updates are legitimate library maintenance and the test failure is coincidental rather than causal. The test should be monitored for flakiness patterns or potentially have its timeout increased or retry logic added if failures persist.
A code change would likely not resolve this issue, so no action was taken.
⚙️ An Nx Cloud workspace admin can disable these reviews in workspace settings.
4f5a7e4
to
eab42c7
Compare
@forgerock/davinci-client
@forgerock/oidc-client
@forgerock/protect
@forgerock/sdk-types
@forgerock/sdk-utilities
@forgerock/iframe-manager
@forgerock/sdk-logger
@forgerock/sdk-oidc
@forgerock/sdk-request-middleware
@forgerock/storage
commit: |
This PR contains the following updates:
0.202.0
->0.206.0
2.0.1
->2.1.0
2.0.1
->2.1.0
2.0.1
->2.1.0
2.0.1
->2.1.0
Release Notes
open-telemetry/opentelemetry-js (@opentelemetry/sdk-logs)
v0.206.0
Compare Source
v0.205.0
Compare Source
v0.204.0
Compare Source
v0.203.0
Compare Source
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.