Minor fixes

Initial code for rsync handler

Fixing user comments

Minor fix at Backupninja_server_realize realization

Another change on backupninja::rdiff backuptag due to the introduction of backupninja_server_realize

Adding back duplicate definition check for backupninja_server_realize

Testing without checking duplicate definition of backupninja_server_realize

Checking for backupninja_server_realize definition / comments

Fixing backupninja::server::sandbox parameter on backupninja::rdiff

Trying a different approach to realize exported resources

Trying another method to override an exported resource if already defined

Reverting last change and trying exported resources override instead

Using namevars to deal with multiple backup destinations

change the parameter values in the sys handler to use true/false
values for the default to be more uniform with the other handler
definitions

Reverting "the lvm and luks sys template should default to 'no' for both of these options. turning them on by default means that you get warnings/errors for systems that do not have either of these. you should opt-in, not opt-out"

I misunderstood the ternary operator here

Merge branch 'master' of git://labs.riseup.net/module_backupninja

fix a couple more bandwidthlimit spelling errors

fix the spelling of the bandwidthlimit option

Avoiding duplicate definition of ssh_authorized_key

Merge branch 'master' of git://labs.riseup.net/module_backupninja

the lvm and luks sys template should default to 'no' for both of these options.
turning them on by default means that you get warnings/errors for systems that
do not have either of these. you should opt-in, not opt-out

Merge branch 'master' of git://labs.riseup.net/module_backupninja

fix syntax error in parameter delineation

add doluks and dolvm parameters to the sys handlers

Merge branch 'master' of git://labs.riseup.net/module_backupninja

Conflicts:
manifests/rdiff.pp

allow for a version variable override for the backupninja package,
instead of hard-coding the 0.9.6 release. This changes the installed
version to be 'installed', and lets you specify a variable if you wish
to install a different version

enclose namevar in curly braces

let the user override the nagios2_description, which is necessary for
multiple backup defines on one host

change the nagios2::passive_service check so that it uses the $name variable instead of the $real_host variable because $real_host prevents us from creating more than one rdiff-backup define on a server because this will cause a duplicate defintion. The $name variable allows for multiple without creating duplicate passive_service checks

Adding $subfolder parameter at backupninja::rdiff

New $subfolder defaulting to rdiff-backup allow configuration
of which inner folder from $home rdiff backup should keep backups.

another important distinction in the README the newer nagios modules
which move us from using nagios2::service to nagios::service (which
support nagios3 and the puppet built-in native types) are not
currently supported

one more note about where things must be to make it work

updated README to include some pre-requisite information required for
nagios backupninja checks

fix the $real_nagios_server variable, this should be $nagios_server as $real_ is not defined anywhere

add a package definition for nsca underneath the $nagios_server block
this makes sure that the nsca package is installed, as it is a
dependency for the nagios checks. It does not configure any of the
nsca bits. In fact this should be done in a nagios::nsca class and
instead this module should require that

Merge remote branch 'anarcat/master'

fixed the README text now that the individual handler defines do the
software installation automatically

Merge branch 'master' of ssh://git.koumbit.net/puppet/modules/backupninja

very small typo

Merge remote branch 'anarcat/master'

update README to provide upgrade information, information about nagios
configuration and a more clear step-by-step process for using the module

fix permissions on  so backupninja doesn't complain loudly

fix rdiff_backup include typo

remove no longer referenced call to rdiff-installed class, and add a check for rsync package installation to avoid duplicate package definitions

move if block up further to the place where the checkbackups script is installed

replace koumbit-specific nagios server configuration with a $nagios_server variable that can be set. updated README to detail this information

Merge remote branch 'anarcat/master'

use the client classes for dependencies instead of directly defining packages

Added class backupninja::client::sys to install neccessary files for backupninja::sys on debian and ubuntu.

move the tag where it belongs (in remote)

fix typo that yields to a weird error

