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: add client config ref #10434
doc: add client config ref #10434
Conversation
There are a few config option descriptions that I'm unsure on (marked by FIXME). Could use some suggestions. |
|
||
``client_max_inline_size`` | ||
|
||
:Description: **FIXME** |
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.
It's the largest a file can be and still get inlined (i.e. stored in the inode instead of in a data object). Only applies if the inline_data flag is set in the mdsmap.
Done, thanks @jcsp. |
|
||
``client_oc`` | ||
|
||
:Description: Enable object caching. |
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 don't think we know if this works, or really expect it to? (Disabling it, that is.) we probably shouldn't document it.
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 suggest marking the config option as ignored and removing the checks in the code.
Do we really want to point people at the monitor-routed inject-arts instead of the admin socket config set? :/ |
09c7336
to
f2ed83e
Compare
Fixes: http://tracker.ceph.com/issues/16743 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
Thanks for the suggestions Greg. I amended the commit. |
Fixes: http://tracker.ceph.com/issues/16743
Signed-off-by: Patrick Donnelly pdonnell@redhat.com