Skip to content

Commit

Permalink
Split letsencrypt commands, so -auto is only used once
Browse files Browse the repository at this point in the history
So we avoid fiddling with the venv every time, and the cron is now
silent.

refs voxpupuliGH-26
  • Loading branch information
lazyfrosch committed Apr 2, 2016
1 parent 6fe18b8 commit 4ba4a54
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion manifests/init.pp
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,11 @@
}

$command = $install_method ? {
'package' => 'letsencrypt',
'vcs' => "${venv_path}/bin/letsencrypt",
}

$command_init = $install_method ? {
'package' => 'letsencrypt',
'vcs' => "${path}/letsencrypt-auto",
}
Expand All @@ -79,8 +84,9 @@
Class['letsencrypt::config'] -> Exec['initialize letsencrypt']
}

# TODO: do we need this command when installing from package?
exec { 'initialize letsencrypt':
command => "${command} -h",
command => "${command_init} -h",
path => $::path,
environment => ["VENV_PATH=${venv_path}"],
refreshonly => true,
Expand Down

0 comments on commit 4ba4a54

Please sign in to comment.