Merge pull request ceph#59328 from mkogan1/wip-d3n-umask

mkogan1 · web-flow · commit aa94b1574dce · 2024-11-11T15:59:59.000+02:00
rgw: d3n: set access permissions for the D3N cache directory when creating
diff --git a/src/rgw/driver/rados/rgw_d3n_datacache.cc b/src/rgw/driver/rados/rgw_d3n_datacache.cc
@@ -86,6 +86,8 @@ void D3nDataCache::init(CephContext *_cct) {
       // create the cache storage directory
       lsubdout(g_ceph_context, rgw, 5) << "D3nDataCache: init: creating the persistent storage directory on start" << dendl;
       efs::create_directories(cache_location);
+      efs::permissions(cache_location, 
+        efs::perms::owner_all | efs::perms::group_all | efs::perms::others_read);
     }
   } catch (const efs::filesystem_error& e) {
     lderr(g_ceph_context) << "D3nDataCache: init: ERROR initializing the cache storage directory '" << cache_location <<

Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,8 @@ void D3nDataCache::init(CephContext *_cct) {`
`86`	`86`	`// create the cache storage directory`
`87`	`87`	`lsubdout(g_ceph_context, rgw, 5) << "D3nDataCache: init: creating the persistent storage directory on start" << dendl;`
`88`	`88`	`efs::create_directories(cache_location);`
	`89`	`+ efs::permissions(cache_location,`
	`90`	`+ efs::perms::owner_all \| efs::perms::group_all \| efs::perms::others_read);`
`89`	`91`	`}`
`90`	`92`	`} catch (const efs::filesystem_error& e) {`
`91`	`93`	`lderr(g_ceph_context) << "D3nDataCache: init: ERROR initializing the cache storage directory '" << cache_location <<`