Skip to content

Add logging to Gitlab::RepositorySetCache#fetch if cache missed

Gavin Hinfeyrequested to merge
517676-add-logging-for-cache-miss into master

What does this MR do and why?

Adds structured logging to Gitlab::RepositorySetCache#fetch on cache miss operations to help investigate potential correlation between cache misses and errors described in #506430.

The logging includes correlation_id and cache key, which will allow us to validate whether specific errors are related to stale cache issues. This data will help determine if fixing the non-atomic cache behavior would resolve the observed problems.

The logging is controlled by the repository_set_cache_logging feature flag and only triggers on cache misses.

References

Related to Add logging to understand impact of SetReposito... (#517676 - closed)

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.

Edited by Gavin Hinfey

Merge request reports