Skip to content

chore(integrations): Gate disabling repositories behind a separate flag#113071

Merged
wedamija merged 2 commits intomasterfrom
danf/scm-auto-sync-remove-flag
Apr 15, 2026
Merged

chore(integrations): Gate disabling repositories behind a separate flag#113071
wedamija merged 2 commits intomasterfrom
danf/scm-auto-sync-remove-flag

Conversation

@wedamija
Copy link
Copy Markdown
Member

Being cautious with this auto sync work and gating a potentially destructive operation behind a deletion flag.

Being cautious with this auto sync work and gating a potentially destructive operation behind a deletion flag.
@wedamija wedamija requested a review from a team April 15, 2026 17:05
@wedamija wedamija requested review from a team as code owners April 15, 2026 17:05
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 15, 2026
@wedamija wedamija enabled auto-merge (squash) April 15, 2026 17:08
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 15, 2026

Backend Test Failures

Failures on 0d1a644 in this run:

tests/sentry/integrations/github/tasks/test_sync_repos_on_install_change.py::SyncReposOnInstallChangeTestCase::test_mixed_add_and_removelog
[gw1] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/sentry/integrations/github/tasks/test_sync_repos_on_install_change.py:114: in test_mixed_add_and_remove
    assert old_repo.status == ObjectStatus.DISABLED
E   AssertionError: assert 0 == 1
E    +  where 0 = <Repository at 0x7f15cc9c34d0: id=21, organization_id=4557973191393312, name='getsentry/old-repo', provider='integrations:github'>.status
E    +  and   1 = ObjectStatus.DISABLED
tests/sentry/integrations/source_code_management/test_sync_repos.py::SyncReposForOrgTestCase::test_disables_removed_reposlog
[gw1] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/sentry/integrations/source_code_management/test_sync_repos.py:94: in test_disables_removed_repos
    assert repo.status == ObjectStatus.DISABLED
E   AssertionError: assert 0 == 1
E    +  where 0 = <Repository at 0x7fa379beeb10: id=29, organization_id=4557973197553696, name='getsentry/old-repo', provider='integrations:github'>.status
E    +  and   1 = ObjectStatus.DISABLED
tests/sentry/integrations/github/test_webhook.py::InstallationRepositoriesEventWebhookTest::test_end_to_end_repos_removedlog
[gw0] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/sentry/integrations/github/test_webhook.py:500: in test_end_to_end_repos_removed
    assert repo.status == ObjectStatus.DISABLED
E   AssertionError: assert 0 == 1
E    +  where 0 = <Repository at 0x7fb1a32dd650: id=23, organization_id=4557973193359376, name='getsentry/old-repo', provider='integrations:github'>.status
E    +  and   1 = ObjectStatus.DISABLED
tests/sentry/integrations/github/tasks/test_sync_repos_on_install_change.py::SyncReposOnInstallChangeTestCase::test_repos_removedlog
[gw0] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/sentry/integrations/github/tasks/test_sync_repos_on_install_change.py:84: in test_repos_removed
    assert repo.status == ObjectStatus.DISABLED
E   AssertionError: assert 0 == 1
E    +  where 0 = <Repository at 0x7fbefe1de780: id=21, organization_id=4557973192835088, name='getsentry/old-repo', provider='integrations:github'>.status
E    +  and   1 = ObjectStatus.DISABLED

@wedamija wedamija merged commit eb04f67 into master Apr 15, 2026
77 checks passed
@wedamija wedamija deleted the danf/scm-auto-sync-remove-flag branch April 15, 2026 18:10
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