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

Conversation

@ajhuh-mdb
Copy link
Collaborator

@ajhuh-mdb ajhuh-mdb commented Nov 14, 2024

DESCRIPTION

  • Add new FAQ entry regarding hardware requirements for destination clusters.
  • Add note on destination cluster disk space requirements in the Quickstart and oplog pages.

STAGING

JIRA

https://jira.mongodb.org/browse/DOCSP-44478

@netlify
Copy link

netlify bot commented Nov 14, 2024

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

Name Link
🔨 Latest commit 99821cc
🔍 Latest deploy log https://app.netlify.com/sites/docs-cluster-to-cluster-sync/deploys/6737a720dc0e9c0008698abf
😎 Deploy Preview https://deploy-preview-488--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.

The number of nodes in the destination replica set does not have
to equal the number of nodes in the source replica set.

.. important::
Copy link
Collaborator Author

@ajhuh-mdb ajhuh-mdb Nov 14, 2024

Choose a reason for hiding this comment

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

for internal reviewer: Do you think there's a better way to format/include this information in this Quickstart section? Product specified that it'd be best here, but the way that the copy flows seems a bit clunky, and I can't really see a better way to format this section given it's within a procedure.

Copy link
Collaborator

@jmd-mongo jmd-mongo Nov 14, 2024

Choose a reason for hiding this comment

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

I understand your concern, but I can't think of a better way to present it. The copy definitely is important and that placement makes sense to me even if it adds some visual noise due to multiple admonitions in the same step. I say go with it. Thanks!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Gotcha, thanks!

@jmd-mongo jmd-mongo self-requested a review November 14, 2024 21:10
Copy link
Collaborator

@jmd-mongo jmd-mongo left a comment

Choose a reason for hiding this comment

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

Thanks for this issue, @ajhuh-mdb! I left a suggestion and responded to your note re the admonition.

Thanks,
Joe

source/faq.txt Outdated
``mongosync`` writes and that the sync can keep up with the source cluster's
workload.

Additionally, the destination cluster must have enough disk storage to
Copy link
Collaborator

Choose a reason for hiding this comment

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

[suggestion]

Consider using includes/fact-oplog-disk-storage.rst here. I think the only difference would be the loss of "Additionally".

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Noted!

The number of nodes in the destination replica set does not have
to equal the number of nodes in the source replica set.

.. important::
Copy link
Collaborator

@jmd-mongo jmd-mongo Nov 14, 2024

Choose a reason for hiding this comment

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

I understand your concern, but I can't think of a better way to present it. The copy definitely is important and that placement makes sense to me even if it adds some visual noise due to multiple admonitions in the same step. I say go with it. Thanks!

@ajhuh-mdb ajhuh-mdb requested a review from jmd-mongo November 15, 2024 16:25
@ajhuh-mdb
Copy link
Collaborator Author

Thanks for the initial review @jmd-mongo ! This should be ready for another pass.

Copy link
Collaborator

@jmd-mongo jmd-mongo left a comment

Choose a reason for hiding this comment

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

Thanks for this updates, @ajhuh-mdb! This one LGTM at this point.

Thanks,
Joe

@ajhuh-mdb ajhuh-mdb requested a review from edobranov November 15, 2024 16:59
@ajhuh-mdb
Copy link
Collaborator Author

Hi @edobranov 👋 Whenever you get the chance, could you review this docs PR regarding hardware specifications for destination clusters? TIA!

Copy link
Contributor

@edobranov edobranov left a comment

Choose a reason for hiding this comment

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

LGTM % a couple optional suggestions, thanks!

data size being migrated and the destination oplog entries from the initial
sync. For example, to migrate 10 GB of data, the destination cluster must have
at least 10 GB available for the data and another 10 GB for the insert oplog
entries.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
entries.
entries from the initial sync.

To reduce the overhead of the destination oplog entries, you can:

- Use the :setting:`~replication.oplogSizeMB` setting to lower the destination
cluster's oplog size
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
cluster's oplog size
cluster's oplog size.

@ajhuh-mdb ajhuh-mdb merged commit b4ee80f into mongodb:master Nov 15, 2024
4 checks passed
ajhuh-mdb added a commit to ajhuh-mdb/docs-cluster-to-cluster-sync that referenced this pull request Nov 15, 2024
* DOCSP-44478-hardware-specs

* important note

* JD feedback

* ED feedback

(cherry picked from commit b4ee80f)
@ajhuh-mdb
Copy link
Collaborator Author

💚 All backports created successfully

Status Branch Result
v1.9
v1.8

Questions ?

Please refer to the Backport tool documentation

ajhuh-mdb added a commit to ajhuh-mdb/docs-cluster-to-cluster-sync that referenced this pull request Nov 15, 2024
* DOCSP-44478-hardware-specs

* important note

* JD feedback

* ED feedback

(cherry picked from commit b4ee80f)
ajhuh-mdb added a commit that referenced this pull request Nov 15, 2024
* DOCSP-44478-hardware-specs

* important note

* JD feedback

* ED feedback

(cherry picked from commit b4ee80f)
ajhuh-mdb added a commit that referenced this pull request Nov 15, 2024
* DOCSP-44478-hardware-specs

* important note

* JD feedback

* ED feedback

(cherry picked from commit b4ee80f)
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