Skip to content

Commit

Permalink
Varnish: install the systemd drop-in file after varnish is installed.
Browse files Browse the repository at this point in the history
This way we don't block the first run of the service because of a
ExecStartPre compiling a vcl that may not be present yet.
  • Loading branch information
jonhattan committed Feb 3, 2018
1 parent 9e71eda commit 7f7c164
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion manifests/varnish.pp
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@
# Restart varnish if the systemd unit has changed.
# Note: system::dropin_file is available in systemd >= 1.0.0
if defined('::systemd') and defined(Class['::systemd']) and defined(Resource['::systemd::dropin_file']) {
Systemd::Dropin_file<| unit == 'varnish.service' |> ~> Class['::varnish::service']
Class['::varnish::install']
-> Systemd::Dropin_file<| unit == 'varnish.service' |>
~> Class['::varnish::service']
}
}

Expand Down

1 comment on commit 7f7c164

@NITEMAN
Copy link
Member

@NITEMAN NITEMAN commented on 7f7c164 Mar 2, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jonhattan confirmed that this can be fixed via hiera

Please sign in to comment.