destroy_all loads all records at once

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

Add documentation for bulk editing of issues and merge requests

Closes #41873

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

Ref contains oid check done by Gitaly

Closes gitaly#884

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

Fix bug with long strings in issue boards

Closes #43367

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

Fix setting Gitlab metrics content types

Closes gitlab-com/infrastructure#3499

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

Resolve "Add nip.io domain suggestion in the Kubernetes cluster details view"

Closes #45584

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

improve basic details on inspecting NFS mounts

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

add values to OpenID configuration test, not only checking for issuer key

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

Fix link for `query_recorder.rb`

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

fix runner requirements

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

The method "content_type" on a controller does not always return the
correct content type. On the other hand, the method "request_format"
does _and_ immediately returns a Symbol (e.g. :json) instead of a
mime-type name (e.g. application/json). With these changes metrics
should again report their action names correctly.

Fixes https://gitlab.com/gitlab-com/infrastructure/issues/3499

Add a unique and not null constraint on the project_features.project_id column

Closes #37882

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

Simplified priority, performance impact should be severity

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

Update Web IDE documentation with recent changes

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

Remove authentication for readonly endpoints in issues API

Closes #40855

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

fix typos Frontend Guide, development_process.md

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

Improve fast specs helper to autoload the library

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

It mentions that explicitly defining dependencies in sources is
preferred over defining such dependencies only in spec files.

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

Does not log failed sign-in attempts when the database is in read-only mode

Closes #46361

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

This commit has two migrations:

1. The first prunes duplicate rows in the project_features table and leaves
   the row with the highest ID.  Since the behavior was indeterministic before
   and depended on which row the database decided to use, this change at least
   makes the permissions consistent. For example, in some cases, the Wiki may
   have been disabled but enabled in another entry.

2. The second adds a non-null constraint on the project_features.project_id
   column.

Closes #37882

Fixes a significant part of gitlab-com/migration#408.

We found that we were overcounting Wikis because of these duplicates.
On GitLab.com, there are 56 rows with duplicate entries by project_id, and 16,661 rows with NULL project_id values.

Add index on ci_runners.runner_type

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

Merge branch '31662-remove-squashed-related-commits-together-from-default-mr-template' into 'master'

Resolve "Remove `Squashed related commits together` from default MR template"

Closes #31662

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

Disable greedy mode when matching filename

Closes #44784

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

We use this to filter runner and will use this increasingly in future
instead of is_shared so it should be indexed

Remove Spinach

Closes #23036

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

Backport Cluster related EE code

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

Disable RBAC on Ingress by default

Closes #46286

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

Signed-off-by: Rémy Coutable <remy@rymai.me>

Update wiki page deletion confirmation

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