Could not create backup-nagios: Parameter type failed: interning empty string

fix directory

Merge commit 'micah/master'

Conflicts:

manifests/mysql.pp
manifests/rdiff.pp
manifests/server.pp

fix vserver lookup script

Adding missing keystore and backupkeystore parameters at rdiff and dup definitions

Adding keytype at backupninja::client::key

Trying a different way to test definitions at backupninja::client::key (3)

Trying a different way to test definitions at backupninja::client::key (2)

Trying a different way to test definitions at backupninja::client::key

New backupninja::client::defaults to avoid duplicates.

Adding part of client.pp code into new class
backupninja::client::defaults in an attempt to avoid duplicate
definitions inside backupninja::client::key.

Defaulting backupkeytype to 'rsa'

Adding additional clause to avoid blank real_keytype

Adding additional clause to avoid blank keytype

Making backupkeytype = false by default to avoided blank keytype

Removing wrong defined() call

Merge branch 'master' of git://labs.riseup.net/module_backupninja

Merge commit '6734c256525e1dd42ba3648db82083bcabca66ed'

Conflicts:
manifests/dup.pp

Merge branch 'master' of git://labs.riseup.net/module_backupninja

Conflicts:
manifests/dup.pp

add a note about the password variable needing to be enclosed in single quotes

the $password variable also must be set, so spit an error if it is not defined

set a requirement that the rdiff-backup directory variable must be set

fix the $desthost variable requirement, was incorrectly set to $host
add a $destdir variable requirement

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.

remove some stray comments

Fixing some definitions at backupninja::server::sandbox (2)

Fixing some definitions at backupninja::server::sandbox

Checking whether an exported resource is defined

Checking whether an exported resource is already defined
at backupninja::server::sandbox, otherwise puppet will try
to redefined them each time a backup resource type is invoked
with the same arguments (such as $directory), raising an error.

add in comment from 05efcc415ed19793e10d71e95a1d412e5d2095d1, didn't
merge this commit, because it also includes the package installation
bits that we are doing a different way

* add duplicity package management class to the client class
* update the README to be more clear about how this works, provide
more examples and included details about the addition of the
backupninja::client::duplicity class

Merge commit '73c59d37ff9f35abf4307fd243e4ed5ec9e78d97'

Trying to avoid duplicate definition of backupninja::client::key on backupninja::rdiff

Avoiding duplicate file definitions at backupninja::client::key

Setting the right default value for backupkeytype

Adding backupkeytype parameter at backupninja::rdiff

Fixing some definitions at backupninja::server::sandbox (2)

Fixing some definitions at backupninja::server::sandbox

Oops

Add rdiff::backup option to config dest user home folder

Merge branch 'master' of git://labs.riseup.net/module_backupninja

Checking whether an exported resource is defined

Checking whether an exported resource is already defined
at backupninja::server::sandbox, otherwise puppet will try
to redefined them each time a backup resource type is invoked
with the same arguments (such as $directory), raising an error.

add some details to the README about the package installation options

fix rdiff-backup class name, "You can use Aa-Zz, 0-9 and underscores in variables, resources and class names." which means no dashes in class names either

remove redundant case statements which tested to see if the package ensure version variables were set
changed the $rdiff-backup_ensure_version variable to $rdiff_backup_ensure_version as dashes are not allowed in variable names

create subclasses for maildir and rdiff-backup. If you are using these features of backupninja, you'll want to switch to doing "include backupninja::client::maildir" for the machines that need it and this will pull in backupninja::client, so you don't need to include that anymore if you were explicitly doing so for those machines (although it shouldn't hurt). Right now what using these new subclasses will get you is dependencies on needed packages, but in the future there might be more.

adding duplicity package as dependency for handler.

fixing file extension from ".duplicity" to ".dup".

adding key type for duplicity.

adding support for keytypes in backupninja server.

adding backup-key-type to duplicity handler.

eliminating some globbing because of server cache.