When filtering issues with a search string in a group, we observed on GitLab.com
that Postgres was using an inefficient query plan, preferring the (global)
trigram indexes on description and title, rather than using a filter on the
restricted set of issues within the group.

Change the callers of the IssuableFinder to use a CTE in this case to fence the
rest of the query from the LIKE filters, so that the optimiser is forced to
perform the filter in the order we prefer.

This will only force the use of a CTE when:

1. The use_cte_for_search params is truthy.
2. We are using Postgres.
3. We have passed the `search` param.

The third item is important - searching issues using the search box does not use
the finder in this way, but contructs a query and appends `full_search` to
that. For some reason, this query does not suffer from the same issue.

Currenly, we only pass this param when filtering issuables (issues or MRs) in a
group context.

We had `item_project_ids` to help make slow queries on the dashboard faster, but
this isn't necessary any more - the queries are plenty fast, and we forbid
searching the dashboard without filters.

Bump grape-path-helpers to solve a bug in GrapePathHelpers::NamedRouteMatcher

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

# Conflicts:
#   Gemfile.lock

CE upstream - 2018-06-06 15:12 UTC

Closes #4811 and gitlab-ce#47280

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

Moving rev-list lfs options to Lfschanges

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

[EE] update net-ssh 4 -> 5, remove rbnacl, rbnacl-libsodium (not needed)

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

Port of 43597-new-navigation-themes to EE

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

If you include this helper with a class that also implements `method_missing`,
this would result in AbstractController::DoubleRenderError.

See https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/19487#note_79096902.

Remove redundant has_internal_id

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

 Conflicts:
   app/models/ci/pipeline.rb

Fix related issues top padding

Closes #6330

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

Fix export issues modal

Closes #6319

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

[ci skip]

[ci skip]

Merge branch '6014-geo-fix-index-for-outdated-projects-on-the-project-repository-states' into 'master'

Resolve "Geo - Revisit the indexes on the project_repository_states table"

Closes #6014

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

Member tabs EE port

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

Docs: Add EKS integration tutorial

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

Add EKS integration tutorial

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

Fix command in license management example

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

Re-add an important '/' that was inadvertently removed

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

# Conflicts:
#	app/views/projects/project_members/index.html.haml
#	config/routes/api.rb
#	doc/api/namespaces.md
#	doc/api/protected_branches.md
#	doc/ci/variables/README.md
#	doc/system_hooks/system_hooks.md
#	doc/user/project/protected_branches.md
#	lib/api/protected_branches.rb
#	locale/gitlab.pot

[ci skip]

Resolve "Rename the `Master` role to `Maintainer`"

Closes #42751

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

Resolve "Rename the `Master` role to `Maintainer`"

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

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

Move GC/Repack to OptOut

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