[Qa-jenkins-scm] [jenkins.debian.net] 01/01: phil should only be a user on jenkins-test-vm

[Holger Levsen]

This is an automated email from the git hooks/post-receive script.

holger pushed a commit to branch master
in repository jenkins.debian.net.

commit 6d99b4e8eefa0af31cb7759df2f9be905d339acf
Author: Holger Levsen <holger at layer-acht.org>
Date:   Thu Apr 28 21:58:18 2016 +0200

    phil should only be a user on jenkins-test-vm
---
 update_jdn.sh | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/update_jdn.sh b/update_jdn.sh
index 0064a14..b06b1ce 100755
--- a/update_jdn.sh
+++ b/update_jdn.sh
@@ -57,19 +57,17 @@ if ! getent passwd jenkins-adm > /dev/null  ; then
 	sudo adduser --system --shell /bin/bash --no-create-home --ingroup jenkins-adm --disabled-login --no-create-home jenkins-adm
 	sudo usermod -G jenkins jenkins-adm
 fi
-for user in helmut holger mattia lunar philh phil ; do
+for user in helmut holger mattia lunar phil ; do
 	if [ "$user" = "lunar" ] && [ "$HOSTNAME" != "jenkins" ] ; then
 		# lunar only wants to configure jekyll
 		continue
 	fi
-	if [ "$user" = "phil" ] && [ "$HOSTNAME" != "jenkins-test-vm" ] ; then
+	if [ "$user" = "phil" ] && [ "$HOSTNAME" = "jenkins-test-vm" ] ; then
 		# phil only wants to test stuff
 		sudo adduser $user libvirt
 		sudo adduser $user libvirt-qemu
 		continue
-	fi
-	if [ "$user" = "philh" ] && [ "$HOSTNAME" != "jenkins-test-vm" ] ; then
-		# philh only wants to test stuff
+	elif [ "$user" = "phil" ] ; then
 		continue
 	fi
 	# actually create the user

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/qa/jenkins.debian.net.git