Skip to content

Commit b9b0853

Browse files
author
GitLab Bot
committed
Add latest changes from gitlab-org/gitlab@master
1 parent b6611ae commit b9b0853

25 files changed

+171
-104
lines changed

app/assets/javascripts/vue_merge_request_widget/components/deployment/deployment_action_button.vue

+2-2
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ export default {
6666
:aria-label="buttonTitle"
6767
:loading="isLoading"
6868
:disabled="isActionInProgress"
69-
:class="`gl-ml-3 ${containerClasses}`"
69+
:class="containerClasses"
7070
:icon="icon"
7171
@click="$emit('click')"
7272
>
@@ -81,7 +81,7 @@ export default {
8181
:aria-label="buttonTitle"
8282
:loading="isLoading"
8383
:disabled="isActionInProgress"
84-
:class="`gl-ml-3 ${containerClasses}`"
84+
:class="containerClasses"
8585
:icon="icon"
8686
@click="$emit('click')"
8787
>

app/assets/javascripts/vue_merge_request_widget/components/deployment/deployment_actions.vue

+1-1
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ export default {
151151
</script>
152152
153153
<template>
154-
<div class="gl-inline-flex">
154+
<div class="gl-inline-flex gl-gap-3">
155155
<deployment-action-button
156156
v-if="canBeManuallyDeployed"
157157
:action-in-progress="actionInProgress"

app/assets/stylesheets/framework/common.scss

-2
Original file line numberDiff line numberDiff line change
@@ -460,9 +460,7 @@ li.note {
460460

461461
copy-code {
462462
position: absolute;
463-
@apply gl-transition-all;
464463
opacity: 0;
465-
466464
top: $gl-spacing-scale-3;
467465
right: $gl-spacing-scale-3;
468466

app/assets/stylesheets/page_bundles/merge_requests.scss

+2-30
Original file line numberDiff line numberDiff line change
@@ -322,25 +322,7 @@
322322
display: flex;
323323
align-items: center;
324324
flex-wrap: wrap;
325-
326-
@include media-breakpoint-up(xs) {
327-
flex-wrap: nowrap;
328-
white-space: nowrap;
329-
}
330-
331-
@container mr-widget-extension (max-width: 600px) {
332-
flex-direction: column;
333-
align-items: flex-start;
334-
335-
.deployment-info {
336-
margin-bottom: $gl-padding-8;
337-
}
338-
339-
// stylelint-disable-next-line gitlab/no-gl-class
340-
.gl-button {
341-
margin-left: 0;
342-
}
343-
}
325+
justify-content: space-between;
344326

345327
> *:not(:last-child) {
346328
margin-right: 0.3em;
@@ -351,19 +333,9 @@
351333
}
352334

353335
.deployment-info {
354-
flex: 1;
355-
white-space: nowrap;
356-
text-overflow: ellipsis;
357-
min-width: 100px;
358-
359-
display: grid;
336+
white-space: nowrap;display: grid;
360337
grid-template-columns: max-content minmax(0, max-content) max-content;
361338
grid-gap: $gl-spacing-scale-2;
362-
363-
@include media-breakpoint-up(xs) {
364-
min-width: 0;
365-
max-width: 100%;
366-
}
367339
}
368340

369341
.dropdown-menu {

db/docs/analytics_cycle_analytics_value_stream_settings.yml

+2-9
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,5 @@ milestone: '16.5'
1010
gitlab_schema: gitlab_main_cell
1111
allow_cross_foreign_keys:
1212
- gitlab_main_clusterwide
13-
desired_sharding_key:
14-
namespace_id:
15-
references: namespaces
16-
backfill_via:
17-
parent:
18-
foreign_key: value_stream_id
19-
table: analytics_cycle_analytics_group_value_streams
20-
sharding_key: group_id
21-
belongs_to: value_stream
13+
sharding_key:
14+
namespace_id: namespaces
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
# frozen_string_literal: true
2+
3+
class AddNamespaceIdToAnalyticsCycleAnalyticsValueStreamSettings < Gitlab::Database::Migration[2.2]
4+
milestone '17.6'
5+
6+
# rubocop:disable Rails/NotNullColumn -- NamespaceId can not have a default value
7+
def change
8+
add_column :analytics_cycle_analytics_value_stream_settings, :namespace_id, :bigint, null: false
9+
end
10+
# rubocop:enable Rails/NotNullColumn
11+
end
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
# frozen_string_literal: true
2+
3+
class IndexAnalyticsCycleAnalyticsValueStreamSettingsOnNamespaceId < Gitlab::Database::Migration[2.2]
4+
milestone '17.6'
5+
disable_ddl_transaction!
6+
7+
INDEX_NAME = 'index_analytics_cycle_analytics_value_stream_settings_on_namesp'
8+
9+
def up
10+
add_concurrent_index :analytics_cycle_analytics_value_stream_settings, :namespace_id, name: INDEX_NAME
11+
end
12+
13+
def down
14+
remove_concurrent_index_by_name :analytics_cycle_analytics_value_stream_settings, INDEX_NAME
15+
end
16+
end
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
# frozen_string_literal: true
2+
3+
class AddAnalyticsCycleAnalyticsValueStreamSettingsNamespaceIdFk < Gitlab::Database::Migration[2.2]
4+
milestone '17.6'
5+
disable_ddl_transaction!
6+
7+
def up
8+
add_concurrent_foreign_key :analytics_cycle_analytics_value_stream_settings, :namespaces, column: :namespace_id,
9+
on_delete: :cascade
10+
end
11+
12+
def down
13+
with_lock_retries do
14+
remove_foreign_key :analytics_cycle_analytics_value_stream_settings, column: :namespace_id
15+
end
16+
end
17+
end
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
# frozen_string_literal: true
2+
3+
class AddAnalyticsCycleAnalyticsValueStreamSettingsNamespaceIdTrigger < Gitlab::Database::Migration[2.2]
4+
milestone '17.6'
5+
6+
def up
7+
install_sharding_key_assignment_trigger(
8+
table: :analytics_cycle_analytics_value_stream_settings,
9+
sharding_key: :namespace_id,
10+
parent_table: :analytics_cycle_analytics_group_value_streams,
11+
parent_sharding_key: :group_id,
12+
foreign_key: :value_stream_id
13+
)
14+
end
15+
16+
def down
17+
remove_sharding_key_assignment_trigger(
18+
table: :analytics_cycle_analytics_value_stream_settings,
19+
sharding_key: :namespace_id,
20+
parent_table: :analytics_cycle_analytics_group_value_streams,
21+
parent_sharding_key: :group_id,
22+
foreign_key: :value_stream_id
23+
)
24+
end
25+
end

db/schema_migrations/20241017124709

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
40db0d47902dbf54c6c2573fb0aa989e56ed2d4ee015f6733145f6d01d0772a5

db/schema_migrations/20241017124710

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
e11c91e8de86f13d78b3edd3ba013d037fcb2dd22f07da270e9ff536cb314e3c

db/schema_migrations/20241017124711

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
576e65acf4940f7194debb21912e5340feb679ebc153770d47ab92127328ec64

db/schema_migrations/20241017124712

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
a1548e36b359a8071b66e66c18e2331dadeabb0948b304ff21377bb9d8128a37

db/structure.sql

+24
Original file line numberDiff line numberDiff line change
@@ -935,6 +935,22 @@ RETURN NEW;
935935
END
936936
$$;
937937

938+
CREATE FUNCTION trigger_0c326daf67cf() RETURNS trigger
939+
LANGUAGE plpgsql
940+
AS $$
941+
BEGIN
942+
IF NEW."namespace_id" IS NULL THEN
943+
SELECT "group_id"
944+
INTO NEW."namespace_id"
945+
FROM "analytics_cycle_analytics_group_value_streams"
946+
WHERE "analytics_cycle_analytics_group_value_streams"."id" = NEW."value_stream_id";
947+
END IF;
948+
949+
RETURN NEW;
950+
951+
END
952+
$$;
953+
938954
CREATE FUNCTION trigger_0da002390fdc() RETURNS trigger
939955
LANGUAGE plpgsql
940956
AS $$
@@ -6050,6 +6066,7 @@ ALTER SEQUENCE analytics_cycle_analytics_stage_event_hashes_id_seq OWNED BY anal
60506066
CREATE TABLE analytics_cycle_analytics_value_stream_settings (
60516067
value_stream_id bigint NOT NULL,
60526068
project_ids_filter bigint[] DEFAULT '{}'::bigint[],
6069+
namespace_id bigint NOT NULL,
60536070
CONSTRAINT project_ids_filter_array_check CHECK (((cardinality(project_ids_filter) <= 100) AND (array_position(project_ids_filter, NULL::bigint) IS NULL)))
60546071
);
60556072

@@ -28161,6 +28178,8 @@ CREATE UNIQUE INDEX index_analytics_ca_group_value_streams_on_group_id_and_name
2816128178

2816228179
CREATE INDEX index_analytics_cycle_analytics_group_stages_custom_only ON analytics_cycle_analytics_group_stages USING btree (id) WHERE (custom = true);
2816328180

28181+
CREATE INDEX index_analytics_cycle_analytics_value_stream_settings_on_namesp ON analytics_cycle_analytics_value_stream_settings USING btree (namespace_id);
28182+
2816428183
CREATE UNIQUE INDEX index_analytics_dashboards_pointers_on_namespace_id ON analytics_dashboards_pointers USING btree (namespace_id);
2816528184

2816628185
CREATE INDEX index_analytics_dashboards_pointers_on_target_project_id ON analytics_dashboards_pointers USING btree (target_project_id);
@@ -34231,6 +34250,8 @@ CREATE TRIGGER trigger_0a1b0adcf686 BEFORE INSERT OR UPDATE ON packages_debian_p
3423134250

3423234251
CREATE TRIGGER trigger_0a29d4d42b62 BEFORE INSERT OR UPDATE ON approval_project_rules_protected_branches FOR EACH ROW EXECUTE FUNCTION trigger_0a29d4d42b62();
3423334252

34253+
CREATE TRIGGER trigger_0c326daf67cf BEFORE INSERT OR UPDATE ON analytics_cycle_analytics_value_stream_settings FOR EACH ROW EXECUTE FUNCTION trigger_0c326daf67cf();
34254+
3423434255
CREATE TRIGGER trigger_0da002390fdc BEFORE INSERT OR UPDATE ON operations_feature_flags_issues FOR EACH ROW EXECUTE FUNCTION trigger_0da002390fdc();
3423534256

3423634257
CREATE TRIGGER trigger_0e13f214e504 BEFORE INSERT OR UPDATE ON merge_request_assignment_events FOR EACH ROW EXECUTE FUNCTION trigger_0e13f214e504();
@@ -35962,6 +35983,9 @@ ALTER TABLE ONLY packages_composer_metadata
3596235983
ALTER TABLE ONLY approval_project_rules_protected_branches
3596335984
ADD CONSTRAINT fk_e6ee913fc2 FOREIGN KEY (project_id) REFERENCES projects(id) ON DELETE CASCADE;
3596435985

35986+
ALTER TABLE ONLY analytics_cycle_analytics_value_stream_settings
35987+
ADD CONSTRAINT fk_e6fcfdeb81 FOREIGN KEY (namespace_id) REFERENCES namespaces(id) ON DELETE CASCADE;
35988+
3596535989
ALTER TABLE ONLY merge_requests
3596635990
ADD CONSTRAINT fk_e719a85f8a FOREIGN KEY (author_id) REFERENCES users(id) ON DELETE SET NULL;
3596735991

doc/administration/geo/setup/two_single_node_external_services.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,7 @@ secondary site is a read-only copy.
317317
1. Select **Geo > Sites**.
318318
1. Select **Add site**.
319319

320-
![Add secondary site](../replication/img/adding_a_secondary_v15_8.png)
320+
![Form to add a new secondary Geo site](../replication/img/adding_a_secondary_v15_8.png)
321321

322322
1. In **Name**, enter the value for `gitlab_rails['geo_node_name']` in
323323
`/etc/gitlab/gitlab.rb`. The values must match exactly.

doc/administration/self_hosted_models/configure_duo_features.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ Configure the GitLab Duo feature to send queries to the configured self-hosted m
108108
- Choose the self-hosted model you want to use, for example, `Mistral`.
109109
1. Select **Save Changes**.
110110

111-
### Configure the features to use GitLab AI Vendor models
111+
### Configure the features to use GitLab AI vendor models
112112

113113
You can choose a GitLab AI vendor to be the GitLab Duo feature's model provider. The
114114
feature then uses the GitLab-hosted model through the GitLab Cloud Connector:

doc/ci/jobs/index.md

+7-7
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ can find the reason:
104104

105105
In each place, if you hover over the failed job you can see the reason it failed.
106106

107-
![Pipeline detail](img/job_failure_reason_v10_7.png)
107+
![A pipeline graph showing a failed job and the failure-reason.](img/job_failure_reason_v10_7.png)
108108

109109
You can also see the reason it failed on the Job detail page.
110110

@@ -148,11 +148,11 @@ to read.
148148
You can automatically group similar jobs together. If the job names are formatted in a certain way,
149149
they are collapsed into a single group in regular pipeline graphs (not the mini graphs).
150150

151-
You can recognize when a pipeline has grouped jobs if you don't see the retry or
152-
cancel button inside them. Hovering over them shows the number of grouped
153-
jobs. Select to expand them.
151+
You can recognize when a pipeline has grouped jobs if you see a number next to a job
152+
name instead of the retry or cancel buttons. The number indicates the amount of grouped
153+
jobs. Hovering over them shows you if all jobs have passed or any has failed. Select to expand them.
154154

155-
![Grouped pipelines](img/pipeline_grouped_jobs_v14_2.png)
155+
![A pipeline graph showing several stages and jobs, including three groups of grouped jobs.](img/pipeline_grouped_jobs_v14_2.png)
156156

157157
To create a group of jobs, in the `.gitlab-ci.yml` file,
158158
separate each job name with a number and one of the following:
@@ -292,7 +292,7 @@ the [variable is overridden](../variables/index.md#override-a-defined-cicd-varia
292292
Any variables overridden by using this process are [expanded](../variables/index.md#prevent-cicd-variable-expansion)
293293
and not [masked](../variables/index.md#mask-a-cicd-variable).
294294

295-
![Manual job variables](img/manual_job_variables_v13_10.png)
295+
![The run manual job page with fields for specifying CI/CD variables.](img/manual_job_variables_v13_10.png)
296296

297297
## Delay a job
298298

@@ -307,7 +307,7 @@ For example, if you start rolling out new code and:
307307
- Users experience trouble with the new code, you can stop the timed incremental rollout by canceling the pipeline
308308
and [rolling](../environments/index.md#retry-or-roll-back-a-deployment) back to the last stable version.
309309

310-
![Pipelines example](img/pipeline_delayed_job_v14_2.png)
310+
![A pipeline graph with a delayed job.](img/pipeline_delayed_job_v14_2.png)
311311

312312
## Deployment jobs
313313

doc/development/observability/index.md

+3-2
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,10 @@ description: "Documentation for developers interested in contributing features o
99

1010
## GitLab Observability development setup
1111

12-
There are two options for developing and debugging GitLab Observability:
12+
There are several options for developing and debugging GitLab Observability:
1313

14-
- [Run GDK locally and connect to the staging instance](#run-gdk-and-connect-to-the-staging-instance-of-gitlab-observability-backend) of [GitLab Observability Backend](https://gitlab.com/gitlab-org/opstrace/opstrace). This is the simplest and recommended approach for those looking to make changes, or verify changes to Rails, Sidekiq or Workhorse.
14+
- [Run GDK and GitLab Observability Backend locally all-in-one](https://gitlab.com/gitlab-org/gitlab-development-kit/-/blob/main/doc/howto/gitlab_observability_backend.md): This is the simplest and recommended approach for those looking to make changes, or verify changes to Rails, Sidekiq or Workhorse.
15+
- [Run GDK locally and connect to the staging instance](#run-gdk-and-connect-to-the-staging-instance-of-gitlab-observability-backend) of [GitLab Observability Backend](https://gitlab.com/gitlab-org/opstrace/opstrace). This is an alternative approach for those looking to make changes, or verify changes to Rails, Sidekiq or Workhorse.
1516
- [Use the purpose built `devvm`](#use-the-purpose-built-devvm). This is more involved but includes a development deployment of the [GitLab Observability Backend](https://gitlab.com/gitlab-org/opstrace/opstrace). This is recommended for those who want to make changes to the GitLab Observability Backend component.
1617
- [Run GDK with mocked Observability data](#run-gdk-with-mocked-observability-data). This could be useful in case you just need to work on a frontend or Rails change and do not need the full stack, or when providing reproduction steps for an MR reviewer, who probably won't want to set up the full stack just for an MR.
1718

Loading

0 commit comments

Comments
 (0)