diff options
author | Sami Hiltunen <shiltunen@gitlab.com> | 2021-08-10 15:46:49 +0300 |
---|---|---|
committer | Sami Hiltunen <shiltunen@gitlab.com> | 2021-08-19 14:41:54 +0300 |
commit | 73eb0f44c0484f4730ca334c9b27e8cd0ad9ec86 (patch) | |
tree | 0bf07af216d618f203feacadd0383103dcd12a83 /internal/gitaly/service/ref/refname_test.go | |
parent | b3bb31655c67e0fcad90de1c31de58686849a393 (diff) |
Don't schedule replication jobs in SetAuthoritativeStorage
SetAuthoritative storage sets a replica's generation to be the latest
one as a way to accept dataloss when all copies of the repository on the
latest generation are lost. To replicate the promoted replica to the other
storages, the method also schedules replication jobs to other storages.
Since the introduction of the automatic reconciler, this is not really
necessary as the reconciler would handle the job scheduling. The method
also doesn't take into account whether the other storages are assigned
hosts of the repository, and instead replicates the repository everywhere.
The method should also be expanded to handle repository ID propagation via
the replication jobs. To solve both problems at once, this commit removes
the explicit job scheduling from SetAuthoritativeStorage and let's it rely
on the reconciler to schedule the jobs correctly.
Changelog: fixed
Diffstat (limited to 'internal/gitaly/service/ref/refname_test.go')
0 files changed, 0 insertions, 0 deletions