Skip to content

feat(m16): retire PipelineRun escape hatches - #21

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/m16-retire-escape-hatches-a538
Sep 7, 2026
Merged

feat(m16): retire PipelineRun escape hatches#21
cursor[bot] merged 1 commit into
mainfrom
cursor/m16-retire-escape-hatches-a538

Conversation

@jmjava

@jmjava jmjava commented Sep 7, 2026

Copy link
Copy Markdown
Owner

Flask always creates StackRun CRs. --pipeline-run and STACKRUN_VIA_CRD=false are removed.

What changed

  • Orchestrator _create_run / GET /api/runs always use StackRuns. STACKRUN_VIA_CRD is ignored.
  • Promote without approved_by is 200 / operator PendingApproval (same as GUI). Newman still gets a pipelinerun alias equal to the StackRun name.
  • generate-run.sh --pipeline-run and GENERATE_PIPELINE_RUN=true exit with a removed-hatch error. The raw PipelineRun YAML path is gone.
  • Helm orchestrationService.stackrunViaCrd is gone; STACKRUN_VIA_CRD is hard-coded "true" for mixed-image rollouts.
  • run-cluster-ci.sh --skip-operator no longer sets STACKRUN_VIA_CRD=false.

Tests

  • Orchestrator route tests assert create_stackrun and StackRun specs.
  • verify-m4-stacks-and-labels.sh checks --pipeline-run is rejected.

Still open in M16: spoken demo narration/MP4 rebuild (docgen rebuild-after-audio); S34 intercept E2E stays parked.

Open in Web Open in Cursor 

Flask always creates StackRuns; STACKRUN_VIA_CRD=false is ignored.
generate-run.sh --pipeline-run is removed. Helm no longer opts out.

Co-authored-by: John Menke <jmjava@gmail.com>
@jmjava
jmjava marked this pull request as ready for review September 7, 2026 16:47
@cursor
cursor Bot merged commit 3b9531b into main Sep 7, 2026
3 checks passed
@cursor
cursor Bot deleted the cursor/m16-retire-escape-hatches-a538 branch September 7, 2026 16:47
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