Skip to content

Add detail to disaster recovery docs

James Ramsay (ex-GitLab) requested to merge jramsay-4399-clarify-dr-docs into master

What does this MR do?

The disaster recovery docs are incomplete and ambiguous. This MR adds more detail.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

From gitlab-org/gitlab-ee#4399:

  • Add instructions for taking down primary if it isn’t completely down
  • ‘Log-in’ is ambiguous in Log-in to your secondary node with.... Explicitly state SSH
  • Specify exactly what gitlab.rb changes are needed to apply to the secondary in order to make it a primary
    • geo_primary_role, geo_secondary_role, listen address, trust_addresses, md5_addresses?
    • Determine if it matters if geo_primary_role['enable'] = false. If it must be true, then we need the trust_auth_cidr_addresses config as well.
    • Decide if we should store a replica of the primary gitlab.rb on the secondary. If so, add instructions during Geo setup (or create an issue to make this happen)
  • Once DNS change to primary has propagated, update URL in gitlab.rb to reflect.

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ee/issues/4399

Edited by Toon Claes

Merge request reports