Skip to content

chore(sentry apps): Bump task duration times for tasks#114101

Merged
Christinarlong merged 1 commit intomasterfrom
crl/bump-task-times-for-sa
Apr 27, 2026
Merged

chore(sentry apps): Bump task duration times for tasks#114101
Christinarlong merged 1 commit intomasterfrom
crl/bump-task-times-for-sa

Conversation

@Christinarlong
Copy link
Copy Markdown
Contributor

@Christinarlong Christinarlong commented Apr 27, 2026

Two tasks are running into errors where the outer operations are taking too long so by the time they get to sending the webhook there's too little time for the hard alarm to be set.

  • send_resource_change_webhook - We're missing the bar by like 0.5-0.8 seconds so bump this task time from 8->12s
  • workflow_notification -> we're missing this by kinda alot, apparently the outer operations are taking like 9s so bump this to 15s . The reason this one takes so long is because we are serializing the group in the workflow_notification task which then can take a while if snuba is taking a while.

For send_resource_change_webhook I think it makes sense to bump. For workflow_notification, I think we could just have it retry and hope snuba is faster next time. I'm not too concerned about this dragging down the pool since these operations should be happening normally it's just we didn't give the task enough time.

@Christinarlong Christinarlong requested review from a team as code owners April 27, 2026 23:03
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 27, 2026
@Christinarlong Christinarlong merged commit 3b372e8 into master Apr 27, 2026
67 checks passed
@Christinarlong Christinarlong deleted the crl/bump-task-times-for-sa branch April 27, 2026 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants