From 6bb42029febc9cc30efad1b2be4deda44fb579a4 Mon Sep 17 00:00:00 2001 From: Maya Raman Date: Tue, 14 Jan 2025 11:48:45 -0500 Subject: [PATCH 1/7] reversal changes --- source/includes/fact-verifier-limitations.rst | 4 ---- source/includes/table-permissions-self-hosted.rst | 15 +++++++++++++++ source/reference/api/reverse.txt | 3 +-- 3 files changed, 16 insertions(+), 6 deletions(-) diff --git a/source/includes/fact-verifier-limitations.rst b/source/includes/fact-verifier-limitations.rst index 93ba3f3a0..646e36a38 100644 --- a/source/includes/fact-verifier-limitations.rst +++ b/source/includes/fact-verifier-limitations.rst @@ -22,10 +22,6 @@ The embedded verifier has the following limitations: verification process restarts from the beginning. This can cause verification to fall substantially behind the migration. -- The :ref:`/reverse ` endpoint disables the - verifier. It remains disabled after additional calls to the - ``/reverse`` endpoint. - - .. include:: /includes/fact-verifier-buildIndexes Unsupported Verification Checks diff --git a/source/includes/table-permissions-self-hosted.rst b/source/includes/table-permissions-self-hosted.rst index eb2c05ea9..b9a82c1f1 100644 --- a/source/includes/table-permissions-self-hosted.rst +++ b/source/includes/table-permissions-self-hosted.rst @@ -48,6 +48,21 @@ - :authrole:`readWriteAnyDatabase` - :authrole:`restore` + * - Multiple Reversals + - - :authrole:`backup` + - :authrole:`clusterManager` + - :authrole:`clusterMonitor` + - :authrole:`dbAdminAnyDatabase` + - :authrole:`readWriteAnyDatabase` + - :authrole:`restore` + + - - :authrole:`backup` + - :authrole:`clusterManager` + - :authrole:`clusterMonitor` + - :authrole:`dbAdminAnyDatabase` + - :authrole:`readWriteAnyDatabase` + - :authrole:`restore` + For details on server roles, see: :ref:`authorization`. To update user permissions, see: :dbcommand:`grantRolesToUser`. diff --git a/source/reference/api/reverse.txt b/source/reference/api/reverse.txt index e2923b2d3..34863909a 100644 --- a/source/reference/api/reverse.txt +++ b/source/reference/api/reverse.txt @@ -159,8 +159,7 @@ Embedded Verifier ~~~~~~~~~~~~~~~~~ The embedded verifier is enabled by default for replica set -migrations and performs verification checks for the forward -direction of reversible sync. When you call the ``/reverse`` +migrations. When you call the ``/reverse`` endpoint, ``mongosync`` disables the verifier. The verifier remains disabled, even after additional calls to the ``/reverse`` endpoint. From 43acb9e2a12df8bf9b2f9ac9daf7475195a8346a Mon Sep 17 00:00:00 2001 From: Maya Raman Date: Tue, 14 Jan 2025 15:33:54 -0500 Subject: [PATCH 2/7] adding info --- source/includes/verify-reversible-migrations.rst | 2 ++ source/reference/api/reverse.txt | 5 +---- source/release-notes/1.10.txt | 10 +++++++++- 3 files changed, 12 insertions(+), 5 deletions(-) create mode 100644 source/includes/verify-reversible-migrations.rst diff --git a/source/includes/verify-reversible-migrations.rst b/source/includes/verify-reversible-migrations.rst new file mode 100644 index 000000000..06e63e0e0 --- /dev/null +++ b/source/includes/verify-reversible-migrations.rst @@ -0,0 +1,2 @@ +In version 1.10, ``mongosync`` enables the embedded verifier for +forward and reverse directions of reversible migrations. \ No newline at end of file diff --git a/source/reference/api/reverse.txt b/source/reference/api/reverse.txt index 34863909a..0e52e8b4a 100644 --- a/source/reference/api/reverse.txt +++ b/source/reference/api/reverse.txt @@ -159,10 +159,7 @@ Embedded Verifier ~~~~~~~~~~~~~~~~~ The embedded verifier is enabled by default for replica set -migrations. When you call the ``/reverse`` -endpoint, ``mongosync`` disables the verifier. The verifier -remains disabled, even after additional calls to the -``/reverse`` endpoint. +migrations. Endpoint Protection ~~~~~~~~~~~~~~~~~~~ diff --git a/source/release-notes/1.10.txt b/source/release-notes/1.10.txt index 45fe720c9..2582b0e5f 100644 --- a/source/release-notes/1.10.txt +++ b/source/release-notes/1.10.txt @@ -17,4 +17,12 @@ Release Notes for mongosync 1.10 .. _1.10.0-c2c-release-notes: This page describes changes and new features introduced in -{+c2c-full-product-name+} 1.10. \ No newline at end of file +{+c2c-full-product-name+} 1.10. + +1.10.0 Release +-------------- + +Upgrades to Embedded Verifier +~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + +- ..include:: /includes/verify-reversible-migrations.rst From 7da753db0559badfd12e8895d7db29d0749cf0ce Mon Sep 17 00:00:00 2001 From: Maya Raman Date: Tue, 14 Jan 2025 16:00:38 -0500 Subject: [PATCH 3/7] updating start --- source/reference/api/start.txt | 5 ----- 1 file changed, 5 deletions(-) diff --git a/source/reference/api/start.txt b/source/reference/api/start.txt index 880f454c6..b28e2678c 100644 --- a/source/reference/api/start.txt +++ b/source/reference/api/start.txt @@ -167,11 +167,6 @@ Request Body Parameters * Reversible sync when ``buildIndexes`` is set to ``never``. - * Reversible sync with the embedded verifier. The - verifier supports the initial forward direction of - reversible sync. When you call the ``/reverse`` - endpoint it disables the verifier. - For more information, see the :ref:`reverse ` endpoint. Default value is ``false``. From 8add1a754668407f6aa5d5e83f199e421ec2de8c Mon Sep 17 00:00:00 2001 From: Maya Raman Date: Thu, 16 Jan 2025 14:39:31 -0500 Subject: [PATCH 4/7] feedback --- source/includes/table-permissions-atlas.rst | 4 ++-- source/release-notes/1.10.txt | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/source/includes/table-permissions-atlas.rst b/source/includes/table-permissions-atlas.rst index cf67d5c7c..e524b6919 100644 --- a/source/includes/table-permissions-atlas.rst +++ b/source/includes/table-permissions-atlas.rst @@ -22,7 +22,7 @@ - atlasAdmin - * - write-blocking or reversing + * - write-blocking or reversing (including multiple reversals) - source cluster - @@ -30,7 +30,7 @@ - backup - bypassWriteBlockMode privilege - * - write-blocking or reversing + * - write-blocking or reversing (including multiple reversals) - destination cluster - diff --git a/source/release-notes/1.10.txt b/source/release-notes/1.10.txt index 2582b0e5f..084000f82 100644 --- a/source/release-notes/1.10.txt +++ b/source/release-notes/1.10.txt @@ -25,4 +25,4 @@ This page describes changes and new features introduced in Upgrades to Embedded Verifier ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ -- ..include:: /includes/verify-reversible-migrations.rst +- .. include:: /includes/verify-reversible-migrations.rst From a8196210e9b49167f855d90487cfe4684cc16196 Mon Sep 17 00:00:00 2001 From: Maya Raman Date: Thu, 16 Jan 2025 16:12:17 -0500 Subject: [PATCH 5/7] stub column, title --- source/includes/table-permissions-atlas.rst | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/source/includes/table-permissions-atlas.rst b/source/includes/table-permissions-atlas.rst index e524b6919..0a102a855 100644 --- a/source/includes/table-permissions-atlas.rst +++ b/source/includes/table-permissions-atlas.rst @@ -4,6 +4,7 @@ .. list-table:: :header-rows: 1 + :stub-columns: 1 * - Sync Type - Target @@ -22,7 +23,7 @@ - atlasAdmin - * - write-blocking or reversing (including multiple reversals) + * - write-blocking, reversing, or multiple reversals - source cluster - @@ -30,7 +31,7 @@ - backup - bypassWriteBlockMode privilege - * - write-blocking or reversing (including multiple reversals) + * - write-blocking, reversing, or multiple reversals - destination cluster - From 9686c1b69f31cc736126b479ac7e6ac9b1c72b83 Mon Sep 17 00:00:00 2001 From: Maya Raman Date: Thu, 16 Jan 2025 16:32:14 -0500 Subject: [PATCH 6/7] table formatting --- source/includes/table-permissions-atlas.rst | 38 ++++++--------------- 1 file changed, 11 insertions(+), 27 deletions(-) diff --git a/source/includes/table-permissions-atlas.rst b/source/includes/table-permissions-atlas.rst index 0a102a855..e51a5947d 100644 --- a/source/includes/table-permissions-atlas.rst +++ b/source/includes/table-permissions-atlas.rst @@ -7,37 +7,21 @@ :stub-columns: 1 * - Sync Type - - Target - - Required Permissions + - Required Source Permissions + - Required Destination Permissions * - default - - source cluster - - - - - atlasAdmin - - backup - - * - default - - destination cluster - - - - - atlasAdmin + - - atlasAdmin + - backup + - - atlasAdmin * - write-blocking, reversing, or multiple reversals - - source cluster - - - - - atlasAdmin - - backup - - bypassWriteBlockMode privilege - - * - write-blocking, reversing, or multiple reversals - - destination cluster - - - - - atlasAdmin - - backup - - bypassWriteBlockMode privilege + - - atlasAdmin + - backup + - bypassWriteBlockMode privilege + - - atlasAdmin + - backup + - bypassWriteBlockMode privilege For details on Atlas roles, see: :atlas:`Atlas User Roles `. From 59a5978023dadc68bb0c2dc1b5a53cfc85604c21 Mon Sep 17 00:00:00 2001 From: Maya Raman Date: Thu, 16 Jan 2025 16:38:05 -0500 Subject: [PATCH 7/7] remove backup --- source/includes/table-permissions-atlas.rst | 3 --- 1 file changed, 3 deletions(-) diff --git a/source/includes/table-permissions-atlas.rst b/source/includes/table-permissions-atlas.rst index e51a5947d..ae44fed69 100644 --- a/source/includes/table-permissions-atlas.rst +++ b/source/includes/table-permissions-atlas.rst @@ -12,15 +12,12 @@ * - default - - atlasAdmin - - backup - - atlasAdmin * - write-blocking, reversing, or multiple reversals - - atlasAdmin - - backup - bypassWriteBlockMode privilege - - atlasAdmin - - backup - bypassWriteBlockMode privilege For details on Atlas roles, see: :atlas:`Atlas User Roles