Merge pull request #73 from scassiba/fix_redhat_support

Fix directory permissions regression

recipes/_redhat_server_native.rb

@@ -22,16 +22,16 @@
 node.set['mariadb']['mysqld']['service_name'] = service_name\
   unless service_name.nil?
 
+package 'mariadb-server' do
+  action :install
+  notifies :enable, 'service[mysql]'
+end
+
 directory '/var/log/mysql' do
   action :create
   user 'mysql'
   group 'mysql'
   mode '0755'
-end
-
-package 'mariadb-server' do
-  action :install
-  notifies :enable, 'service[mysql]'
   notifies :start, 'service[mysql]', :immediately
   notifies :run, 'execute[change first install root password]', :immediately
 end

spec/centos_spec.rb

@@ -80,13 +80,16 @@
 
     it 'Install os shipped package' do
       expect(chef_run).to install_package('mariadb-server')
-      expect(os_package).to notify('service[mysql]').to(:start).immediately
     end
 
     it 'Create Log directory' do
       expect(chef_run).to create_directory('/var/log/mysql')
     end
 
+    it 'Does not restart mysql service' do
+      expect(chef_run).to_not restart_service('mysql')
+    end
+
     it 'Server service with the correct name' do
       expect(os_service.service_name).to eq 'mariadb'
     end