Removes unnecessary attributes and
adds snapshot spec.

Merge branch '376089-follow-up-to-remove-the-use-of-vulnerability-states-somesymbol-from-service-classes' into 'master'

Remove use of Vulnerability.states[:somesymbol] from service classes

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126772



Merged-by: Brett Walker <bwalker@gitlab.com>
Approved-by: Brett Walker <bwalker@gitlab.com>
Approved-by: Sylvester Chin <schin@gitlab.com>
Co-authored-by: Michael Becker <11881043-wandering_person@users.noreply.gitlab.com>

We can just use the enum states directly as symbols rather than
calling `.states` on the class

This was brought up in a comment on another MR ([here][0]):

> I noticed a lot of indexing and explicit references to the
> Vulnerability State enum which Rails will conveniently magic away for
> us anyways. I've added in suggestions to let Rails handle that
> behaviour more for us

This commit removes those aforementioned explicit usages of the enum

[0]:https://gitlab.com/gitlab-org/gitlab/-/merge_requests/99142#note_1118961906

Update summarize diff prompt to return more correct responses

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126683



Merged-by: Kerri Miller <kerrizor@kerrizor.com>
Approved-by: Patrick Bajao <ebajao@gitlab.com>
Approved-by: Kerri Miller <kerrizor@kerrizor.com>
Reviewed-by: Patrick Bajao <ebajao@gitlab.com>
Co-authored-by: Phil Hughes <me@iamphill.com>

Autofocus the editor when commenting on a diff

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126978



Merged-by: Phil Hughes <me@iamphill.com>
Approved-by: Phil Hughes <me@iamphill.com>
Co-authored-by: Himanshu Kapoor <hkapoor@gitlab.com>

Allow pasting a markdown block

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126993



Merged-by: Enrique Alcántara <ealcantara@gitlab.com>
Approved-by: Enrique Alcántara <ealcantara@gitlab.com>
Co-authored-by: Himanshu Kapoor <hkapoor@gitlab.com>

Add allowed_to_be_slow to RSpec JSON formatter

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126959



Merged-by: Brett Walker <bwalker@gitlab.com>
Approved-by: Brett Walker <bwalker@gitlab.com>
Approved-by: Dzmitry Meshcharakou <12459192-dmeshcharakou@users.noreply.gitlab.com>
Reviewed-by: Dzmitry Meshcharakou <12459192-dmeshcharakou@users.noreply.gitlab.com>
Co-authored-by: Alina Mihaila <amihaila@gitlab.com>

Update View runner statistics topic to make median info more prominent

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/127025



Merged-by: Marcin Sedlak-Jakubowski <msedlakjakubowski@gitlab.com>
Approved-by: Marcin Sedlak-Jakubowski <msedlakjakubowski@gitlab.com>
Co-authored-by: Fiona Neill <fneill@gitlab.com>

Add elastic doc for BackfillArchivedOnWorkItems

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/127029



Merged-by: Mayra Cabrera <mcabrera@gitlab.com>
Approved-by: Mayra Cabrera <mcabrera@gitlab.com>
Co-authored-by: Terri Chu <tchu@gitlab.com>

Test scaling security->canonical sync with a fast CI

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/125862



Merged-by: Jennifer Li <jli@gitlab.com>
Approved-by: Reuben Pereira <2967854-rpereira2@users.noreply.gitlab.com>
Approved-by: Jennifer Li <jli@gitlab.com>
Approved-by: Nao Hashizume <nhashizume@gitlab.com>
Reviewed-by: Reuben Pereira <2967854-rpereira2@users.noreply.gitlab.com>
Reviewed-by: Nao Hashizume <nhashizume@gitlab.com>
Co-authored-by: Reuben Pereira <2967854-rpereira2@users.noreply.gitlab.com>
Co-authored-by: Alessio Caiazza <acaiazza@gitlab.com>

doc/api: Clarify group storage moves API use case

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/127019



Merged-by: Ashraf Khamis <akhamis@gitlab.com>
Approved-by: Ashraf Khamis <akhamis@gitlab.com>
Co-authored-by: Justin Tobler <jtobler@gitlab.com>

The group storage moves API is used to migrate group wiki repositories
from one storage to another. Projects contained within a group are not
migrated as part of this operation. In the documentation, this is not
immediately clear to the reader and it would be reasonable to assume
that moving a group to a new storage would also move all associated
project repositories.

Clarify the group storage moves API documentation introduction to
explicitly explain that a group's project repositories are not moved
through the use of this API.

Redirecting SaaS to SM comparison

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/125936



Merged-by: Phillip Wells <pwells@gitlab.com>
Approved-by: Phillip Wells <pwells@gitlab.com>
Co-authored-by: Suzanne Selhorn <sselhorn@gitlab.com>

Related to: https://gitlab.com/gitlab-org/gitlab/-/issues/416707

PlantUML troubleshooting section

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/124806



Merged-by: Marcin Sedlak-Jakubowski <msedlakjakubowski@gitlab.com>
Approved-by: Marcin Sedlak-Jakubowski <msedlakjakubowski@gitlab.com>
Co-authored-by: John Gaughan <jgaughan@gitlab.com>

Add a link to a click-through demo

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126958



Merged-by: Suzanne Selhorn <sselhorn@gitlab.com>
Co-authored-by: Itzik Gan Baruch <iganbarouch@gitlab.com>

