docs(mxc): document Windows host preparation (NVBug 6842834) - #3892
Conversation
Signed-off-by: Prekshi Vyas <prekshiv@nvidia.com>
Signed-off-by: Shailendra Singh <shailendras@nvidia.com>
|
/ok to test 0cea165 |
shailendra-nv
left a comment
There was a problem hiding this comment.
Re-reviewed the complete five-file documentation diff at 0cea165 against windows, including the MXC launch/error paths, upstream tier-selection and host-preparation documentation, shipped example consistency, and architecture/skill references. No remaining blocking findings.
The system-drive ACL preparation is now conditional on the AppContainer + DACL tier and the probe recommendation. All five copies explain its scope and link the upstream verification/rollback guide. Generic Access Denied and exit code 1 no longer imply that a host ACL change is needed.
Local Markdown/Mermaid, pinned Fern, standalone license, Rust formatting, and diff checks passed. The full local pre-commit attempt was limited by the unchanged grpcio-tools build on Windows ARM64; this is recorded in the PR. No live host ACL changes or MXC qualification were performed for this documentation-only update.
Current-head CI is green: 30 passed checks, 34 skipped, no failures or pending checks. Verified a clean merge against windows at 4cb6054. READY_TO_MERGE.
Summary
Document Windows host preparation for live MXC ProcessContainer runs. Use a standalone
wxc-exec.exeoutside MSIX/WindowsApps package directories, and apply system-drive ACL preparation only when MXC selects the AppContainer + DACL fallback tier and its probe recommends the change.Related Issue
Changes
wxc-exec.exe --probebefore host preparation. BaseContainer and AppContainer + BFS do not require the documented system-drive ACL change.Testing
Current-head Branch Checks passed at
0cea165bbaaf188925dbe95410b453697d65d730, including all three Rust platforms. PR rollup: 30 passed, 34 skipped, no failing or pending checks.mise run markdown:lint: 139 Markdown files, zero errors; 185 Mermaid-scanned files, all diagrams valid.Repository-pinned Fern 5.112.0
check: zero errors, three existing warnings.uv run --no-project python scripts/update_license_headers.py --check: all 991 files passed.Rust formatting checks completed successfully during pre-commit.
git diff --check.All five edited files reviewed against upstream MXC host-preparation documentation and tier-selection behavior.
mise run pre-commitpasses. Attempted on native Windows ARM64; the aggregate failed while building the unchangedgrpcio-tools==1.78.0Python dependency for CPython 3.14. The focused checks above passed. The follow-up documentation commit bypassed only the repeated local hook invocation after recording this failure; current-head GitHub CI remains the merge gate.Unit and E2E tests: not applicable to this documentation-only change. No runtime code or Windows build/test workflows changed.
The host-preparation command was not executed because it persistently changes host ACLs. Documentation checks do not claim live MXC qualification on either x64 or ARM64.
Checklist