feat(installer): support DGX OS 7.6 on Station - #7419
Conversation
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughStation Express now recognizes qualifying no-OTA DGX OS 7.6.x DGX Station releases, updates PackageKit gating, expands classification tests, and revises DGX Station and managed-vLLM documentation. ChangesDGX Station 7.6.x support
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant StationExpress
participant ReleaseClassifier
participant PackageStateCheck
StationExpress->>ReleaseClassifier: inspect DGX OS release metadata
ReleaseClassifier-->>StationExpress: return supported-dgx-os or unsupported-dgx-os
StationExpress->>PackageStateCheck: validate package-manager state
PackageStateCheck-->>StationExpress: allow factory-preserving profile or reject active PackageKit state
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
PR Review Advisor — InformationalAdvisor assessment: Informational / medium confidence Model lanes
Nemotron output stays in workflow artifacts and does not change the assessment above. E2E guidanceAdvisory only. E2E / PR Gate selects and runs jobs independently. Recommended E2E: This automated review informs maintainers. Warnings and suggestions do not require a response. A maintainer decides whether to merge. |
|
Physical DGX Station evidence for review: the committed Station preparation helper at |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@scripts/prepare-dgx-station-host.sh`:
- Around line 261-264: Update the no-OTA branch around
dgx_station_release_schema_is_valid to reject any present DGX_OTA_* metadata,
not only DGX_OTA_DATE, before continuing to DGX_PRETTY_NAME and version parsing.
Preserve the existing return behavior for valid no-OTA metadata, and add a
regression fixture covering a partial OTA marker such as DGX_OTA_PRETTY_NAME
alone.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 8a30c54a-3cf8-4779-a0a6-4eae1c42028b
📒 Files selected for processing (11)
ci/platform-matrix.jsondocs/get-started/dgx-station-preparation.mdxdocs/get-started/prerequisites.mdxdocs/inference/choose-inference-provider.mdxdocs/reference/platform-support.mdxscripts/install.shscripts/prepare-dgx-station-host.shtest/install-express-prompt.test.tstest/install-station-dgx-os.test.tstest/install-station-package-state.test.tstest/station-doc-ownership.test.ts
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
|
PR #7419 is synchronized with Changes:
Verification:
Physical read-only check of the final preparation-helper bytes: The read-only check stopped at the expected existing-agent-workload guard because that Station runs OpenShell workloads. It did not stop containers or change host state. Evidence is preserved at Review disposition:
|
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/get-started/prerequisites.mdx`:
- Line 97: Reformat the DGX OS (Station) qualification text so each sentence
occupies its own MDX source line, moving the detailed prose outside the table if
needed while preserving the existing meaning and link.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: b8fa81e2-0c71-4475-abf3-0bcfb92ce2b1
📒 Files selected for processing (8)
ci/platform-matrix.jsondocs/get-started/dgx-station-preparation.mdxdocs/get-started/prerequisites.mdxdocs/inference/choose-inference-provider.mdxdocs/reference/platform-support.mdxscripts/install.shtest/install-express-prompt.test.tstest/station-doc-ownership.test.ts
🚧 Files skipped from review as they are similar to previous changes (7)
- docs/inference/choose-inference-provider.mdx
- ci/platform-matrix.json
- docs/get-started/dgx-station-preparation.mdx
- test/station-doc-ownership.test.ts
- scripts/install.sh
- docs/reference/platform-support.mdx
- test/install-express-prompt.test.ts
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
Signed-off-by: Senthil Ravichandran <senthilr@nvidia.com>
|
Addressed the final advisor warning in |
<!-- markdownlint-disable MD041 --> ## Summary Adds the canonical dated `## v0.0.93` release entry to `docs/changelog/2026-07-23.mdx`. The entry records user-visible behavior, release validation, and documentation controls merged after `v0.0.92`, while preserving the pending DGX OS `7.6.x` Station Express qualification caveat. ## Changes - Adds the parser-safe dated release entry with a summary, grouped details, and published-route links. - Reconciles the `v0.0.92..origin/main` commit range with merged `v0.0.93` PRs. - Records that no-OTA DGX OS `7.6.x` passed bounded host preflight, while full Station Express end-to-end qualification remains pending. - Leaves existing product pages unchanged because the source PRs already document their supported behavior. ### Source summary - #7285 -> `docs/changelog/2026-07-23.mdx`: Records the existing-vLLM ownership choice and resumable Station handoff. - #7419 -> `docs/changelog/2026-07-23.mdx`: Records bounded no-OTA DGX OS `7.6.x` recognition and its pending end-to-end qualification. - #7268 -> `docs/changelog/2026-07-23.mdx`: Records optional Hugging Face authentication, output sanitization, and resumable HTTP `429` recovery. - #7442 -> `docs/changelog/2026-07-23.mdx`: Records clean SIGINT handling at hidden credential prompts. - #7299 -> `docs/changelog/2026-07-23.mdx`: Records Intel macOS rejection before ref resolution or network work. - #7296 -> `docs/changelog/2026-07-23.mdx`: Records the DGX Spark non-interactive local-vLLM selection order. - #7342 -> `docs/changelog/2026-07-23.mdx`: Records delegated protected E2E approvals in the grouped release-validation bullet. - #7373 -> `docs/changelog/2026-07-23.mdx`: Records base-image publication gating before final-main fanout. - #7388 -> `docs/changelog/2026-07-23.mdx`: Records semantic phase runtime summaries. - #7397 -> `docs/changelog/2026-07-23.mdx`: Records progress coverage hardening. - #7391 -> `docs/changelog/2026-07-23.mdx`: Records centralized larger-runner routing. - #7423 -> `docs/changelog/2026-07-23.mdx`: Records one retry for confirmed hosted-runner loss. - #7399 -> `docs/changelog/2026-07-23.mdx`: Records runner-comparison telemetry. - #7270 -> `docs/changelog/2026-07-23.mdx`: Records staging Brev Launchable validation. - #7426 -> `docs/changelog/2026-07-23.mdx`: Records filtering of irrelevant base-image run history. - #7333 -> `docs/changelog/2026-07-23.mdx`: Records aligned Quickstart platform guidance. - #7343 -> `docs/changelog/2026-07-23.mdx`: Records documentation-writer receipt collection. - #7400 -> `docs/changelog/2026-07-23.mdx`: Records the documentation-writer receipt requirement for docs-only PRs. - #7413 -> `docs/changelog/2026-07-23.mdx`: Records removal of redundant receipt PR metadata. - #7405 -> `docs/changelog/2026-07-23.mdx`: Records corrected inference CLI references. - #7389 -> `docs/changelog/2026-07-23.mdx`: Records completion of the v0.0.91 documentation audit. `#7384` is an internal refactor with no intended runtime behavior change. `#7401` updates internal CodeQL Actions dependencies. `#7376` is already contained in `v0.0.92`, so it is outside the release-entry scan range despite its retained planning label. ## Type of Change - [ ] Code change (feature, bug fix, or refactor) - [ ] Code change with doc updates - [x] Doc only (prose changes, no code sample modifications) - [ ] Doc only (includes code sample changes) ## Quality Gates - [ ] Tests added or updated for changed behavior - [x] Existing tests cover changed behavior — justification: `test/changelog-docs.test.ts` validates dated changelog structure, SPDX syntax, and version headings. - [ ] Tests not applicable — justification: - [x] Docs updated for user-facing behavior changes - [ ] Docs not applicable — justification: - [ ] Sensitive paths changed (security, policy, credentials, preflight, onboarding, inference, runner, sandbox, or messaging) - [ ] Sensitive-path review completed or maintainer-approved waiver recorded — reviewer/approval link/justification: - [ ] Non-success, skipped, or missing CI check accepted by maintainer — check name, approval link, and follow-up issue: ## Documentation Writer Review - [x] Documentation writer subagent reviewed the completed changes - Result: `docs-updated` - Evidence: Reviewed `docs/changelog/2026-07-23.mdx` against `WRITING.md`, `docs/CONTRIBUTING.md`, `docs/.docs-skip`, `docs/index.yml`, the six user-visible source PRs, and the remaining grouped release commits. The review corrected an ambiguous qualification claim, confirmed all published routes, preserved the DGX OS `7.6.x` caveat, and found no remaining action. - Agent: Codex Desktop <!-- docs-review-head-sha: ec0a866 --> <!-- docs-review-agents-blob-sha: 9c9b36d --> ## DGX Station Hardware Evidence - [ ] Tested on DGX Station - Tested commit: Not applicable. This PR does not change `scripts/prepare-dgx-station-host.sh`. - Station profile/scenario: Not applicable. - Result: Not applicable. - Supporting evidence: Not applicable. ## Verification - [x] PR description includes a `Signed-off-by:` line and every commit appears as `Verified` in GitHub - [x] Normal `pre-commit`, `commit-msg`, and `pre-push` hooks passed, or `npm run check:diff` passed when hooks were skipped or unavailable - [x] Targeted behavior tests pass for the current change set, or tests are marked not applicable above — `npx vitest run test/changelog-docs.test.ts`: 1 file and 6 tests passed. - [ ] Applicable broad gate passed — Not applicable to one native changelog file. - [x] Quality Gates section completed with required justifications or waivers - [x] No secrets, API keys, or credentials committed - [ ] `npm run docs` builds without warnings (doc changes only) — completed with 0 errors and 2 existing Fern warnings. - [x] Doc pages follow the [style guide](https://github.com/NVIDIA/NemoClaw/blob/main/docs/CONTRIBUTING.md) (doc changes only) - [ ] New doc pages include SPDX header and frontmatter (new pages only) — not applicable because native changelog entries use a parser-safe MDX SPDX comment without frontmatter. --- Signed-off-by: Carlos Villela <cvillela@nvidia.com> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Documentation** * Added the v0.0.93 changelog covering onboarding and validation improvements. * Documented support for additional DGX Station Express workstation releases and clearer handling of existing vLLM workloads. * Added guidance for optional Hugging Face authentication, resumable rate-limit recovery, and DGX Spark provider selection. * Clarified installer behavior on Intel macOS, release validation requirements, hosted-runner retries, documentation checks, and supported CLI quickstart paths. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
Summary
Recognize stock DGX OS
7.6.xas a stable DGX Station GB300 release family instead of requiring--force-station-installfor each build. The existing hardware and runtime gates remain unchanged, future release families remain fail-closed, and package-preserving factory profiles now tolerate an idle PackageKit daemon observed on the physical 7.6 image.Related Issue
Fixes #7417
Changes
7.2.0,7.4.0,7.5.0, and numeric7.6.xreleases. The Station classifier consumes it because exact build dates and point-release tuples cannot represent the observed 7.6 release family; classifier tests cover OTA, no-OTA, malformed, future, and wrong-lineage cases.DGX_PRETTY_NAME,DGX_SWBUILD_VERSION,DGX_SWBUILD_DATE, andDGX_PLATFORMestablish the workstation release lineage. Driver, ECC, Docker, CDI, container GPU visibility, and exactly-one-GB300 checks still gate preparation.packagekitdonly on stock DGX OS and AI Developer Tools profiles that preserve factory packages. Generic Ubuntu still quiesces PackageKit before package mutation, BaseOS remains process-gated, and all real APT/dpkg activity remains blocked.Type of Change
Quality Gates
Documentation Writer Review
blocked— documentation wording is complete, but canonical publication remains blocked until a maintainer accepts issue Support DGX OS 7.6.x in DGX Station Express #7417's product scope.ci/platform-matrix.json,docs/get-started/dgx-station-preparation.mdx,docs/get-started/prerequisites.mdx,docs/inference/choose-inference-provider.mdx, anddocs/reference/platform-support.mdxDGX Station Hardware Evidence
ddfd4857e55f2b564dc93587a48cc78cf413bf127.6.0; SMBIOSDGX_Station_GB300; one GB300 plus one auxiliary RTX PRO 6000; fresh login without Docker-group membership.supported-dgx-os. Passwordless-sudo--checkmade no persistent changes and returnedCHECK_RESULT=READY_FOR_FACTORY_RUNTIME_PREPARATION; GB300 driver610.43.03, ECC0/0, zero containers, no inference workload, and failed units none. Full Station Express end-to-end qualification remains pending.Verification
Signed-off-by:line and every commit appears asVerifiedin GitHubpre-commit,commit-msg, andpre-pushhooks passed, ornpm run check:diffpassed when hooks were skipped or unavailablenpm testfor broad runtime/test-harness changes;npm run checkfor repo-wide validation/coverage changes — not applicable to this focused classifier/preflight compatibility change; repository structural checks passednpm run docsbuilds without warnings (doc changes only) — equivalent generation, route, variant, and Fern checks passed with zero errors; Fern reported two existing environment/style warningsSigned-off-by: Senthil Ravichandran senthilr@nvidia.com
Summary by CodeRabbit