fai LOGUSER

[Waldemar Brodkorb]

Hi,

I found another issues in FAI 4.0beta3 on Debian 6.0.3.
When using the quickstart guide, I am creating a fai user with
faisetup -v. This user does have /sbin/false as a shell.

While installling the fai client following code in
lib/subroutines always returns 1:
$frsh true >/dev/null 2>&1

Isn't a real shell for the fai user required?

I am proposing following patch:

From bc26e5066d0313c4c3528e113866aee6b06e3119 Mon Sep 17 00:00:00 2001
From: Waldemar Brodkorb <mail at waldemar-brodkorb.de>
Date: Fri, 18 Nov 2011 15:44:39 +0100
Subject: [PATCH] add a shell to the fai user, so that fai-chboot remote calls are possible


Signed-off-by: Waldemar Brodkorb <mail at waldemar-brodkorb.de>
---
 bin/fai-setup |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/bin/fai-setup b/bin/fai-setup
index be5e5f1..190bc48 100644
--- a/bin/fai-setup
+++ b/bin/fai-setup
@@ -72,7 +72,7 @@ add_fai_account() {
 	return
     fi
 
-    adduser --system --disabled-password --home /var/log/fai --gecos "FAI account for log files" $LOGUSER
+    adduser --system --disabled-password --home /var/log/fai --gecos "FAI account for log files" --shell /bin/sh  $LOGUSER
     # get the home dir of a user in a variable; do not exit when set -e is used
     # loguserhome is unset if $LOGUSER does not exists
     # this is not a evil hack, it's a very clever piece of code
-- 
1.7.2.5