Merge RC2 into 9.3 (actually-stable)
Checklist
Please refer to https://gitlab.com/gitlab-org/gitlab-ce/snippets/1664866 for the diff changes when check the files below
-
@lbennett please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/assets/javascripts/issue_show/components/edited.vue -
@timothyandrew please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/assets/javascripts/notes.js -
@brycepj please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/assets/javascripts/pipelines/components/graph/stage_column_component.vue -
@ClemMakesApps please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/assets/javascripts/users_select.js -
@brycepj please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/assets/stylesheets/framework/sidebar.scss -
@psimyn please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/assets/stylesheets/pages/merge_requests.scss -
@rspeicher please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/controllers/autocomplete_controller.rb -
@iamphill please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/controllers/projects/issues_controller.rb -
@iamphill please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/helpers/application_helper.rb -
@ayufan please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/models/commit_status.rb -
@lbennett please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/models/concerns/editable.rb -
@smcgivern please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/models/concerns/elastic/application_search.rb -
@lbennett please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/models/concerns/issuable.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/models/note.rb -
@lbennett please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/models/snippet.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/policies/project_snippet_policy.rb -
@dbalexandre please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/services/geo/repository_sync_service.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/uploaders/file_uploader.rb -
@ayufan please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/app/uploaders/gitlab_uploader.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/config/routes/uploads.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/db/migrate/20170316163800_rename_system_namespaces.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/db/migrate/20170316163845_move_uploads_to_system_dir.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/db/post_migrate/20170317162059_update_upload_paths_to_system.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/db/post_migrate/20170406111121_clean_upload_symlinks.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/db/post_migrate/20170606202615_move_appearance_to_system_dir.rb -
@nick.thomas please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/doc/ci/README.md -
@eReGeBe please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/doc/development/README.md -
@JobV please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/doc/user/admin_area/settings/usage_statistics.md -
@SeanPackham (GitLab) please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/doc/user/project/issues/img/issues_main_view_numbered.png -
@brycepj please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/features/project/merge_requests/accept.feature -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/features/steps/groups.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/features/steps/profile/profile.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/features/steps/project/project.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/banzai/reference_parser/base_parser.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/banzai/reference_parser/commit_parser.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/banzai/reference_parser/commit_range_parser.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/banzai/reference_parser/external_issue_parser.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/banzai/reference_parser/label_parser.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/banzai/reference_parser/merge_request_parser.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/banzai/reference_parser/milestone_parser.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/banzai/reference_parser/snippet_parser.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/banzai/reference_parser/user_parser.rb -
@smcgivern please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/gitlab/elastic/search_results.rb -
@timothyandrew please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/gitlab/regex.rb -
@smcgivern please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/lib/gitlab/uploads_transfer.rb -
@rspeicher please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/scripts/static-analysis -
@smcgivern please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/elastic_integration/global_search_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/factories/uploads.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/features/admin/admin_appearance_spec.rb -
@ClemMakesApps please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/features/issues/form_spec.rb -
@brycepj please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/features/merge_requests/merge_when_pipeline_succeeds_spec.rb -
@godfat please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/features/security/project/private_access_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/features/uploads/user_uploads_avatar_to_group_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/features/uploads/user_uploads_avatar_to_profile_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/helpers/application_helper_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/helpers/emails_helper_spec.rb -
@timothyandrew please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/helpers/groups_helper_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/helpers/page_layout_helper_spec.rb -
@lbennett please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/javascripts/issue_show/components/edited_spec.js -
@timothyandrew please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/javascripts/notes_spec.js -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/javascripts/vue_shared/components/commit_spec.js -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/lib/banzai/reference_parser/base_parser_spec.rb -
@selfup please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/lib/banzai/reference_parser/snippet_parser_spec.rb -
@smcgivern please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/lib/gitlab/uploads_transfer_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/migrations/clean_upload_symlinks_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/migrations/move_uploads_to_system_dir_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/migrations/rename_system_namespaces_spec.rb -
@kushalpandya please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/migrations/update_retried_for_ci_builds_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/migrations/update_upload_paths_to_system_spec.rb -
@tiagonbotelho please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/models/application_setting_spec.rb -
@lbennett please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/models/concerns/editable_spec.rb -
oswaldo please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/models/license_spec.rb -
@mkozono please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/models/merge_request_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/models/namespace_spec.rb -
@ayufan please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/services/geo/repository_sync_service_spec.rb -
@ClemMakesApps please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/support/prometheus_helpers.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/uploaders/attachment_uploader_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/uploaders/avatar_uploader_spec.rb -
@DouweM please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/uploaders/file_uploader_spec.rb -
@mkozono please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/workers/admin_emails_worker_spec.rb -
@nick.thomas please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/workers/project_cache_worker_spec.rb -
@mkozono please make sure this file looks correct https://gitlab.com/gitlab-org/gitlab-ee/blob/9-3-actually-stable-ee-rc2/spec/workers/update_all_mirrors_worker_spec.rb
If any files need to be modified, please push (no force-push!) to the 9-3-actually-stable-ee-rc2
branch.
Steps taken
-
git merge -Xtheirs 86051969bd10f2c75540add61424aa12bb21e265
(https://gitlab.com/gitlab-org/gitlab-ee/commit/088eab2d420d29836a5f136e3ac98edf29e9bf15) -
timeline.scss was missing a brace, so I fixed it in (https://gitlab.com/gitlab-org/gitlab-ee/commit/6eec80fe718f01bd534f0a084ca352f538a5a259)
Step 1 also led to:
deleted by them: app/assets/javascripts/issue_show/issue_title_description.vue
deleted by them: app/assets/javascripts/projects/settings_service_desk/service_desk_root.js
deleted by them: app/workers/geo_backfill_worker.rb
deleted by them: spec/javascripts/commit/pipelines/mock_data.js
deleted by them: spec/javascripts/issue_show/issue_title_description_spec.js
deleted by them: spec/support/wait_for_vue_resource.rb
deleted by them: spec/workers/geo/geo_backfill_worker_spec.rb
deleted by them: spec/workers/repository_update_mirror_dispatch_worker_spec.rb
-
So I removed those files in https://gitlab.com/gitlab-org/gitlab-ee/commit/c0175b1a3b7a955b41b67d515df45692fc8105b4
-
Found a missing
end
in a spec (bad merge conflict), so it got fixed in https://gitlab.com/gitlab-org/gitlab-ee/commit/3bcabf1fb606492f19394db8b27b4b1069555dc2 -
Removed duplicate prometheus_ping_url as seen in commit
-
Remove duplicate key in graph_component.vue
-
Fix bad merge of users_select.js
-
Cherry pick https://gitlab.com/gitlab-org/gitlab-ee/commit/c86d9e1026250c7fa0d69d20319854d9f6101b16