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

Conversation

@ajhuh-mdb
Copy link
Collaborator

@ajhuh-mdb ajhuh-mdb commented Feb 27, 2024

DESCRIPTION

  • Moved on-prem to Atlas cluster example to separate include since there are differences when connecting to Atlas
  • Updated cluster1 hosts and ports to better resemble an Atlas cluster
  • Changed connection strings to use SRV format
  • Moved note on Atlas string requirements (SRV or tls option) higher up for better visibility

STAGING

JIRA

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

BUILD

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

Copy link
Collaborator

@sarah-olson-mongodb sarah-olson-mongodb 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 these updates @ajhuh-mdb! LGTM

Copy link

@mattt416 mattt416 left a comment

Choose a reason for hiding this comment

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

I've suggested a few minor changes. Thanks @ajhuh-mdb !

On a side note, would we want to make similar changes to the Connect Two Atlas Clusters section?

@ajhuh-mdb ajhuh-mdb force-pushed the DOCSP-23343-connect-self-managed-cluster-atlas branch from 3aa71af to 9d4f673 Compare February 27, 2024 20:27
@ajhuh-mdb ajhuh-mdb force-pushed the DOCSP-23343-connect-self-managed-cluster-atlas branch from 9d4f673 to f213590 Compare February 27, 2024 21:34
@ajhuh-mdb ajhuh-mdb force-pushed the DOCSP-23343-connect-self-managed-cluster-atlas branch from b6b8815 to 796a7ec Compare February 27, 2024 22:41
@ajhuh-mdb ajhuh-mdb requested a review from mattt416 February 28, 2024 17:30
@ajhuh-mdb
Copy link
Collaborator Author

Thanks for taking a look @mattt416 ! This should be ready for another round of review.

Copy link

@mattt416 mattt416 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @ajhuh-mdb !

@jeff-allen-mongo jeff-allen-mongo merged commit b2b56cb into mongodb:master Mar 1, 2024
ajhuh-mdb added a commit to ajhuh-mdb/docs-cluster-to-cluster-sync that referenced this pull request Mar 1, 2024
* DOCSP-23343 Improve Atlas Connection String Example

* text edits

* move note

* MT feedback

* test

* tiny changes

* *

* fix staging
jeff-allen-mongo pushed a commit that referenced this pull request Mar 1, 2024
* DOCSP-23343 Improve Atlas Connection String Example

* text edits

* move note

* MT feedback

* test

* tiny changes

* *

* fix staging
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