use only_if instead of if

rubygems · Aug 5, 2014 · fcd6c45 · fcd6c45
1 parent 9b4173f
commit fcd6c45
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 12 deletions.
diff --git a/Berksfile.lock b/Berksfile.lock
@@ -213,7 +213,7 @@ GRAPH
     user (>= 0.0.0)
   rubygems-apt (0.0.13)
     apt (>= 0.0.0)
-  rubygems-backups (0.0.16)
+  rubygems-backups (0.0.17)
     chef-vault (>= 0.0.0)
     rubygems-ruby (>= 0.0.0)
   rubygems-balancer (0.0.24)

diff --git a/cookbooks/rubygems-backups/metadata.rb b/cookbooks/rubygems-backups/metadata.rb
@@ -1,6 +1,6 @@
 name 'rubygems-backups'
 
-version '0.0.16'
+version '0.0.17'
 
 depends 'chef-vault'
 depends 'rubygems-ruby'

diff --git a/cookbooks/rubygems-backups/recipes/postgresql.rb b/cookbooks/rubygems-backups/recipes/postgresql.rb
@@ -52,14 +52,13 @@
   )
 end
 
-if node.chef_environment == 'production'
-  cron 'postgresql-public-dump' do
-    hour '21'
-    minute '21'
-    day '*'
-    month '*'
-    weekday '1'
-    command "backup perform --trigger public_postgresql --config-file #{File.join(node['rubygems']['backups']['config_dir'], 'public_postgresql.rb')}"
-    user 'root'
-  end
+cron 'postgresql-public-dump' do
+  hour '21'
+  minute '21'
+  day '*'
+  month '*'
+  weekday '1'
+  command "backup perform --trigger public_postgresql --config-file #{File.join(node['rubygems']['backups']['config_dir'], 'public_postgresql.rb')}"
+  user 'root'
+  only_if { node.chef_environment == 'production' }
 end