delta/gitlab/gitlab-ce.git/app/services/groups/destroy_service.rb, branch docs-pages-https

Remove soft removals related code

2018-01-08T16:04:45+00:00

This removes all usage of soft removals except for the "pending delete"
system implemented for projects. This in turn simplifies all the query
plans of the models that used soft removals. Since we don't really use
soft removals for anything useful there's no point in keeping it around.

This _does_ mean that hard removals of issues (which only admins can do
if I'm not mistaken) can influence the "iid" values, but that code is
broken to begin with. More on this (and how to fix it) can be found in
https://gitlab.com/gitlab-org/gitlab-ce/issues/31114.

Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/37447

Make sure repository's removal work for legacy and hashed storages

2017-08-22T04:33:20+00:00

Remove Mattermost team with GitLab group

2017-07-28T07:09:55+00:00

When destroying a group, now an API call is made to the Mattermost
server to request the deletion of the project. Actual team deletion on
the Mattermost side happens async, so the runtime shouldn't increase by
more than a second.

Defer project destroys within a namespace in Groups::DestroyService#async_execute

2017-06-29T16:23:31+00:00

Group#destroy would actually hard-delete all associated projects even
though the acts_as_paranoia gem is used, preventing Projects::DestroyService
from doing any work.

We first noticed this while trying to log all projects deletion to the Geo
log.

Fix current feature related specs

2017-06-28T10:32:34+00:00

Make nested groups deletion synchronous to avoid potential race

2017-02-27T11:09:55+00:00

Fix a number of race conditions that can occur during namespace deletion

2017-02-16T07:56:40+00:00

There are two problems in the current implementation:

1. If a project is marked for deletion via the `pending_delete` flag
and then the namespace was quickly deleted, it's possible that the
namespace skips over that project and leaves that project in
an orphaned state.

2. Before namespace deletion, the namespace attempts to clean up
all the relevant storage paths. However, if all projects have been
removed synchronously, then the namespace will not be able to clean anything.
To prevent this, we should load the paths to be deleted before
actually destroying projects.

The specs were missing this second case due to a permission issue
that caused project removal never to happen.

Fix inconsistent naming for services that delete things

2017-02-08T08:16:43+00:00

* Changed name of delete_user_service and worker to destroy
* Move and change delete_group_service to Groups::DestroyService
* Rename Notes::DeleteService to Notes::DestroyService