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

osd: start pool IDs from 1 #16626

Closed
wants to merge 1 commit into from
Closed

osd: start pool IDs from 1 #16626

wants to merge 1 commit into from

Conversation

jcsp
Copy link
Contributor

@jcsp jcsp commented Jul 27, 2017

CephFS doesn't like id=0 pools, which didn't used to
matter because id 0 was always consumed by RBD. Now,
there is no default rbd pool, so cephfs users
encounter this problem.

Fixes: http://tracker.ceph.com/issues/20792
Signed-off-by: John Spray john.spray@redhat.com

CephFS doesn't like id=0 pools, which didn't used to
matter because id 0 was always consumed by RBD.  Now,
there is no default rbd pool, so cephfs users
encounter this problem.

Fixes: http://tracker.ceph.com/issues/20792
Signed-off-by: John Spray <john.spray@redhat.com>
@jcsp jcsp added the core label Jul 27, 2017
@jcsp jcsp requested a review from liewegas July 27, 2017 14:16
@jcsp jcsp added this to the luminous milestone Jul 27, 2017
@xiexingguo
Copy link
Member

@jcsp IMHO, #16624 is a better fix.

@liewegas
Copy link
Member

I could have sworn @gregsfortytwo fixed this several weeks back as part of his no-rbd-pool series... did we break something?

@liewegas
Copy link
Member

.. when i do a vstart, i have no pools, and when i create one i get

pool 1 'foo' replicated size 1 min_size 1 crush_rule 0 object_hash rjenkins pg_num 1 pgp_num 1 last_change 6 flags hashpspool stripe_width 0

@jcsp
Copy link
Contributor Author

jcsp commented Jul 27, 2017

Huh, yeah, I'm finding (minus patch) that my pools start from 1 too. So I am confused :-/

@jcsp
Copy link
Contributor Author

jcsp commented Jul 27, 2017

Still don't know what was going on, but on second look this fix seems definitely wrong, I can see the new_pool_max is pre incremented when used during pool creation.

@jcsp jcsp closed this Jul 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants