BUGFIX: chown using the full path instead of the relative path

Signed-off-by: Tim Ramlot <[email protected]>

Author: inteon

storage/filesystem.go

@@ -263,11 +263,17 @@ func (f *Filesystem) WriteFiles(meta metadata.Metadata, files map[string][]byte)
 			return nil
 		}
 
-		// If a fsGroup is defined, Chown all files in the timestamped directory.
+		// Change ownership of the data directory, the timestamp directory and all files within it.
+		paths := []string{
+			filepath.Join(f.dataPathForVolumeID(meta.VolumeID), tsDirName),
+		}
 		for filename := range files {
-			// Set the uid to -1 which means don't change ownership in Go.
-			if err := os.Chown(filepath.Join(tsDirName, filename), -1, int(*fsGroup)); err != nil {
-				return err
+			paths = append(paths, filepath.Join(f.dataPathForVolumeID(meta.VolumeID), tsDirName, filename))
+		}
+
+		for _, path := range paths {
+			if err := os.Lchown(path, -1, int(*fsGroup)); err != nil {
+				return fmt.Errorf("failed to chown data dir to gid %v: %w", *fsGroup, err)
 			}
 		}