Skip to content

docs: document add/remove SKU admin UI action#3263

Open
wminckler wants to merge 1 commit into
NVIDIA:mainfrom
wminckler:docs/sku-webui-admin-ui
Open

docs: document add/remove SKU admin UI action#3263
wminckler wants to merge 1 commit into
NVIDIA:mainfrom
wminckler:docs/sku-webui-admin-ui

Conversation

@wminckler

Copy link
Copy Markdown
Contributor

Documents the machine-detail-page SKU assign/remove action that was added to the admin WebUI in 97b58d8 (#2295).

  • docs/operations/debug_webui.md — the Machines row of the "Available Views" table now notes the assign/remove SKU write action on hosts, matching how other write-capable views are flagged.
  • docs/provisioning/sku-validation.md — adds a note that admins can manually assign/remove a host's SKU via either the nico-admin-cli sku assign/unassign commands or the Debug WebUI machine detail page.

Related issues

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Breaking Changes

  • This PR contains breaking changes

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

Additional Notes

Documentation-only follow-up to the SKU admin UI feature (#2295).

🤖 Generated with Claude Code

Document the machine detail page SKU assign/remove action (added in
97b58d8) in the Debug WebUI manual, and note the WebUI as an
alternative to the CLI for manual SKU assignment in the SKU validation
docs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@wminckler wminckler requested a review from polarweasel as a code owner July 8, 2026 18:18
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: dfd1c91a-288d-4a61-a186-0dac18835807

📥 Commits

Reviewing files that changed from the base of the PR and between f884199 and c66f771.

📒 Files selected for processing (2)
  • docs/operations/debug_webui.md
  • docs/provisioning/sku-validation.md

Summary by CodeRabbit

  • Documentation
    • Clarified that the Machines view in the Debug WebUI now supports write actions, including assigning and removing SKUs on hosts.
    • Added guidance showing that admins can manage SKUs on individual hosts from the command line or directly in the Debug WebUI machine details page.

Walkthrough

Documentation updates describing that admins can manually assign or remove a SKU on an individual host via nico-admin-cli sku assign/unassign or the NICo Debug WebUI machine detail page, with the Debug WebUI docs updated to reflect this write capability for the Machines view.

Changes

SKU Assignment Documentation

Layer / File(s) Summary
Document manual SKU assign/remove capability
docs/operations/debug_webui.md, docs/provisioning/sku-validation.md
The Machines view description is updated to note write actions (assign/remove SKU), and a new note in the SKU validation behavior section describes manual per-host SKU assignment/removal via CLI or Debug WebUI.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and accurately describes the docs update about adding SKU assign/remove admin UI actions.
Description check ✅ Passed The description matches the documented changes and clearly summarizes the Debug WebUI and SKU validation docs updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

🔍 Container Scan Summary

No Grype artifacts were found to aggregate.

@wminckler wminckler enabled auto-merge (squash) July 8, 2026 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant