From edd183a633915eacd9b73cab43ea839a4cd42bf6 Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Thu, 22 Oct 2020 12:08:41 +0000 Subject: [PATCH] Add latest changes from gitlab-org/gitlab@master --- .haml-lint_todo.yml | 2 - .rubocop_todo.yml | 10 -- GITALY_SERVER_VERSION | 2 +- .../components/edit_meta_controls.vue | 51 ++----- .../components/edit_meta_modal.vue | 14 +- .../static_site_editor/constants.js | 2 + .../components/states/ready_to_merge.vue | 86 +++++------ .../components/local_storage_sync.vue | 10 ++ .../stylesheets/fontawesome_custom.scss | 4 - app/assets/stylesheets/framework/spinner.scss | 4 + app/controllers/jwks_controller.rb | 26 ++++ app/finders/merge_requests_finder.rb | 13 ++ app/graphql/types/issue_type.rb | 4 + app/helpers/search_helper.rb | 21 ++- app/models/application_setting.rb | 4 + app/models/ci/build.rb | 2 +- app/models/commit_status.rb | 9 +- app/models/members/group_member.rb | 1 + app/validators/rsa_key_validator.rb | 27 ++++ app/views/notify/_note_email.html.haml | 2 +- .../projects/mirrors/_mirror_repos.html.haml | 2 +- .../_remote_mirror_update_button.html.haml | 8 +- .../214607-ci-jwt-signing-key-jwks-take-3.yml | 5 + ...-only-one-result-redirect-to-that-page.yml | 5 + ...man-readable-time-estimate-via-graphql.yml | 5 + .../unreleased/gy-fix-load-perf-docs.yml | 5 + .../jduplessis_issue_comment_link.yml | 5 + .../mw-replace-fa-refresh-icons.yml | 5 + .../ph-filterTargetBranchByNotEquals.yml | 5 + ...nd_for_non_unique_project_fingerprints.yml | 5 + .../development/ci_jwt_signing_key.yml | 7 + .../development/one_dimensional_matrix.yml | 7 - config/routes.rb | 5 +- .../development/02_application_settings.rb | 3 + db/fixtures/production/010_settings.rb | 4 + ...jwt_signing_key_to_application_settings.rb | 13 ++ ...ettings_encrypted_ci_jwt_signing_key_iv.rb | 17 +++ ...01008013434_generate_ci_jwt_signing_key.rb | 32 ++++ ...441_add_position_into_security_findings.rb | 19 +++ ...an_id_and_position_of_security_findings.rb | 18 +++ db/schema_migrations/20201008011523 | 1 + db/schema_migrations/20201008011937 | 1 + db/schema_migrations/20201008013434 | 1 + db/schema_migrations/20201008224441 | 1 + db/schema_migrations/20201019091307 | 1 + db/structure.sql | 6 + .../graphql/reference/gitlab_schema.graphql | 20 +++ doc/api/graphql/reference/gitlab_schema.json | 56 +++++++ doc/api/graphql/reference/index.md | 4 + doc/ci/yaml/README.md | 3 - .../load_performance_testing.md | 2 +- doc/user/search/index.md | 1 + .../ci/config/entry/product/variables.rb | 6 +- lib/gitlab/ci/features.rb | 4 - lib/gitlab/ci/jwt.rb | 16 +- .../usage_data_counters/hll_redis_counter.rb | 4 +- locale/gitlab.pot | 9 ++ qa/qa/page/merge_request/show.rb | 1 - spec/controllers/groups_controller_spec.rb | 30 ++-- spec/controllers/jwks_controller_spec.rb | 36 +++++ spec/db/production/settings_spec.rb | 7 + .../user_merges_immediately_spec.rb | 2 +- ...user_merges_when_pipeline_succeeds_spec.rb | 13 -- .../user_filters_by_target_branch_spec.rb | 10 ++ .../components/edit_meta_controls_spec.js | 12 -- .../components/edit_meta_modal_spec.js | 51 +++++-- .../components/local_storage_sync_spec.js | 26 ++++ spec/graphql/types/issue_type_spec.rb | 2 +- spec/helpers/search_helper_spec.rb | 36 ++++- .../ci/config/entry/product/matrix_spec.rb | 138 ++++-------------- .../ci/config/entry/product/variables_spec.rb | 71 ++------- spec/lib/gitlab/ci/jwt_spec.rb | 69 ++++++--- .../hll_redis_counter_spec.rb | 10 ++ spec/mailers/notify_spec.rb | 1 + .../generate_ci_jwt_signing_key_spec.rb | 42 ++++++ spec/models/application_setting_spec.rb | 17 +++ spec/models/ci/build_spec.rb | 17 ++- spec/models/commit_status_spec.rb | 129 +++++----------- spec/models/members/group_member_spec.rb | 14 +- spec/routing/openid_connect_spec.rb | 5 - spec/routing/routing_spec.rb | 7 + ...heck_installation_progress_service_spec.rb | 6 +- .../services/issues/zoom_link_service_spec.rb | 22 ++- spec/support/helpers/snowplow_helpers.rb | 6 +- spec/support/snowplow.rb | 2 +- spec/validators/rsa_key_validator_spec.rb | 40 +++++ 86 files changed, 919 insertions(+), 508 deletions(-) create mode 100644 app/controllers/jwks_controller.rb create mode 100644 app/validators/rsa_key_validator.rb create mode 100644 changelogs/unreleased/214607-ci-jwt-signing-key-jwks-take-3.yml create mode 100644 changelogs/unreleased/233278-in-global-search-if-there-is-only-one-result-redirect-to-that-page.yml create mode 100644 changelogs/unreleased/37102-expose-human-readable-time-estimate-via-graphql.yml create mode 100644 changelogs/unreleased/gy-fix-load-perf-docs.yml create mode 100644 changelogs/unreleased/jduplessis_issue_comment_link.yml create mode 100644 changelogs/unreleased/mw-replace-fa-refresh-icons.yml create mode 100644 changelogs/unreleased/ph-filterTargetBranchByNotEquals.yml create mode 100644 changelogs/unreleased/workaround_for_non_unique_project_fingerprints.yml create mode 100644 config/feature_flags/development/ci_jwt_signing_key.yml delete mode 100644 config/feature_flags/development/one_dimensional_matrix.yml create mode 100644 db/migrate/20201008011523_add_ci_jwt_signing_key_to_application_settings.rb create mode 100644 db/migrate/20201008011937_add_text_limit_to_application_settings_encrypted_ci_jwt_signing_key_iv.rb create mode 100644 db/migrate/20201008013434_generate_ci_jwt_signing_key.rb create mode 100644 db/migrate/20201008224441_add_position_into_security_findings.rb create mode 100644 db/migrate/20201019091307_add_unique_index_on_scan_id_and_position_of_security_findings.rb create mode 100644 db/schema_migrations/20201008011523 create mode 100644 db/schema_migrations/20201008011937 create mode 100644 db/schema_migrations/20201008013434 create mode 100644 db/schema_migrations/20201008224441 create mode 100644 db/schema_migrations/20201019091307 create mode 100644 spec/controllers/jwks_controller_spec.rb create mode 100644 spec/migrations/generate_ci_jwt_signing_key_spec.rb create mode 100644 spec/validators/rsa_key_validator_spec.rb diff --git a/.haml-lint_todo.yml b/.haml-lint_todo.yml index 78b5f3fb88b..6d6a5d2a813 100644 --- a/.haml-lint_todo.yml +++ b/.haml-lint_todo.yml @@ -329,7 +329,6 @@ linters: - 'ee/app/views/errors/kerberos_denied.html.haml' - 'ee/app/views/groups/ee/_settings_nav.html.haml' - 'ee/app/views/groups/group_members/_ldap_sync.html.haml' - - 'ee/app/views/groups/group_members/_sync_button.html.haml' - 'ee/app/views/groups/hooks/edit.html.haml' - 'ee/app/views/groups/ldap_group_links/index.html.haml' - 'ee/app/views/layouts/nav/ee/admin/_new_monitoring_sidebar.html.haml' @@ -362,7 +361,6 @@ linters: - 'ee/app/views/projects/services/gitlab_slack_application/_help.html.haml' - 'ee/app/views/projects/services/gitlab_slack_application/_slack_integration_form.html.haml' - 'ee/app/views/projects/settings/slacks/edit.html.haml' - - 'ee/app/views/shared/_mirror_update_button.html.haml' - 'ee/app/views/shared/epic/_search_bar.html.haml' - 'ee/app/views/shared/issuable/_approvals.html.haml' - 'ee/app/views/shared/issuable/_board_create_list_dropdown.html.haml' diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index d7df1264fe3..19115b7bedb 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -186,31 +186,21 @@ RSpec/ExpectChange: # Offense count: 47 RSpec/ExpectGitlabTracking: Exclude: - - 'ee/spec/controllers/groups/analytics/coverage_reports_controller_spec.rb' - 'ee/spec/controllers/projects/settings/operations_controller_spec.rb' - - 'ee/spec/controllers/registrations_controller_spec.rb' - 'ee/spec/requests/api/visual_review_discussions_spec.rb' - 'ee/spec/services/epics/issue_promote_service_spec.rb' - 'spec/controllers/groups/registry/repositories_controller_spec.rb' - - 'spec/controllers/groups_controller_spec.rb' - 'spec/controllers/projects/registry/repositories_controller_spec.rb' - 'spec/controllers/projects/registry/tags_controller_spec.rb' - 'spec/controllers/projects/settings/operations_controller_spec.rb' - 'spec/controllers/registrations_controller_spec.rb' - 'spec/lib/api/helpers_spec.rb' - - 'spec/lib/gitlab/experimentation_spec.rb' - - 'spec/mailers/notify_spec.rb' - - 'spec/models/project_services/prometheus_service_spec.rb' - 'spec/requests/api/project_container_repositories_spec.rb' - - 'spec/services/clusters/applications/check_installation_progress_service_spec.rb' - - 'spec/services/issues/zoom_link_service_spec.rb' - - 'spec/support/helpers/snowplow_helpers.rb' - 'spec/support/shared_examples/controllers/trackable_shared_examples.rb' - 'spec/support/shared_examples/requests/api/container_repositories_shared_examples.rb' - 'spec/support/shared_examples/requests/api/discussions_shared_examples.rb' - 'spec/support/shared_examples/requests/api/packages_shared_examples.rb' - 'spec/support/shared_examples/requests/api/tracking_shared_examples.rb' - - 'spec/support/snowplow.rb' # Offense count: 751 RSpec/ExpectInHook: diff --git a/GITALY_SERVER_VERSION b/GITALY_SERVER_VERSION index 5059b30b278..4f781d9e61f 100644 --- a/GITALY_SERVER_VERSION +++ b/GITALY_SERVER_VERSION @@ -1 +1 @@ -19109e7090eb6dab2b8c0c168bbef76ceca79a31 +0ebfb705b79a8baecc1db46f31761f83f4e471f9 diff --git a/app/assets/javascripts/static_site_editor/components/edit_meta_controls.vue b/app/assets/javascripts/static_site_editor/components/edit_meta_controls.vue index 9f75c65a316..fb80b158b46 100644 --- a/app/assets/javascripts/static_site_editor/components/edit_meta_controls.vue +++ b/app/assets/javascripts/static_site_editor/components/edit_meta_controls.vue @@ -1,6 +1,5 @@ @@ -75,6 +81,12 @@ export default { @secondary="onSecondary" @hide="() => $emit('hide')" > + import { isEmpty } from 'lodash'; -import { GlIcon, GlButton, GlSprintf, GlLink } from '@gitlab/ui'; +import { + GlIcon, + GlButton, + GlButtonGroup, + GlDropdown, + GlDropdownItem, + GlSprintf, + GlLink, + GlTooltipDirective, +} from '@gitlab/ui'; import readyToMergeMixin from 'ee_else_ce/vue_merge_request_widget/mixins/ready_to_merge'; import simplePoll from '~/lib/utils/simple_poll'; import { __ } from '~/locale'; @@ -36,6 +45,9 @@ export default { GlSprintf, GlLink, GlButton, + GlButtonGroup, + GlDropdown, + GlDropdownItem, MergeTrainHelperText: () => import('ee_component/vue_merge_request_widget/components/merge_train_helper_text.vue'), MergeImmediatelyConfirmationDialog: () => @@ -43,6 +55,9 @@ export default { 'ee_component/vue_merge_request_widget/components/merge_immediately_confirmation_dialog.vue' ), }, + directives: { + GlTooltip: GlTooltipDirective, + }, mixins: [readyToMergeMixin], props: { mr: { type: Object, required: true }, @@ -283,7 +298,7 @@ export default {
- + {{ mergeButtonText }} - - - + + + {{ __('Merge immediately') }} + + + +