Skip to content

Conversation

@brumi1024
Copy link
Member

@brumi1024 brumi1024 commented Nov 12, 2025

Description of PR

Expose the Capacity Scheduler UI, similar to UI2.

How was this patch tested?

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?

@brumi1024 brumi1024 changed the title YARN-11886: Introduce Capacity Scheduler UI. YARN-11888. Serve the Capacity Scheduler UI. Nov 12, 2025
@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 25s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 3s No case conflicting files found.
+0 🆗 codespell 0m 3s codespell was not available.
+0 🆗 detsecrets 0m 3s detect-secrets was not available.
+0 🆗 shelldocs 0m 3s Shelldocs was not available.
+0 🆗 xmllint 0m 0s xmllint was not available.
+0 🆗 markdownlint 0m 0s markdownlint was not available.
+0 🆗 jsonlint 0m 0s jsonlint was not available.
+0 🆗 jshint 0m 0s jshint was not available.
+1 💚 @author 0m 1s The patch does not contain any @author tags.
-1 ❌ test4tests 0m 0s The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
+0 🆗 mvndep 7m 18s Maven dependency ordering for branch
+1 💚 mvninstall 16m 15s trunk passed
+1 💚 compile 9m 5s trunk passed with JDK Ubuntu-21.0.7+6-Ubuntu-0ubuntu120.04
+1 💚 compile 9m 21s trunk passed with JDK Ubuntu-17.0.15+6-Ubuntu-0ubuntu120.04
+1 💚 checkstyle 1m 28s trunk passed
-1 ❌ mvnsite 5m 22s /branch-mvnsite-root.txt root in trunk failed.
+1 💚 javadoc 5m 9s trunk passed with JDK Ubuntu-21.0.7+6-Ubuntu-0ubuntu120.04
+1 💚 javadoc 4m 52s trunk passed with JDK Ubuntu-17.0.15+6-Ubuntu-0ubuntu120.04
+0 🆗 spotbugs 0m 15s branch/hadoop-assemblies no spotbugs output file (spotbugsXml.xml)
-1 ❌ spotbugs 7m 31s /branch-spotbugs-hadoop-yarn-project_hadoop-yarn-warnings.html hadoop-yarn-project/hadoop-yarn in trunk has 4206 extant spotbugs warnings.
-1 ❌ spotbugs 1m 24s /branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-api-warnings.html hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api in trunk has 87 extant spotbugs warnings.
-1 ❌ spotbugs 1m 13s /branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common-warnings.html hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common in trunk has 1100 extant spotbugs warnings.
-1 ❌ spotbugs 1m 25s /branch-spotbugs-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager-warnings.html hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager in trunk has 1299 extant spotbugs warnings.
-1 ❌ spotbugs 6m 53s /branch-spotbugs-hadoop-yarn-project-warnings.html hadoop-yarn-project in trunk has 4206 extant spotbugs warnings.
-1 ❌ spotbugs 19m 20s /branch-spotbugs-root-warnings.html root in trunk has 9242 extant spotbugs warnings.
+1 💚 shadedclient 33m 51s branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+0 🆗 mvndep 0m 20s Maven dependency ordering for patch
+1 💚 mvninstall 20m 30s the patch passed
+1 💚 compile 9m 46s the patch passed with JDK Ubuntu-21.0.7+6-Ubuntu-0ubuntu120.04
+1 💚 javac 9m 46s the patch passed
+1 💚 compile 9m 43s the patch passed with JDK Ubuntu-17.0.15+6-Ubuntu-0ubuntu120.04
+1 💚 javac 9m 43s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
-0 ⚠️ checkstyle 1m 42s /results-checkstyle-root.txt root: The patch generated 1 new + 242 unchanged - 0 fixed = 243 total (was 242)
-1 ❌ mvnsite 3m 57s /patch-mvnsite-root.txt root in the patch failed.
+1 💚 shellcheck 0m 0s No new issues.
-1 ❌ javadoc 5m 23s /results-javadoc-javadoc-root-jdkUbuntu-21.0.7+6-Ubuntu-0ubuntu120.04.txt root-jdkUbuntu-21.0.7+6-Ubuntu-0ubuntu120.04 with JDK Ubuntu-21.0.7+6-Ubuntu-0ubuntu120.04 generated 5 new + 46254 unchanged - 0 fixed = 46259 total (was 46254)
-1 ❌ javadoc 4m 41s /results-javadoc-javadoc-root-jdkUbuntu-17.0.15+6-Ubuntu-0ubuntu120.04.txt root-jdkUbuntu-17.0.15+6-Ubuntu-0ubuntu120.04 with JDK Ubuntu-17.0.15+6-Ubuntu-0ubuntu120.04 generated 5 new + 43037 unchanged - 0 fixed = 43042 total (was 43037)
+0 🆗 spotbugs 0m 12s hadoop-assemblies has no data from spotbugs
+0 🆗 spotbugs 0m 14s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui has no data from spotbugs
+1 💚 shadedclient 32m 39s patch has no errors when building and testing our client artifacts.
_ Other Tests _
-1 ❌ unit 222m 39s /patch-unit-root.txt root in the patch failed.
-1 ❌ asflicense 0m 43s /results-asflicense.txt The patch generated 4 ASF License warnings.
449m 29s
Reason Tests
Failed junit tests hadoop.yarn.server.nodemanager.containermanager.logaggregation.TestLogAggregationService
hadoop.security.ssl.TestDelegatingSSLSocketFactory
hadoop.hdfs.tools.TestDFSAdmin
Subsystem Report/Notes
Docker ClientAPI=1.52 ServerAPI=1.52 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8082/2/artifact/out/Dockerfile
GITHUB PR #8082
Optional Tests dupname asflicense codespell detsecrets shellcheck shelldocs compile javac javadoc mvninstall mvnsite unit shadedclient xmllint spotbugs checkstyle markdownlint jsonlint jshint
uname Linux 94384b6bf6cf 5.15.0-156-generic #166-Ubuntu SMP Sat Aug 9 00:02:46 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/bin/hadoop.sh
git revision trunk / 072724d
Default Java Ubuntu-17.0.15+6-Ubuntu-0ubuntu120.04
Multi-JDK versions /usr/lib/jvm/java-21-openjdk-amd64:Ubuntu-21.0.7+6-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-17-openjdk-amd64:Ubuntu-17.0.15+6-Ubuntu-0ubuntu120.04
Test Results https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8082/2/testReport/
Max. process+thread count 4120 (vs. ulimit of 5500)
modules C: hadoop-assemblies hadoop-yarn-project/hadoop-yarn hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-capacity-scheduler-ui hadoop-yarn-project . U: .
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8082/2/console
versions git=2.25.1 maven=3.9.11 spotbugs=4.9.7 shellcheck=0.7.0
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

