Skip to content

Fix Tekton multi-PVC compile TaskRuns - #52

Merged
jmjava merged 18 commits into
cursor/close-e2e-audit-gaps-fc5ffrom
cursor/fix-tekton-multi-pvc-coschedule-fc5f
Sep 14, 2026
Merged

jmjava merged 18 commits into
cursor/close-e2e-audit-gaps-fc5ffrom
cursor/fix-tekton-multi-pvc-coschedule-fc5f

Conversation

@jmjava

@jmjava jmjava commented Sep 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • configure Tekton with coschedule: disabled so compile TaskRuns may bind source and build-cache PVCs
  • wait for Pipeline and Triggers admission deployments before applying webhook-dependent resources
  • retry Task and interceptor applies for transient admission startup races
  • give all compile-image Pipeline parameters a valid ubuntu:22.04 default instead of overriding Task defaults with an empty image
  • add static acceptance coverage for these contracts

Live failure evidence

The first run confirmed the PVC change in TaskRun provenance. It then exposed two later blockers:

  • Pipeline and Triggers manifests intermittently failed with admission webhook connection refused because pod-selector waits could succeed before pods existed and ignored timeout failures.
  • A successful setup reached bootstrap, where compile TaskRuns failed with missing field(s): steps[0].Image; Pipeline defaults passed an empty value to Tasks and overrode their valid image defaults.

Verification

  • bash -n scripts/install-tekton.sh
  • targeted M17 intercept and hook tests — 9 passed
  • Ruff passed
  • all Pipeline and Task YAML parsed successfully
  • bash scripts/run-regression-agent.shregression exit code: 0 (local-only because this environment has no kubectl context; Tekton/Newman/Results were not run locally)

The live intercept and strict cluster workflows remain the cluster-backed acceptance gates.

Open in Web Open in Cursor 

jmjava and others added 17 commits September 14, 2026 11:16
docs(m17): track and close end-to-end audit gaps
* ci(e2e): automate intercept product paths

Co-authored-by: jmjava <jmjava@gmail.com>

* fix(e2e): limit product flow help text

Co-authored-by: jmjava <jmjava@gmail.com>

* docs(m17): record intercept automation evidence

Co-authored-by: jmjava <jmjava@gmail.com>

* fix(e2e): prepare intercept namespace and results evidence

Co-authored-by: jmjava <jmjava@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* ci(results): schedule strict database regression

Co-authored-by: jmjava <jmjava@gmail.com>

* docs(m17): record Results automation evidence

Co-authored-by: jmjava <jmjava@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* test(operator): add domain reconciliation integration

Co-authored-by: jmjava <jmjava@gmail.com>

* fix(operator): define integration list GVK

Co-authored-by: jmjava <jmjava@gmail.com>

* build(operator): upgrade golangci-lint for Go 1.23

Co-authored-by: jmjava <jmjava@gmail.com>

* chore(operator): resolve lint gate findings

Co-authored-by: jmjava <jmjava@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* fix(ci): retry Tekton controller apply races

Co-authored-by: jmjava <jmjava@gmail.com>

* ci(operator): gate Tekton installer changes

Co-authored-by: jmjava <jmjava@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* fix(rbac): default pipeline bootstrap to least privilege

Co-authored-by: jmjava <jmjava@gmail.com>

* ci(rbac): require strict cluster regression

Co-authored-by: jmjava <jmjava@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* fix(test): isolate Helm RBAC rendering

Co-authored-by: jmjava <jmjava@gmail.com>

* fix(rbac): permit orchestrator PipelineRun creation

Co-authored-by: jmjava <jmjava@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* ci(rbac): isolate deterministic cluster gate

Co-authored-by: jmjava <jmjava@gmail.com>

* ci(rbac): gate cluster workflow changes

Co-authored-by: jmjava <jmjava@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor Agent <cursoragent@cursor.com>
* ci(e2e): gate RBAC changes on intercept paths

Co-authored-by: jmjava <jmjava@gmail.com>

* fix(e2e): clone public apps without SSH secret

Co-authored-by: jmjava <jmjava@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: jmjava <jmjava@gmail.com>
Co-authored-by: jmjava <jmjava@gmail.com>
@jmjava
jmjava marked this pull request as ready for review September 14, 2026 18:43
Co-authored-by: jmjava <jmjava@gmail.com>
@jmjava
jmjava merged commit f7c54db into cursor/close-e2e-audit-gaps-fc5f Sep 14, 2026
7 of 9 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.

2 participants