From d31904a5280209aefec146dac37f060642089009 Mon Sep 17 00:00:00 2001 From: Manish Goregaokar Date: Thu, 9 Aug 2018 14:44:42 -0700 Subject: [PATCH] Remove require: user: servo This only works if we have a `servo: user.present` state somewhere; but we don't, so we get the following error: ``` Comment: The following requisites were not found: require: user: servo ``` We assume the user exists elsewhere anyway, though perhaps we could add the user.present in case (but when i tried i wasn't able to make that work) --- buildbot/master/init.sls | 3 --- 1 file changed, 3 deletions(-) diff --git a/buildbot/master/init.sls b/buildbot/master/init.sls index b273c398..36b47ad4 100644 --- a/buildbot/master/init.sls +++ b/buildbot/master/init.sls @@ -21,7 +21,6 @@ buildbot-master: # Buildbot must be restarted manually! See 'Buildbot administration' on the # wiki and https://github.com/servo/saltfs/issues/304. - require: - - user: servo - pip: buildbot-master - file: ownership-{{ common.servo_home }}/buildbot/master - file: /etc/init/buildbot-master.conf @@ -41,8 +40,6 @@ deploy-{{ common.servo_home }}/buildbot/master: homu: {{ homu }} buildbot_credentials: {{ pillar['buildbot']['credentials'] }} wpt_sync_credentials: {{ pillar['wpt-sync'] }} - - require: - - user: servo ownership-{{ common.servo_home }}/buildbot/master: file.directory: