-
Notifications
You must be signed in to change notification settings - Fork 35
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 recommended values? #103
Comments
@iranzo we can't adjust it after the deployment via t-h-t can we? I thought this has to be done with ceph. Would you like to create a test script for those values? This is handled in https://github.com/ceph/ceph-medic |
Eventually we could take a look on those checks for ceph and if they follow same return codes logic re-use them. |
Seems to be made in python, if they don't return the same logic we can always write a 'wrapper' to call its functions or implement individual ones in the meantime, open for discussion, would be nice to get feedback from ceph guys |
min_size by default is configured by ceph, if not specified. if specified it will be in ceph.conf or other cluster configuration, this is wrong and we should keep using the ceph for deciding the value: https://review.gerrithub.io/#/c/377120/ |
min_size merged, pg_num still left. |
As starting point it's a good one. https://access.redhat.com/labs/cephpgc/ |
This is upstream version for those who don't have access to redhat portal http://ceph.com/pgcalc/ |
pg_num draft is here https://review.gerrithub.io/#/c/377317/ |
pg_num check merged |
"pool_name":"test"
"min_size":1 //<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< Needs to be increased to 2
"pg_num":100 //<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< Needs to be a power of two
Should we perform a check for hints like this?
The text was updated successfully, but these errors were encountered: