ceph-fuse: load supplementary groups by default

Partially addresses POSIX test failures in [1] due to the config setting being
false by default.

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1480182#c6

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>

diff --git a/src/client/fuse_ll.cc b/src/client/fuse_ll.cc
index 1fdd3289a938189b30d34f6d578c137a8c93adc1..cbb0bb0762e3b017f2b420eaa4efdc248fc5b414 100644 (file)
--- a/src/client/fuse_ll.cc
+++ b/src/client/fuse_ll.cc
@@ -134,7 +134,7 @@ static int getgroups_cb(void *handle, gid_t **sgids)
 }
 
 #define GET_GROUPS(perms, req) {                               \
-  if (cfuse->client->cct->_conf->fuse_set_user_groups) {       \
+  if (g_conf->get_val<bool>("fuse_set_user_groups")) { \
     gid_t *gids = NULL;                                                \
     int count = getgroups(req, &gids);                         \
     perms.init_gids(gids, count);                              \

diff --git a/src/common/legacy_config_opts.h b/src/common/legacy_config_opts.h
index 7222ef6ca7ea76f47e96c1ecd948b097b2432331..f680d68a92fbc9208da2606396b784da2262845d 100644 (file)
--- a/src/common/legacy_config_opts.h
+++ b/src/common/legacy_config_opts.h
@@ -406,7 +406,6 @@ OPTION(fuse_debug, OPT_BOOL)
 OPTION(fuse_multithreaded, OPT_BOOL)
 OPTION(fuse_require_active_mds, OPT_BOOL) // if ceph_fuse requires active mds server
 OPTION(fuse_syncfs_on_mksnap, OPT_BOOL)
-OPTION(fuse_set_user_groups, OPT_BOOL) // if ceph_fuse fills in group lists or not
 
 OPTION(client_try_dentry_invalidate, OPT_BOOL) // the client should try to use dentry invaldation instead of remounting, on kernels it believes that will work for
 OPTION(client_die_on_failed_remount, OPT_BOOL)

diff --git a/src/common/options.cc b/src/common/options.cc
index e44d6b974e6479cc575e17e1bc9e4062b8beef59..95ca71916fd788c9ed297503787dcdd972f38d35 100644 (file)
--- a/src/common/options.cc
+++ b/src/common/options.cc
@@ -5918,8 +5918,8 @@ std::vector<Option> get_mds_client_options() {
     .set_description(""),
 
     Option("fuse_set_user_groups", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
-    .set_default(false)
-    .set_description(""),
+    .set_default(true)
+    .set_description("check for ceph-fuse to consider supplementary groups for permissions"),
 
     Option("client_try_dentry_invalidate", Option::TYPE_BOOL, Option::LEVEL_ADVANCED)
     .set_default(true)