Merge pull request #204 from eshamow/fix_undefined_jenkins_port

Fix for #174 allows setting $jenkins::port
voxpupuli · Oct 24, 2014 · da1c743 · da1c743
2 parents 6411d6b + 06504c0
commit da1c743
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -12,6 +12,9 @@
 # lts = true
 #   Use LTS verison of jenkins
 #
+# port = 8080 (default)
+#   Sets firewall port to 8080 if puppetlabs-firewall module is installed
+#
 # repo = true (Default)
 #   install the jenkins repo.
 #
@@ -109,6 +112,7 @@
   $proxy_port         = undef,
   $no_proxy_list      = undef,
   $cli                = undef,
+  $port               = $jenkins::params::port,
   $libdir             = $jenkins::params::libdir,
 ) inherits jenkins::params {
 

diff --git a/manifests/params.pp b/manifests/params.pp
@@ -10,6 +10,7 @@
   $install_java               = true
   $swarm_version              = '1.17'
   $default_plugins_host       = 'http://updates.jenkins-ci.org'
+  $port                       = '8080'
 
   case $::osfamily {
     'Debian': {