Fix cache walker to only walk each path once (!1769) · Merge requests · GitLab.org / gitaly

Paul Okstad (ex-GitLab) requested to merge po-cache-walker-one-per-path into master Jan 20, 2020

Currently, we limit cache walkers based on the storage name. This is a problem because the storage name is always unique on gitlab.com, but the path is not (see #2292 (comment 274006482)). This means that the file walkers will compete against each other and create lots of noise in our telemetry.

This changes the behavior so that each only one cache walker is spawned per storage path.

Part of #2388 (closed)

Edited Nov 18, 2021 by GitLab Release Tools Bot

Fix cache walker to only walk each path once

Merge request reports