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/OSDMap: do not create erasure rule by default #1132

Merged
1 commit merged into from Jan 23, 2014
Merged

Conversation

liewegas
Copy link
Member

If we do, we will require the v2 feature bit from clients.

We could only include feature bits for rules that are actually referenced by
pools, but for now making the user create the rule is simpler. There is no
need to create this rule ahead of time.

Signed-off-by: Sage Weil sage@inktank.com

@gregsfortytwo
Copy link
Member

Is there some reason we want to leave the command in a comment, instead of just removing it?

@liewegas
Copy link
Member Author

i suppose not :)

On Thu, 23 Jan 2014, Gregory Farnum wrote:

Is there some reason we want to leave the command in a comment, instead of
just removing it?

?
Reply to this email directly or view it onGitHub.[433031__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcwNjAyODI
zNywiZGF0YSI6eyJpZCI6MjQxMjMzNTB9fQ==--469d5c8e5542c35fa6951bffc3dc1c30d188
a9f0.gif]

If we do, we will require the v2 feature bit from clients.

We could only include feature bits for rules that are actually referenced
by pools, but for now making the user create the rule is simpler.  There is
no need to create this rule ahead of time.

Signed-off-by: Sage Weil <sage@inktank.com>
@ghost
Copy link

ghost commented Jan 23, 2014

rebased because make check fails on bufferlist ( the branch is ~190 commits behind ) : running make check again

@ghost
Copy link

ghost commented Jan 23, 2014

LGTM but master is broken and I'm not able to compile & check 👍

@ghost
Copy link

ghost commented Jan 23, 2014

make check running against new master rebase ;-)

ghost pushed a commit that referenced this pull request Jan 23, 2014
osd/OSDMap: do not create erasure rule by default

Reviewed-by: Greg Farnum <greg@inktank.com>
Reviewed-by: Loic Dachary <loic@dachary.org>
@ghost ghost merged commit 102c83a into next Jan 23, 2014
@ghost
Copy link

ghost commented Jan 23, 2014

Realized I merged against next (and not master) after the fact :-( Running make check again after a rebase on next ( 12 commits behind only ).

@ghost
Copy link

ghost commented Jan 23, 2014

make check says all is well on next :-)

@liewegas liewegas deleted the wip-erasure-rule branch January 27, 2014 16:33
liewegas pushed a commit that referenced this pull request Dec 14, 2016
rgw: we don't support btrfs any more
This pull request was closed.
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