Fix bug where systemd was always assumed, if not windows

EncoreTechnologies · May 22, 2020 · 962af0d · 962af0d
1 parent dba7056
commit 962af0d
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file.
 
+## Release 0.1.1
+
+**Bugfixes**
+
+* Change from using `$facts['os']['family']` to `$facts['service_provider']` from `puppetlabs/stdlib`.
+  This allows `service_autorestart::generic` to "do the right thing" when detecting systemd.
+
 ## Release 0.1.0
 
 **Features**

diff --git a/manifests/generic.pp b/manifests/generic.pp
@@ -4,12 +4,13 @@
 # to tweak settings,  you'll need to declare the `service_autorestart::windows`
 # or `service_autorestart::systemd` directly.
 define service_autorestart::generic () {
-  case $facts['os']['family'] {
+  case $facts['service_provider'] {
     'windows': {
       service_autorestart::windows { $title: }
     }
-    default: {
+    'systemd': {
       service_autorestart::systemd { $title: }
     }
+    default: {}
   }
 }
diff --git a/metadata.json b/metadata.json
@@ -1,13 +1,17 @@
 {
   "name": "encore-service_autorestart",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "author": "Encore Technologies",
   "summary": "Puppet module to manage auto-restarting services (Windows and SystemD), and Windows Service Recovery options",
   "license": "Apache-2.0",
   "source": "https://github.com/EncoreTechnologies/puppet-service_autorestart",
   "project_page": "https://github.com/EncoreTechnologies/puppet-service_autorestart",
   "issues_url": "https://github.com/EncoreTechnologies/puppet-service_autorestart/issues",
   "dependencies": [
+    {
+      "name": "puppetlabs/stdlib",
+      "version_requirement": ">= 4.0.0 < 7.0.0"
+    },
     {
       "name": "puppetlabs/inifile",
       "version_requirement": ">= 3.0.0 < 5.0.0"