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
octopus: mon: calculate min_size on osd pool set size #34528
Conversation
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.
LGTM
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.
@smithfarm we don't want to backport #33281, see https://tracker.ceph.com/issues/44025#note-8. Is there a reason why you think we should?
@neha-ojha My understanding was that it should not be backported to nautilus. But if it's a pacific-only feature, then we shouldn't backport it to octopus, either, and I'll remove it from this PR.
|
currently `osd pool set size` only modifies min_size when it is above new size, while it should be recalculated unconditionally. fixes: https://tracker.ceph.com/issues/44862 Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com> (cherry picked from commit b69dcaa)
verify whether min_size is recalculated when osd pool size is changed. fixes: https://tracker.ceph.com/issues/44862 Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com> (cherry picked from commit 0a7325a) Conflicts: qa/workunits/mon/pool_ops.sh - master/pacific needs "--yes-i-really-mean-it" when setting pool size to 1
8a35a23
to
d2ae92c
Compare
@neha-ojha Please have another look - I removed the feature cherry-pick. |
@smithfarm looks good, thanks! |
backport tracker:
backport of
parent tracker:
this backport was staged using ceph-backport.sh version 15.1.1.389
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh