(maint) Remove unnecessary condition

Commit 2e24fb7 ensures we no longer
push nil on the context. As a result, we can remove the nil check inside
the block, and specify the `master_used` in the else clause.

Author: joshcooper

lib/puppet/configurer.rb

@@ -222,14 +222,12 @@ def run(options = {})
           found = find_functional_server()
           server = found[:server]
           if server.nil?
-            raise Puppet::Error, _("Could not select a functional puppet master from server_list: '%{server_list}'") % { server_list: Puppet.settings[:server_list] }
+            raise Puppet::Error, _("Could not select a functional puppet master from server_list: '%{server_list}'") % { server_list: Puppet[:server_list] }
+          else
+            Puppet.debug "Selected master: #{server[0]}:#{server[1]}"
+            report.master_used = "#{server[0]}:#{server[1]}"
           end
           Puppet.override(:server => server[0], :serverport => server[1]) do
-            if !server.first.nil?
-              Puppet.debug "Selected master: #{server[0]}:#{server[1]}"
-              report.master_used = "#{server[0]}:#{server[1]}"
-            end
-
             completed = run_internal(options.merge(:node => found[:node]))
           end
         else