Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/adblockplus/manifests/host.pp

Issue 29366814: Noissue - Fix adblockplus::host $ensure = "absent" support (Closed)
Patch Set: Created Dec. 5, 2016, 10:49 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | modules/private-stub/hiera/hosts.yaml » ('j') | modules/private-stub/hiera/hosts.yaml » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: modules/adblockplus/manifests/host.pp
diff --git a/modules/adblockplus/manifests/host.pp b/modules/adblockplus/manifests/host.pp
index a2441b3095615256653f29acd20e51d49066abc0..2e5267bfd36472f1d09f3e8b12f1cf46b76609f5 100644
--- a/modules/adblockplus/manifests/host.pp
+++ b/modules/adblockplus/manifests/host.pp
@@ -58,7 +58,7 @@ define adblockplus::host (
$ensure = 'present',
$fqdn = "$name.$adblockplus::authority",
$groups = [],
- $ips,
+ $ips = [],
$public_key = undef,
$role = undef,
) {
@@ -84,7 +84,7 @@ define adblockplus::host (
@host {$title:
ensure => $ensure,
- ip => pick($ips),
+ ip => pick($ips[0], '0.0.0.0'),
name => $fqdn,
tag => ['adblockplus::host'],
}
« no previous file with comments | « no previous file | modules/private-stub/hiera/hosts.yaml » ('j') | modules/private-stub/hiera/hosts.yaml » ('J')

Powered by Google App Engine
This is Rietveld