[Synthetics] Upgrade Nodejs to v22.22.2#49815
Conversation
Upgrade NodeJS version to v22.22.2 on heartbeat
🤖 GitHub commentsJust comment with:
|
|
Pinging @elastic/obs-ds-hosted-services (Team:obs-ds-hosted-services) |
|
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
📝 WalkthroughWalkthroughThe pull request upgrades Node.js to version 22.22.2 across the heartbeat component. Changes include a new changelog fragment documenting the dependency upgrade and modifications to the Dockerfile template. The Wolfi-based setup stage updates nodejs-22 from 22.22.0-r0 to 22.22.2-r0 and npm from 11.4.2-r1 to 11.12.0-r1. The ubi9-minimal stage updates the NODE_VERSION environment variable from 22.22.0 to 22.22.2. These changes align the pinned versions used during Docker image builds. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@dev-tools/packaging/templates/docker/Dockerfile.tmpl`:
- Line 136: The Dockerfile template pins Node.js to 22.22.2
(nodejs-22=22.22.2-r0) but CI pipelines still pin 22.20.0 via
ASDF_NODEJS_VERSION and NODEJS_VERSION; update the CI variable values to match
the Dockerfile or alternatively downgrade the Dockerfile pin to 22.20.0 so all
environments align. Locate the apk add line with nodejs-22=22.22.2-r0 and the CI
variables ASDF_NODEJS_VERSION and NODEJS_VERSION in your pipeline configs and
make them consistent (same major/minor/patch), then run CI to confirm no
runtime/test divergence.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 64b52d4b-461d-4571-b5cd-42b565b9d4d2
📒 Files selected for processing (2)
changelog/fragments/1774969420-dependency-upgrade-nodejs-v22.22.2.yamldev-tools/packaging/templates/docker/Dockerfile.tmpl
|
@Mergifyio backport 8.19 9.2 9.3 |
✅ Backports have been createdDetails
|
Upgrade NodeJS version to v22.22.2 on heartbeat (cherry picked from commit 96df70e)
Upgrade NodeJS version to v22.22.2 on heartbeat (cherry picked from commit 96df70e)
Upgrade NodeJS version to v22.22.2 on heartbeat (cherry picked from commit 96df70e)
Proposed commit message
Upgrade NodeJS version to v22.22.2 on heartbeat
Checklist
stresstest.shscript to run them under stress conditions and race detector to verify their stability../changelog/fragmentsusing the changelog tool.Disruptive User Impact
Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs