Skip to content

mon,osd: add require_min_compat_client setting to enforce and clarify client compatibility#14959

Merged
liewegas merged 15 commits intoceph:masterfrom
liewegas:wip-crush-compat
May 9, 2017
Merged

mon,osd: add require_min_compat_client setting to enforce and clarify client compatibility#14959
liewegas merged 15 commits intoceph:masterfrom
liewegas:wip-crush-compat

Conversation

@liewegas
Copy link
Member

@liewegas liewegas commented May 4, 2017

  • tests
  • qa suite update
  • rename config option to include 'require'

@liewegas liewegas requested a review from a user May 4, 2017 18:06
@ghost
Copy link

ghost commented May 4, 2017

Having a feature bit makes things easier for http://tracker.ceph.com/issues/19836, great !

@liewegas liewegas force-pushed the wip-crush-compat branch from d49cffc to 54bb699 Compare May 4, 2017 19:02
@ghost
Copy link

ghost commented May 4, 2017

Should we add a CrushWrapper method encode_with_choose_args_pre_luminuous_if_possible to be called instead of encode if the client is pre-luminous ? Or should that be done outside of CrushWrapper ?

@liewegas
Copy link
Member Author

liewegas commented May 4, 2017 via email

@ghost
Copy link

ghost commented May 4, 2017

Ok.

@liewegas liewegas force-pushed the wip-crush-compat branch from 54bb699 to 4ab50d2 Compare May 5, 2017 22:48
@ghost
Copy link

ghost commented May 8, 2017

The UNIX domain socket path /home/jenkins-build/build/workspace/ceph-pull-requests/build/src/test/td/t-7202/out/client.xx-profile-ro.15033.asok is too long

jenkins test this please

1 similar comment
@ghost
Copy link

ghost commented May 8, 2017

The UNIX domain socket path /home/jenkins-build/build/workspace/ceph-pull-requests/build/src/test/td/t-7202/out/client.xx-profile-ro.15033.asok is too long

jenkins test this please

@ghost
Copy link

ghost commented May 8, 2017

#15002 has the modified CrushWrapper::encode

@liewegas
Copy link
Member Author

liewegas commented May 9, 2017

Calculate the oldest client we allow to connect.  Provide both a release
name (e.g., firefly) and a version string (e.g., "0.80").

Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas force-pushed the wip-crush-compat branch from 1ef0c0e to 630a23e Compare May 9, 2017 16:10
liewegas added 13 commits May 9, 2017 11:32
This is a persistent record of the compatibility for old clients that
we will enforce.  Once set, the mon will eventually prevent newer features
of the OSDMap or CRUSH maps from being used if they violate this
requirement.

Signed-off-by: Sage Weil <sage@redhat.com>
…lusters

Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Fill in this field once require_luminous is set.

Signed-off-by: Sage Weil <sage@redhat.com>
…compat_client

If require_min_compat_client is set (we've finished luminous upgrade), use
that to enforce primary_temp and primary_affinity settings.  Otherwise,
use the legacy config option.

Drop the assert in update_from_paxos.

After luminous we can remove the config options entirely.

Signed-off-by: Sage Weil <sage@redhat.com>
…ient instead

Signed-off-by: Sage Weil <sage@redhat.com>
…compat_client

Signed-off-by: Sage Weil <sage@redhat.com>
Missed this with the remap -> upmap rename a few weeks back.

Signed-off-by: Sage Weil <sage@redhat.com>
We'll use this for feature bit compat checks with clients and osds.

Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
For cases where we are selecting crush tunables beyond the default
min of hammer.

Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas force-pushed the wip-crush-compat branch from 2ddb0ef to c018ee3 Compare May 9, 2017 16:33
Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas force-pushed the wip-crush-compat branch from c018ee3 to 48648c4 Compare May 9, 2017 16:34
@liewegas
Copy link
Member Author

liewegas commented May 9, 2017

squashed commits and fixed cli test!

@liewegas liewegas merged commit 623c04e into ceph:master May 9, 2017
@liewegas liewegas deleted the wip-crush-compat branch May 9, 2017 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant