Skip to content

Zoekt search enable should respect experiment setting

Terri Chu requested to merge 457917-change-reallocation-logic-to-use-expirement-setting into master

What does this MR do and why?

Related to #457917 (closed)

Will likely merge this in place of Draft: Zoekt should respect the experiment sett... (!150692 - closed)

This change will affect Zoekt rollout after 2024-04-25.

Any zoekt enabled namespaces which were created before 2024-04-25 will have Zoekt search enabled by the rollout scheduling task. All other zoekt enabled namespaces will respect the experiment setting in the root namespace.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

N/A

How to set up and validate locally

I did not have time to manually test this but you'd also have to emulate SaaS locally.

The steps should be something like:

  1. emulate SAAS
  2. create a new group
  3. set the experiment setting to false
  4. create a zoekt enabled namespace for the group
  5. run the scheduler
  6. verify the search is not enabled
Edited by Terri Chu

Merge request reports