diff --git a/manifests/site.pp b/manifests/site.pp index a96909f713..00e1b8c7d9 100644 --- a/manifests/site.pp +++ b/manifests/site.pp @@ -229,16 +229,9 @@ node 'zuul.openstack.org' { url_pattern => 'http://logs.openstack.org/{change.number}/{change.patchset}/{pipeline.name}/{job.name}/{build.number}', sysadmins => hiera('sysadmins'), statsd_host => 'graphite.openstack.org', - gearman_workers => [ - 'jenkins.openstack.org', - 'jenkins-dev.openstack.org', - ], } # co-host gearman-job-server include gearman - class { 'gearman': - listen => '::', - } } # A bare machine, but with a jenkins user diff --git a/modules/openstack_project/manifests/jenkins.pp b/modules/openstack_project/manifests/jenkins.pp index c8dd59429d..23311f97c8 100644 --- a/modules/openstack_project/manifests/jenkins.pp +++ b/modules/openstack_project/manifests/jenkins.pp @@ -9,7 +9,7 @@ class openstack_project::jenkins ( $sysadmins = [] ) { class { 'openstack_project::server': - iptables_public_tcp_ports => [80, 443], + iptables_public_tcp_ports => [80, 443, 4155], sysadmins => $sysadmins, } diff --git a/modules/openstack_project/manifests/jenkins_dev.pp b/modules/openstack_project/manifests/jenkins_dev.pp index cf91c2520a..1a332c99f2 100644 --- a/modules/openstack_project/manifests/jenkins_dev.pp +++ b/modules/openstack_project/manifests/jenkins_dev.pp @@ -4,7 +4,7 @@ class openstack_project::jenkins_dev ( $sysadmins = [] ) { class { 'openstack_project::server': - iptables_public_tcp_ports => [80, 443], + iptables_public_tcp_ports => [80, 443, 4155], sysadmins => $sysadmins, } include bup diff --git a/modules/openstack_project/manifests/zuul.pp b/modules/openstack_project/manifests/zuul.pp index bc3c0ab572..74d42bbd5c 100644 --- a/modules/openstack_project/manifests/zuul.pp +++ b/modules/openstack_project/manifests/zuul.pp @@ -11,20 +11,14 @@ class openstack_project::zuul( $zuul_ssh_private_key = '', $url_pattern = '', $sysadmins = [], - $statsd_host = '', - $gearman_workers = [] + $statsd_host = '' ) { - # Turn a list of hostnames into a list of iptables rules - $iptables_rules6 = regsubst ($gearman_workers, '^(.*)$', '-m state --state NEW -m tcp -p tcp --dport 4730 -s \1 -j ACCEPT') - $iptables_rules4 = $iptables_rules6 - - $iptables_rules4 += [ "-m state --state NEW -m tcp -p tcp --dport 8001 -s ${jenkins_host} -j ACCEPT" ] + $rules = [ "-m state --state NEW -m tcp -p tcp --dport 8001 -s ${jenkins_host} -j ACCEPT" ] class { 'openstack_project::server': iptables_public_tcp_ports => [80], - iptables_rules6 => $iptables_rules6, - iptables_rules4 => $iptables_rules4, + iptables_rules4 => $rules, sysadmins => $sysadmins, }