From f304ba02fd5602319c58c538d980fe3e70276766 Mon Sep 17 00:00:00 2001 From: Clark Boylan Date: Fri, 12 Oct 2012 15:26:05 -0700 Subject: [PATCH] Cleanup openstack_project::jenkins. There were a few extra []'s and some missing commas. Also leave a comment that 'gerrig' is not a typo. Change-Id: I04a5fa026fbd715ac370e070ff75b474fbe6335f Reviewed-on: https://review.openstack.org/14414 Reviewed-by: Paul Belanger Reviewed-by: Monty Taylor Approved: James E. Blair Reviewed-by: James E. Blair Tested-by: Jenkins --- modules/openstack_project/manifests/jenkins.pp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/modules/openstack_project/manifests/jenkins.pp b/modules/openstack_project/manifests/jenkins.pp index 92d5190e68..2759b71265 100644 --- a/modules/openstack_project/manifests/jenkins.pp +++ b/modules/openstack_project/manifests/jenkins.pp @@ -8,7 +8,7 @@ class openstack_project::jenkins ( ) { class { 'openstack_project::server': iptables_public_tcp_ports => [80, 443, 4155], - sysadmins => $sysadmins + sysadmins => $sysadmins, } $vhost_name = 'jenkins.openstack.org' @@ -27,7 +27,7 @@ class openstack_project::jenkins ( if manage_jenkins_jobs == true { class { '::jenkins::job_builder': url => "https://${vhost_name}/", - username => 'gerrig', + username => 'gerrig', # This is not a typo, well it isn't anymore. password => $jenkins_jobs_password, } @@ -37,13 +37,13 @@ class openstack_project::jenkins ( group => 'root', mode => '0755', recurse => true, - source => ['puppet:///modules/openstack_project/jenkins_job_builder/config'], - notify => Exec['jenkins_jobs_update'] + source => 'puppet:///modules/openstack_project/jenkins_job_builder/config', + notify => Exec['jenkins_jobs_update'], } file { '/etc/default/jenkins': ensure => present, - source => 'puppet:///modules/openstack_project/jenkins/jenkins.default' + source => 'puppet:///modules/openstack_project/jenkins/jenkins.default', } } }