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

Conversation

@ltran-mdb2
Copy link
Collaborator

@ltran-mdb2 ltran-mdb2 commented Apr 30, 2024

DESCRIPTION

Recommend that users upgrade to the latest server patch prior to migration

STAGING

https://preview-mongodbltranmdb2.gatsbyjs.io/cluster-sync/DOCSP-39091/release-notes/1.7/#minimum-supported-version
https://preview-mongodbltranmdb2.gatsbyjs.io/cluster-sync/DOCSP-39091/connecting/onprem-to-onprem/
https://preview-mongodbltranmdb2.gatsbyjs.io/cluster-sync/DOCSP-39091/connecting/onprem-to-atlas/

JIRA

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

BUILD LOG

https://workerpool-boxgs.mongodbstitch.com/pages/job.html?collName=queue&jobId=66312de3b67dbf804aa92668

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
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.

LGTM % a suggestion!

@@ -0,0 +1,3 @@
For best performance, upgrade your source and destination clusters to
the most recent server patch release prior to migration. For more
information, see :ref:`MongoDB Patch Releases <versions-patch-release>`. No newline at end of file
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe this page might be slightly better to link to:

https://www.mongodb.com/docs/manual/tutorial/upgrade-revision/

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done! Thanks for this suggestion.


.. note::

.. include:: /includes/migration-upgrade-recommendation.rst
Copy link
Collaborator

Choose a reason for hiding this comment

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

For on prem to atlas, the atlas cluster will be updated by mongodb in a rolling manner, so they only need to worry about the on-prem cluster. I think we should make that clear.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done!

Copy link
Collaborator

Choose a reason for hiding this comment

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

The atlas cluster can either be the source or destination. I think that we should specify that the on prem cluster is the one that needs to be updated.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed!

@ltran-mdb2
Copy link
Collaborator Author

Thanks for your feedback @rohshar ! Can you take another look when you get a chance?

@ltran-mdb2 ltran-mdb2 requested a review from rohshar May 1, 2024 15:04
.. note::

For best performance, upgrade your on self-managed cluster to the
most recent server patch release prior to migration. For more
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
most recent server patch release prior to migration. For more
most recent MongoDB Server patch release prior to migration. For more

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done!

@@ -0,0 +1,3 @@
For best performance, upgrade your source and destination clusters to
the most recent server patch release prior to migration. For more
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
the most recent server patch release prior to migration. For more
the most recent MongoDB Server patch release prior to migration. For more

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done!

@mdb-ashley mdb-ashley merged commit a79219c into mongodb:master May 1, 2024
ltran-mdb2 added a commit to ltran-mdb2/docs-cluster-to-cluster-sync that referenced this pull request May 1, 2024
…ongodb#310)

* DOCSP-39091  Recommend that users upgrade to the latest server patch version before migrations

* DOCSP-39091  Recommend that users upgrade to the latest server patch version before migrations

* DOCSP-39091 copy review

* DOCSP-39091 tech edit

* DOCSP-39091 tech edit 2

* DOCSP-39091 edits from Ashley
mdb-ashley pushed a commit that referenced this pull request May 1, 2024
…310) (#313)

* DOCSP-39091  Recommend that users upgrade to the latest server patch version before migrations

* DOCSP-39091  Recommend that users upgrade to the latest server patch version before migrations

* DOCSP-39091 copy review

* DOCSP-39091 tech edit

* DOCSP-39091 tech edit 2

* DOCSP-39091 edits from Ashley
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.

4 participants