Skip to content

Fix GPU image smoke launches with the restricted workload identity - #921

Merged
bradhilton merged 1 commit into
mainfrom
hayek/fix-image-smoke
Sep 17, 2026
Merged

bradhilton merged 1 commit into
mainfrom
hayek/fix-image-smoke

Conversation

@bradhilton

@bradhilton bradhilton commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

ART main images build and push, but the post-build SkyPilot smoke launch fails before a pod is provisioned, preventing the downstream Caladan image build from being dispatched. The smoke workflow omits the workload identity already configured by ART's trainer CI and Caladan's image workflow. SkyPilot 0.12 consequently tries to restore its legacy wildcard RBAC, which cks-wb3 admission policy has blocked since September 3.

Configure the existing skypilot-workload service account for the smoke launch. Also print the provision log on failure so the underlying Kubernetes error is visible rather than only ResourcesUnavailableError.

Validation

  • CI passes at 5a9561440: 783 Megatron lightweight tests and 1,517 unit tests, with 73 skips; lint/type checks are green.

  • Compared recent failed image runs with the last successful September 3 run and the cluster admission-policy creation time. Builds/pushes succeed; smoke provisioning fails.

  • Confirmed the denial with a Kubernetes server-side dry run; no RBAC was modified. Inspected the workflow-pinned SkyPilot 0.12 bootstrap path to verify the explicit identity skips legacy RBAC setup.

  • Parsed the workflow, checked the smoke step with bash -n, and passed the existing GPU image build-context test.

  • Live validation passed against the previously published immutable ART image (7ef066b5) on free cks-wb3/H200 infrastructure using the hosted SkyPilot service. The pod used serviceAccountName: skypilot-workload and printed ART_IMAGE_SMOKE_OK after checking the image revision, SkyPilot/Ray paths, and uv. The validation cluster was explicitly terminated; Kubernetes confirms no owned pods or services remain.

This change does not modify the image, cluster admission rules, or service-account permissions.

@bradhilton
bradhilton merged commit ca661bc into main Sep 17, 2026
7 checks passed
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