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

Conversation

@irajdeep
Copy link
Contributor

Minor fixes to documentation
Also resolves: #223

All Submissions:

  • [ n/a] Have you opened an Issue before filing this PR?
  • [ n/a] Have you signed our CLA?
  • [ n/a] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • [ n/a] Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

@irajdeep irajdeep requested a review from bznein October 20, 2020 11:53
Copy link
Contributor

@bznein bznein left a comment

Choose a reason for hiding this comment

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

LGTM

mongodb://<metadata.name of the MongoDB resource>-svc.<namespace>.svc.cluster.local:27017/?replicaSet=<replica set name>
mongo "mongodb://<service-object-name>.<namespace>.svc.cluster.local:27017/?replicaSet=<replica-set-name>"
```
<em>NOTE: You can access the mongodb instance only from within the pod running in the cluster.</em>
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
<em>NOTE: You can access the mongodb instance only from within the pod running in the cluster.</em>
<em>NOTE: You can access the mongodb instance only from within a pod running in the cluster.</em>

@irajdeep irajdeep force-pushed the irajdeep/documentation-fixes branch from 6e1a179 to 0c8bdb8 Compare October 20, 2020 11:57
@irajdeep irajdeep merged commit 76214a6 into master Oct 20, 2020
@irajdeep irajdeep deleted the irajdeep/documentation-fixes branch October 20, 2020 12:01
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.

Install document Needed a correction ?

3 participants