Skip to content
This repository was archived by the owner on May 1, 2025. It is now read-only.

DOCSP-48525-disaster-recovery-faq-entry #679

Conversation

amalhotra-mdb
Copy link
Collaborator

@amalhotra-mdb amalhotra-mdb commented Mar 18, 2025

Copy link

netlify bot commented Mar 18, 2025

Deploy Preview for docs-cluster-to-cluster-sync ready!

Name Link
🔨 Latest commit eb2f63f
🔍 Latest deploy log https://app.netlify.com/sites/docs-cluster-to-cluster-sync/deploys/67db2f3fdbe3e90008a73151
😎 Deploy Preview https://deploy-preview-679--docs-cluster-to-cluster-sync.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Collaborator

@gmiller-mdb gmiller-mdb left a comment

Choose a reason for hiding this comment

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

LGTM with one suggestion!

source/faq.txt Outdated

No, you can't currently set up a Disaster Recovery cluster with ``mongosync``,
since ``mongosync`` must commit in order to safely accept traffic to the
Copy link
Collaborator

Choose a reason for hiding this comment

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

[s] Do you want to add the bold to 'must commit' here like in the ticket?

Copy link

@mvankeulen94 mvankeulen94 left a comment

Choose a reason for hiding this comment

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

thank you for the speedy turnaround! a few changes from my end - I am also socializing this change with internal stakeholders

@@ -23,6 +23,8 @@ Can I change the load level while ``mongosync`` is syncing?
Yes, you can adjust the cluster workload level during a migration by
following the steps in :ref:`c2c-reconfigure-mid-migration`.

.. _c2c-faq-reads-writes-mongosync:

Can I perform reads or writes to my destination cluster while ``mongosync`` is syncing?

Choose a reason for hiding this comment

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

can we add this as the question right under "Can I perform reads or writes to my destination cluster while mongosync is syncing?" ?

source/faq.txt Outdated
efficiently across the destination cluster’s shards after migration.

Can I use ``mongosync`` to set up a Disaster Recovery cluster?

Choose a reason for hiding this comment

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

on second thought, it's better to say "maintain" instead of "set up" here, since the key issue is the lack of an ability to keep the cluster up to date with the source

source/faq.txt Outdated
--------------------------------------------------------------

No, you can't currently set up a Disaster Recovery cluster with ``mongosync``,

Choose a reason for hiding this comment

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

"set up" -> "maintain"

source/faq.txt Outdated

No, you can't currently set up a Disaster Recovery cluster with ``mongosync``,
since ``mongosync`` **must commit** in order to safely accept traffic to the

Choose a reason for hiding this comment

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

@@ -0,0 +1,4 @@
Until you've called commit on ``mongosync`` and ``canWrite`` successfully

Choose a reason for hiding this comment

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

@@ -219,6 +219,10 @@ For any continuous synchronization use cases with ``mongosync``, ensure that
``mongosync`` commits before cutting over from the source to the

Choose a reason for hiding this comment

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

this line starting with "For any continuous synchronization" is now redundant with the new textbox so we can remove it.

@amalhotra-mdb
Copy link
Collaborator Author

Thank you for your review @mvankeulen94 ! Back to you with the updates.

Copy link

@mvankeulen94 mvankeulen94 left a comment

Choose a reason for hiding this comment

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

lgtm, thank you!

@amalhotra-mdb amalhotra-mdb merged commit 9036320 into mongodb:master Mar 24, 2025
4 checks passed
amalhotra-mdb added a commit to amalhotra-mdb/docs-cluster-to-cluster-sync that referenced this pull request Mar 24, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue

(cherry picked from commit 9036320)
amalhotra-mdb added a commit to amalhotra-mdb/docs-cluster-to-cluster-sync that referenced this pull request Mar 24, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue

(cherry picked from commit 9036320)
amalhotra-mdb added a commit to amalhotra-mdb/docs-cluster-to-cluster-sync that referenced this pull request Mar 24, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue

(cherry picked from commit 9036320)
amalhotra-mdb added a commit to amalhotra-mdb/docs-cluster-to-cluster-sync that referenced this pull request Mar 24, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue

(cherry picked from commit 9036320)
@amalhotra-mdb
Copy link
Collaborator Author

💚 All backports created successfully

Status Branch Result
v1.11
v1.10
v1.9
v1.8

Questions ?

Please refer to the Backport tool documentation

amalhotra-mdb added a commit that referenced this pull request Mar 24, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue

(cherry picked from commit 9036320)
amalhotra-mdb added a commit that referenced this pull request Mar 24, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue

(cherry picked from commit 9036320)
amalhotra-mdb added a commit that referenced this pull request Mar 24, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue

(cherry picked from commit 9036320)
amalhotra-mdb added a commit that referenced this pull request Mar 24, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue

(cherry picked from commit 9036320)
amalhotra-mdb added a commit to amalhotra-mdb/docs-cluster-to-cluster-sync that referenced this pull request Mar 25, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue
amalhotra-mdb added a commit that referenced this pull request Mar 25, 2025
* DOCSP-48525-disaster-recovery-faq-entry

* wording

* build error

* reviewer request

* add important

* monospace

* reviewer updates

* render issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants