Skip to content

Commit 8906b66

Browse files
committed
Updated monitoring_plan variable to be a hash instead of an array that now includes an overridable monitoring target
1 parent a306365 commit 8906b66

2 files changed

Lines changed: 21 additions & 9 deletions

File tree

REFERENCE.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1344,8 +1344,10 @@ Disable monitoring for targets in multiple services
13441344
vars:
13451345
patching_monitoring_plan: 'patching::monitoring_multiple'
13461346
patching_monitoring_plan_multiple:
1347-
- 'patching::monitoring_solarwinds'
1348-
- 'patching::monitoring_prometheus'
1347+
- plan: 'patching::monitoring_solarwinds'
1348+
target: 'solarwinds'
1349+
- plan: 'patching::monitoring_prometheus'
1350+
target: 'prometheus'
13491351
13501352
groups:
13511353
- name: solarwinds

plans/monitoring_multiple.pp

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,10 @@
1616
# vars:
1717
# patching_monitoring_plan: 'patching::monitoring_multiple'
1818
# patching_monitoring_plan_multiple:
19-
# - 'patching::monitoring_solarwinds'
20-
# - 'patching::monitoring_prometheus'
19+
# - plan: 'patching::monitoring_solarwinds'
20+
# target: 'solarwinds'
21+
# - plan: 'patching::monitoring_prometheus'
22+
# target: 'prometheus'
2123
#
2224
# groups:
2325
# - name: solarwinds
@@ -48,14 +50,22 @@
4850
plan patching::monitoring_multiple (
4951
TargetSpec $targets,
5052
Enum['enable', 'disable'] $action,
51-
Array $monitoring_plan = get_targets($targets)[0].vars['patching_monitoring_plan_multiple'],
53+
Array[Hash] $monitoring_plans = get_targets($targets)[0].vars['patching_monitoring_plan_multiple'],
5254
Boolean $noop = false,
5355
) {
5456

5557
# Loop over and run each monitoring plan
56-
$monitoring_plan.each |String $plan| {
57-
run_plan($plan, $targets,
58-
action => $action,
59-
noop => $noop)
58+
$monitoring_plans.each |Hash $plan_hash| {
59+
if $plan_hash['target'] {
60+
run_plan($plan_hash['plan'], $targets,
61+
action => $action,
62+
monitoring_target => $plan_hash['target'],
63+
noop => $noop)
64+
}
65+
else {
66+
run_plan($plan_hash['plan'], $targets,
67+
action => $action,
68+
noop => $noop)
69+
}
6070
}
6171
}

0 commit comments

Comments
 (0)