summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMicah Anderson <micah@riseup.net>2009-08-27 17:05:36 -0400
committerMicah Anderson <micah@riseup.net>2009-08-27 17:05:36 -0400
commit1ec565f20a431ab8b44d655cb59ab01706b5312f (patch)
tree345fe0525602a2fae053ca67cad788d503ff7ab6
parent264f3ef1216d1e1e4ab0171e7b503bbb8c4658e1 (diff)
I unset the duplicity options that are set by default by backupninja
already (it is unecessary to set them when they are already set to the defaults): testconnect, sign, keep, bandwidthlimit I also unset because they were not set to the defaults: $tmpdir = "/var/tmp/duplicity", $nicelevel = 19. The defaults for these are: $tmpdir = "/tmp", and $nicelevel = 0. The example configuration files included in backupninja are confusing about these, so I've just clarified those in the upstream source. Then there were a couple options that were set to values that I think should not be set: $password = "a_very_complicated_passphrase" -- I dont think this should be set by default at all, because people will accidentally use this passphrase by default (yuck). People should be forced to set this. I also reset the $destdir = "/backups" setting to be false. This value must be set, but I dont think it should be set by default.
-rw-r--r--manifests/dup.pp16
1 files changed, 8 insertions, 8 deletions
diff --git a/manifests/dup.pp b/manifests/dup.pp
index 26514ad..afcdab2 100644
--- a/manifests/dup.pp
+++ b/manifests/dup.pp
@@ -37,14 +37,14 @@ define backupninja::duplicity( $order = 90,
$ensure = present,
# options to the config file
$options = false,
- $nicelevel = 0,
- $testconnect = "yes",
- $tmpdir = "/var/tmp/duplicity",
+ $nicelevel = false,
+ $testconnect = false,
+ $tmpdir = false,
# [gpg]
- $sign = "no",
+ $sign = false,
$encryptkey = false,
$signkey = false,
- $password = "a_very_complicated_passphrase",
+ $password = false,
# [source]
$include = [ "/var/spool/cron/crontabs",
"/var/backups",
@@ -67,10 +67,10 @@ define backupninja::duplicity( $order = 90,
$vsinclude = false,
# [dest]
$incremental = "yes",
- $keep = 60,
- $bandwithlimit = "0",
+ $keep = false,
+ $bandwithlimit = false,
$sshoptions = false,
- $destdir = "/backups",
+ $destdir = false,
$desthost = false,
$destuser = false,
# configs to backupninja client