Resolve "ProjectSyncWorker should skip projects that have a broken Gitaly shard"
requested to merge 4957-projectsyncworker-should-skip-projects-that-have-a-broken-gitaly-shard into master
What does this MR do?
This MR aims to skip syncing for a given Project within the Geo::ProjectSyncWorker
worker if the storage associated to the Project is unhealthy.
Are there points in the code the reviewer needs to double check?
N/A
Why was this MR needed?
To address #4957 (closed)
Screenshots (if relevant)
N/A
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary [ ] Documentation created/updated[ ] API support added-
Tests added for this feature/bug - Review
[ ] Has been reviewed by UX[ ] Has been reviewed by Frontend-
Has been reviewed by Backend [ ] Has been reviewed by Database
-
EE specific content should be in the top level /ee
folder -
Conform by the merge request performance guides -
Conform by the style guides -
Squashed related commits together [ ] Internationalization required/considered-
If paid feature, have we considered GitLab.com plan and how it works for groups and is there a design for promoting it to users who aren't on the correct plan -
End-to-end tests pass ( package-qa
manual pipeline job)
What are the relevant issue numbers?
Closes #4957 (closed)
Edited by Nick Thomas