[ci skip]

Fix an order-dependent spec failure in spec/migrations/schedule_sync_issuables_state_id_spec.rb

Closes #60029

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

Change the way it is checked if the user is last group owner

Closes #38564

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

Chnage method used in model to make it
more efficient database-wise
Add additional spec

Revert "Merge branch 'fix/missing-border' into 'master'"

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

Resolve "Package repository group endpoint URL encoding"

Closes #10832

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

Monitor GraphQL with Prometheus (try 2)

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

Extends graphql's platform tracing class to observe duration
of graphql methods.

In graphql 1.8.11 is added prometheus class but it's not very useful
for us because it uses prometheus_exporter to export results.

Exempt release automation MRs from Danger rules

Closes gitlab-org/release/framework#269

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

Support multi-line suggestions

Closes #53310

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

- added suggestions to mock data
- fixed props to be not required

Fix pipeline bridge serialization error

Closes gitlab-ce#59221

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

Backport: Ignore merge if the status of the merge request pipeline is stale

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

Remove EE-specific parts from non-EE BoardSidebar component

Closes #10010

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

Fix webpack dev-server crash caused by 1.5gb limit

Closes #59075

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

Speed up avatar URLs with object storage

Closes #43065

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

Add v2 to reserved top level routes

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

Add v2 to reserved top level routes [EE]

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

Make EE prometheus args behavior default -> metrics time window

Closes #10894

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

GraphQL Type authorization

Closes #54417

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

Fix user agent string for Hosted Jira

Closes #10737

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

Add testing to feature proposal template

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

Filters branch and path labels for metrics

Closes #48090

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

With object storage enabled, calling `#filename` on an upload does this:

1. Call the `#filename` method on the CarrierWave object.
2. Generate the URL for that object.
3. If the uploader isn't public, do so by generating an authenticated
   URL, including signing that request.

That's all correct behaviour, but for the case where we use `#filename`,
it's typically to generate a GitLab URL. That URL doesn't need to be
signed because we do our own auth.

Signing the URLs can be very expensive, especially in batch (say, we
need to get the avatar URLs for 150 users in one request). It's all
unnecessary work. If we used the `RecordsUploads` concern, we have
already recorded a `path` in the database. That `path` is actually
generated from CarrierWave's `#filename` at upload time, so we don't
need to recompute it - we can just use it and strip off the prefix if
it's available.

On a sample users autocomplete URL, at least 10% of the time before this
change went to signing URLs. After this change, we spend no time in URL
signing, and still get the correct results.