Skip to content

Optionally start the CI k8s shipit check next to the shipit label - #38

Merged
asheth-eightfold merged 1 commit into
masterfrom
feat/shipit-k8s-dispatch
Oct 30, 2026
Merged

asheth-eightfold merged 1 commit into
masterfrom
feat/shipit-k8s-dispatch

Conversation

@asheth-eightfold

Copy link
Copy Markdown
Contributor

Summary

Optionally start the CI k8s shipit check next to the shipit label, the same way CI_K8S_TRIGGER does for needs_ci (#35) and SANDBOX_K8S_TRIGGER for sandbox (#36).

With SHIPIT_K8S_TRIGGER=true (passed in by the calling workflow), every shipit / :shipit: / sudo shipit comment also dispatches shipit.yml with trigger=comment (or trigger=sudo).

It dispatches even when the PR already has shipit: a label change made with GITHUB_TOKEN starts no workflow, and commenting shipit again is how an author asks for a fresh merge check (for example after resolving a review thread).

Behaviour change: none unless the caller sets the variable. A failed dispatch never fails the action. Labels are added exactly as today.

Test

entrypoint.sh run locally with curl stubbed:

comment labels already on trigger calls made
shipit none true adds needs_ci, adds shipit, dispatches comment
shipit none false adds needs_ci, adds shipit (as today)
shipit shipit, ci_verified true dispatches comment only
:shipit: ci_verified true adds shipit, dispatches comment
sudo shipit hotfix shipit true adds needs_ci, dispatches sudo
sudo shipit shipit false adds needs_ci (as today)
needs_ci none true adds needs_ci only
shipit please none true none

🤖 Generated with Claude Code

With SHIPIT_K8S_TRIGGER=true (passed by the calling workflow), every shipit / sudo shipit
comment also dispatches shipit.yml (trigger comment or sudo), even when the PR already has
`shipit`: a GITHUB_TOKEN label change starts no workflow, and commenting again is how an
author asks for a fresh merge check.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Files

None

@asheth-eightfold
asheth-eightfold merged commit e8b64c7 into master Oct 30, 2026
1 check 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.

1 participant