Skip to content

Commit

Permalink
Merge pull request #197 from ronivay/fix/nfs-remote-permissions
Browse files Browse the repository at this point in the history
fix: do not run chown recursively to mounts dir with non-root user
  • Loading branch information
ronivay authored Oct 30, 2023
2 parents dd2ea64 + 4beae1a commit 806f6c1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion xo-install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -801,7 +801,7 @@ function InstallXO {
printinfo "Changing default mountsDir in xo-server configuration file"
runcmd "sed -i \"s%#mountsDir.*%mountsDir = '$INSTALLDIR/mounts'%\" $INSTALLDIR/xo-builds/xen-orchestra-$TIME/packages/xo-server/sample.config.toml"
runcmd "mkdir -p $INSTALLDIR/mounts"
runcmd "chown -R $XOUSER:$XOUSER $INSTALLDIR/mounts"
runcmd "chown $XOUSER:$XOUSER $INSTALLDIR/mounts"
fi

if [[ -n "$SYSLOG_TARGET" ]]; then
Expand Down

0 comments on commit 806f6c1

Please sign in to comment.