diff --git a/CHANGELOG.md b/CHANGELOG.md index 6888e92fc..5f3ad7bcf 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1098,7 +1098,7 @@ https://github.com/voxpupuli/puppet-nginx/pull/1385 changes the default behaviou - 'undef' from left operand of 'in' expression is not a string at /etc/puppet/modules/nginx/manifests/params.pp:23 [\#601](https://github.com/voxpupuli/puppet-nginx/issues/601) - \[WIP\] Improve SSL support [\#599](https://github.com/voxpupuli/puppet-nginx/issues/599) - ssl vhost gives error [\#585](https://github.com/voxpupuli/puppet-nginx/issues/585) -- class ::nginx::config has not been evaluated [\#580](https://github.com/voxpupuli/puppet-nginx/issues/580) +- class nginx::config has not been evaluated [\#580](https://github.com/voxpupuli/puppet-nginx/issues/580) - vagrant vhost files [\#577](https://github.com/voxpupuli/puppet-nginx/issues/577) - How to set document root in server block using hiera? [\#576](https://github.com/voxpupuli/puppet-nginx/issues/576) - Configure passenger through hiera. [\#568](https://github.com/voxpupuli/puppet-nginx/issues/568) diff --git a/HISTORY.md b/HISTORY.md index 79db9d702..b15042da0 100644 --- a/HISTORY.md +++ b/HISTORY.md @@ -607,7 +607,7 @@ - 'undef' from left operand of 'in' expression is not a string at /etc/puppet/modules/nginx/manifests/params.pp:23 [\#601](https://github.com/voxpupuli/puppet-nginx/issues/601) - \[WIP\] Improve SSL support [\#599](https://github.com/voxpupuli/puppet-nginx/issues/599) - ssl vhost gives error [\#585](https://github.com/voxpupuli/puppet-nginx/issues/585) -- class ::nginx::config has not been evaluated [\#580](https://github.com/voxpupuli/puppet-nginx/issues/580) +- class nginx::config has not been evaluated [\#580](https://github.com/voxpupuli/puppet-nginx/issues/580) - vagrant vhost files [\#577](https://github.com/voxpupuli/puppet-nginx/issues/577) - How to set document root in server block using hiera? [\#576](https://github.com/voxpupuli/puppet-nginx/issues/576) - Configure passenger through hiera. [\#568](https://github.com/voxpupuli/puppet-nginx/issues/568) diff --git a/manifests/resource/server.pp b/manifests/resource/server.pp index 267264c0a..84961943b 100644 --- a/manifests/resource/server.pp +++ b/manifests/resource/server.pp @@ -639,9 +639,9 @@ } } - create_resources('::nginx::resource::map', $string_mappings) - create_resources('::nginx::resource::geo', $geo_mappings) - create_resources('::nginx::resource::location', $locations, { + create_resources('nginx::resource::map', $string_mappings) + create_resources('nginx::resource::geo', $geo_mappings) + create_resources('nginx::resource::location', $locations, { ensure => $ensure, server => $name_sanitized, ssl => $ssl, diff --git a/spec/defines/resource_geo_spec.rb b/spec/defines/resource_geo_spec.rb index 59f527ff7..9ec0f812d 100644 --- a/spec/defines/resource_geo_spec.rb +++ b/spec/defines/resource_geo_spec.rb @@ -14,7 +14,7 @@ let :pre_condition do [ - 'include ::nginx' + 'include nginx' ] end diff --git a/spec/defines/resource_location_spec.rb b/spec/defines/resource_location_spec.rb index f52021395..7f0fe2a1c 100644 --- a/spec/defines/resource_location_spec.rb +++ b/spec/defines/resource_location_spec.rb @@ -15,7 +15,7 @@ end let :pre_condition do [ - 'include ::nginx' + 'include nginx' ] end diff --git a/spec/defines/resource_mailhost_spec.rb b/spec/defines/resource_mailhost_spec.rb index b111619ed..e2257b4f4 100644 --- a/spec/defines/resource_mailhost_spec.rb +++ b/spec/defines/resource_mailhost_spec.rb @@ -16,7 +16,7 @@ ipv6_enable: true } end - let(:pre_condition) { ['include ::nginx'] } + let(:pre_condition) { ['include nginx'] } describe 'os-independent items' do describe 'basic assumptions' do @@ -689,7 +689,7 @@ facts.merge(nginx_version: '1.16.0') end - let(:pre_condition) { ['include ::nginx'] } + let(:pre_condition) { ['include nginx'] } it 'has `ssl` at end of listen directive' do content = catalogue.resource('concat::fragment', "#{title}-ssl").send(:parameters)[:content] diff --git a/spec/defines/resource_map_spec.rb b/spec/defines/resource_map_spec.rb index 4f431e0b4..6cd03b2d2 100644 --- a/spec/defines/resource_map_spec.rb +++ b/spec/defines/resource_map_spec.rb @@ -26,7 +26,7 @@ let :pre_condition do [ - 'include ::nginx' + 'include nginx' ] end diff --git a/spec/defines/resource_server_spec.rb b/spec/defines/resource_server_spec.rb index 29e3954db..212beb91e 100644 --- a/spec/defines/resource_server_spec.rb +++ b/spec/defines/resource_server_spec.rb @@ -23,7 +23,7 @@ let :pre_condition do [ - 'include ::nginx' + 'include nginx' ] end diff --git a/spec/defines/resource_stream_spec.rb b/spec/defines/resource_stream_spec.rb index bdeb25205..8508cf6dd 100644 --- a/spec/defines/resource_stream_spec.rb +++ b/spec/defines/resource_stream_spec.rb @@ -20,7 +20,7 @@ let :pre_condition do [ - 'include ::nginx' + 'include nginx' ] end diff --git a/spec/defines/resource_upstream_spec.rb b/spec/defines/resource_upstream_spec.rb index 620137d94..af3a50b28 100644 --- a/spec/defines/resource_upstream_spec.rb +++ b/spec/defines/resource_upstream_spec.rb @@ -34,7 +34,7 @@ let :pre_condition do [ - 'include ::nginx' + 'include nginx' ] end diff --git a/templates/server/server_header.erb b/templates/server/server_header.erb index 2b6b74ddd..c641818ee 100644 --- a/templates/server/server_header.erb +++ b/templates/server/server_header.erb @@ -42,7 +42,7 @@ server { <% elsif @access_log == 'off' -%> access_log off; <% elsif not @access_log -%> - access_log <%= scope['::nginx::config::log_dir'] %>/<%= @name_sanitized %>.access.log<% if @format_log %> <%= @format_log%><% end %>; + access_log <%= scope['nginx::config::log_dir'] %>/<%= @name_sanitized %>.access.log<% if @format_log %> <%= @format_log%><% end %>; <% else -%> access_log <%= @access_log %><% if @format_log %> <%= @format_log%><% end %>; <% end -%> @@ -52,7 +52,7 @@ server { <%- end -%> <% elsif @error_log == 'absent' -%> <% elsif not @error_log -%> - error_log <%= scope['::nginx::config::log_dir'] %>/<%= @name_sanitized %>.error.log; + error_log <%= scope['nginx::config::log_dir'] %>/<%= @name_sanitized %>.error.log; <% else -%> error_log <%= @error_log %>; <% end -%> @@ -194,7 +194,7 @@ server { <% elsif @access_log == 'off' -%> access_log off; <% elsif not @access_log -%> - access_log <%= scope['::nginx::config::log_dir'] %>/<%= @name_sanitized %>.access.log<% if @format_log %> <%= @format_log%><% end %>; + access_log <%= scope['nginx::config::log_dir'] %>/<%= @name_sanitized %>.access.log<% if @format_log %> <%= @format_log%><% end %>; <% else -%> access_log <%= @access_log %><% if @format_log %> <%= @format_log%><% end %>; <% end -%> @@ -204,7 +204,7 @@ server { <%- end -%> <% elsif @error_log == 'absent' -%> <% elsif not @error_log -%> - error_log <%= scope['::nginx::config::log_dir'] %>/<%= @name_sanitized %>.error.log; + error_log <%= scope['nginx::config::log_dir'] %>/<%= @name_sanitized %>.error.log; <% else -%> error_log <%= @error_log %>; <% end -%> diff --git a/templates/server/server_ssl_header.erb b/templates/server/server_ssl_header.erb index 47774ada5..4f496943a 100644 --- a/templates/server/server_ssl_header.erb +++ b/templates/server/server_ssl_header.erb @@ -25,7 +25,7 @@ server { <% elsif @access_log == 'off' -%> access_log off; <% elsif not @access_log -%> - access_log <%= scope['::nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.access.log<% if @format_log %> <%= @format_log%><% end %>; + access_log <%= scope['nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.access.log<% if @format_log %> <%= @format_log%><% end %>; <% else -%> access_log <%= @access_log %><% if @format_log %> <%= @format_log%><% end %>; <% end -%> @@ -35,7 +35,7 @@ server { <%- end -%> <% elsif @error_log == 'absent' -%> <% elsif not @error_log -%> - error_log <%= scope['::nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.error.log; + error_log <%= scope['nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.error.log; <% else -%> error_log <%= @error_log %>; <% end -%> @@ -123,7 +123,7 @@ server { <% elsif @access_log == 'off' -%> access_log off; <% elsif not @access_log -%> - access_log <%= scope['::nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.access.log<% if @format_log %> <%= @format_log%><% end %>; + access_log <%= scope['nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.access.log<% if @format_log %> <%= @format_log%><% end %>; <% else -%> access_log <%= @access_log %><% if @format_log %> <%= @format_log%><% end %>; <% end -%> @@ -133,7 +133,7 @@ server { <%- end -%> <% elsif @error_log == 'absent' -%> <% elsif not @error_log -%> - error_log <%= scope['::nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.error.log; + error_log <%= scope['nginx::config::log_dir'] %>/ssl-<%= @name_sanitized %>.error.log; <% else -%> error_log <%= @error_log %>; <% end -%>