Skip to content

Notify users about duplicate jobs#780

Merged
f-f merged 1 commit into
masterfrom
notify-duplicate-jobs
Jul 22, 2026
Merged

Notify users about duplicate jobs#780
f-f merged 1 commit into
masterfrom
notify-duplicate-jobs

Conversation

@thomashoneyman

@thomashoneyman thomashoneyman commented Jul 22, 2026

Copy link
Copy Markdown
Member

Closes #748. We now notify GitHub issue submitters when their operation matches an existing job instead of polling that job and replaying its previous logs or failure.

API behavior

Job submission endpoints now distinguish new and duplicate submissions using the HTTP status:

  • 201 Created when a new job is inserted
  • 200 OK when an existing duplicate job is returned

Both responses retain the existing V1 JSON body:

{
  "jobId": "..."
}

This keeps the V1 wire format compatible while allowing clients to distinguish the two outcomes using standard HTTP semantics. The behavior applies consistently to publish, unpublish, transfer, and package-set submissions.

GitHub issue behavior

When the GitHub issue client receives a duplicate response, it:

  • posts a duplicate-job notice
  • links to the existing job logs
  • does not poll or replay the existing job's logs or result
  • leaves the submitted issue open

New jobs continue through the existing polling and reporting workflow.

@thomashoneyman
thomashoneyman requested a review from f-f July 22, 2026 00:46
@f-f
f-f force-pushed the notify-duplicate-jobs branch from 5045537 to 09a40f2 Compare July 22, 2026 09:21
@f-f
f-f force-pushed the notify-duplicate-jobs branch from 09a40f2 to 0159443 Compare July 22, 2026 09:38
@f-f
f-f merged commit 5eedc08 into master Jul 22, 2026
7 checks passed
@f-f
f-f deleted the notify-duplicate-jobs branch July 22, 2026 10:19
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.

Notify users when a job is a duplicate

2 participants