Skip to content

Add helpful 400 message when enabling Zoekt indexing when FF is disabled

Dylan Griffith requested to merge 407331-zoekt-api-cleanups-2 into master

What does this MR do and why?

Add helpful 400 message when enabling Zoekt indexing when FF is disabled

This was raised in #407331 (closed) . If a user tries to enabling indexing for a namespace or explicitly trigger indexing for a project they will just get back 200 from this API but nothing will happen. This may be confusing for self-managed admins setting this up and we could save them some time debugging if we return a 400 early and just say that it can't be done.

We still want to allow the other APIs in here when the feature flag is disabled because they may still reasonably want to disable a specific namespace or figure out what is being indexed before they turn a feature flag back on.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #407331 (closed)

Merge request reports