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

Conversation

morales-mario-mdb
Copy link
Collaborator

@morales-mario-mdb morales-mario-mdb commented Feb 5, 2025

DESCRIPTION

Removed the second sentence in "You can't enable the enableUserWriteBlocking option in the /start request. Ensure that no writes are made to the source or destination cluster during the migration." which is inaccurate as users can actually write to the source cluster/destination during migrations.

STAGING

The change is done under Pre 6.0 limitations section: https://deploy-preview-591--docs-cluster-to-cluster-sync.netlify.app/reference/limitations/#pre-6.0-migrations

JIRA

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

Copy link

netlify bot commented Feb 5, 2025

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

Name Link
🔨 Latest commit 246aefc
🔍 Latest deploy log https://app.netlify.com/sites/docs-cluster-to-cluster-sync/deploys/67a3f1aebeb44600086aa4e4
😎 Deploy Preview https://deploy-preview-591--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.

@morales-mario-mdb morales-mario-mdb changed the title (DOCSP-46983) Removes sentence in limitations for pre 6 migrations DOCSP-46983-removes-sentence-in-limitations-for-pre-6-migrations Feb 7, 2025
Copy link
Collaborator

@jeff-allen-mongo jeff-allen-mongo left a comment

Choose a reason for hiding this comment

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

This update LGTM from a copy perspective, but I wanted to call out that on the /commit endpoint page we still have a bullet under requirements that states:

Stop your application to prevent any further writes to the source cluster. During commit, you may still read from the source cluster.

Warning

If you write to the source cluster during commit, you might experience data loss.

I'm wondering if we need to remove that bullet as well? It would be good to get product / engineering input on this.

@morales-mario-mdb
Copy link
Collaborator Author

morales-mario-mdb commented Feb 10, 2025

The PR appears to no longer be required. Asked Jane Tio to review and she replied:

Hey Mario! I actually don't think we need the PR anymore, since the docs were changed from when I created the ticket. The sentence about being able to write to the source post /commit is accurate now.

Should we close the ticket? @jeff-allen-mongo

@morales-mario-mdb
Copy link
Collaborator Author

@jeff-allen-mongo Just confirmed with Jane, this PR is no longer required. The docs have changed to:

Ensure that no writes are made to the source cluster after you call the /commit endpoint.

The sentence that was originally wrong was:

Ensure that no writes are made to the source or destination cluster during the migration.

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.

2 participants