As previously implemented the code was technically buggy and
could cause confusion in the future. That is because the call
to self.identity, if an update were necessary, would fail to
update user identities association and would not be auto-saved.
Actually, it wouldn't be possible for the identity to be found
in this manner if the extern_uid had changed. Make it clearer
by using a guard to indicate nothing is happening in this case.

Changelog: fixed

Fix labels picker text overflow

See merge request gitlab-org/gitlab!61349

Fix N+1 in issues API when loading parent epics

See merge request gitlab-org/gitlab!61852

Prepare DevOps Adoption DB structure for migration

See merge request gitlab-org/gitlab!60733

Send email campaign usage data

See merge request gitlab-org/gitlab!56752

E2E: Validate subgroup migration

See merge request gitlab-org/gitlab!61446

Add link to MR when draft status changed

See merge request gitlab-org/gitlab!61891

Add duplicated gin trigram index on notes note

See merge request gitlab-org/gitlab!61430

Add histogram and track latency

See merge request gitlab-org/gitlab!61582

Enable snippets to use reference filter cache

See merge request gitlab-org/gitlab!61076

Turn `vue_issuables_list` FF off for feature specs

See merge request gitlab-org/gitlab!61627

Namespace `ChatMessage` classes as `Integrations::`

See merge request gitlab-org/gitlab!61559

Edited Epics topic for CTRT

See merge request gitlab-org/gitlab!61303

New structure will separate "display namespace" and
"target namespace" concepts for devops adoption

Changelog: added

Enable api_caching_rate_limit_branches by default [RUN ALL RSPEC] [RUN AS-IF-FOSS]

See merge request gitlab-org/gitlab!61723

Update Gitaly version

See merge request gitlab-org/gitlab!61908

We identified that the gin trigram index on notes (note)
can lead to very slow insert times on notes. It appears that
reindexing the index solves the problem. We have decided to
reindex it through migrations, first by creating a duplicate
index, then by dropping the old index.

Original MR thread: gitlab.com/gitlab-org/gitlab/-/issues/218410#note_565624409
Change MR: gitlab.com/gitlab-org/gitlab/-/merge_requests/61430

Mount vulnerability list on pipeline dashboard

See merge request gitlab-org/gitlab!61536

Fix spelling of Action Cable

See merge request gitlab-org/gitlab!61804

Observe "limit to hours" for issuable time tracking report

See merge request gitlab-org/gitlab!61849

- update test
- update translations

Merge branch '325286-write-documentation-for-running-gitlab-pages-without-nfs-in-beta-2' into 'master'

Add guide for migrating GitLab Pages to 14.0

See merge request gitlab-org/gitlab!60107

Fix incorrect attributions of audit events feature

See merge request gitlab-org/gitlab!61755

Remove .processed scope from package model and finders

See merge request gitlab-org/gitlab!61732

Resolve "Fix inconsistent drag and drop message in Markdown and Designs"

See merge request gitlab-org/gitlab!61782

Remove issuable destroy service related FFs [RUN ALL RSPEC] [RUN AS-IF-FOSS]

See merge request gitlab-org/gitlab!61764

Merge branch '325286-write-documentation-for-running-gitlab-pages-without-nfs-in-beta' into 'master'

Document disabling pages local_store

See merge request gitlab-org/gitlab!59980

Reindexing: Lock-based retries for DROP INDEX

See merge request gitlab-org/gitlab!61110

This avoids a long running query: Instead of setting a high statement
timeout, we use rather long lock timeouts and retries in between
attempts. This should gracefully resolve the situation when a vacuum
operation is blocking the `DROP INDEX` statement.

Still, if we run out of retries, we force the execution
without lock timeout (and potentially see a long running query then).

Relates to https://gitlab.com/gitlab-org/gitlab/-/issues/329724