Pass proxy_server only when defined (#133)

deric · Jun 27, 2019 · fefb0fb · fefb0fb
1 parent d545245
commit fefb0fb
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 1 deletion.
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -22,6 +22,7 @@
   String                                     $archive_symlink_name    = $::zookeeper::params::archive_symlink_name,
   String                                     $archive_version         = $::zookeeper::params::archive_version,
   Optional[String]                           $proxy_server            = $::zookeeper::params::proxy_server,
+  Optional[String]                           $proxy_type              = $::zookeeper::params::proxy_type,
   Optional[String]                           $cdhver                  = $::zookeeper::params::cdhver,
   Boolean                                    $install_java            = $::zookeeper::params::install_java,
   String                                     $install_method          = $::zookeeper::params::install_method,

diff --git a/manifests/install/archive.pp b/manifests/install/archive.pp
@@ -33,7 +33,6 @@
     user          => 'root',
     group         => 'root',
     source        => $download_url,
-    proxy_server  => $::zookeeper::proxy_server,
     checksum      => $::zookeeper::archive_checksum['hash'],
     checksum_type => $::zookeeper::archive_checksum['type'],
     extract_path  => $::zookeeper::archive_install_dir,
@@ -46,6 +45,13 @@
     notify        => Exec['chown_zookeeper_directory'],
   }
 
+  if $::zookeeper::proxy_server {
+    Archive<| title == $archive_file |> {
+      proxy_server => $::zookeeper::proxy_server,
+      proxy_type   => $::zookeeper::proxy_type,
+    }
+  }
+
   $symlink_require = Archive["${::zookeeper::archive_install_dir}/${filename}.tar.gz"]
 
   exec { 'chown_zookeeper_directory':

diff --git a/manifests/params.pp b/manifests/params.pp
@@ -84,6 +84,7 @@
   $java_package = undef
   $repo = undef
   $proxy_server = undef
+  $proxy_type = undef
 
   # service options
   $manage_service = true