Merge pull request #16 from PWx21/master

add ability to save snapshots of db

manifests/server.pp

@@ -61,6 +61,8 @@
 #   Configure Redis slave replication timeout
 # [*repl_ping_slave_period*]
 #   Configure Redis replication ping slave period
+# [*save*]
+#   Configure Redis save snapshotting. Example: [[900, 1], [300, 10]]. Default: []
 #
 define redis::server (
   $redis_name              = $name,
@@ -92,6 +94,7 @@
   $slave_read_only         = true,
   $repl_timeout            = 60,
   $repl_ping_slave_period  = 10,
+  $save                    = [],
 ) {
 
   $redis_install_dir = $::redis::install::redis_install_dir
@@ -131,7 +134,7 @@
       require => Class['redis::install'],
     }
   }
-  
+
   file { "${redis_dir}/redis_${redis_name}":
     ensure  => directory,
     require => Class['redis::install'],

templates/etc/redis.conf.erb

@@ -84,9 +84,9 @@ databases <%= @redis_nr_dbs %>
 #
 #   Note: you can disable saving at all commenting all the "save" lines.
 
-#save 900 1
-#save 300 10
-#save 60 10000
+<% @save.each do |seconds, changes| %>
+  save <%= seconds %> <%= changes %>
+<% end %>
 
 # Compress string objects using LZF when dump .rdb databases?
 # For default that's set to 'yes' as it's almost always a win.