Skip to content
Snippets Groups Projects

Repo Sync: If all errors are warnings, delete repos with bad permissions

Merged Warren Gifford requested to merge iv/fix-non-fatal-errors-stopping-sync into main

Created by: varsanojidan

Fixes https://github.com/sourcegraph/sourcegraph/issues/40449

If all errors during repo sync are warnings of either Unauthorized/Forbidden, we should remove the related repos. Warning errors are generally reserved for smaller portions of repo-syncs (i.e. just bitbucket projectKeys atm), and if all the errors during a sync are just permissions based (401 or 403) its fair to assume that there was a change of permissions and we can properly complete the sync and delete any not-seen repos.

Test plan

Tested manually against bitbucket.sgdev.org

Merge request reports

Merged by avatar (Apr 22, 2025 7:15am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading