Merge pull request #212 from kscherer/suse_support

Add support of OpenSuSE and SLED
voxpupuli · Jan 6, 2016 · 4e93a9b · 4e93a9b
2 parents 57b0e94 + e9c0798
commit 4e93a9b
Show file tree

Hide file tree

Showing 3 changed files with 56 additions and 2 deletions.
diff --git a/manifests/params.pp b/manifests/params.pp
@@ -55,8 +55,14 @@
     }
   } elsif $::operatingsystem == 'Archlinux' {
     $init_style = 'systemd'
-  } elsif $::operatingsystem == 'SLES' {
-    $init_style = 'sles'
+  } elsif $::operatingsystem == 'OpenSuSE' {
+    $init_style = 'systemd'
+  } elsif $::operatingsystem =~ /SLE[SD]/ {
+    if versioncmp($::operatingsystemrelease, '12.0') < 0 {
+      $init_style = 'sles'
+    } else {
+      $init_style = 'systemd'
+    }
   } elsif $::operatingsystem == 'Darwin' {
     $init_style = 'launchd'
   } elsif $::operatingsystem == 'Amazon' {

diff --git a/metadata.json b/metadata.json
@@ -77,6 +77,27 @@
         "20",
         "21"
       ]
+    },
+    {
+      "operatingsystem": "OpenSuSE",
+      "operatingsystemrelease": [
+        "13.1",
+        "13.2"
+      ]
+    },
+    {
+      "operatingsystem": "SLES",
+      "operatingsystemrelease": [
+        "11.4",
+        "12.0"
+      ]
+    },
+    {
+      "operatingsystem": "SLED",
+      "operatingsystemrelease": [
+        "11.4",
+        "12.0"
+      ]
     }
   ]
 }
diff --git a/spec/classes/init_spec.rb b/spec/classes/init_spec.rb
@@ -690,6 +690,33 @@
     it { should contain_class('consul').with_init_style('debian') }
   end
 
+  context "On opensuse" do
+    let(:facts) {{
+      :operatingsystem => 'OpenSuSE',
+      :operatingsystemrelease => '13.1'
+    }}
+
+    it { should contain_class('consul').with_init_style('systemd') }
+  end
+
+  context "On SLED" do
+    let(:facts) {{
+      :operatingsystem => 'SLED',
+      :operatingsystemrelease => '11.4'
+    }}
+
+    it { should contain_class('consul').with_init_style('sles') }
+  end
+
+  context "On SLES" do
+    let(:facts) {{
+      :operatingsystem => 'SLES',
+      :operatingsystemrelease => '12.0'
+    }}
+
+    it { should contain_class('consul').with_init_style('systemd') }
+  end
+
   # Config Stuff
   context "With extra_options" do
     let(:params) {{