Skip to content

Commit

Permalink
Update the resource ordering for the new Splunk types.
Browse files Browse the repository at this point in the history
  • Loading branch information
Nick Perry authored and Ziaunys committed Mar 16, 2016
1 parent 47d2383 commit 78ecf59
Showing 1 changed file with 57 additions and 24 deletions.
81 changes: 57 additions & 24 deletions manifests/init.pp
Original file line number Diff line number Diff line change
Expand Up @@ -134,32 +134,65 @@
realize(Package[$package_name])
realize(Service[$virtual_service])

Exec <| tag == 'splunk_server' |> {
require +> Package[$package_name],
before +> Service[$virtual_service],
} ->
File <| tag == 'splunk_server' |> {
require +> Package[$package_name],
before +> Service[$virtual_service],
}
Package <| title == $package_name |> ->
Exec <| tag == 'splunk_server' |> ->
File <| tag == 'splunk_server' |> ->
Service <| title == $virtual_service |>

File <| tag == 'splunk_server' |> {
require +> Package[$package_name],
notify +> Service[$virtual_service],
} ->
Splunk_input <| tag == 'splunk_server' |> {
require +> Package[$package_name],
notify +> Service[$virtual_service],
}
Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_authentication <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

File <| tag == 'splunk_server' |> {
require +> Package[$package_name],
notify +> Service[$virtual_service],
} ->
Splunk_output <| tag == 'splunk_server' |> {
require +> Package[$package_name],
notify +> Service[$virtual_service],
}
Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_authorize <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_distsearch <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_indexes <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_input <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_limits <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_output <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_props <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_server <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_transforms <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

Package <| title == $package_name |> ->
File <| tag == 'splunk_server' |> ->
Splunk_web <| tag == 'splunk_server' |> ~>
Service <| title == $virtual_service |>

# Validate: if both Splunk and Splunk Universal Forwarder are installed on
# the same system, then they must use different admin ports.
Expand Down

1 comment on commit 78ecf59

@Vincent--
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure we can use an array in a resource collector, see https://docs.puppet.com/puppet/4.8/reference/lang_collectors.html#search-expressions

== (equality search)

This operator is non-symmetric:

The left operand (attribute) must be the name of a resource attribute or the word title (which searches on the resource’s title).
The right operand (search key) must be a string, boolean, number, resource reference, or undef. The behavior of arrays and hashes in the right operand is undefined in this version of Puppet.
For a given resource, this operator will match if the value of the attribute (or one of the value’s members, if the value is an array) is identical to the search key.

I'm pretty sure this approach doesn't restart the service when it's expected

Please sign in to comment.