gitlab-org--gitlab-foss/spec/workers
Stan Hu cb8a425ba4 Fix bug where destroying a namespace would not always destroy projects
There is a race condition in DestroyGroupService now that projects are deleted asynchronously:

1. User attempts to delete group
2. DestroyGroupService iterates through all projects and schedules a Sidekiq job to delete each Project
3. DestroyGroupService destroys the Group, leaving all its projects without a namespace
4. Projects::DestroyService runs later but the can?(current_user,
   :remove_project) is `false` because the user no longer has permission to
   destroy projects with no namespace.
5. This leaves the project in pending_delete state with no namespace/group.

Projects without a namespace or group also adds another problem: it's not possible to destroy the container
registry tags, since container_registry_path_with_namespace is the wrong value.

The fix is to destroy the group asynchronously and to run execute directly on Projects::DestroyService.

Closes #17893
2016-08-11 15:36:35 -07:00
..
repository_check Do not check repos without push events 2016-06-15 19:11:25 +02:00
build_email_worker_spec.rb
delete_user_worker_spec.rb
email_receiver_worker_spec.rb so we use separate classes to handle different tasks 2016-05-18 17:25:45 -05:00
emails_on_push_worker_spec.rb Fix emails on push for new and deleted branches 2016-07-21 10:53:38 +01:00
expire_build_artifacts_worker_spec.rb Make sure that artifacts_file is nullified after removing artifacts 2016-06-17 12:58:26 +02:00
git_garbage_collect_worker_spec.rb Refresh branch cache after git gc 2016-07-13 06:49:58 -07:00
group_destroy_worker_spec.rb Fix bug where destroying a namespace would not always destroy projects 2016-08-11 15:36:35 -07:00
merge_worker_spec.rb Unify check branch name exist 2016-06-20 21:44:21 +02:00
post_receive_spec.rb Pre-create all builds for Pipeline when a trigger is received 2016-08-11 15:22:35 +02:00
project_cache_worker_spec.rb Enable Style/EmptyLines cop, remove redundant ones 2016-07-01 21:56:17 +02:00
project_destroy_worker_spec.rb Fix skip_repo parameter being ignored when destroying a namespace 2016-08-03 17:07:38 -07:00
repository_fork_worker_spec.rb Fix a bug where forking a project from a repository storage to another would fail 2016-07-26 17:22:13 -04:00
repository_import_worker_spec.rb Mask credentials from URL when import of project has failed. 2016-05-18 21:16:36 -05:00
stuck_ci_builds_worker_spec.rb Don't fail builds for projects that are deleted when they are stuck 2016-06-12 15:15:58 +02:00