Mailinglist Archive: opensuse-buildservice (207 mails)

< Previous Next >
[opensuse-buildservice] Re: [PATCH] [appliance] Chown all directories on appliance change.
  • From: Adrian Schröter <adrian@xxxxxxx>
  • Date: Thu, 24 Mar 2011 08:05:25 +0100
  • Message-id: <2068138.r6Lmkg0gv4@scherben>
Am Mittwoch, 23. März 2011, 23:23:13 schrieb OBS build-service:
From: Jan-Simon Möller <jsmoeller@xxxxxxxxxxxxxxxxxxx>

Signed-off-by: Jan-Simon Möller <jsmoeller@xxxxxxxxxxxxxxxxxxx>
---
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@xxxxxxx

--
To unsubscribe, e-mail: opensuse-buildservice+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: opensuse-buildservice+help@xxxxxxxxxxxx

< Previous Next >
Follow Ups