Skip to content
Snippets Groups Projects

search: only marshal config for repos that have changed

Merged Administrator requested to merge k/search-subset into main

Created by: keegancsmith

There was a subtle but benign bug in the previous code when using the new "only get config for what has changed" issue. We would return a 404 error for every repository that hasn't changed. This would be ignored by zoekt and as a side effect would just cause log and metric spam.

We fix the code path by only marshalling configuration for repositories that have changed.

Merge request reports

Merged by avatar (Jul 5, 2025 1:43pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading