Merge branch 'master' of github.com:jfryman/puppet-nginx
This commit is contained in:
commit
398d0c9ae7
2 changed files with 18 additions and 8 deletions
|
@ -27,23 +27,33 @@ Magically, it's all done! Work through these until the deprecation notices go aw
|
|||
|
||||
Maybe for some reason, Hiera isn't being used in your organization. Or, you like to keep a certain amount of composibilty in you modules. Or, hidden option #3! Regardless, the recommended path is to instantiate your own copy of Class[nginx::config] and move on with life. Let's do another example.
|
||||
|
||||
Assume the same code block as before:
|
||||
Assume you have the following code block:
|
||||
|
||||
```ruby
|
||||
class { 'nginx':
|
||||
gzip => false,
|
||||
class { 'nginx' :
|
||||
manage_repo => false,
|
||||
confd_purge => true,
|
||||
vhost_purge => true,
|
||||
}
|
||||
```
|
||||
|
||||
Should become...
|
||||
This should become...
|
||||
|
||||
```ruby
|
||||
include nginx
|
||||
class { 'nginx::config':
|
||||
gzip => false,
|
||||
Anchor['nginx::begin']
|
||||
->
|
||||
class { 'nginx::config' :
|
||||
confd_purge => true,
|
||||
vhost_purge => true,
|
||||
}
|
||||
|
||||
class { 'nginx' :
|
||||
manage_repo => false,
|
||||
}
|
||||
```
|
||||
|
||||
The order in which this commands are parsed is important, since nginx looks for nginx::config via a defined(nginx::config) statement, which as of puppet 3.x is still parse-order dependent.
|
||||
|
||||
# Why again are you doing this?
|
||||
|
||||
Well, the fact of the matter, the old Package/Config/Service pattern has served us well, but times are a-changin. Many users are starting to manage their packages and service seperately outside of the traditional pattern (Docker, anyone?). This means that in order to stay true to the goals of Configuration Management, it is becoming necessary to make less assumptions about how an organizations graph is composed, and allow the end-user additional flexibility. This is requring a re-think about how to best consume this module.
|
||||
|
|
|
@ -530,7 +530,7 @@ define nginx::resource::vhost (
|
|||
|
||||
if $use_default_location == true {
|
||||
# Create the default location reference for the vHost
|
||||
::nginx::resource::location {"${name_sanitized}-default":
|
||||
nginx::resource::location {"${name_sanitized}-default":
|
||||
ensure => $ensure,
|
||||
vhost => $name_sanitized,
|
||||
ssl => $ssl,
|
||||
|
|
Loading…
Reference in a new issue