Skip to content

PSMDB-1284 Doc: File copy based initial sync #912

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

Merged
merged 13 commits into from
Jul 28, 2025
Merged

Conversation

nastena1606
Copy link
Contributor

new file: docs/initial-sync.md
modified: docs/psmdb-pro.md
modified: mkdocs-base.yml

new file:   docs/initial-sync.md
	modified:   docs/psmdb-pro.md
	modified:   mkdocs-base.yml
Copy link

render bot commented Aug 23, 2024

Copy link
Contributor

@igorsol igorsol left a comment

Choose a reason for hiding this comment

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

Looks good but exact versions should be reviewed everywhere when we are ready for release

Addressed comments
Added a high-level workflow
Added confog parameters description
Added limitations to not send users to upstream docs
Co-authored-by: radoslawszulgo <[email protected]>
@it-percona it-percona temporarily deployed to PSMDB-1284-Doc-FCBIS-7.0 - psmdb-docs-7.0 PR #912 July 24, 2025 11:24 — with Render Destroyed
Co-authored-by: radoslawszulgo <[email protected]>
@it-percona it-percona temporarily deployed to PSMDB-1284-Doc-FCBIS-7.0 - psmdb-docs-7.0 PR #912 July 24, 2025 11:24 — with Render Destroyed
Co-authored-by: radoslawszulgo <[email protected]>
@radoslawszulgo
Copy link
Contributor

I'd change the file name to fcbis-initial-sync.md or something. As there's also logical initial sync - the name of the file (and the URL) might be misleading.

* Don't run backups on either sync source or syncing nodes
* Don't write to the `local` database on the syncing node
* You cannot use the same sync source for multiple target nodes simultaneously because only one backup cursor can exist at any moment.
* If you're using encrypted storage, Percona Server for MongoDB applies the encryption key from the sync source node to secure the data on the syncing node.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add initial sync doesn't work for in-memory storage engine

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Percona Memory Engine?

@it-percona it-percona temporarily deployed to PSMDB-1284-Doc-FCBIS-7.0 - psmdb-docs-7.0 PR #912 July 24, 2025 13:44 — with Render Destroyed

When a new member joins the replica set, it receives the data from the existing replica set node via the initial sync.

In Percona Server for MongoDB, you can choose a file copy-based initial sync for a new node. You must have WiredTiger defined as the storage.
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
In Percona Server for MongoDB, you can choose a file copy-based initial sync for a new node. You must have WiredTiger defined as the storage.
In Percona Server for MongoDB, you can choose a file copy-based initial sync for a new node.

| `fileBasedInitialSyncMaxLagSec` | integer | 300 | Specifies the max lag in seconds between the syncing node and the sync source to mark the file copy based initial sync as done successfully |
| `fileBasedInitialSyncMaxCyclesWithoutProgress` | integer | 3 | Specifies the max number of cycles to clone updates while the lag between the syncing node and the sync source is higher than `fileBasedInitialSyncMaxLagSec` |


Copy link
Contributor

Choose a reason for hiding this comment

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

I'd appreciate a full configuration example with all the options

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As discussed with @igorsol we don't want users to mess with these params. yes, we list it. And the defaults should be enough. The way how to enable FCBIS is given.
@igorsol please confirm

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree that its is enough to list parameters with descriptions. By default FCBIS should work with default parameters. Only dbadmins with good understanding of the topic should try to alter those parameters. I assume they know how to put parameter into config file.

@nastena1606 nastena1606 merged commit 8fe454c into 7.0 Jul 28, 2025
@nastena1606 nastena1606 deleted the PSMDB-1284-Doc-FCBIS-7.0 branch July 28, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants