update logging

Author: mschuchard

provisioner/command.go

@@ -144,7 +144,7 @@ func (provisioner *Provisioner) determineExecCmd(ui packer.Ui) (*exec.Cmd, *pack
 	if !localExec {
 		communication, err := provisioner.determineCommunication(ui)
 		if err != nil {
-			ui.Say("could not accurately determine communication configuration")
+			ui.Error("could not accurately determine communication configuration")
 			return nil, nil, err
 		}
 
@@ -155,7 +155,7 @@ func (provisioner *Provisioner) determineExecCmd(ui packer.Ui) (*exec.Cmd, *pack
 	// pytest path
 	pytestPath, err := interpolate.Render(provisioner.config.PytestPath, &provisioner.config.ctx)
 	if err != nil {
-		ui.Sayf("error parsing config for PytestPath: %v", err.Error())
+		ui.Errorf("error parsing config for PytestPath: %v", err.Error())
 		return nil, nil, err
 	}
 
@@ -167,7 +167,7 @@ func (provisioner *Provisioner) determineExecCmd(ui packer.Ui) (*exec.Cmd, *pack
 	// keyword
 	keyword, err := interpolate.Render(provisioner.config.Keyword, &provisioner.config.ctx)
 	if err != nil {
-		ui.Sayf("error parsing config for Keyword: %v", err.Error())
+		ui.Errorf("error parsing config for Keyword: %v", err.Error())
 		return nil, nil, err
 	}
 	if len(keyword) > 0 {
@@ -176,7 +176,7 @@ func (provisioner *Provisioner) determineExecCmd(ui packer.Ui) (*exec.Cmd, *pack
 	// marker
 	marker, err := interpolate.Render(provisioner.config.Marker, &provisioner.config.ctx)
 	if err != nil {
-		ui.Sayf("error parsing config for Marker: %v", err.Error())
+		ui.Errorf("error parsing config for Marker: %v", err.Error())
 		return nil, nil, err
 	}
 	if len(marker) > 0 {

provisioner/communication.go

@@ -182,15 +182,15 @@ func (provisioner *Provisioner) determineUserAddr(connectionType string, ui pack
 		port, ok = provisioner.generatedData["Port"].(int)
 
 		if !ok || port == 0 {
-			ui.Error("host port could not be determined from available Packer data")
+			ui.Errorf("host port could not be determined from available Packer data: %d", port)
 			return "", "", errors.New("unknown host port")
 		}
 	}
 
 	// string format connection endpoint
 	httpAddr := fmt.Sprintf("%s:%d", ipaddress, port)
 
-	ui.Sayf("user determined to be %s and connection endpoint determined to be %s", user, httpAddr)
+	log.Printf("user determined to be %s and connection endpoint determined to be %s", user, httpAddr)
 
 	return user, httpAddr, nil
 }