Merge branch '241138-remove-the-ci_expand_names_for_cross_pipeline_artifacts-in-gitlab-ci-features' into 'master'

Remove FF helper method ci_expand_names_for_cross_pipeline_artifacts?

Closes #241138

See merge request gitlab-org/gitlab!40385

Closure policy for inactive bugs

See merge request gitlab-org/gitlab!39693

Refactor(VSA): Separate query and state management

See merge request gitlab-org/gitlab!39603

Separate filter bar store and actions from app
store and actions to prepare for a general
analytics filter bar solution

Use url-sync component instead of mixin to
separate concerns between base and filter-bar

Rename service model scopes

See merge request gitlab-org/gitlab!40418

Create security_on_demand_scans_scanner_profiles feature flag

See merge request gitlab-org/gitlab!39250

Update merge request diff files_count cache when removing diffs

Closes #240921

See merge request gitlab-org/gitlab!40384

Move Pages directory in Sidekiq when renaming a project

See merge request gitlab-org/gitlab!40087

Specify custom path for Vault auth method

Closes #241130

See merge request gitlab-org/gitlab!40366

Fixed batch comments styles not loading in FOSS

Closes #240235

See merge request gitlab-org/gitlab!40479

Closes https://gitlab.com/gitlab-org/gitlab/-/issues/240235

Override the parallel diff lines property with new data

See merge request gitlab-org/gitlab!40131

Doing it this way allows us to:
1. Allow all existing code to work without changing a lot
2. Migrate all the code away from this property
once this feature has been fully rolled out

https://gitlab.com/gitlab-org/gitlab/-/issues/236143

Update factory doc with common change information

See merge request gitlab-org/gitlab!40308

The files_count cache column was created with the assumption that the
number of files never changes throughout the life of a diff. However,
we have discovered the DeleteDiffFilesWorker, which removes them (and
sets the state of the diff to :without_files) in some limited cases.

This MR updates the cache columns (both commits_count and files_count)
so they will be re-recorded on every update. Since updates are rare,
the additional cost of this should be very low.

Improve error message for job configs that fail to match an entry

See merge request gitlab-org/gitlab!40120

This is behind the async_pages_move_project_rename feature flag. When
it's enabled and a project is renamed, we do not move the Pages
directory in Puma - instead, we schedule a Sidekiq job to perform the
move.

We already weren't checking the return value of
`Gitlab::PagesTransfer#rename_project`, so this shouldn't introduce any
failure cases that weren't there already.

Fix: already initialized constant warnings for HTTPError

Closes #239380

See merge request gitlab-org/gitlab!40142

Currently Vault Secrets are using partially hardcoded configuration
for the authentication method. While the Vault server URL and the
role name for JWT authentication method can be configured with the
variables, the authentication method path is hardoced to `jwt`.
This may limit the usability of our solution.

This change makes this value configurable with the `VAULT_AUTH_PATH`
variable (similar to how `VAULT_SERVER_URL` and `VAULT_AUTH_ROLE`
are being used already) and ensures that in case when the variable
is not defined by the user, it will fall-back to the `jwt` value
that we have hardcoded now.

Make Gitlab::PagesTransfer possible to call async

See merge request gitlab-org/gitlab!39856

Update the jobs timings according to the latest data

See merge request gitlab-org/gitlab!39978

Ensure issue status counts are scoped

Closes #238569

See merge request gitlab-org/gitlab!39914

Add tests to ensure this

Refactor file locking docs

See merge request gitlab-org/gitlab!35212

Fix underscores vs hyphens content in Doc Style Guide

See merge request gitlab-org/gitlab!38099

Add note for how all CE applies to EE

See merge request gitlab-org/gitlab!38931

Use active user count instead of historical max

Closes #224681

See merge request gitlab-org/gitlab!37916

"User count threshold" banner should use
active user count instead of historical max
to reflect the current state.

The worker specs should just test the worker's behaviour, which is
calling the service. The service specs should test the operations on
disk.

When we rename a project, we call Gitlab::PagesTransfer to move any
pages directory on disk. This makes it convenient to use that class
through Sidekiq instead, so that Puma no longer needs access to Pages
NFS mounts.

Fix a link to the Slash commands page

See merge request gitlab-org/gitlab!40480

Fix all_list board param

See merge request gitlab-org/gitlab!40006

When filtering board issues, EE version of the list_service can
add additional filters which were not covered in CE version of the
method.

Replace isVueInstance with alternate options

See merge request gitlab-org/gitlab!40218