ObjectStore uploader requires presence of associated `uploads` record
when deleting the upload file (through the carrierwave's after_commit
hook) because we keep info whether file is LOCAL or REMOTE in `upload`
object.

For this reason we can not destroy uploads as "dependent: :destroy" hook
because these would be deleted too soon. Instead we rely on
carrierwave's hook to destroy `uploads` in after_commit hook.

But in before_destroy hook we still have to delete not-mounted uploads
(which don't use carrierwave's destroy hook). This has to be done in
before_Destroy instead of after_commit because `FileUpload` requires
existence of model's object on destroy action.

This is not ideal state of things, in a next step we should investigate
how to unify model dependencies so we can use same workflow for all
uploads.

Related to #45425

5943 improve license management tests

Closes #5943

See merge request gitlab-org/gitlab-ee!5718

CE upstream - 2018-05-15 09:46 UTC

See merge request gitlab-org/gitlab-ee!5711

Does not rescue from GRPC::DataLoss errors

See merge request gitlab-org/gitlab-ee!5716

Geo: Fix repo, wiki, and upload replication when renaming a namespace that has subgroups, without hashed storage enabled

Closes #6012

See merge request gitlab-org/gitlab-ee!5704

EE: Display help text below auto devops domain with nip.io domain name (#45561)

See merge request gitlab-org/gitlab-ee!5691

This limit is (and already was) different in compare to CE
version, bumping this now to reflect recent change introduced by
gitlab-ce!18735.

Resolve "Interact with security reports should be marked as 'alpha' instead of 'experimental'"

Closes #6016

See merge request gitlab-org/gitlab-ee!5708

Merge branch '46361-does-not-log-failed-sign-in-attempts-when-the-database-is-in-read-only-mode-ee' into 'master'

Port of 46361-does-not-log-failed-sign-in-attempts-when-the-database-is-in-read-only-mode to EE

See merge request gitlab-org/gitlab-ee!5703

# Conflicts:
#	.gitlab-ci.yml
#	doc/development/testing_guide/ci.md

[ci skip]

Remove Spinach

Closes #23036

See merge request gitlab-org/gitlab-ce!18869

[CE port] Remove Spinach

See merge request gitlab-org/gitlab-ee!5688

Improve security dismissal documentation

See merge request gitlab-org/gitlab-ee!5705

Don't rebase if not needed in ee-specific-lines-check

See merge request gitlab-org/gitlab-ee!5698

Add forgotten IgnorableColumn in Geo::RepositoryDeletedEvent

See merge request gitlab-org/gitlab-ee!5710

Backport Cluster related EE code

See merge request gitlab-org/gitlab-ce!18931

Resolve "Extract EE specific files/lines for clusters related files"

Closes #5968

See merge request gitlab-org/gitlab-ee!5687

Disable RBAC on Ingress by default

Closes #46286

See merge request gitlab-org/gitlab-ce!18947

Update wiki page deletion confirmation

See merge request gitlab-org/gitlab-ce!18712

Enable update_(build|pipeline) for maintainers

Closes #45905

See merge request gitlab-org/gitlab-ce!18735

In gitlab-org/gitlab-ee!5389 the use of repository_storage_path was
removed. We ignored those columns in the geo event log tables, but for
the `geo_repository_deleted_events` table, this was forgotten.

gitlab-org/gitlab-ee!5468 is prepared to actually remove the columns
with a migration.

Fix link to MR for Maintainer Access docs

See merge request gitlab-org/gitlab-ce!18951

Fix minor typo of main docs page

See merge request gitlab-org/gitlab-ce!18952

Remove license management docs

See merge request gitlab-org/gitlab-ee!5706

when the namespace has subgroups with projects.

Resolve "Link for interacting with security reports is wrong"

Closes #6015

See merge request gitlab-org/gitlab-ee!5707

Geo - Calculate the checksum for outdated projects with less activity first

See merge request gitlab-org/gitlab-ee!5702