Merge pull request #307 from phlipper/old_passwords

Define old_passwords only when there is a value defined. #304

README.md

@@ -732,6 +732,7 @@ Many thanks go to the following [contributors](https://github.com/phlipper/chef-
 * **[@bitpusher-real](https://github.com/bitpusher-real)**
     * add `binlog_ignore_db attribute` attribute
     * add version restrictions on three MySQL directives
+    * only set `old_passwords` only when a value defined
 
 
 ## License

templates/default/my.cnf.cluster.erb

@@ -91,10 +91,12 @@ innodb_locks_unsafe_for_binlog = <%= node["percona"]["cluster"]["innodb_locks_un
 innodb_autoinc_lock_mode       = <%= node["percona"]["cluster"]["innodb_autoinc_lock_mode"] %>
 
 
+<% if @old_passwords %>
 #
 # For compatibility to other Debian packages that still use
 # libmysqlclient10 and libmysqlclient12.
 old_passwords = <%= @old_passwords %>
+<% end %>
 
 #
 # Instead of skip-networking the default is now to listen only on

templates/default/my.cnf.main.erb

@@ -74,10 +74,12 @@ net_read_timeout = <%= node["percona"]["server"]["net_read_timeout"] %>
 connect_timeout  = <%= node["percona"]["server"]["connect_timeout"] %>
 wait_timeout     = <%= node["percona"]["server"]["wait_timeout"] %>
 
+<% if @old_passwords %>
 #
 # For compatibility to other Debian packages that still use
 # libmysqlclient10 and libmysqlclient12.
 old_passwords = <%= @old_passwords %>
+<% end %>
 
 #
 # Instead of skip-networking the default is now to listen only on