address code review feedback

lib/kitchen/command.rb

@@ -170,7 +170,6 @@ def run_action(action, instances, *args)
         concurrency.times do
           threads << Thread.new do
             while instance = queue.pop
-              puts "running #{instance.name}"
               run_action_in_thread(action, instance, *args)
             end
           end

lib/kitchen/provisioner/base.rb

@@ -33,8 +33,8 @@ class Base
       default_config :ftp_proxy, nil
 
       default_config :retry_on_exit_code, []
-      default_config :max_retries, nil
-      default_config :wait_for_retry, nil
+      default_config :max_retries, 1
+      default_config :wait_for_retry, 30
 
       default_config :root_path do |provisioner|
         provisioner.windows_os? ? "$env:TEMP\\kitchen" : "/tmp/kitchen"

lib/kitchen/transport/base.rb

@@ -117,16 +117,14 @@ def execute(command)
         # @param wait_time [Fixnum] number of seconds to wait before retrying command
         # @raise [TransportFailed] if the command does not exit successfully,
         #   which may vary by implementation
-        def execute_with_retry(command, retryable_exit_codes = [], max_retries, wait_time)
-          max_retries = 1 if max_retries.nil?
-          wait_time = 30 if wait_time.nil?
+        def execute_with_retry(command, retryable_exit_codes = [], max_retries = 1, wait_time = 30)
           tries = 0
           begin
             tries += 1
             debug("Attempting to execute command - try #{tries} of #{max_retries}.")
             execute(command)
           rescue Kitchen::Transport::TransportFailed => e
-            if retry? tries, max_retries, retryable_exit_codes, e.exit_code
+            if retry?(tries, max_retries, retryable_exit_codes, e.exit_code)
               close
               sleep wait_time
               retry