Skip to content

feat(alerts): Set up prune_old_open_period_activity#112243

Open
kcons wants to merge 2 commits intomasterfrom
kcons/tasker
Open

feat(alerts): Set up prune_old_open_period_activity#112243
kcons wants to merge 2 commits intomasterfrom
kcons/tasker

Conversation

@kcons
Copy link
Copy Markdown
Member

@kcons kcons commented Apr 3, 2026

Our backlog is only 17m or so rows, but by reusing a task pattern we've already used before we should be able to safely get that quickly cleaned up in all regions.

A job or post-deploy migration would also be an option, but this is no harder and more or less guarantees that next time we look it'll be dealt with.

@kcons kcons requested review from a team as code owners April 3, 2026 22:48
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 3, 2026
Copy link
Copy Markdown
Member

@wedamija wedamija left a comment

Choose a reason for hiding this comment

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

Does this cleanup need to happen in self hosted?

@kcons
Copy link
Copy Markdown
Member Author

kcons commented Apr 6, 2026

Does this cleanup need to happen in self hosted?

Depends on what we mean by "need". Self hosted would also have a backlog that needs to be cleared, so this will do useful work, but if self-hosted also runs incinerator stuff (I honestly don't know) chances are the backlog is small enough that incinerator could handle it.

@wedamija
Copy link
Copy Markdown
Member

wedamija commented Apr 6, 2026

Does this cleanup need to happen in self hosted?

Depends on what we mean by "need". Self hosted would also have a backlog that needs to be cleared, so this will do useful work, but if self-hosted also runs incinerator stuff (I honestly don't know) chances are the backlog is small enough that incinerator could handle it.

Yeah, if incinerator will handle it then we don't need to worry about it. Mostly I was wondering if we should use a migration so that it runs explicitly, because I assume we won't keep this task around forever, and so not every self-hosted install will end up seeing it.

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