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

Approval is optional

Merged by avatar (Jul 25, 2025 2:43am UTC)

Merge details

  • Changes merged into main with 72231deb.
  • Deleted the source branch.

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