Skip to content

Conversation

@armenzg
Copy link
Member

@armenzg armenzg commented Nov 24, 2025

This is a follow-up to #103820.

A failure to delete a group should be re-processed within 24 hours.

Changes included:

  • Schedule re-processing task every N hours instead of odd schedule
  • Re-process groups pending deletion that are older than 6 hours rather than midnight from the day before

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 24, 2025
This is a follow-up to #103820.

A failure to delete a group should be re-processed within 24 hours.

Changes included:
* Schedule re-processing task every 6 hours
* Re-process groups pending deletion that are older than 6 hours rather than midnight from the day before
@armenzg armenzg force-pushed the 11_24/only_report_on_last_attempt/armenzg branch from f863251 to 9c17b92 Compare November 25, 2025 13:05
@armenzg armenzg self-assigned this Nov 25, 2025
@armenzg armenzg changed the title 11 24/only report on last attempt/armenzg chore(deletions): Change schedule & date ranges Nov 25, 2025
"task": "deletions:sentry.tasks.delete_pending_groups",
# Runs every 2 hours during 9am-5pm Eastern Time (EST: UTC-5)
# 9am, 11am, 1pm, 3pm, 5pm EST = 14:00, 16:00, 18:00, 20:00, 22:00 UTC
"schedule": task_crontab("0", "14,16,18,20,22", "*", "*", "*"),
Copy link
Member Author

Choose a reason for hiding this comment

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

This was done so we could observe during work hours.

Copy link
Member

Choose a reason for hiding this comment

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

When you switch schedules, you may miss an interval in the new schedule depending on when the last run was. But the schedule should align to the new schedule after that.

@armenzg armenzg marked this pull request as ready for review November 25, 2025 13:11
@armenzg armenzg requested review from markstory and yuvmen November 25, 2025 14:28
"task": "deletions:sentry.tasks.delete_pending_groups",
# Runs every 2 hours during 9am-5pm Eastern Time (EST: UTC-5)
# 9am, 11am, 1pm, 3pm, 5pm EST = 14:00, 16:00, 18:00, 20:00, 22:00 UTC
"schedule": task_crontab("0", "14,16,18,20,22", "*", "*", "*"),
Copy link
Member

Choose a reason for hiding this comment

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

When you switch schedules, you may miss an interval in the new schedule depending on when the last run was. But the schedule should align to the new schedule after that.

@armenzg armenzg merged commit d586c61 into master Nov 25, 2025
69 checks passed
@armenzg armenzg deleted the 11_24/only_report_on_last_attempt/armenzg branch November 25, 2025 18:39
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.

3 participants