ci: run on the acng-butler pool and do a real kpt fn render #1

Merged

rcsheets merged 2 commits from feat/ci-docker-kpt-render into main

2026-07-20 01:33:36 +00:00

rcsheets commented

2026-07-19 23:09:35 +00:00

Owner

Switch runs-on to the repo's own runner pool (brooktrails/acng-butler),
which is dind. Install the docker CLI + kpt and point DOCKER_HOST at the
in-pod daemon so validate.sh can run the actual kpt fn render (via the
apply-setters function container) instead of only the deterministic
setter/marker wiring check. The render targets a copy so base/ is never
mutated, and stays a clean skip when no docker daemon is present.

Depends on the acng-butler RunnerPool (infra) being deployed first.

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

Switch runs-on to the repo's own runner pool (brooktrails/acng-butler), which is dind. Install the docker CLI + kpt and point DOCKER_HOST at the in-pod daemon so validate.sh can run the actual `kpt fn render` (via the apply-setters function container) instead of only the deterministic setter/marker wiring check. The render targets a copy so base/ is never mutated, and stays a clean skip when no docker daemon is present. Depends on the acng-butler RunnerPool (infra) being deployed first. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

rcsheets added 1 commit

2026-07-19 23:09:35 +00:00

ci: run on the acng-butler pool and do a real kpt fn render

ci / validate (pull_request) Failing after 1m28s

Details

cc5ca40ae3

Switch runs-on to the repo's own runner pool (brooktrails/acng-butler),
which is dind. Install the docker CLI + kpt and point DOCKER_HOST at the
in-pod daemon so validate.sh can run the actual `kpt fn render` (via the
apply-setters function container) instead of only the deterministic
setter/marker wiring check. The render targets a copy so base/ is never
mutated, and stays a clean skip when no docker daemon is present.

Depends on the acng-butler RunnerPool (infra) being deployed first.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

rcsheets added 1 commit

2026-07-20 01:32:29 +00:00

ci: drop docker-from-job; keep the deterministic kpt wiring check

ci / validate (pull_request) Successful in 24s

Details

21c9ea66da

The dind pools don't wire the docker daemon into job containers (jobs run
in their own netns, not host-networked — the same reason pr-reviewer's
build uses buildah, not docker-in-job), so DOCKER_HOST=tcp://localhost:2375
is unreachable from the job and the real `kpt fn render` can't run here.

Remove the docker CLI, kpt, DOCKER_HOST, and the wait-for-docker step. The
setter/marker wiring check in validate.sh already validates the kpt path
deterministically without a container runtime. Keep runs-on: acng-butler
(its own pool) and the kubectl-based Kustomize render.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

rcsheets merged commit 1c11211f9c into main

2026-07-20 01:33:36 +00:00

rcsheets deleted branch feat/ci-docker-kpt-render

2026-07-20 01:33:37 +00:00

rcsheets referenced this pull request from a commit

2026-07-20 01:33:38 +00:00

Merge pull request 'ci: run on the acng-butler pool and do a real kpt fn render' (#1) from feat/ci-docker-kpt-render into main