Repository navigation
feat(signal): run one production 15-check certification workflow - #963
Kevin Cho (chokevin) wants to merge 42 commits into
Conversation
|
Typed workload policy, Pod queue labeling, MPI worker isolation, and focused contract tests (15 seconds): signal-workload-policy-evidence.mp4 |
|
Copilot-generated 20-second fframes explainer. The source excerpts remain present in the production cleanup fixes. The recorded integration is explicitly archival, before the final production-only trim: 15 task nodes, 10 Pass and 5 Inconclusive. This is not a new live run or evidence that the updated head passed certification. The updated implementation is being validated separately; the live cluster and soak were not modified. signal-pr963-20s.mp4 |
|
Updated 20-second fframes architecture explainer with local narration: why Signal is needed, how the controller/CRDs/RBAC/adapter template are deployed alongside Argo, operator-owned scripts and Profile, the exact Node request label, immutable admission, serial tasks, and script execution in Pods/MPIJobs. This is an architecture animation, not a live execution recording. Placeholder deployment commands were not executed. It supersedes the archival clip as the architecture overview, not as runtime validation. signal-deployment-trigger-20s.mp4 |
One immutable
CertificationRunreserves its participant set once and executes exactly 15 ordered checks through one serial Argo Workflow. Argo completion is not certification success: aggregation requires identity-bound task outputs and durable typed backend evidence.PodandMPIJobexecution through one fixed adapter WorkflowTemplate; Kueue is intentionally deferred.spec.release: true.Evidence
Architecture overview: narrated 20-second fframes animation covering deployment prerequisites, the operator-owned Profile/scripts, Node request label, immutable admission, serial Argo tasks, and Pod/MPIJob execution.
signal-deployment-trigger-20s.mp4
This is a code-backed architecture animation, not a live execution recording. Deployment commands contain placeholders and were not executed. Signal does not install Argo, MPI Operator, operator scripts, runner images, or Profiles.
The earlier archival integration explainer showed 15 task nodes, 10 Pass and 5 Inconclusive, before the production-only reduction and lifecycle hardening. Neither video establishes successful certification of the current head. No new live Argo/MPI/GPU run or soak was performed.
Verification
Current head
cd641dc76b58a1839d93f83de6aa69b7175e8b24:main.[no test files].go vet,go generate ./api/signal/v1alpha1,make signal-manifests, andgit diff --checkpass.Remaining limits and review guide
No committed Signal regression suite and no new real-cluster verification. Foreground garbage collection/RBAC, Argo output persistence, MPI Operator cleanup ordering, and the fixed adapter image's required shell/hash/MPI tooling still need deployment-level validation.
Review
api/signal/v1alpha1/for the immutable CR contracts;internal/signal/controller.go,evidence.go, andprojection.gofor admission, recovery, audit, cleanup, and release;internal/signaladapter/for backend execution and evidence capture; anddeploy/signal/for RBAC, templates, and operator documentation.