Skip to content

ci: fix broken setup-kind action with helm/kind-action#445

Merged
sanketsudake merged 2 commits into
masterfrom
hotfix-setup-kind
Jun 6, 2026
Merged

ci: fix broken setup-kind action with helm/kind-action#445
sanketsudake merged 2 commits into
masterfrom
hotfix-setup-kind

Conversation

@sanketsudake
Copy link
Copy Markdown
Member

What

Hotfix for a regression that #436 merged: engineerd/setup-kind@v0.6.2 fails with File not found: .../dist/main/index.js (the tagged commit doesn't contain the compiled action bundle), which breaks the setup-cluster step for every env CI job. Surfaced on #438's first post-merge run.

  • setup-cluster: switch to the maintained helm/kind-action@v1.14.0 (SHA-pinned), passing node_image/version/config through and cluster_name: kind so the kind-kind kubectl context and kind load docker-image defaults keep working.
  • collect-fission-dump: best-effort now — a missing fission CLI (setup failed before installing it) or an empty dump no longer masks the original job failure (if-no-files-found: ignore).

Why it slipped through

#436 touched no env directories, so the path-filtered jobs all skipped and the composite-action change was never exercised pre-merge. To avoid repeating that, this PR includes a small genuine fix under perl/ (broken examples link in the README) so the perl job runs setup-cluster end-to-end on this PR — green CI here means the new kind action actually works.

Verification

  • YAML validated locally
  • perl CI job on this PR exercises: helm install → kind-action cluster create (kind v0.32.0, node v1.34.8) → fission v1.25.0 install → skaffold build/deploy

🤖 Generated with Claude Code

sanketsudake and others added 2 commits June 6, 2026 20:29
engineerd/setup-kind v0.6.2 fails with 'File not found:
dist/main/index.js' (the tagged commit does not ship the compiled
action), breaking the setup-cluster step for every env job. Switch to
the maintained helm/kind-action v1.14.0 with cluster_name kind so the
kind-kind kubectl context and `kind load docker-image` defaults keep
working.

Also make collect-fission-dump best effort: a missing fission CLI
(setup failed before installing it) or empty dump no longer masks the
original job failure.

The broken bump in #436 went unnoticed because that PR touched no env
directories, so no job exercised setup-cluster pre-merge; this PR
includes a perl README link fix so the perl job validates the action
end to end.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Same fix as #442/#443/#444: hiberbee/github-action-skaffold pins
skaffold 2.3.1, which cannot parse the repo's skaffold/v4beta13
config. Needed here so the perl job can validate the kind-action
hotfix end to end (identical hunk to #444, merges cleanly).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sanketsudake sanketsudake merged commit 303e89b into master Jun 6, 2026
15 checks passed
sanketsudake added a commit that referenced this pull request Jun 6, 2026
…n from master which supersedes this branch's jvm-job hunk
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