Zoekt: Rollout should skip the namespaces which can not be indexed

Ravi Kumarrequested to merge
530423-zoekt-rolloutworker-should-skip-the-namespaces into master

What does this MR do and why?

During the Zoekt rollout, we realized that a lot of times the same namespace was coming into the batch which was just tried before and failed. In this MR, adding the metadata last_rollout_failed_at on the failure. And in the SelectionService, skipping this namespace which has this metadata set. I am not adding delay, in the SelectionService, completely ignoring the failed namespaces. I think we can completely ignore and let the worker run until the last namespace. We can later investigate how to proceed with the failed namespaces. May be we can try to rollout all failed namespaces with batch size 1. Or something else.

References

Screenshots or screen recordings

Before After

How to set up and validate locally

MR acceptance checklist

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

Related to #530423 (closed)

Edited by Ravi Kumar

Merge request reports