Skip to content

Conversation

@kirrg001
Copy link
Contributor

@kirrg001 kirrg001 commented Nov 13, 2025

refs https://jsw.ibm.com/browse/INSTA-64143
refs https://jsw.ibm.com/browse/INSTA-64277

  • https://nodejs.org/api/deprecations.html#dep0161-process_getactiverequests-and-process_getactivehandles

  • We previously used the internal methods to get active requests and active handles

  • These functions are officially deprecated and the official public fn getActiveResourcesInfo exposes the signals together.

  • Its too much effort to determine which signal belongs to which category (request or handle).

  • Decided to just return the full number of active resources

  • BE + client migration first

  • manually test if that breaks anything (backend / UI)

aryamohanan and others added 3 commits November 13, 2025 11:42
- No updates or releases in over 5 years
- Very few downloads (around 100+)
- No Instana customers using it
- Supports only Node.js up to v14, which we’ve already dropped in v4
- Already we deprecated an year ago the warning also added.

ref: https://jsw.ibm.com/browse/INSTA-63921
BREAKING CHANGE:
The following environment variables and configuration options, previously used to disable tracing, have been removed:
- INSTANA_DISABLED_TRACERS
- INSTANA_DISABLE_TRACING
- tracing.disabledTracers

Migration Recommendations:
Replace INSTANA_DISABLED_TRACERS and INSTANA_DISABLE_TRACING with INSTANA_TRACING_DISABLE.
Replace tracing.disabledTracers with tracing.disable.
…iveResources

refs https://jsw.ibm.com/browse/INSTA-64143

BREAKING CHANGE:
- https://nodejs.org/api/deprecations.html#dep0161-process_getactiverequests-and-process_getactivehandles
- We previously used the internal methods to get active requests and active handles
- These functions are officially deprecated and the official public fn getActiveResourcesInfo exposes the signals
  together.
- Its too much effort to determine which signal belongs to which category (request or handle).
- Decided to just return the full number of active resources
@kirrg001 kirrg001 added the v5 v5 release items label Nov 13, 2025
@kirrg001 kirrg001 mentioned this pull request Nov 13, 2025
13 tasks
@kirrg001 kirrg001 added merge later do not merge and removed v5 v5 release items labels Nov 14, 2025
@kirrg001
Copy link
Contributor Author

Migration path:

  • BE receives activeRequests + activeHandles OR activeResources and provides one single metric to the ui client
  • UI client dashboard metric only shows the activeResources
  • If both changes are live, we can merge this fix without a major release!

@kirrg001 kirrg001 changed the base branch from v5-release to main November 14, 2025 10:05
@kirrg001 kirrg001 changed the title fix(shared-metrics): merged activeHandles and activeRequests into activeResources Q1 2026: Merge activeHandles and activeRequests into activeResources Nov 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants