Skip to content

fix(observability): accept authenticated pre-webhook start - #208

Merged
rldyourmnd merged 1 commit into
mainfrom
fix/pre-webhook-short-job-correlation
Aug 26, 2026
Merged

fix(observability): accept authenticated pre-webhook start#208
rldyourmnd merged 1 commit into
mainfrom
fix/pre-webhook-short-job-correlation

Conversation

@rldyourmnd

Copy link
Copy Markdown
Contributor

Closes #206

Natural evidence behind this refinement

The first post-#207 short job (setup-systems run 33001825473, job pins) reached the authenticated job-start hook before GARM delivered JobStarted. Waiting for a running row is therefore circular: the hook blocks before job steps while the lifecycle event may arrive later.

Change

  • use the verified one-job claim pool plus exact repository and JobAvailable job identity
  • promote exactly one assigned/acquiring/acquired intent to running from the authenticated hook
  • refuse zero or multiple exact candidates
  • preserve the existing exact-runner idempotent path

Verification

  • make verify
  • repeated queue/cachebroker tests
  • HTTP claim integration now starts from an acquired intent with no runner identity

@rldyourmnd
rldyourmnd enabled auto-merge August 26, 2026 19:12
@rldyourmnd
rldyourmnd merged commit 8dfb61c into main Aug 26, 2026
10 checks passed
@rldyourmnd
rldyourmnd deleted the fix/pre-webhook-short-job-correlation branch August 28, 2026 10:00
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.

Short downstream direct-JIT jobs finish without run-scoped queue.running correlation

1 participant