[opensuse-buildservice] Re: [PATCH] [appliance] Chown all directories on appliance change.

Am Mittwoch, 23. März 2011, 23:23:13 schrieb OBS build-service:

From: Jan-Simon Möller

Signed-off-by: Jan-Simon Möller --- dist/obsstoragesetup | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/dist/obsstoragesetup b/dist/obsstoragesetup index eae6e93..ca7c7a9 100755 --- a/dist/obsstoragesetup +++ b/dist/obsstoragesetup @@ -258,7 +258,7 @@ case "$1" in [ -d /obs/info ] || mkdir /obs/info [ -d /obs/projects ] || mkdir /obs/projects [ -d /obs/workers ] || mkdir /obs/workers - chown obsrun.obsrun /obs/{,log,run,build,events,jobs,info,projects,workers} + chown -R obsrun:obsrun /obs/{,build,db,events,info,jobs,log,projects,remotecache,repos,repos_sync,run,sources,trees,upload,workers}

Uh, isn't that a bit too agressive ? Someone might have done a chown to root to be sure that OBS does not touch certain binaries anymore. Actually the entire chown should not be needed, the appliances should have always the same userid. If that is not the case we have a bug ...

else echo "Setup local storage" if [ -e /obs -a ! -L /obs ]; then

-- Adrian Schroeter SUSE Linux Products GmbH email: adrian@suse.de -- To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-buildservice+help@opensuse.org