From 533ff342283be8cb228d4997435a0a0cbceff2c3 Mon Sep 17 00:00:00 2001
From: Tim Flink <tflink@fedoraproject.org>
Date: Wed, 2 Jul 2014 15:04:11 +0000
Subject: [PATCH] correcting use of sudo so that buildmaster stuff is created
 with correct user

---
 roles/taskotron/buildmaster-configure/tasks/main.yml | 10 ++++++++--
 roles/taskotron/buildmaster/tasks/main.yml           |  2 +-
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/roles/taskotron/buildmaster-configure/tasks/main.yml b/roles/taskotron/buildmaster-configure/tasks/main.yml
index 5fc1b8be08..73673cf7bb 100644
--- a/roles/taskotron/buildmaster-configure/tasks/main.yml
+++ b/roles/taskotron/buildmaster-configure/tasks/main.yml
@@ -1,10 +1,16 @@
 - name: create master
+  sudo: true
+  sudo_user: "{{ master_user }}"
   command: creates={{ buildmaster_dir }} buildbot create-master -r {{ buildmaster_dir }}
 
 - name: upload master config
+  sudo: true
+  sudo_user: "{{ master_user }}"
   template: src={{ buildmaster_template }} dest={{ buildmaster_dir }}/master.cfg owner={{ master_user }} group={{ master_user }}
 
 - name: check master config
+  sudo: true
+  sudo_user: "{{ master_user }}"
   command: buildbot checkconfig {{ buildmaster_dir }}
 
 #- name: upgrade master
@@ -12,11 +18,11 @@
 
 - name: generate buildmaster service file
   template: src=buildmaster.service.j2 dest=/lib/systemd/system/buildmaster.service owner=root group=root mode=0744
-  sudo_user: root
 
 - name: start and enable buildmaster service
   service: name=buildmaster enabled=yes state=started
-  sudo_user: root
 
 - name: reconfig master
+  sudo: true
+  sudo_user: "{{ master_user }}"
   command: buildbot reconfig {{ buildmaster_dir }}
diff --git a/roles/taskotron/buildmaster/tasks/main.yml b/roles/taskotron/buildmaster/tasks/main.yml
index fc1a6a85fa..b9202a6670 100644
--- a/roles/taskotron/buildmaster/tasks/main.yml
+++ b/roles/taskotron/buildmaster/tasks/main.yml
@@ -6,7 +6,7 @@
     - libsemanage-python
 
 - name: add the buildmaster user
-  user: name=buildmaster
+  user: name={{ master_user }}
 
 - name: allow httpd tcp connections with selinux
   seboolean: name=httpd_can_network_connect state=true persistent=yes