New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc/architecture: edit "High Avail. Monitors" #53444
doc/architecture: edit "High Avail. Monitors" #53444
Conversation
doc/architecture.rst
Outdated
|
||
For details on configuring monitors, see the `Monitor Config Reference`_. | ||
A Ceph Client must contact a Ceph Monitor and obtain a current copy of the | ||
cluster map in order to be permitted to read data from or to write data to the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/be permitted to//
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good eye. Accepted.
doc/architecture.rst
Outdated
Ceph cluster. | ||
|
||
It is possible for a Ceph cluster to function properly with only a single | ||
monitor in it, but a Ceph cluster that has only a single monitor in it has a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/ in it//g
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good eye here, too. Accepted.
doc/architecture.rst
Outdated
single point of failure: if the monitor goes down, Ceph clients will be unable | ||
to read data from or write data to the cluster. | ||
|
||
Ceph supports a cluster of monitors in order to increase reliability and fault |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/supports/leverages/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is that rarest of cases: the time when I think that "leveraged" is, in fact, the right word.
Accepted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ever since I read that "utilize" doesn't mean what I think it means I've shied away from it.
Improve the sentence structure in the "High Availability Monitors" section of doc/architecture.rst. Co-authored-by: Anthony D'Atri <anthony.datri@gmail.com> Signed-off-by: Zac Dover <zac.dover@proton.me>
6c82ceb
to
57019c3
Compare
Improve the sentence structure in the "High Availability Monitors" section of doc/architecture.rst.
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard cephadm
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox
jenkins test windows