Skip to content

fix(ci): make the consumer canary runnable#96

Merged
ErenAri merged 1 commit into
mainfrom
ci/canary-fix
Jul 21, 2026
Merged

fix(ci): make the consumer canary runnable#96
ErenAri merged 1 commit into
mainfrom
ci/canary-fix

Conversation

@ErenAri

@ErenAri ErenAri commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

The canary's first real dispatch failed on both jobs — the example .bpf.o objects are gitignored build outputs, so a fresh checkout has nothing to validate. Found by dispatching the lane, not by review, which is the canary earning its keep on day one (against itself, admittedly).

  • source-build: runs make examples; its dependency set already includes the toolchain.
  • prebuilt: drops the artifact and validates in command mode, needing no compiler at all. Building the fixture there would have pulled in exactly the toolchain that job exists to prove is unnecessary — so this makes the job a stricter check than before, not a weaker one. It still covers the full consumer path: resolve pin → release → download + checksum assets → boot → execute in-guest → report.

Its first real run failed on both jobs: the example .bpf.o objects are
gitignored build outputs, so a fresh checkout has no artifact to
validate. Caught by dispatching the lane rather than by review.

The source-build job now runs 'make examples' (its dependency set
already includes the toolchain). The prebuilt job instead drops the
artifact entirely and validates in command mode, which needs no
compiler at all -- building the fixture there would have pulled in the
toolchain that job exists to prove is unnecessary. It still exercises
the whole consumer path: resolve the pin to a release, download and
checksum the assets, boot, execute in-guest, report.

Signed-off-by: ErenAri <erenari27@gmail.com>
Copilot AI review requested due to automatic review settings July 21, 2026 16:04

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ErenAri
ErenAri merged commit 548ce89 into main Jul 21, 2026
7 checks passed
@ErenAri
ErenAri deleted the ci/canary-fix branch July 21, 2026 16:06
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