Skip to content

Make user account and organization deletion grace periods consistent#109834

Draft
geoffg-sentry wants to merge 1 commit intomasterfrom
update-user-and-org-deletion-grace-period
Draft

Make user account and organization deletion grace periods consistent#109834
geoffg-sentry wants to merge 1 commit intomasterfrom
update-user-and-org-deletion-grace-period

Conversation

@geoffg-sentry
Copy link
Contributor

@geoffg-sentry geoffg-sentry commented Mar 3, 2026

User account and organization grace periods vary by 29 days. One day grace periods for org deletion have been challenging for support and investigation needs and a 30 days user activation is exceptionally long.

Currently reviewing all documentation locations that need to be updated in tandem.

@github-actions github-actions bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Mar 3, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

Backend Test Failures

Failures on ec07500 in this run:

tests/sentry/users/api/endpoints/test_user_details_analytics.py::UserDetailsDeleteAnalyticsTest::test_soft_delete_timestamps_differ_by_30_dayslog
tests/sentry/users/api/endpoints/test_user_details_analytics.py:120: in test_soft_delete_timestamps_differ_by_30_days
    assert delta >= timedelta(days=29, hours=23, minutes=59)
E   assert datetime.timedelta(days=14) >= datetime.timedelta(days=29, seconds=86340)
E    +  where datetime.timedelta(days=29, seconds=86340) = timedelta(days=29, hours=23, minutes=59)
tests/sentry/users/api/endpoints/test_user_details_analytics.py::UserDetailsDeleteAnalyticsTest::test_soft_delete_records_analytics_and_securitylog
tests/sentry/users/api/endpoints/test_user_details_analytics.py:49: in test_soft_delete_records_analytics_and_security
    assert deletion_scheduled >= deletion_request + timedelta(days=29)
E   assert datetime.datetime(2026, 3, 17, 21, 41, 25, 284579, tzinfo=datetime.timezone.utc) >= (datetime.datetime(2026, 3, 3, 21, 41, 25, 284579, tzinfo=datetime.timezone.utc) + datetime.timedelta(days=29))
E    +  where datetime.timedelta(days=29) = timedelta(days=29)

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 Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant