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

Conversation

amalhotra-mdb
Copy link
Collaborator

@amalhotra-mdb amalhotra-mdb commented Jan 22, 2025

DESCRIPTION

Update the oplog sizing page to indicate that replSetResizeOplog is incompatible in Atlas. Add a note that redirects the user to the Atlas page for resizing their oplog.

STAGING

https://deploy-preview-558--docs-cluster-to-cluster-sync.netlify.app/reference/oplog-sizing/#validate-oplog-size

JIRA

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

SELF-REVIEW CHECKLIST

  • Does each file have 3-5 taxonomy facet tags?
    See the taxonomy tagging instructions and this example PR
  • Is this free of any warnings or errors in the RST?
  • Is this free of spelling errors?
  • Is this free of grammatical errors?
  • Is this free of staging / rendering issues?
  • Are all the links working?

EXTERNAL REVIEW REQUIREMENTS

What's expected of an external reviewer?

Copy link

netlify bot commented Jan 22, 2025

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

Name Link
🔨 Latest commit bc61d51
🔍 Latest deploy log https://app.netlify.com/sites/docs-cluster-to-cluster-sync/deploys/6792acc41b32510008abfbb2
😎 Deploy Preview https://deploy-preview-558--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

@ltran-mdb2 ltran-mdb2 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 work @amalhotra-mdb ! I just left a small question/suggestion

Comment on lines 67 to 70
.. note::

:dbcommand:`replSetResizeOplog` is unsupported in Atlas. To resize
the oplog in Atlas, see :ref:`set-oplog-size`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

[question/suggestion] Since this note pertains specifically to the above bullet, should it be indented under the bullet (as opposed to in line with the bullet)?

Suggested change
.. note::
:dbcommand:`replSetResizeOplog` is unsupported in Atlas. To resize
the oplog in Atlas, see :ref:`set-oplog-size`.
.. note::
:dbcommand:`replSetResizeOplog` is unsupported in Atlas. To resize
the oplog in Atlas, see :ref:`set-oplog-size`.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This makes sense @ltran-mdb2 ! Thanks for this suggestion, fixed!

Copy link
Collaborator

@ltran-mdb2 ltran-mdb2 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @amalhotra-mdb !

Copy link
Collaborator

@JuliaMongo JuliaMongo 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, Annika!

to set ``minRetentionHours`` greater than the current ``oplog``
window.

.. note::
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do c2c docs use a replacement for Atlas, like |service|? If so, use it here too. If not, then leave Atlas.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Please review with the dev team as well, cc Ankit Luthra (via slack).

@ajhuh-mdb ajhuh-mdb merged commit 1175079 into mongodb:master Jan 28, 2025
4 checks passed
amalhotra-mdb added a commit to amalhotra-mdb/docs-cluster-to-cluster-sync that referenced this pull request Jan 29, 2025
* DOCSP-46629-replSetResizeOplog-unsupported-Atlas

* indent

* reviewer changes

* undo some changes

(cherry picked from commit 1175079)
amalhotra-mdb added a commit to amalhotra-mdb/docs-cluster-to-cluster-sync that referenced this pull request Jan 29, 2025
* DOCSP-46629-replSetResizeOplog-unsupported-Atlas

* indent

* reviewer changes

* undo some changes

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

💚 All backports created successfully

Status Branch Result
v1.9
v1.8

Questions ?

Please refer to the Backport tool documentation

amalhotra-mdb added a commit to amalhotra-mdb/docs-cluster-to-cluster-sync that referenced this pull request Jan 29, 2025
* DOCSP-46629-replSetResizeOplog-unsupported-Atlas

* indent

* reviewer changes

* undo some changes

(cherry picked from commit 1175079)
amalhotra-mdb added a commit to amalhotra-mdb/docs-cluster-to-cluster-sync that referenced this pull request Jan 29, 2025
* DOCSP-46629-replSetResizeOplog-unsupported-Atlas

* indent

* reviewer changes

* undo some changes

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

💚 All backports created successfully

Status Branch Result
v1.9
v1.8

Questions ?

Please refer to the Backport tool documentation

jmd-mongo pushed a commit that referenced this pull request Jan 29, 2025
* DOCSP-46629-replSetResizeOplog-unsupported-Atlas

* indent

* reviewer changes

* undo some changes

(cherry picked from commit 1175079)
jmd-mongo pushed a commit that referenced this pull request Jan 29, 2025
* DOCSP-46629-replSetResizeOplog-unsupported-Atlas

* indent

* reviewer changes

* undo some changes

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

6 participants