From fb19b392e2331e7663c3d65c05aeb81a1db3630d Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Mon, 9 May 2022 15:07:50 +0000 Subject: [PATCH] Add latest changes from gitlab-org/gitlab@master --- .../layout/space_in_lambda_literal.yml | 1 - .../background_migration_base_class.yml | 1 - .../migration/background_migration_record.yml | 1 - .../rails/skips_model_validations.yml | 1 - .rubocop_todo/style/redundant_self.yml | 1 - .../components/delete_branch_modal.vue | 37 ++++----- .../components/value_stream_filters.vue | 57 +------------ .../components/import_a_project_modal.vue | 2 +- .../date_time_picker/date_time_picker.vue | 2 +- .../dropdown_contents_create_view.vue | 2 +- app/helpers/invite_members_helper.rb | 2 +- app/models/container_repository.rb | 2 +- app/models/namespace.rb | 2 + app/services/concerns/group_linkable.rb | 2 +- .../application_settings/general.html.haml | 2 +- .../migration/enqueuer_worker.rb | 7 +- ...egistry_migration_phase2_enqueue_twice.yml | 8 ++ .../updated_diff_expansion_buttons.yml | 2 +- ...schedule_artifact_expiry_backfill_again.rb | 38 +-------- doc/administration/geo/setup/database.md | 2 +- doc/api/graphql/reference/index.md | 2 +- doc/ci/pipelines/merged_results_pipelines.md | 2 +- .../specification_guide/index.md | 10 ++- .../settings/account_and_limit_settings.md | 2 +- .../application_security/policies/index.md | 5 +- .../policies/scan-execution-policies.md | 5 +- .../policies/scan-result-policies.md | 5 +- doc/user/project/issues/csv_import.md | 45 +++++----- doc/user/project/protected_branches.md | 36 +++++--- .../internal/container_registry/migration.rb | 2 +- lib/container_registry/migration.rb | 4 + .../backfill_artifact_expiry_date.rb | 58 ------------- lib/gitlab/ci/config/entry/job.rb | 20 +---- lib/gitlab/ci/config/entry/root.rb | 23 +----- lib/gitlab/process_supervisor.rb | 20 ++--- locale/gitlab.pot | 33 ++------ metrics_server/metrics_server.rb | 2 - sidekiq_cluster/cli.rb | 14 +--- spec/commands/sidekiq_cluster/cli_spec.rb | 26 ++---- spec/controllers/groups_controller_spec.rb | 2 +- spec/features/groups/group_settings_spec.rb | 2 +- .../components/delete_branch_modal_spec.js | 4 +- spec/frontend/cycle_analytics/base_spec.js | 3 - .../value_stream_filters_spec.js | 54 ------------ .../date_time_picker/date_time_picker_spec.js | 2 +- spec/helpers/invite_members_helper_spec.rb | 4 +- spec/lib/container_registry/migration_spec.rb | 14 ++++ .../backfill_artifact_expiry_date_spec.rb | 82 ------------------- spec/lib/gitlab/ci/config/entry/job_spec.rb | 2 +- spec/lib/gitlab/ci/config/entry/root_spec.rb | 37 +-------- spec/lib/gitlab/ci/yaml_processor_spec.rb | 10 +-- spec/lib/gitlab/process_supervisor_spec.rb | 76 ++++++++++++++++- spec/metrics_server/metrics_server_spec.rb | 27 ++---- ...ule_artifact_expiry_backfill_again_spec.rb | 38 --------- spec/models/container_repository_spec.rb | 14 ++++ spec/models/namespace_spec.rb | 9 ++ spec/requests/api/graphql/ci/config_spec.rb | 18 ---- .../container_registry/migration_spec.rb | 6 ++ spec/requests/api/lint_spec.rb | 13 --- .../inviting_groups_shared_examples.rb | 2 +- .../migration/enqueuer_worker_spec.rb | 17 ++++ 61 files changed, 296 insertions(+), 626 deletions(-) create mode 100644 config/feature_flags/development/container_registry_migration_phase2_enqueue_twice.yml delete mode 100644 lib/gitlab/background_migration/backfill_artifact_expiry_date.rb delete mode 100644 spec/lib/gitlab/background_migration/backfill_artifact_expiry_date_spec.rb delete mode 100644 spec/migrations/20210413132500_reschedule_artifact_expiry_backfill_again_spec.rb diff --git a/.rubocop_todo/layout/space_in_lambda_literal.yml b/.rubocop_todo/layout/space_in_lambda_literal.yml index 7a2bd752b6e..2377553ccdb 100644 --- a/.rubocop_todo/layout/space_in_lambda_literal.yml +++ b/.rubocop_todo/layout/space_in_lambda_literal.yml @@ -351,7 +351,6 @@ Layout/SpaceInLambdaLiteral: - 'lib/container_registry/base_client.rb' - 'lib/container_registry/client.rb' - 'lib/file_size_validator.rb' - - 'lib/gitlab/background_migration/backfill_artifact_expiry_date.rb' - 'lib/gitlab/background_migration/populate_latest_pipeline_ids.rb' - 'lib/gitlab/background_migration/remove_all_trace_expiration_dates.rb' - 'lib/gitlab/cache/helpers.rb' diff --git a/.rubocop_todo/migration/background_migration_base_class.yml b/.rubocop_todo/migration/background_migration_base_class.yml index 9307aba4685..8d7d8e954e3 100644 --- a/.rubocop_todo/migration/background_migration_base_class.yml +++ b/.rubocop_todo/migration/background_migration_base_class.yml @@ -2,7 +2,6 @@ Migration/BackgroundMigrationBaseClass: Exclude: - 'lib/gitlab/background_migration/add_primary_email_to_emails_if_user_confirmed.rb' - - 'lib/gitlab/background_migration/backfill_artifact_expiry_date.rb' - 'lib/gitlab/background_migration/backfill_ci_namespace_mirrors.rb' - 'lib/gitlab/background_migration/backfill_ci_project_mirrors.rb' - 'lib/gitlab/background_migration/backfill_ci_queuing_tables.rb' diff --git a/.rubocop_todo/migration/background_migration_record.yml b/.rubocop_todo/migration/background_migration_record.yml index 6ae518a86fb..cbf39285a7d 100644 --- a/.rubocop_todo/migration/background_migration_record.yml +++ b/.rubocop_todo/migration/background_migration_record.yml @@ -2,7 +2,6 @@ Migration/BackgroundMigrationRecord: Exclude: - lib/gitlab/background_migration/add_primary_email_to_emails_if_user_confirmed.rb - - lib/gitlab/background_migration/backfill_artifact_expiry_date.rb - lib/gitlab/background_migration/backfill_ci_namespace_mirrors.rb - lib/gitlab/background_migration/backfill_ci_project_mirrors.rb - lib/gitlab/background_migration/backfill_ci_queuing_tables.rb diff --git a/.rubocop_todo/rails/skips_model_validations.yml b/.rubocop_todo/rails/skips_model_validations.yml index 9b908b7b997..e6402ef27f2 100644 --- a/.rubocop_todo/rails/skips_model_validations.yml +++ b/.rubocop_todo/rails/skips_model_validations.yml @@ -387,7 +387,6 @@ Rails/SkipsModelValidations: - 'lib/api/commit_statuses.rb' - 'lib/api/usage_data.rb' - 'lib/gitlab/background_migration/add_primary_email_to_emails_if_user_confirmed.rb' - - 'lib/gitlab/background_migration/backfill_artifact_expiry_date.rb' - 'lib/gitlab/background_migration/backfill_ci_queuing_tables.rb' - 'lib/gitlab/background_migration/backfill_draft_status_on_merge_requests.rb' - 'lib/gitlab/background_migration/backfill_jira_tracker_deployment_type2.rb' diff --git a/.rubocop_todo/style/redundant_self.yml b/.rubocop_todo/style/redundant_self.yml index 6b5991e9b3c..e363f83761b 100644 --- a/.rubocop_todo/style/redundant_self.yml +++ b/.rubocop_todo/style/redundant_self.yml @@ -297,7 +297,6 @@ Style/RedundantSelf: - 'lib/gitlab/auth/ldap/dn.rb' - 'lib/gitlab/auth/o_auth/provider.rb' - 'lib/gitlab/auth/result.rb' - - 'lib/gitlab/background_migration/backfill_artifact_expiry_date.rb' - 'lib/gitlab/background_migration/backfill_ci_queuing_tables.rb' - 'lib/gitlab/background_migration/backfill_project_repositories.rb' - 'lib/gitlab/background_migration/job_coordinator.rb' diff --git a/app/assets/javascripts/branches/components/delete_branch_modal.vue b/app/assets/javascripts/branches/components/delete_branch_modal.vue index 14c2badeb3f..383fa5f7512 100644 --- a/app/assets/javascripts/branches/components/delete_branch_modal.vue +++ b/app/assets/javascripts/branches/components/delete_branch_modal.vue @@ -32,17 +32,10 @@ export default { return sprintf(modalTitle, { branchName: this.branchName }); }, - message() { - const modalMessage = this.isProtectedBranch + modalMessage() { + return this.isProtectedBranch ? this.$options.i18n.modalMessageProtectedBranch : this.$options.i18n.modalMessage; - - return sprintf(modalMessage, { branchName: this.branchName }); - }, - unmergedWarning() { - return sprintf(this.$options.i18n.unmergedWarning, { - defaultBranchName: this.defaultBranchName, - }); }, undoneWarning() { return sprintf(this.$options.i18n.undoneWarning, { @@ -92,17 +85,15 @@ export default { i18n: { modalTitle: s__('Branches|Delete branch. Are you ABSOLUTELY SURE?'), modalTitleProtectedBranch: s__('Branches|Delete protected branch. Are you ABSOLUTELY SURE?'), - modalMessage: s__( - "Branches|You're about to permanently delete the branch %{strongStart}%{branchName}.%{strongEnd}", - ), + modalMessage: s__("Branches|You're about to permanently delete the branch %{branchName}."), modalMessageProtectedBranch: s__( - "Branches|You're about to permanently delete the protected branch %{strongStart}%{branchName}.%{strongEnd}", + "Branches|You're about to permanently delete the protected branch %{branchName}.", ), unmergedWarning: s__( - 'Branches|This branch hasn’t been merged into %{defaultBranchName}. To avoid data loss, consider merging this branch before deleting it.', + "Branches|This branch hasn't been merged into %{defaultBranchName}. To avoid data loss, consider merging this branch before deleting it.", ), undoneWarning: s__( - 'Branches|Once you confirm and press %{strongStart}%{buttonText},%{strongEnd} it cannot be undone or recovered.', + 'Branches|After you confirm and select %{strongStart}%{buttonText},%{strongEnd} you cannot recover this branch.', ), cancelButtonText: s__('Branches|Cancel, keep branch'), confirmationText: s__( @@ -119,13 +110,19 @@ export default {
- - - {{ branchName }} + {{ branchName }} -import { GlIcon, GlToggle, GlTooltipDirective } from '@gitlab/ui'; -import { s__ } from '~/locale'; +import { GlTooltipDirective } from '@gitlab/ui'; import DateRange from '~/analytics/shared/components/daterange.vue'; import ProjectsDropdownFilter from '~/analytics/shared/components/projects_dropdown_filter.vue'; import { DATE_RANGE_LIMIT, PROJECTS_PER_PAGE } from '~/analytics/shared/constants'; import FilterBar from './filter_bar.vue'; -export const AGGREGATION_TOGGLE_LABEL = s__('CycleAnalytics|Filter by stop date'); -export const AGGREGATION_DESCRIPTION = s__( - 'CycleAnalytics|When enabled, the results show items with a stop event within the date range. When disabled, the results show items with a start event within the date range.', -); - export default { name: 'ValueStreamFilters', components: { - GlIcon, - GlToggle, DateRange, ProjectsDropdownFilter, FilterBar, @@ -57,21 +49,6 @@ export default { required: false, default: null, }, - canToggleAggregation: { - type: Boolean, - required: false, - default: false, - }, - isAggregationEnabled: { - type: Boolean, - required: false, - default: false, - }, - isUpdatingAggregationData: { - type: Boolean, - required: false, - default: false, - }, }, computed: { projectsQueryParams() { @@ -81,19 +58,8 @@ export default { }; }, }, - methods: { - onUpdateAggregation(ev) { - if (!this.isUpdatingAggregationData) { - this.$emit('toggleAggregation', ev); - } - }, - }, multiProjectSelect: true, maxDateRange: DATE_RANGE_LIMIT, - i18n: { - AGGREGATION_TOGGLE_LABEL, - AGGREGATION_DESCRIPTION, - }, };