diff --git a/.gitlab/merge_request_templates/Security Release.md b/.gitlab/merge_request_templates/Security Release.md index af3839a96a4..f852bebae95 100644 --- a/.gitlab/merge_request_templates/Security Release.md +++ b/.gitlab/merge_request_templates/Security Release.md @@ -19,7 +19,7 @@ See [the general developer security release guidelines](https://gitlab.com/gitla - [ ] A [CHANGELOG entry](https://docs.gitlab.com/ee/development/changelog.html) is added without a `merge_request` value, with `type` set to `security` - [ ] Assign to a reviewer and maintainer, per our [Code Review process]. - [ ] For the MR targeting `master`: - - [ ] Ping appsec team member who created the issue and ask for a non-blocking review with `Please review this MR`. + - [ ] Ask for a non-blocking review from the AppSec team member associated to the issue in the [Canonical repository](https://gitlab.com/gitlab-org/gitlab). If you're unsure who to ping, ask on `#sec-appsec` Slack channel. - [ ] Ensure it's approved according to our [Approval Guidelines]. - [ ] Merge request _must not_ close the corresponding security issue, _unless_ it targets `master`. diff --git a/.rubocop.yml b/.rubocop.yml index 333baadb958..707e1a3ea36 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -359,9 +359,6 @@ RSpec/LeakyConstantDeclaration: - 'spec/lib/gitlab/database/obsolete_ignored_columns_spec.rb' - 'spec/lib/gitlab/database/with_lock_retries_spec.rb' - 'spec/lib/gitlab/git/diff_collection_spec.rb' - - 'spec/lib/gitlab/graphql/pagination/keyset/connection_spec.rb' - - 'spec/lib/gitlab/health_checks/master_check_spec.rb' - - 'spec/lib/gitlab/import_export/attribute_configuration_spec.rb' - 'spec/lib/gitlab/import_export/import_test_coverage_spec.rb' - 'spec/lib/gitlab/import_export/project/relation_factory_spec.rb' - 'spec/lib/gitlab/jira_import/issues_importer_spec.rb' @@ -372,17 +369,14 @@ RSpec/LeakyConstantDeclaration: - 'spec/lib/gitlab/sidekiq_middleware/server_metrics_spec.rb' - 'spec/lib/gitlab/view/presenter/factory_spec.rb' - 'spec/lib/marginalia_spec.rb' - - 'spec/lib/system_check/simple_executor_spec.rb' - 'spec/mailers/notify_spec.rb' - 'spec/migrations/20191125114345_add_admin_mode_protected_path_spec.rb' - - 'spec/migrations/encrypt_plaintext_attributes_on_application_settings_spec.rb' - 'spec/models/concerns/batch_destroy_dependent_associations_spec.rb' - 'spec/models/concerns/bulk_insert_safe_spec.rb' - 'spec/models/concerns/bulk_insertable_associations_spec.rb' - 'spec/models/concerns/triggerable_hooks_spec.rb' - 'spec/models/repository_spec.rb' - 'spec/requests/api/graphql/tasks/task_completion_status_spec.rb' - - 'spec/rubocop/cop/rspec/env_assignment_spec.rb' - 'spec/serializers/commit_entity_spec.rb' - 'spec/services/clusters/applications/check_installation_progress_service_spec.rb' - 'spec/services/clusters/applications/check_uninstall_progress_service_spec.rb' diff --git a/app/assets/javascripts/clusters_list/components/clusters.vue b/app/assets/javascripts/clusters_list/components/clusters.vue index eb575b9ed6c..af3f1437c64 100644 --- a/app/assets/javascripts/clusters_list/components/clusters.vue +++ b/app/assets/javascripts/clusters_list/components/clusters.vue @@ -1,22 +1,32 @@