Merge pull request #1 from mgagliardo/repository_bug

repository and wait_for_completion variables not working

README.md

@@ -148,7 +148,7 @@ Default: false
 String.  Path to configuration file. You must ensure that the directory path exists.
 Default: '/root/.curator/curator.yml'
 
-#####`action_file`
+#####`actions_file`
 String.  Path to actions file. You must ensure that the directory path exists.
 Default: '/root/.curator/actions.yml'
 

manifests/action.pp

@@ -126,15 +126,15 @@
     fail('$key can be set only for action = allocation')
   }
 
-  if $repository and !member(['delete_snapshots', 'snapshot',], $repository) {
+  if $repository and !member(['delete_snapshots', 'snapshot',], $action) {
     fail('$repository can be set only for action = delete_snapshots or snapshot')
   }
 
   if ($retry_count or $retry_interval) and $action != 'delete_snapshots' {
     fail('$retry_count can be set only for action = delete_snapshots')
   }
 
-  if $wait_for_completion and !member(['allocation', 'replicas', 'restore', 'snapshot'], $wait_for_completion) {
+  if $wait_for_completion and !member(['allocation', 'replicas', 'restore', 'snapshot'], $action) {
     fail('$wait_for_completion can be set only for action = allocation or replicas or restore or snapshot')
   }
 

manifests/init.pp

@@ -27,7 +27,7 @@
 #   String.  Path to configuration file. You must ensure that the directory path exists.
 #   Default: '/root/.curator/curator.yml'
 #
-# [*action_file*]
+# [*actions_file*]
 #   String.  Path to actions file. You must ensure that the directory path exists.
 #   Default: '/root/.curator/actions.yml'
 #