Merge pull request #6 from kupferk/master

Improved compatibility with CentOS

Author: kupferk

.gitignore

@@ -0,0 +1,2 @@
+pkg/*
+

Modulefile

@@ -1,5 +1,5 @@
 name 'dimajix-impala'
-version '0.1.8'
+version '0.1.9'
 source 'https://github.com/dimajix/puppet-impala'
 author 'Kaya Kupferschmidt <k.kupferschmidt@dimajix.de>'
 license 'MIT'

manifests/params.pp

@@ -4,8 +4,8 @@
 # It sets variables according to platform
 #
 class impala::params {
-  case $::osfamily {
-    'Debian': {
+  case "${::osfamily}-${::operatingsystem}" {
+    /RedHat-Fedora/: {
       $packages = {
         common => [ 'impala' ],
         catalog => 'impala-catalog',
@@ -19,7 +19,7 @@
         server => 'impala-server',
       }
     }
-    'RedHat': {
+    /Debian|RedHat/: {
       $packages = {
         common => [ 'impala' ],
         catalog => 'impala-catalog',
@@ -42,9 +42,9 @@
     debian => 'cluster',
     redhat => undef,
   }
-  $confdir = $::osfamily ? {
-    debian => '/etc/impala/conf',
-    redhat => '/etc/impala',
+  $confdir = "${::osfamily}-${::operatingsystem}" ? {
+    /Fedora-RedHat/ => '/etc/hive',
+    /Debian|RedHat/ => '/etc/hive/conf',
   }
   $config = '/etc/default/impala'
 

metadata.json

@@ -1,6 +1,6 @@
 {
   "name": "dimajix/impala",
-  "version": "0.1.8",
+  "version": "0.1.9",
   "author": "Kaya Kupferschmidt <k.kupferschmidt@dimajix.de>",
   "summary": "Puppet module for managing Cloudera Impala",
   "license": "Apache 2.0",