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

OSDMonitor: be a little nicer about letting users do pg splitting #802

Merged
merged 0 commits into from Nov 1, 2013

Conversation

gregsfortytwo
Copy link
Member

I tested this by running vstart with two OSDs. Doing split on a pool while a different pool was creating succeeded; trying to split a still-creating pool did not.

gregf@kai:/ceph/src [next]$ ./ceph osd pool create foo 100 && ./ceph osd pool set data pg_num 50
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
pool 'foo' created
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
set pool 0 pg_num to 50
gregf@kai:
/ceph/src [next]$ ./ceph osd pool set data pg_num 100
*** DEVELOPER MODE: setting PATH, PYTHONPATH and LD_LIBRARY_PATH ***
Error EAGAIN: currently creating pgs, wait

@dzafman dzafman merged this pull request into next Nov 1, 2013
@dzafman dzafman deleted the wip-6673b branch November 1, 2013 23:04
@dzafman dzafman restored the wip-6673b branch November 1, 2013 23:05
dzafman pushed a commit that referenced this pull request Nov 1, 2013
(manually merged after next branch rebuilt)

OSDMonitor: be a little nicer about letting users do pg splitting

Reviewed-by: David Zafman <david.zafman@inktank.com>
@dzafman
Copy link
Contributor

dzafman commented Nov 2, 2013

This pull request ended up having to be merged again (e043b6d) because next branch was rebuilt.

liewegas pushed a commit that referenced this pull request Dec 14, 2016
Whitelisted 'wrongly marked me down'

Reviewed-by: Samuel Just <sjust@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants