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

Am Donnerstag, 24. März 2011, 08:58:43 schrieb Jan-Simon Moeller:

Am Donnerstag, 24. März 2011, 08:05:25 schrieb Adrian Schröter:

...

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 ...

We have a bug. Lets me revert this commit and hunt the other. On appliance change, we had a lot of messagebus:messagebus ownerships. This might depend on installation-order in kiwi ? May be better to find a fixed numeric id to assign to obs user.

I do so in the kiwi file, unfortunatly to id 103 for historic reasons, which can conflict meanwhile. However, it should still work, I will test it using a 2.1 appliance and update afterwards with 2.2 .. bye adrian -- 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