diff --git a/manifests/executor.pp b/manifests/executor.pp index f3ce82c..521464c 100644 --- a/manifests/executor.pp +++ b/manifests/executor.pp @@ -19,7 +19,6 @@ class zuul::executor ( $ensure = undef, $manage_log_conf = true, - $site_variable_yaml_file = undef, ) { include ::apt apt::ppa { 'ppa:openstack-ci-core/bubblewrap': } @@ -59,18 +58,6 @@ class zuul::executor ( } } - if $site_variable_yaml_file != undef { - file { '/etc/zuul/site-variables.yaml': - ensure => file, - group => 'zuul', - mode => '0644', - owner => 'zuul', - replace => true, - require => File['/etc/zuul'], - source => $site_variable_yaml_file, - } - } - include ::logrotate ::logrotate::file { 'executor.log': log => '/var/log/zuul/executor.log', diff --git a/manifests/init.pp b/manifests/init.pp index 707716a..a2989cf 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -86,6 +86,7 @@ class zuul ( $gearman_ssl_ca = undef, $zuul_web_url = 'http://127.0.0.1:9000', $zuul_scheduler_url = 'http://127.0.0.1:8001', + $site_variables_yaml_file = undef, ) { include ::httpd include ::pip @@ -549,6 +550,18 @@ class zuul ( } } + if $site_variables_yaml_file != undef { + file { '/etc/zuul/site-variables.yaml': + ensure => file, + group => 'zuul', + mode => '0644', + owner => 'zuul', + replace => true, + require => File['/etc/zuul'], + source => $site_variables_yaml_file, + } + } + ::httpd::vhost { $vhost_name: port => 443, # Is required despite not being used. docroot => 'MEANINGLESS ARGUMENT', diff --git a/templates/zuulv3.conf.erb b/templates/zuulv3.conf.erb index 5ecc556..df69eaf 100644 --- a/templates/zuulv3.conf.erb +++ b/templates/zuulv3.conf.erb @@ -48,6 +48,9 @@ git_user_name=<%= @git_name %> [executor] log_config=/etc/zuul/executor-logging.conf +<% if @site_variables_yaml_file != nil -%> +variables=/etc/zuul/site-variables.yaml +<% end -%> <% if @worker_private_key_file != "" -%> private_key_file=<%= @worker_private_key_file %> <% end -%>