[HWORKS-2761] Compute Resources Usage view should only show Kueue-reachable nodes#576
Open
o-alex wants to merge 4 commits into
Open
[HWORKS-2761] Compute Resources Usage view should only show Kueue-reachable nodes#576o-alex wants to merge 4 commits into
o-alex wants to merge 4 commits into
Conversation
…chable nodes Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds end-user and admin documentation for the Hopsworks Compute Resources Usage view, clarifying how per-project node visibility is determined (Kueue reachability) and how to interpret/filter the UI.
Changes:
- Add a new user guide explaining the Compute Resources Usage card (totals, per-node breakdown, Queue/Labels filtering, access notice, and non-Kueue fallback).
- Add a new admin guide explaining the LocalQueue → ClusterQueue → ResourceFlavor mapping and required RBAC, plus troubleshooting.
- Update
mkdocs.ymlnavigation to include both new pages.
Reviewed changes
Copilot reviewed 5 out of 8 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| mkdocs.yml | Adds nav entries for the new user and admin documentation pages. |
| docs/user_guides/projects/scheduling/compute_resources.md | New end-user guide for reading/filtering the Compute Resources Usage view. |
| docs/setup_installation/admin/compute_resources.md | New admin guide describing how Kueue objects map to node visibility and required RBAC. |
| .claude/docs/README.md | Updates contributor command guidance (mike deploy/serve) and preview notes. |
| .claude/CLAUDE.md | Updates contributor command guidance (mike deploy/serve) and preview notes. |
Comments suppressed due to low confidence (1)
docs/setup_installation/admin/compute_resources.md:79
- These See also links use relative
.mdpaths. Prefer MkDocs autorefs/heading IDs (e.g.,[Kueue][kueue]) to avoid fragile links under mike versioning.
- [Compute Resources Usage](../../user_guides/projects/scheduling/compute_resources.md) — the end-user view this configuration drives.
- [Kueue](../../user_guides/projects/scheduling/kueue_details.md) — overview of the Kueue abstractions referenced above.
…chable nodes Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…chable nodes Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…chable nodes Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
| uv pip install "git+https://github.com/logicalclocks/hopsworks-api.git@main#subdirectory=python" # needed for Python API section | ||
| touch docs/javadoc; uv run mkdocs serve; rm docs/javadoc # preview with live reload | ||
| touch docs/javadoc; uv run mkdocs build -s; rm docs/javadoc # build in strict mode | ||
| uv run mike deploy <version> latest --update-alias # build a versioned bundle (use repo's current version, e.g. 4.4); first time only: `uv run mike set-default latest` |
| uv pip install "git+https://github.com/logicalclocks/hopsworks-api.git@main#subdirectory=python" # install Python API (needed for API docs section) | ||
| touch docs/javadoc; uv run mkdocs build -s; rm docs/javadoc # build (strict) | ||
| uv run mkdocs serve # preview with live reload | ||
| uv run mike deploy <version> latest --update-alias # build a versioned bundle to the gh-pages worktree (use repo's current version, e.g. 4.4); first time only: `uv run mike set-default latest` |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
user_guides/projects/scheduling/compute_resources.mdexplaining the Compute Resources Usage card: the totals, the per-node breakdown, the Queue: filter dropdown (any+ per-LocalQueue), the access-notice tooltip, and the fallback when Kueue is not in use.setup_installation/admin/compute_resources.mdcovering the LocalQueue → ClusterQueue → ResourceFlavor walk, the node-label / node-taint matching rules, the Kueue ClusterRole RBAC, and a troubleshooting table.Kubernetes Scheduling(user) andSetup and Administration > Administration(admin).Test plan
mkdocs build -sis clean (no new errors; pre-existing warnings aboutdatabricks/*.mdnot being in nav are unchanged).markdownlint-cli2clean on both new files.🤖 Generated with Claude Code