windmill/ansible-role-nodepool
Code Issues Proposed changes

Remove PREFIX from service files

Browse Source 
This was really to work around the need not to template our service
files.  Mostly because I was scared how large and unmanageable they
would become.  Let ansible do its job and remove the hacks we had in
our systemd unit file.

Change-Id: I92a4167900bbf48d564207017a9790d4102f937f
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This commit is contained in:
Paul Belanger 2018-04-04 20:22:31 -04:00
parent 9105363441
commit 663e9ce11c
No known key found for this signature in database
GPG Key ID: 611A80832067AF38
2 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
templates/etc/systemd/system/nodepool-builder.service
View File

@ -6,12 +6,9 @@ After=syslog.target network.target
Type=simple
# Options to pass to nodepool-builder.
Environment="ARGS=-l /etc/nodepool/builder-logging.conf"
# The prefix used when nodepool-builder was installed. Be sure to update this
# value if you are using a virtualenv.
Environment="PREFIX=/usr/local"
Group=nodepool
User=nodepool
ExecStart=/bin/sh -c "${PREFIX}/bin/nodepool-builder -d ${ARGS}"
ExecStart={{ nodepool_pip_virtualenv | default('/usr/local') }}/bin/nodepool-builder -d $ARGS
[Install]
WantedBy=multi-user.target

5
templates/etc/systemd/system/nodepool-launcher.service
View File

@ -6,12 +6,9 @@ After=syslog.target network.target
Type=simple
# Options to pass to nodepool-launcher.
Environment="ARGS=-l /etc/nodepool/launcher-logging.conf --no-webapp"
# The prefix used when nodepool-launcher was installed. Be sure to update this
# value if you are using a virtualenv.
Environment="PREFIX=/usr/local"
Group=nodepool
User=nodepool
ExecStart=/bin/sh -c "${PREFIX}/bin/nodepool-launcher -d ${ARGS}"
ExecStart={{ nodepool_pip_virtualenv | default('/usr/local') }}/bin/nodepool-launcher -d $ARGS
[Install]
WantedBy=multi-user.target