Skip to content

fix(orchestrator): accept schedule-suffixed workflow IDs for the active scan#88

Closed
awolden wants to merge 1 commit into
block:mainfrom
awolden:awolden/fix-scheduled-scan-workflow-id
Closed

fix(orchestrator): accept schedule-suffixed workflow IDs for the active scan#88
awolden wants to merge 1 commit into
block:mainfrom
awolden:awolden/fix-scheduled-scan-workflow-id

Conversation

@awolden

@awolden awolden commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Why

The singleton workflow-ID check added in #86's merge ("Enforce singleton orchestrator workflow ID") rejects every schedule-started run: Temporal schedules start the action workflow as {Action.ID}-{nominal start time}, so the daily scan arrives as version-guard-active-scan-2026-06-11T08:00:00Z and fails the exact match. Staging's 2026-06-11 08:00 UTC scan failed in 4 seconds and produced no snapshot; manual scans are unaffected since they use the bare ID.

What

  • Match the workflow ID on prefix instead of equality. Arbitrary IDs are still rejected (existing test unchanged); a new test covers the schedule-suffixed form.

Generated with Claude Code

…ve scan

Temporal schedules start the action workflow as "{Action.ID}-{nominal
start time}", so scheduled runs arrive as
version-guard-active-scan-2026-06-11T08:00:00Z and the exact-match
singleton check rejected every scheduled scan (staging produced no
snapshot on 2026-06-11). Match on prefix instead; arbitrary workflow
IDs are still rejected.
@awolden awolden closed this Jun 11, 2026
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