Handle project deletions in Geo::LogCursor
The simplest change here is to remove projects with the cursor, and this duplicates the current functionality used in system hooks.
However, until we solve gitlab-org/gitlab-ce#28283, there's a chance we could accidentally delete another project that was created with this name.
Merge request reports
Activity
changed milestone to %9.5
@dbalexandre Can you do an initial review here? I'm going to hold off on merging this until we know more about gitlab-org/gitlab-ce#28283.
assigned to @dbalexandre
- Resolved by Stan Hu
- Resolved by Stan Hu
- Resolved by Stan Hu
@stanhu Thanks! The code looks good! I left some minor comments.
assigned to @stanhu
- Resolved by Stan Hu
added 381 commits
- 8a6f510d...b6326824 - 379 commits from branch
master
- 08aaebaa - Handle project deletions in Geo::LogCursor
- 7619ed3e - Add specs for deleting projects via Geo::LogCursor
- 8a6f510d...b6326824 - 379 commits from branch
added 155 commits
- 90521d63...c96dadf9 - 154 commits from branch
master
- bc3ec5d9 - Handle project deletions in Geo::LogCursor
- 90521d63...c96dadf9 - 154 commits from branch
assigned to @rspeicher
assigned to @stanhu
- Resolved by Stan Hu
- Resolved by Stan Hu
added 526 commits
- bc3ec5d9...5a9b6b5e - 525 commits from branch
master
- b1288722 - Handle project deletions in Geo::LogCursor
- bc3ec5d9...5a9b6b5e - 525 commits from branch
assigned to @rspeicher
- Resolved by Robert Speicher
@stanhu LGTM, just needs a rebase.
assigned to @stanhu
assigned to @rspeicher
mentioned in commit 6b2fde7e
mentioned in issue #2827 (closed)