Skip to content

Include Elasticsearch query params in Performance Bar

Dylan Griffith requested to merge add-query-params-to-performance-bar into master

What does this MR do?

Include Elasticsearch query params in Performance Bar. These were missed when we introduced this and they are valuable for understanding the Elasticsearch queries being made since (as the included test shows) we use routing keys in the query param which improve the performance of the searches.

Screenshots

Screen_Shot_2020-07-15_at_10.46.33_am

Does this MR meet the acceptance criteria?

Conformity

Availability 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
Edited by Dylan Griffith

Merge request reports