Use search_service to determine type
Merged
requested to merge 434899-fix-search_type-reporting-for-blobs-api-search-in-zoekt into master
What does this MR do and why?
Related to #434899 (closed)
Customer reported issues in Slack thread (internal) with seeing Advanced search syntax for group level API searches (when they should be seeing Zoekt). I noticed in Kibana (internal) that the search_type
is being reported incorrectly as advanced_search
for API searches but there are zoekt_calls
present.
This MR fixes the reporting and adds a spec to test it out
Screenshots or screen recordings
N/A
How to set up and validate locally
To test functionality for Zoekt search and index:
- enable zoekt in gdk
- configure zoekt in your environment
- enable it for a namespace
- perform an API group level search for that namespace
- verify in
log/api_json.log
that themeta.search.type
is reported correctly aszoekt
. On master it will report asadvanced_search
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Terri Chu
Merge request reports
Activity
Please register or sign in to reply