Skip to content

Resolve "Move Elasticsearch admin options to top level"

Denys Mishunov requested to merge 13533-elastic-search-1 into master

Note: There's a CE port that has to be merged as well when this one gets merged

What does this MR do?

  1. Moves Admin area > Integrations > Elasticsearch to the top-level Admin area > Elasticsearch
  2. Splits settings into two views: "GitLab index" and "Settings"
GitLab index Settings
Screenshot_2019-08-28_at_13.00.23 Screenshot_2019-08-28_at_13.01.33
  1. In "GitLab index" view adds the ability to show/hide AWS credentials area via a checkbox

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Performance and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

References #13533 (closed)

Edited by Markus Koller

Merge request reports