Skip to content

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

Edited by Bob Van Landuyt

Merge request reports