Skip to content

feat(taskbroker): Add delay_on_retry to SetTaskStatusRequest#262

Merged
untitaker merged 4 commits into
mainfrom
feat/taskbroker-delay-on-retry
May 21, 2026
Merged

feat(taskbroker): Add delay_on_retry to SetTaskStatusRequest#262
untitaker merged 4 commits into
mainfrom
feat/taskbroker-delay-on-retry

Conversation

@untitaker
Copy link
Copy Markdown
Member

@untitaker untitaker commented May 20, 2026

Summary

Add delay_on_retry field to SetTaskStatusRequest so workers can communicate retry backoff delays for tasks from raw topics that don't have retry_state embedded in the message.

Without this field, retry delays configured via Retry(delay=...) are lost for raw topic tasks, causing retries to run immediately instead of respecting the configured backoff.

Related: getsentry/taskbroker#630 (comment)

Ref STREAM-981

🤖 Generated with Claude Code

Add delay_on_retry field to SetTaskStatusRequest so workers can
communicate retry backoff delays for tasks from raw topics.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@untitaker untitaker requested a review from a team as a code owner May 20, 2026 17:35
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 20, 2026

STREAM-981

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 20, 2026

The latest Buf updates on your PR. Results from workflow ci / buf-checks (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedMay 20, 2026, 6:18 PM

Comment thread proto/sentry_protos/taskbroker/v1/taskbroker.proto
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 0fb622b. Configure here.

Comment thread Cargo.lock
@untitaker untitaker merged commit d5a53b5 into main May 21, 2026
15 checks 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.

3 participants