Skip to content

ci(vm): cleanup vm build infra#1186

Merged
drew merged 2 commits intomainfrom
dn/consolidate-vm-ci
May 6, 2026
Merged

ci(vm): cleanup vm build infra#1186
drew merged 2 commits intomainfrom
dn/consolidate-vm-ci

Conversation

@drew
Copy link
Copy Markdown
Collaborator

@drew drew commented May 6, 2026

Summary

Publish openshell-driver-vm from the normal dev and tagged release workflows, while leaving VM runtime tarballs on the on-demand vm-runtime workflow/release. The legacy openshell-vm crate remains in the repository but is excluded from workspace builds and active CI paths.

Related Issue

None.

Changes

  • Add reusable macOS VM driver build workflow and wire Linux/macOS driver artifacts into dev and tagged releases.
  • Move runtime pins and kernel config under crates/openshell-driver-vm/runtime.
  • Publish runtime tarballs from the on-demand VM runtime workflow to the rolling vm-runtime tag/release.
  • Align VM Linux runtime and macOS driver host jobs with the shared CPU runner labels; keep the native macOS runtime job on macos-latest-xlarge.
  • Add GitHub artifact attestations for VM runtime tarballs and VM driver release tarballs.
  • Remove VM-specific checksum files from releases; keep the general openshell-checksums-sha256.txt used by install-dev.sh.
  • Use gh release download instead of curl/API URL plumbing for VM runtime release assets consumed by driver workflows.
  • Remove the old release-vm-dev.yml, standalone VM wrapper/tasks, and install-vm.sh.
  • Remove the install-dev.sh Linux package snippet from generated dev/tag release notes.
  • Make install-dev.sh install the selected Debian package via OPENSHELL_VERSION, including the gateway and VM driver.
  • Update canary coverage, docs, and stale references for the new release shape.

Testing

  • mise run pre-commit passes
  • Shell syntax checks for changed VM scripts and installers
  • Workflow YAML parse checks for changed release workflows
  • cargo check -p openshell-driver-vm
  • cargo metadata confirms openshell-vm is excluded from workspace members
  • Reference checks for removed install-vm, release-vm-dev, and stale VM paths
  • Manually triggered release-vm-kernel.yml on dn/consolidate-vm-ci: run 25421146257 passed, refreshed the vm-runtime prerelease without checksum assets, and generated GitHub artifact attestations before the final runner-label change
  • gh attestation verify vm-runtime-linux-x86_64.tar.zst -R NVIDIA/OpenShell succeeds for the refreshed vm-runtime asset

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

@drew drew requested review from a team, derekwaynecarr, maxamillion and mrunalp as code owners May 6, 2026 05:05
@drew drew force-pushed the dn/consolidate-vm-ci branch from 96bc482 to f3b0df9 Compare May 6, 2026 05:13
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

@drew drew force-pushed the dn/consolidate-vm-ci branch 4 times, most recently from 26c9266 to fe39b7f Compare May 6, 2026 06:14
@drew drew changed the title ci(vm): publish driver in normal releases ci(vm): cleanup vm build infra May 6, 2026
@drew drew force-pushed the dn/consolidate-vm-ci branch from fe39b7f to 571064d Compare May 6, 2026 06:27
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 6, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@drew drew force-pushed the dn/consolidate-vm-ci branch from 571064d to 31fe874 Compare May 6, 2026 06:59
@drew drew force-pushed the dn/consolidate-vm-ci branch from 31fe874 to 254004a Compare May 6, 2026 14:57
@drew drew merged commit 4d388d2 into main May 6, 2026
9 checks passed
@drew drew deleted the dn/consolidate-vm-ci branch May 6, 2026 15:58
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.

2 participants