YARN-11902.Fix build failure caused by mktemp@2.0.2#8108
YARN-11902.Fix build failure caused by mktemp@2.0.2#8108slfan1989 merged 1 commit intoapache:trunkfrom
Conversation
|
💔 -1 overall
This message was automatically generated. |
ayushtkn
left a comment
There was a problem hiding this comment.
LGTM
Can u assign the jira ticket on your name or drop a comment on the jira with your id, so we can assign the ticket and attribute the contribution to you
Just adding a quick follow-up — since my ASF Jira account setup is still underway, I’m not yet able to self-assign or comment directly on the Jira issue. When convenient, could you please proceed with merging this change? I appreciate your assistance. |
Just a quick follow-up — my ASF Jira account setup is still in progress, so I’m unable to self-assign or comment directly on the Jira issue at the moment. When you get a chance, could you please go ahead and merge this change? Thanks for your help! @slfan1989 @ayushtkn |
|
@dilrajsourabh Thanks for the contribution! Merged into trunk. |
|
do we need this in 3.4.x? |
Yes. I think we need to add this in 3.4.x line as well. |
Description of PR
JIRA: YARN-11902. Fix Hadoop Yarn UI build failure caused by mktemp@2.0.2.
A build failure occurs in the Yarn UI module during dependency installation due to an incompatible version of mktemp@2.0.2, which enforces a Node.js engine requirement of 20 || 22 || 24.
The current Hadoop build environment uses Node.js 12.22.1, resulting in the following error:
error mktemp@2.0.2: The engine "node" is incompatible with this module. Expected version "20 || 22 || 24". Got "12.22.1" error Found incompatible module.Root Cause
The mktemp package was pulled in as a transitive dependency.
Version 2.x introduced a strict Node.js engine requirement (≥20), which is incompatible with the current Node 12 environment.
Fix
Pinned the mktemp dependency to a compatible version (0.4.0) using Yarn's resolutions block in package.json:
"resolutions": { "mktemp": "0.4.0" }This resolves the build failure without requiring a Node.js upgrade.
How was this patch tested?
ci
For code changes:
LICENSE,LICENSE-binary,NOTICE-binaryfiles?