]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commitdiff
debian: service: fix `ensure` parameter usage
authorDamon Breeden <damon.breeden@ahrefs.com>
Fri, 2 Dec 2022 16:04:56 +0000 (16:04 +0000)
committerDamon Breeden <damon.breeden@ahrefs.com>
Fri, 2 Dec 2022 16:04:56 +0000 (16:04 +0000)
`ensure` is defined but not used anywhere, no clear way to ensure the service is running
(and makes it impossible to do so outside the module)

manifests/linux/debian.pp

index c8363d22118c19de723d28a61d081eab5f0061ab..b8d6cdec7ba9af2c438aafe054d99be888a6d513 100644 (file)
@@ -34,7 +34,7 @@ class firewall::linux::debian (
   # This isn't a real service/daemon. The start action loads rules, so just
   # needs to be called on system boot.
   service { $service_name:
-    ensure    => undef,
+    ensure    => $ensure,
     enable    => $enable,
     hasstatus => true,
     require   => Package[$package_name],