Ceph » CephFS

BTW, the only material difference between this test run and a normal FS suite run is that ceph-fuse is turned off. Compare config to:

http://qa-proxy.ceph.com/teuthology/jspray-2016-11-22_10:40:59-fs-wip-jcsp-testing-20161121-distro-basic-smithi/568578/teuthology.log

This chmod may be the bit we're missing:
https://github.com/ceph/ceph-qa-suite/blob/master/tasks/cephfs/fuse_mount.py#L190

We could do a similar thing with ceph_ll_setattr and passing UserPerm constructed to be root?

I think the right solution is to add a new function for setting the default UserPerm for the cmount, and have those jobs set that when they run. I'll have a look at doing that soon...

Hmm...now that I look, I like John's suggestion better -- do a ceph_ll_setattr as root, and change the permissions on the root to 01777. We might still want to add an interface to allow setting the default UserPerm on a mount, but I don't think that's really the right fix here.

Actually, now that I look further, it turns out that access.cc looks like it has just the function we need:

static int update_root_mode()                                                   
{                                                                               
  struct ceph_mount_info *admin;                                                
  int r = ceph_create(&admin, NULL);                                            
  if (r < 0)                                                                    
    return r;                                                                   
  ceph_conf_read_file(admin, NULL);                                             
  ceph_conf_parse_env(admin, NULL);                                             
  ceph_conf_set(admin, "client_permissions", "false");                          
  r = ceph_mount(admin, "/");                                                   
  if (r < 0)                                                                    
    goto out;                                                                   
  r = ceph_chmod(admin, "/", 0777);                                             
out:                                                                            
  ceph_shutdown(admin);                                                         
  return r;                                                                     
}                                                         

Maybe the best thing would be to combine ceph_test_libcepfs and ceph_test_libcephfs_access into a single binary, and use the main() function defined in access.cc. Then we can let it fix up the permissions at the start of the test run.

Hmmm...actually, a simple interim workaround might be to just reverse the order of the tests in qa/workunits/libcephfs/test.sh. ceph_test_libcephfs_access will fix permissions first thing, and that should persist when the other binary is run.

Sounds reasonable to me Jeff.

Potential fix up here. Still needs testing, but it seems to do the right thing on my box:

https://github.com/ceph/ceph/pull/12228