Skip to content

YARN-11918. Fix YARN UI v2 build in Ubuntu 24.04 ARM#8218

Merged
aajisaka merged 2 commits intoapache:trunkfrom
aajisaka:fix-yarn-ui-build-only
Feb 3, 2026
Merged

YARN-11918. Fix YARN UI v2 build in Ubuntu 24.04 ARM#8218
aajisaka merged 2 commits intoapache:trunkfrom
aajisaka:fix-yarn-ui-build-only

Conversation

@aajisaka
Copy link
Member

@aajisaka aajisaka commented Jan 30, 2026

Description of PR

YARN-11918 Fix YARN UI v2 build in Ubuntu 24.04 ARM

  • Use sass instead of node-sass to support Python 3.12 to fix open(build_file_path, 'rU').read() error
  • Upgrade ember-cli dependencies to eliminate graceful-fs 3.0.5 to fix ReferenceError: primordials is not defined error

How was this patch tested?

Ran the following commands in M4 MBP

./start-build-env.sh ubuntu_24
mvn clean install -DskipTests -Pyarn-ui

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
  • Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
  • If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

AI Tooling

If an AI tool was used:

@hadoop-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 15m 8s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 0s codespell was not available.
+0 🆗 detsecrets 0m 0s detect-secrets was not available.
+0 🆗 jshint 0m 0s jshint was not available.
+0 🆗 jsonlint 0m 0s jsonlint was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ trunk Compile Tests _
+1 💚 mvninstall 41m 12s trunk passed
+1 💚 shadedclient 69m 19s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 💚 mvninstall 1m 53s the patch passed
+1 💚 blanks 0m 1s The patch has no blanks issues.
+1 💚 shadedclient 29m 6s patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 💚 asflicense 0m 50s The patch does not generate ASF License warnings.
119m 16s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8218/1/artifact/out/Dockerfile
GITHUB PR #8218
Optional Tests dupname asflicense shadedclient codespell detsecrets jshint jsonlint
uname Linux c7b4f9384054 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 37af168
Max. process+thread count 639 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8218/1/console
versions git=2.25.1 maven=3.9.11
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

Copy link
Contributor

@cnauroth cnauroth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1. Thank you for this @aajisaka ! I just noticed this while attempting dry-run release builds on ARM. (For some reason, it only repros on ARM for me.)

I don't actively use hadoop-yarn-ui, and I'm not too familiar with the npm ecosystem. I wonder if anyone else would like to review. @iwasakims @GauthamBanasandra , I see you have done work here. What do you think?

@cnauroth
Copy link
Contributor

+1. Thank you for this @aajisaka ! I just noticed this while attempting dry-run release builds on ARM. (For some reason, it only repros on ARM for me.)

I don't actively use hadoop-yarn-ui, and I'm not too familiar with the npm ecosystem. I wonder if anyone else would like to review. @iwasakims @GauthamBanasandra , I see you have done work here. What do you think?

I forgot to mention that I applied this patch locally and it fixed the build for me, which is I gave +1.

@aajisaka aajisaka force-pushed the fix-yarn-ui-build-only branch from 37af168 to 224e59d Compare February 2, 2026 07:43
@aajisaka
Copy link
Member Author

aajisaka commented Feb 2, 2026

Discussed with @LamiumAmplexicaule offline, we can enable minifyCSS without processImport. ProcessImport is disabled by default in ember-cli >= 2.12.0 ember-cli/ember-cli#5874

Also he suggested to update LICENSE-binary. Thanks.

@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 14m 6s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 1s No case conflicting files found.
+0 🆗 codespell 0m 1s codespell was not available.
+0 🆗 detsecrets 0m 1s detect-secrets was not available.
+0 🆗 jshint 0m 1s jshint was not available.
+0 🆗 jsonlint 0m 1s jsonlint was not available.
+0 🆗 shelldocs 0m 1s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ trunk Compile Tests _
+0 🆗 mvndep 8m 7s Maven dependency ordering for branch
+1 💚 mvninstall 36m 40s trunk passed
+1 💚 shadedclient 27m 20s branch has no errors when building and testing our client artifacts.
-0 ⚠️ patch 27m 51s Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 28s Maven dependency ordering for patch
+1 💚 mvninstall 37m 8s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
+1 💚 shellcheck 0m 0s No new issues.
+1 💚 shadedclient 27m 7s patch has no errors when building and testing our client artifacts.
_ Other Tests _
-1 ❌ asflicense 1m 12s /results-asflicense.txt The patch generated 2 ASF License warnings.
156m 18s
Subsystem Report/Notes
Docker ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8218/2/artifact/out/Dockerfile
Optional Tests dupname asflicense shadedclient codespell detsecrets jshint jsonlint shellcheck shelldocs
uname Linux 9cf5a42bc0f5 5.15.0-164-generic #174-Ubuntu SMP Fri Nov 14 20:25:16 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / f79bca8
Max. process+thread count 627 (vs. ulimit of 5500)
modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui . U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8218/2/console
versions git=2.43.0 maven=3.9.11 shellcheck=0.9.0
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@aajisaka aajisaka merged commit 6392c6b into apache:trunk Feb 3, 2026
1 of 3 checks passed
@aajisaka aajisaka deleted the fix-yarn-ui-build-only branch February 3, 2026 01:52
@aajisaka
Copy link
Member Author

aajisaka commented Feb 3, 2026

Merged. Thank you @cnauroth @slfan1989

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.

4 participants