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

test,rgw: fix rgw placement rule pool config option #16380

Merged
merged 1 commit into from Jul 18, 2017

Conversation

Projects
None yet
2 participants
@mikulely
Contributor

mikulely commented Jul 18, 2017

actually we're checking --index-pool & --max-buckets during the argument parsing.

same as 3af3f4e

Signed-off-by: Jiaying Ren jiaying.ren@umcloud.com

@mikulely

This comment has been minimized.

Contributor

mikulely commented Jul 18, 2017

@joscollin This's same as #16084 , mind to take a look?

@joscollin joscollin self-requested a review Jul 18, 2017

@joscollin

joscollin requested changes Jul 18, 2017 edited

Could you fix '--max_buckets' too ? So that we eliminate all _ mismatches in these files.

test,rgw: fix rgw placement rule pool config option
actually we're checking '--index-pool' & `--max-buckets` during
the argument parsing.

same as 3af3f4e

Signed-off-by: Jiaying Ren jiaying.ren@umcloud.com
@mikulely

This comment has been minimized.

Contributor

mikulely commented Jul 18, 2017

@joscollin Of course, repushed.

@joscollin joscollin merged commit 78511bf into ceph:master Jul 18, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details

@mikulely mikulely deleted the mikulely:doc-index-typos branch Jul 18, 2017

@mikulely

This comment has been minimized.

Contributor

mikulely commented Jul 18, 2017

Thanks @joscollin for the quick review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment