Merge branch 'bugfix/27-backup_notifications' into 'master'

Fix backup gem notifications failing

Closes #27

See merge request kosmos/chef!13

site-cookbooks/backup/recipes/default.rb

@@ -45,30 +45,40 @@ template "#{backup_dir}/config.rb" do
             mail_from: "backups@kosmos.org"
 end
 
-if node["backup"]["default_model"]
-  template "#{backup_dir}/models/default.rb" do
+template "#{backup_dir}/models/default.rb" do
   source    "backup.rb.erb"
   mode      0640
-  end
+end
 
-  cron "default backup model" do
+cron "default backup model" do
   hour node['backup']['cron']['hour']
   minute node['backup']['cron']['minute']
-    command "/usr/bin/env HOME=/home/user PATH=/usr/local/bin:/usr/local/ruby/bin:/usr/bin:/bin:$PATH /bin/sh -l -c 'backup perform -t default --root-path #{backup_dir} >> /var/log/backup.log 2>&1'"
-  end
+  command "/usr/bin/env HOME=/root /bin/sh -l -c '/usr/local/bin/backup perform -t default --root-path #{backup_dir} >> /var/log/backup.log 2>&1'"
+end
 
-  include_recipe 'logrotate'
-  unless node["backup"]["mysql"]["databases"].empty?
-    # Install MySQL client (includes mysqldump)
-    mysql_client 'default' do
+include_recipe 'logrotate'
+# Install MySQL client (includes mysqldump)
+mysql_client 'default' do
   action :create
-    end
-  end
+  not_if { node["backup"]["mysql"]["databases"].empty? }
+end
 
-  logrotate_app 'backup' do
+# Write the credentials file to allow dumps without password for the root
+# user (https://dev.mysql.com/doc/refman/5.7/en/option-files.html)
+file "/root/.my.cnf" do
+  mode "600"
+  content lazy { <<-EOF
+[client]
+user=#{node["backup"]["mysql"]["username"]}
+password=#{node["backup"]["mysql"]["password"]}
+  EOF
+  }
+  not_if { node["backup"]["mysql"]["password"].nil? }
+end
+
+logrotate_app 'backup' do
   path '/var/log/backup.log'
   frequency 'daily'
   rotate 10
   create '640 root root'
-  end
 end

site-cookbooks/backup/templates/default/config.rb.erb

@@ -40,7 +40,6 @@ end
 Database::MySQL.defaults do |db|
   db.host               = "<%= node["backup"]["mysql"]["host"] %>"
   db.username           = "<%= node["backup"]["mysql"]["username"] %>"
-  db.password           = "<%= node["backup"]["mysql"]["password"] %>"
   db.additional_options = ['--quick', '--single-transaction']
 end
 <%- end -%>