Use Usage Statistics Component in Statistics Seats Card

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/125658



Merged-by: Vitaly Slobodin <vslobodin@gitlab.com>
Approved-by: Valerie Burton <vburton@gitlab.com>
Approved-by: Tim Noah <tnoah@gitlab.com>
Approved-by: Vitaly Slobodin <vslobodin@gitlab.com>
Approved-by: Kos Palchyk <kpalchyk@gitlab.com>
Reviewed-by: Angelo Gulina <agulina@gitlab.com>
Reviewed-by: Kos Palchyk <kpalchyk@gitlab.com>
Reviewed-by: Valerie Burton <vburton@gitlab.com>
Reviewed-by: Ankit Panchal <apanchal@gitlab.com>
Co-authored-by: Angelo Gulina <agulina@gitlab.com>

Add guideline for hyphens in titles

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126982



Merged-by: Suzanne Selhorn <sselhorn@gitlab.com>
Co-authored-by: Lorena Ciutacu <lciutacu@gitlab.com>

Adds target_scheduling_duration_s in Sidekiq logs

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126994



Merged-by: Douglas Barbosa Alexandre <dbalexandre@gmail.com>
Approved-by: Douglas Barbosa Alexandre <dbalexandre@gmail.com>
Co-authored-by: Gregorius Marco <gmarco@gitlab.com>

Fix links in Geo documentation code comments

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126927



Merged-by: Suzanne Selhorn <sselhorn@gitlab.com>
Co-authored-by: Evan Read <eread@gitlab.com>

Expand /completions endpoint to support Self-Managed

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/125563



Merged-by: Matthias Käppler <mkaeppler@gitlab.com>
Approved-by: Matthias Käppler <mkaeppler@gitlab.com>
Reviewed-by: Matthias Käppler <mkaeppler@gitlab.com>
Reviewed-by: Roy Zwambag <rzwambag@gitlab.com>
Co-authored-by: Roy Zwambag <rzwambag@gitlab.com>

This MR updates the completions endpoint to allow self-managed users to
use it as well.

Changelog: other
EE: true

Clear GroupIndexStatus in clear_index_status rake task

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/127007



Merged-by: Dmitry Gruzd <dgruzd@gitlab.com>
Approved-by: Dmitry Gruzd <dgruzd@gitlab.com>
Co-authored-by: Terri Chu <tchu@gitlab.com>

Hide analytics dashboards from guest

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126388



Merged-by: Luke Duncalfe <lduncalfe@gitlab.com>
Approved-by: Joe Woodward <jwoodward@gitlab.com>
Approved-by: Luke Duncalfe <lduncalfe@gitlab.com>
Reviewed-by: Luke Duncalfe <lduncalfe@gitlab.com>
Reviewed-by: Halil Coban <hcoban@gitlab.com>
Co-authored-by: Halil Coban <hcoban@gitlab.com>

Ensure components are ingested in a consistent order

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126898



Merged-by: Mehmet Emin INAC <minac@gitlab.com>
Approved-by: mo khan <mo@mokhan.ca>
Approved-by: Mehmet Emin INAC <minac@gitlab.com>
Co-authored-by: Brian Williams <bwilliams@gitlab.com>

Removes namespace_limits_admin_dashboard FF

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126547



Merged-by: Madelein van Niekerk <mvanniekerk@gitlab.com>
Approved-by: Prabakaran Murugesan <pmurugesan@gitlab.com>
Approved-by: Madelein van Niekerk <mvanniekerk@gitlab.com>
Reviewed-by: Kos Palchyk <kpalchyk@gitlab.com>
Co-authored-by: Vijay Hawoldar <vhawoldar@gitlab.com>

Backfill archived for work_items

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126714



Merged-by: Terri Chu <tchu@gitlab.com>
Approved-by: Terri Chu <tchu@gitlab.com>
Approved-by: Ravi Kumar <rkumar@gitlab.com>
Reviewed-by: Terri Chu <tchu@gitlab.com>
Co-authored-by: Dmitry Gruzd <dgruzd@gitlab.com>

EE: true
Changelog: changed

Refactor audit events for PasswordsController

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/123187



Merged-by: Harsimar Sandhu <hsandhu@gitlab.com>
Approved-by: Harsimar Sandhu <hsandhu@gitlab.com>
Approved-by: Smriti Garg <sgarg@gitlab.com>
Co-authored-by: Aaron Huntsman <ahuntsman@gitlab.com>

- set audit scope to ::Gitlab::Audit::UnauthenticatedAuthor
  when password reset is requested for nonexistent email
- use have_attributes in specs

Remove the ignore rule for chat_names. integration_id column

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/126186



Merged-by: Omar Qunsul <oqunsul@gitlab.com>
Approved-by: Vasilii Iakliushin <viakliushin@gitlab.com>
Approved-by: Carla Drago <cdrago@gitlab.com>
Approved-by: Omar Qunsul <oqunsul@gitlab.com>
Reviewed-by: Vasilii Iakliushin <viakliushin@gitlab.com>
Reviewed-by: Omar Qunsul <oqunsul@gitlab.com>
Reviewed-by: Bojan Marjanovic <bmarjanovic@gitlab.com>
Co-authored-by: bmarjanovic <bmarjanovic@gitlab.com>

It removes ignored column `integration_id` from the `chat_names` table.

Changelog: changed