Skip to content
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: rgw: correct get usage parameter default value #14372

Merged
merged 1 commit into from Apr 21, 2017

Conversation

Projects
None yet
2 participants
@hrchu
Copy link
Contributor

hrchu commented Apr 6, 2017

show-entries and show-summary are default in True instead of False, ref the corresponding code section here: https://github.com/ceph/ceph/blob/master/src/rgw/rgw_rest_usage.cc#L38

hrchu
doc: rgw: correct get usage parameter default value
Signed-off-by: hrchu <petertc@gmail.com>
@hrchu

This comment has been minimized.

Copy link
Contributor Author

hrchu commented Apr 6, 2017

Behind the scene: I got into trouble when working with radosgw admin APIs, especially that docs are a bit confusing and inconsist with the code base. What I can do are these two things: 1. Correct the section that ran me into problems 2. Code a client library and hope that people can done their jobs more smoothly with it. 🍰

@hrchu

This comment has been minimized.

Copy link
Contributor Author

hrchu commented Apr 17, 2017

@tchaikov should I provide additional information about this?

@cbodley cbodley merged commit 307060e into ceph:master Apr 21, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.