ci(windows): exercise MXC inference demos with mock API - #3780
prekshivyas wants to merge 3 commits into
Conversation
Signed-off-by: Prekshi Vyas <prekshiv@nvidia.com>
|
@elezar CI note: this PR changes the Windows lane, but the author account cannot apply repository labels. Please add the |
shailendra-nv
left a comment
There was a problem hiding this comment.
Requesting changes for two blocking coverage gaps:
- The harness can report /api/tags coverage even if the sandbox workload stops making that request, because the host-side prerequisite check already records the same GET.
- This changes the Windows CI/task contract without updating the contributor skill that documents and validates that contract.
The currently executed checks are green, but the Windows MSVC matrix jobs that exercise this path were skipped by the test:windows gate.
Signed-off-by: Prekshi Vyas <prekshiv@nvidia.com>
|
@shailendra-nv The two requested changes are addressed in |
Signed-off-by: Prekshi Vyas <prekshiv@nvidia.com>
|
@shailendra-nv Fixed the hosted Windows regression in Root cause: Native ARM64 qualification:
The labeled x64 and ARM64 hosted lanes are rerunning on this commit. Please re-review once they are green. |
Summary
Runs both shipped MXC inference demos end to end in hosted Windows CI using the real gateway and CLI, the in-process MXC mock, and one local API stub. The lane covers Ollama and cloud-inference wiring on x64 and ARM64 while explicitly making no claim of real MXC or AppContainer enforcement.
Related Issue
Related to #2050. Complements #3769, which adds the broader Windows control-plane conformance lane.
Changes
run-inference-test.ps1with a synthetic sandbox-scoped credential.wxcmock launch children with exactlyMxcProcess.env, matching the real exec contract and preventing host-environment inheritance./api/tagsGETs (host prerequisite plus sandbox workload), one/api/generatePOST, and one authenticated/v1/chat/completionsPOST.Testing
mise run pre-commitpassesValidated locally on native Windows ARM64 at
442d6f0f:mise run --skip-tools windows:lint:arm64: PASSmise run --skip-tools windows:build:arm64: PASScargo test --target aarch64-pc-windows-msvc -p openshell-driver-mxc mock_exec_uses_only_the_mxc_process_environment: 1 passedcargo test --target aarch64-pc-windows-msvc -p openshell-driver-mxc --test demo_examples: 5 passedmise run --skip-tools windows:e2e:mxc:inference-mock:arm64: PASS for both demosGET /api/tags, onePOST /api/generate, one authenticatedPOST /v1/chat/completionsmise run --skip-tools windows:test:arm64: 5,020/5,021 passed; one unrelated DNS test hit a transient Windows UDP bind denial (os error 10013) and passed immediately when rerun alonecargo fmt --all -- --check,git diff --check, PowerShell parser checks, andmise run --skip-tools markdown:lint: PASSThe hosted matrix will provide the corresponding x64 result and rerun the full ARM64 suite.
Checklist