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
ceph: Allow an even number of mons #8636
Conversation
e97c1ba
to
6d1b1d8
Compare
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.
I like the doc changes.
While we are revising this, should we set |
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.
rook/pkg/apis/ceph.rook.io/v1/cluster.go
Line 58 in d577fea
if updatedCephCluster.Spec.Mon.Count > 0 && updatedCephCluster.Spec.Mon.Count%2 == 0 { |
Is this API should also be updated with the change?
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.
What for the case for automatic adding of mons if new nodes are added, is this we need to keep the scenario the same? add mons auto if there are an odd number of nodes present
.
Not sure what you mean? We are relaxing the check so we allow an even number of mons now.
If new nodes are added, the behavior would still be the same as before. We usually don't add new mons with new nodes though. |
6d1b1d8
to
afc436f
Compare
Sorry, I just thought this place was missed.
Okay :) And just Check if this warning needs to be updated rook/pkg/operator/ceph/cluster/mon/health.go Lines 600 to 602 in 8af24cf
|
While an even number of mons can cause lower availability of mon quorum, it also can provide higher durability for the cluster. Mon quorum can be restored from a single mon according to the disaster recovery guide, so there may be scenarios where an even number of mons may be preferable. Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
Typical clusters only require 3 or maybe 5 mons. If someone really wants to go crazy they can add more mons, but over 9 mons really should not be used. Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
afc436f
to
849f434
Compare
@parth-gr Yeah, removed the message from the external cluster, thanks! |
ceph: Allow an even number of mons (backport #8636)
Description of your changes:
While an even number of mons can cause lower availability of mon quorum, it also can provide higher durability for the cluster. Mon quorum can be restored from a single mon according to the disaster recovery guide, so there may be scenarios where an even number of mons may be preferable.
Reverts the check for an even number of mons added in #6420.
Checklist:
make codegen
) has been run to update object specifications, if necessary.