Skip to content

Commit 6918db8

Browse files
author
GitLab Bot
committed
Add latest changes from gitlab-org/gitlab@master
1 parent 051d825 commit 6918db8

File tree

25 files changed

+232
-242
lines changed

25 files changed

+232
-242
lines changed

app/assets/javascripts/ci/mixins/pipeline_variables_permissions_mixin.js

Lines changed: 0 additions & 62 deletions
This file was deleted.

app/assets/javascripts/ci/pipeline_new/components/pipeline_new_form.vue

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import { createAlert } from '~/alert';
77
import glFeatureFlagsMixin from '~/vue_shared/mixins/gl_feature_flags_mixin';
88
import SafeHtml from '~/vue_shared/directives/safe_html';
99
import PipelineInputsForm from '~/ci/common/pipeline_inputs/pipeline_inputs_form.vue';
10-
import PipelineVariablesPermissionsMixin from '~/ci/mixins/pipeline_variables_permissions_mixin';
1110
import createPipelineMutation from '../graphql/mutations/create_pipeline.mutation.graphql';
1211
import RefsDropdown from './refs_dropdown.vue';
1312
import PipelineVariablesForm from './pipeline_variables_form.vue';
@@ -38,9 +37,10 @@ export default {
3837
import('ee_component/vue_shared/components/pipeline_account_verification_alert.vue'),
3938
},
4039
directives: { SafeHtml },
41-
mixins: [glFeatureFlagsMixin(), PipelineVariablesPermissionsMixin],
40+
mixins: [glFeatureFlagsMixin()],
4241
inject: [
4342
'canViewPipelineEditor',
43+
'canSetPipelineVariables',
4444
'pipelineEditorPath',
4545
'pipelinesPath',
4646
'projectPath',
@@ -266,7 +266,7 @@ export default {
266266
@update-inputs="handleInputsUpdated"
267267
/>
268268
<pipeline-variables-form
269-
v-if="canViewPipelineVariables"
269+
v-if="canSetPipelineVariables"
270270
:file-params="fileParams"
271271
:is-maintainer="isMaintainer"
272272
:project-path="projectPath"

app/assets/javascripts/ci/pipeline_new/index.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ const mountPipelineNewForm = (el) => {
1414
projectPath,
1515
projectRefsEndpoint,
1616
userRole,
17+
canSetPipelineVariables,
1718

1819
// props
1920
defaultBranch,
@@ -39,6 +40,7 @@ const mountPipelineNewForm = (el) => {
3940
apolloProvider,
4041
provide: {
4142
canViewPipelineEditor: parseBoolean(canViewPipelineEditor),
43+
canSetPipelineVariables: parseBoolean(canSetPipelineVariables),
4244
identityVerificationPath,
4345
// Normally this will have a value from a helper. In this case, this is
4446
// set to true because the alert that uses this field is dynamically

app/helpers/ci/pipelines_helper.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,8 @@ def new_pipeline_data(project)
9090
project_refs_endpoint: refs_project_path(project, sort: 'updated_desc'),
9191
settings_link: project_settings_ci_cd_path(project),
9292
max_warnings: ::Gitlab::Ci::Warnings::MAX_LIMIT,
93-
user_role: project.team.human_max_access(current_user&.id)
93+
user_role: project.team.human_max_access(current_user&.id),
94+
can_set_pipeline_variables: Ability.allowed?(current_user, :set_pipeline_variables, project).to_s
9495
}
9596
end
9697

app/models/ci/resource_group.rb

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ class ResourceGroup < Ci::ApplicationRecord
1717
enum process_mode: {
1818
unordered: 0,
1919
oldest_first: 1,
20-
newest_first: 2
20+
newest_first: 2,
21+
newest_ready_first: 3
2122
}
2223

2324
##
@@ -53,6 +54,9 @@ def upcoming_processables
5354
elsif newest_first?
5455
processables.waiting_for_resource_or_upcoming
5556
.order(Arel.sql("commit_id DESC, #{sort_by_job_status}"))
57+
elsif newest_ready_first?
58+
processables.waiting_for_resource
59+
.order(Arel.sql("commit_id DESC, #{sort_by_job_status}"))
5660
else
5761
Ci::Processable.none
5862
end

db/docs/batched_background_migrations/backfill_issue_email_participants_namespace_id.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,4 +5,4 @@ feature_category: service_desk
55
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/174506
66
milestone: '17.7'
77
queued_migration_version: 20241203075620
8-
finalized_by: # version of the migration that finalized this BBM
8+
finalized_by: '20250421231526'
Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# frozen_string_literal: true
2+
3+
class FinalizeHkBackfillIssueEmailParticipantsNamespaceId < Gitlab::Database::Migration[2.2]
4+
milestone '18.0'
5+
6+
disable_ddl_transaction!
7+
8+
restrict_gitlab_migration gitlab_schema: :gitlab_main_cell
9+
10+
def up
11+
ensure_batched_background_migration_is_finished(
12+
job_class_name: 'BackfillIssueEmailParticipantsNamespaceId',
13+
table_name: :issue_email_participants,
14+
column_name: :id,
15+
job_arguments: [:namespace_id, :issues, :namespace_id, :issue_id],
16+
finalize: true
17+
)
18+
end
19+
20+
def down; end
21+
end
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
ea58f13a7a8e80d42684aeec2795ee912687951bd58580658a720255564f7c78

doc/administration/settings/slack_app.md

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -136,6 +136,26 @@ To enable the GitLab for Slack app functionality, your network must allow inboun
136136
- For [Slack notifications](../../user/project/integrations/gitlab_slack_application.md#slack-notifications), the GitLab instance must be able to send requests to `https://slack.com`.
137137
- For [Slash commands](../../user/project/integrations/gitlab_slack_application.md#slash-commands) and other features, the GitLab instance must be able to receive requests from `https://slack.com`.
138138

139+
## Enable support for multiple workspaces
140+
141+
By default, you can [install the GitLab for Slack app](../../user/project/integrations/gitlab_slack_application.md#install-the-gitlab-for-slack-app) in only one Slack workspace.
142+
An administrator selects this workspace when they [create a GitLab for Slack app](#create-a-gitlab-for-slack-app).
143+
144+
To enable support for multiple Slack workspaces, you must configure the GitLab for Slack app
145+
as an [unlisted distributed app](https://api.slack.com/distribution#unlisted-distributed-apps).
146+
An unlisted distributed app:
147+
148+
- Is not published in the Slack App Directory.
149+
- Can only be used with your GitLab instance and not by other sites.
150+
151+
To configure the GitLab for Slack app as an unlisted distributed app:
152+
153+
1. Go to the [**Your Apps**](https://api.slack.com/apps) page on Slack and select your GitLab for Slack app.
154+
1. Select **Manage Distribution**.
155+
1. In the **Share Your App with Other Workspaces** section, expand **Remove Hard Coded Information**.
156+
1. Select the **I've reviewed and removed any hard-coded information** checkbox.
157+
1. Select **Activate Public Distribution**.
158+
139159
## Troubleshooting
140160

141161
When administering the GitLab for Slack app, you might encounter the following issues.

doc/api/openapi/openapi_v2.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49771,6 +49771,7 @@ definitions:
4977149771
- unordered
4977249772
- oldest_first
4977349773
- newest_first
49774+
- newest_ready_first
4977449775
description: Edit an existing resource group
4977549776
postApiV4ProjectsIdRunners:
4977649777
type: object

0 commit comments

Comments
 (0)