Skip to content

Commit 0ea2d99

Browse files
author
GitLab Bot
committed
Add latest changes from gitlab-org/gitlab@master
1 parent f602da8 commit 0ea2d99

File tree

14 files changed

+105
-40
lines changed

14 files changed

+105
-40
lines changed

GITALY_SERVER_VERSION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
ca638e23ca921cf2f2f3cdc8a6ff033af667170b
1+
3fc66dc23581de48bdbbf1b5a5d5ca9faf5f925b

app/assets/javascripts/issues/show/components/incidents/incident_tabs.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ export default {
101101
>
102102
<gl-tab :title="s__('Incident|Summary')">
103103
<highlight-bar :alert="alert" />
104-
<description-component v-bind="$attrs" />
104+
<description-component v-bind="$attrs" v-on="$listeners" />
105105
</gl-tab>
106106
<incident-metric-tab />
107107
<gl-tab

app/assets/javascripts/pages/shared/wikis/components/wiki_content.vue

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ import { GlSkeletonLoader, GlSafeHtmlDirective, GlAlert } from '@gitlab/ui';
33
import createFlash from '~/flash';
44
import { __ } from '~/locale';
55
import axios from '~/lib/utils/axios_utils';
6+
import { handleLocationHash } from '~/lib/utils/common_utils';
67
import { renderGFM } from '../render_gfm_facade';
78
89
export default {
@@ -43,6 +44,7 @@ export default {
4344
this.$nextTick()
4445
.then(() => {
4546
renderGFM(this.$refs.content);
47+
handleLocationHash();
4648
})
4749
.catch(() =>
4850
createFlash({

app/assets/javascripts/runner/components/registration/registration_token_reset_dropdown_item.vue

Lines changed: 22 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -8,14 +8,16 @@ import runnersRegistrationTokenResetMutation from '~/runner/graphql/list/runners
88
import { captureException } from '~/runner/sentry_utils';
99
import { INSTANCE_TYPE, GROUP_TYPE, PROJECT_TYPE } from '../../constants';
1010
11+
const i18n = {
12+
modalAction: s__('Runners|Reset token'),
13+
modalCancel: __('Cancel'),
14+
modalCopy: __('Are you sure you want to reset the registration token?'),
15+
modalTitle: __('Reset registration token'),
16+
};
17+
1118
export default {
1219
name: 'RunnerRegistrationTokenReset',
13-
i18n: {
14-
modalAction: s__('Runners|Reset token'),
15-
modalCancel: __('Cancel'),
16-
modalCopy: __('Are you sure you want to reset the registration token?'),
17-
modalTitle: __('Reset registration token'),
18-
},
20+
i18n,
1921
components: {
2022
GlDropdownItem,
2123
GlLoadingIcon,
@@ -68,6 +70,18 @@ export default {
6870
return null;
6971
}
7072
},
73+
actionPrimary() {
74+
return {
75+
text: i18n.modalAction,
76+
attributes: [{ variant: 'danger' }],
77+
};
78+
},
79+
actionSecondary() {
80+
return {
81+
text: i18n.modalCancel,
82+
attributes: [{ variant: 'default' }],
83+
};
84+
},
7185
},
7286
methods: {
7387
handleModalPrimary() {
@@ -115,14 +129,8 @@ export default {
115129
<gl-modal
116130
size="sm"
117131
:modal-id="$options.modalId"
118-
:action-primary="/* eslint-disable @gitlab/vue-no-new-non-primitive-in-template */ {
119-
text: $options.i18n.modalAction,
120-
attributes: [{ variant: 'danger' }],
121-
} /* eslint-enable @gitlab/vue-no-new-non-primitive-in-template */"
122-
:action-secondary="/* eslint-disable @gitlab/vue-no-new-non-primitive-in-template */ {
123-
text: $options.i18n.modalCancel,
124-
attributes: [{ variant: 'default' }],
125-
} /* eslint-enable @gitlab/vue-no-new-non-primitive-in-template */"
132+
:action-primary="actionPrimary"
133+
:action-secondary="actionSecondary"
126134
:title="$options.i18n.modalTitle"
127135
@primary="handleModalPrimary"
128136
>

app/assets/javascripts/vue_merge_request_widget/components/extensions/telemetry.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ function simplifyWidgetName(componentName) {
6565
function baseRedisEventName(extensionName) {
6666
const redisEventName = extensionName.replace(/([A-Z])/g, '_$1').toLowerCase();
6767

68-
return `i_merge_request_widget_${redisEventName}`;
68+
return `i_code_review_merge_request_widget_${redisEventName}`;
6969
}
7070

7171
function whenable(bus) {

app/views/projects/incidents/show.html.haml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,6 @@
22
- add_to_breadcrumbs _("Incidents"), project_incidents_path(@project)
33
- breadcrumb_title @issue.to_reference
44
- page_title "#{@issue.title} (#{@issue.to_reference})", _("Incidents")
5+
- add_page_specific_style 'page_bundles/issues_show'
56

67
= render 'projects/issuable/show', issuable: @issue
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
---
2+
name: hard_failure_for_mirrors_without_license
3+
introduced_by_url: https://gitlab.com/gitlab-org/gitlab/-/merge_requests/92422
4+
rollout_issue_url: https://gitlab.com/gitlab-org/gitlab/-/issues/367851
5+
milestone: '15.2'
6+
type: development
7+
group: group::source code
8+
default_enabled: false

doc/development/contributing/issue_workflow.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ labels, you can _always_ add the type, stage, group, and often the category/feat
6868
Type labels are very important. They define what kind of issue this is. Every
6969
issue should have one and only one.
7070

71-
The current type labels are [available in the handbook](https://about.gitlab.com/handbook/engineering/metrics/#work-type-classification)
71+
The SSOT for type and subtype labels is [available in the handbook](https://about.gitlab.com/handbook/engineering/metrics/#work-type-classification).
7272

7373
A number of type labels have a priority assigned to them, which automatically
7474
makes them float to the top, depending on their importance.

doc/user/packages/package_registry/reduce_package_registry_storage.md

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,3 +50,39 @@ To delete package files in the UI, from your group or project:
5050
1. Expand the ellipsis and select **Delete file**.
5151

5252
The package files are permanently deleted.
53+
54+
## Cleanup policy
55+
56+
> - [Introduced](https://gitlab.com/gitlab-org/gitlab/-/issues/346153) in GitLab 15.2.
57+
58+
Depending on the number of packages to remove, the process of manually deleting the packages can take a long time to finish.
59+
A cleanup policy defines a set of rules that, applied to a project, defines which package files you can automatically delete.
60+
61+
### Enable the cleanup policy
62+
63+
By default, the packages cleanup policy is disabled. To enable it:
64+
65+
1. Go to your project **Settings > Packages & Registries**.
66+
1. Expand **Manage storage used by package assets**.
67+
1. Set the rules appropriately.
68+
69+
NOTE:
70+
To access these project settings, you must be at least a maintainer on the related project.
71+
72+
### Available rules
73+
74+
- `Number of duplicated assets to keep`. The number of duplicated assets to keep. Some package formats allow you
75+
to upload more than one copy of an asset. You can limit the number of duplicated assets to keep and automatically
76+
delete the oldest files once the limit is reached.
77+
78+
### Set cleanup limits to conserve resources
79+
80+
A background process executes the package-cleanup policies. This process can take a long time to finish and consumes
81+
server resources while it is running.
82+
83+
You can use the following setting to limit the number of cleanup workers:
84+
85+
- `package_registry_cleanup_policies_worker_capacity`: the maximum number of cleanup workers running concurrently.
86+
This number must be greater than or equal to `0`.
87+
We recommend starting with a low number and increasing it after monitoring the resources used by the background workers.
88+
To remove all workers and not execute the cleanup policies, set this to `0`. The default value is `2`.

locale/gitlab.pot

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4803,16 +4803,13 @@ msgstr ""
48034803
msgid "ApprovalSettings|Prevent approval by author"
48044804
msgstr ""
48054805

4806-
msgid "ApprovalSettings|Prevent approval by author."
4807-
msgstr ""
4808-
48094806
msgid "ApprovalSettings|Prevent approvals by users who add commits"
48104807
msgstr ""
48114808

4812-
msgid "ApprovalSettings|Prevent approvals by users who add commits."
4809+
msgid "ApprovalSettings|Prevent editing approval rules in merge requests"
48134810
msgstr ""
48144811

4815-
msgid "ApprovalSettings|Prevent editing approval rules in merge requests"
4812+
msgid "ApprovalSettings|Prevent editing approval rules in projects and merge requests"
48164813
msgstr ""
48174814

48184815
msgid "ApprovalSettings|Prevent editing approval rules in projects and merge requests."
@@ -29533,7 +29530,7 @@ msgstr ""
2953329530
msgid "Prevent auto-stopping"
2953429531
msgstr ""
2953529532

29536-
msgid "Prevent editing approval rules in projects and merge requests."
29533+
msgid "Prevent editing approval rules in projects and merge requests"
2953729534
msgstr ""
2953829535

2953929536
msgid "Prevent environment from auto-stopping"

qa/qa/specs/features/browser_ui/4_verify/pipeline/pipeline_editor_lint_spec.rb

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,12 @@
22

33
module QA
44
RSpec.describe 'Verify' do
5-
describe 'Pipeline editor', :reliable do
5+
# TODO: Remove this test when feature flag is removed
6+
# Flag rollout issue https://gitlab.com/gitlab-org/gitlab/-/issues/364257
7+
describe 'Pipeline editor', :reliable, feature_flag: {
8+
name: :simulate_pipeline,
9+
scope: :global
10+
} do
611
let(:project) do
712
Resource::Project.fabricate_via_api! do |project|
813
project.name = 'pipeline-editor-project'
@@ -37,7 +42,7 @@ module QA
3742
end
3843

3944
before do
40-
Runtime::Feature.disable(:simulate_pipeline)
45+
Runtime::Feature.disable(:simulate_pipeline) if Runtime::Feature.enabled?(:simulate_pipeline)
4146

4247
Flow::Login.sign_in
4348
project.visit!

spec/frontend/pages/shared/wikis/components/wiki_content_spec.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,10 @@ import { renderGFM } from '~/pages/shared/wikis/render_gfm_facade';
77
import axios from '~/lib/utils/axios_utils';
88
import httpStatus from '~/lib/utils/http_status';
99
import waitForPromises from 'helpers/wait_for_promises';
10+
import { handleLocationHash } from '~/lib/utils/common_utils';
1011

1112
jest.mock('~/pages/shared/wikis/render_gfm_facade');
13+
jest.mock('~/lib/utils/common_utils');
1214

1315
describe('pages/shared/wikis/components/wiki_content', () => {
1416
const PATH = '/test';
@@ -76,6 +78,12 @@ describe('pages/shared/wikis/components/wiki_content', () => {
7678

7779
expect(renderGFM).toHaveBeenCalledWith(wrapper.element);
7880
});
81+
82+
it('handles hash after render', async () => {
83+
await nextTick();
84+
85+
expect(handleLocationHash).toHaveBeenCalled();
86+
});
7987
});
8088

8189
describe('when loading content fails', () => {

spec/frontend/vue_mr_widget/extentions/terraform/index_spec.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,11 +142,11 @@ describe('Terraform extension', () => {
142142

143143
expect(api.trackRedisHllUserEvent).toHaveBeenCalledTimes(1);
144144
expect(api.trackRedisHllUserEvent).toHaveBeenCalledWith(
145-
'i_merge_request_widget_terraform_click_full_report',
145+
'i_code_review_merge_request_widget_terraform_click_full_report',
146146
);
147147
expect(api.trackRedisCounterEvent).toHaveBeenCalledTimes(1);
148148
expect(api.trackRedisCounterEvent).toHaveBeenCalledWith(
149-
'i_merge_request_widget_terraform_count_click_full_report',
149+
'i_code_review_merge_request_widget_terraform_count_click_full_report',
150150
);
151151
});
152152
});

spec/frontend/vue_mr_widget/mr_widget_options_spec.js

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1080,17 +1080,17 @@ describe('MrWidgetOptions', () => {
10801080
// The default working extension is a "warning" type, which generates a second - more specific - telemetry event for expansions
10811081
expect(api.trackRedisHllUserEvent).toHaveBeenCalledTimes(2);
10821082
expect(api.trackRedisHllUserEvent).toHaveBeenCalledWith(
1083-
'i_merge_request_widget_test_extension_expand',
1083+
'i_code_review_merge_request_widget_test_extension_expand',
10841084
);
10851085
expect(api.trackRedisHllUserEvent).toHaveBeenCalledWith(
1086-
'i_merge_request_widget_test_extension_expand_warning',
1086+
'i_code_review_merge_request_widget_test_extension_expand_warning',
10871087
);
10881088
expect(api.trackRedisCounterEvent).toHaveBeenCalledTimes(2);
10891089
expect(api.trackRedisCounterEvent).toHaveBeenCalledWith(
1090-
'i_merge_request_widget_test_extension_count_expand',
1090+
'i_code_review_merge_request_widget_test_extension_count_expand',
10911091
);
10921092
expect(api.trackRedisCounterEvent).toHaveBeenCalledWith(
1093-
'i_merge_request_widget_test_extension_count_expand_warning',
1093+
'i_code_review_merge_request_widget_test_extension_count_expand_warning',
10941094
);
10951095
});
10961096
});
@@ -1160,11 +1160,11 @@ describe('MrWidgetOptions', () => {
11601160

11611161
expect(api.trackRedisHllUserEvent).toHaveBeenCalledTimes(1);
11621162
expect(api.trackRedisHllUserEvent).toHaveBeenCalledWith(
1163-
'i_merge_request_widget_test_extension_view',
1163+
'i_code_review_merge_request_widget_test_extension_view',
11641164
);
11651165
expect(api.trackRedisCounterEvent).toHaveBeenCalledTimes(1);
11661166
expect(api.trackRedisCounterEvent).toHaveBeenCalledWith(
1167-
'i_merge_request_widget_test_extension_count_view',
1167+
'i_code_review_merge_request_widget_test_extension_count_view',
11681168
);
11691169
});
11701170

@@ -1183,17 +1183,17 @@ describe('MrWidgetOptions', () => {
11831183
// The default working extension is a "warning" type, which generates a second - more specific - telemetry event for expansions
11841184
expect(api.trackRedisHllUserEvent).toHaveBeenCalledTimes(2);
11851185
expect(api.trackRedisHllUserEvent).toHaveBeenCalledWith(
1186-
'i_merge_request_widget_test_extension_expand',
1186+
'i_code_review_merge_request_widget_test_extension_expand',
11871187
);
11881188
expect(api.trackRedisHllUserEvent).toHaveBeenCalledWith(
1189-
'i_merge_request_widget_test_extension_expand_warning',
1189+
'i_code_review_merge_request_widget_test_extension_expand_warning',
11901190
);
11911191
expect(api.trackRedisCounterEvent).toHaveBeenCalledTimes(2);
11921192
expect(api.trackRedisCounterEvent).toHaveBeenCalledWith(
1193-
'i_merge_request_widget_test_extension_count_expand',
1193+
'i_code_review_merge_request_widget_test_extension_count_expand',
11941194
);
11951195
expect(api.trackRedisCounterEvent).toHaveBeenCalledWith(
1196-
'i_merge_request_widget_test_extension_count_expand_warning',
1196+
'i_code_review_merge_request_widget_test_extension_count_expand_warning',
11971197
);
11981198
});
11991199

@@ -1236,11 +1236,11 @@ describe('MrWidgetOptions', () => {
12361236

12371237
expect(api.trackRedisHllUserEvent).toHaveBeenCalledTimes(1);
12381238
expect(api.trackRedisHllUserEvent).toHaveBeenCalledWith(
1239-
'i_merge_request_widget_test_extension_click_full_report',
1239+
'i_code_review_merge_request_widget_test_extension_click_full_report',
12401240
);
12411241
expect(api.trackRedisCounterEvent).toHaveBeenCalledTimes(1);
12421242
expect(api.trackRedisCounterEvent).toHaveBeenCalledWith(
1243-
'i_merge_request_widget_test_extension_count_click_full_report',
1243+
'i_code_review_merge_request_widget_test_extension_count_click_full_report',
12441244
);
12451245
});
12461246

0 commit comments

Comments
 (0)