Merge pull request #8 from jpuskar/0_1_8

Added manage_host_entry parameter to make the host resource optional.
jpuskar · Jun 10, 2017 · 0aced6f · 0aced6f
2 parents 92cf5db + 864fa02
commit 0aced6f
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 7 deletions.
diff --git a/Changelog.md b/Changelog.md
@@ -1,5 +1,8 @@
 # puppet-cognos changelog
 
+## v0.1.8
++ Added manage_host_entry parameter to make the host resource optional.
+
 ## v0.1.7
 + Fixed another cert database syntax error.
 

diff --git a/manifests/init.pp b/manifests/init.pp
@@ -76,6 +76,7 @@
   Optional[Array]   $ldaps_public_certs_to_trust  = [],
   Optional[String]  $license_type                 = 'production',
   Optional[Boolean] $manage_firewall              = true,
+  Optional[Boolean] $manage_host_entry            = true,
   Optional[Boolean] $manage_ldaps_cert_db         = false,
   Optional[Integer] $pmp_max_memory               = 1024,
   Optional[String]  $release_type                 = 'ga',

diff --git a/manifests/install/prep.pp b/manifests/install/prep.pp
@@ -26,10 +26,11 @@
   }
 
   # Cognos won't start without this host entry
-  ensure_resource(
-    'host',
-    $::fqdn,
-    { ip => '127.0.0.1' }
-  )
-
+  if $cognos::manage_host_entry {
+    ensure_resource(
+      'host',
+      $::fqdn,
+      { ip => '127.0.0.1' }
+    )
+  }
 }
diff --git a/metadata.json b/metadata.json
@@ -1,6 +1,6 @@
 {
   "name": "jpuskar-cognos",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "author": "jpuskar",
   "summary": "Installs and configures Cognos.",
   "license": "Apache-2.0",