Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add DR with active-passive and external storage #3679

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

aneta-petrova
Copy link
Member

What changes are you introducing?

Adding a new disaster recovery scenario: Active and passive server with external storage.

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

A follow-up to #3558
The new scenario follows the structure established in #3615 for consistency

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

Now that I'm working on the second scenario, I can already see opportunities for reuse or snippets. However, I think it will be better to deal with this after all the planned scenarios are in place.

Checklists

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.13/Katello 4.15 (EL9 only)
  • Foreman 3.12/Katello 4.14 (Satellite 6.16)
  • Foreman 3.11/Katello 4.13 (orcharhino 6.11 on EL8 only; orcharhino 7.0 on EL8+EL9)
  • Foreman 3.10/Katello 4.12
  • Foreman 3.9/Katello 4.11 (Satellite 6.15; orcharhino 6.8/6.9/6.10)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • We do not accept PRs for Foreman older than 3.7.

@github-actions github-actions bot added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective Needs testing Requires functional testing labels Feb 18, 2025
@aneta-petrova aneta-petrova removed the Needs testing Requires functional testing label Feb 18, 2025
Copy link

github-actions bot commented Feb 18, 2025

The PR preview for 9aba517 is available at theforeman-foreman-documentation-preview-pr-3679.surge.sh

The following output files are affected by this PR:

show diff

show diff as HTML

@aneta-petrova
Copy link
Member Author

@ehelms Can you please review another DR scenario? I am once again making a lot of barely-educated guesses and I added several questions as comments so please keep that in mind and read carefully :) Also, I'm not sure what level of detail we are aiming for with this scenario: Do we want to keep things high-level like the virtualization scenario or can we/should we provide more detailed instructions?


In this scenario, you maintain two {ProjectServer}s and you store critical data externally on a network-attached storage (NAS).
The primary server is active while the secondary server remains passive.
If the primary server fails, you attach the NAS to your secondary server and turn it into your new primary server.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@evgeni They could attach it to both, and then flip it to write for the passive server when it needs to become the primary? Or is this a better /safer method?

. Stop the active server to mimic failure.
// How? foreman-maintain service stop?
. Start the passive server.
// How? foreman-maintain service start?
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can see two scenarios:

  1. A passive server that is running all the time but inaccessible. In this case the user would simply ensure storage is attached properly and flip DNS.
  2. A passive server that is something like an image, that gets turned up, storage attached and then update is run.
  3. A machine that is spun up, restore is run with storage attached.

@evgeni Vet me here

@aneta-petrova aneta-petrova requested a review from rh-max February 19, 2025 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs style review Requires a review from docs style/grammar perspective Needs tech review Requires a review from the technical perspective
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants