Mark mirrors as failed 1 hour after they started
What does this MR do?
We call Project#mark_stuck_remote_mirrors_as_failed!
from the
Git::BaseHooksService
. So that gets called every time we push tags
or branches.
Before this would only mark started mirrors as failed if they had been started 24 hours ago. A push would never take 24 hours, especially not when we run it so often.
Lowering that threshold 1 hour should at least allow us to retry broken mirrors more often on pushes.
The timeout for the initial push is set somewhat longer to accommodate for pushing large repos. Both numbers are currently picked arbitrarily.
This is the first step in improving reliability of push mirrors in https://gitlab.com/gitlab-org/gitlab-ce/issues/63178
Does this MR meet the acceptance criteria?
Conformity
-
Changelog entry for user-facing changes, or community contribution. Check the link for other scenarios. - [-] Documentation created/updated or follow-up review issue created
-
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content
Edited by Bob Van Landuyt