@K0K0V0K
Copy link
Contributor

K0K0V0K commented Nov 14, 2025

Hi @brumi1024 !

Thanks for this new feature, seems very fascinating.
All seems good, i just have one NIT:
Maybe we should extend the LICENSE-binary with the React and Tailwind licence?

@brumi1024
Copy link
Member Author

brumi1024 commented Nov 21, 2025

Thanks @K0K0V0K! Good point, I've added the missing licenses to the original PR: #8076. Will rebase this one once that get's a green build.

YETUS='yetus'
// Branch or tag name. Yetus release tags are 'rel/X.Y.Z'
YETUS_VERSION='rel/0.14.0'
YETUS_VERSION='a7d29a6a72750a0c5c39512f33945e773e69303e'
Copy link
Contributor

Choose a reason for hiding this comment

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

why version is committer id?

Copy link
Member Author

@brumi1024 brumi1024 Nov 22, 2025

Choose a reason for hiding this comment

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

It's a temporary workaround, the PR is to big, without this Yetus won't build it. It can be reverted after a successful build.

$class: 'GitSCM',
branches: [[name: "${env.YETUS_VERSION}"]],
userRemoteConfigs: [[ url: 'https://github.com/apache/yetus.git']]]
userRemoteConfigs: [[ url: 'https://github.com/ayushtkn/yetus.git']]]
Copy link
Contributor

Choose a reason for hiding this comment

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

Why use a personal repository?

Copy link
Member Author

Choose a reason for hiding this comment

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

Same as above.

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