diff --git a/builder/amazon/common/step_encrypted_ami.go b/builder/amazon/common/step_encrypted_ami.go index 40e985322..6a4c2967e 100644 --- a/builder/amazon/common/step_encrypted_ami.go +++ b/builder/amazon/common/step_encrypted_ami.go @@ -25,7 +25,7 @@ func (s *StepCreateEncryptedAMICopy) Run(state multistep.StateBag) multistep.Ste // Encrypt boot not set, so skip step if !s.EncryptBootVolume { if kmsKeyId != "" { - log.Printf(fmt.Sprintf("Ignoring KMS Key ID: %s, encrypted=false", kmsKeyId)) + log.Printf("Ignoring KMS Key ID: %s, encrypted=false", kmsKeyId) } return multistep.ActionContinue } diff --git a/builder/qemu/step_configure_vnc.go b/builder/qemu/step_configure_vnc.go index d7bace29e..bda624f89 100644 --- a/builder/qemu/step_configure_vnc.go +++ b/builder/qemu/step_configure_vnc.go @@ -2,11 +2,12 @@ package qemu import ( "fmt" - "github.com/mitchellh/multistep" - "github.com/mitchellh/packer/packer" "log" "math/rand" "net" + + "github.com/mitchellh/multistep" + "github.com/mitchellh/packer/packer" ) // This step configures the VM to enable the VNC server. @@ -28,7 +29,7 @@ func (stepConfigureVNC) Run(state multistep.StateBag) multistep.StepAction { // best. msg := fmt.Sprintf("Looking for available port between %d and %d on %s", config.VNCPortMin, config.VNCPortMax, config.VNCBindAddress) ui.Say(msg) - log.Printf(msg) + log.Print(msg) var vncPort uint portRange := int(config.VNCPortMax - config.VNCPortMin) for { diff --git a/provisioner/powershell/provisioner.go b/provisioner/powershell/provisioner.go index 4d5ffadf7..8448cf819 100644 --- a/provisioner/powershell/provisioner.go +++ b/provisioner/powershell/provisioner.go @@ -317,7 +317,7 @@ func (p *Provisioner) retryable(f func() error) error { // Create an error and log it err = fmt.Errorf("Retryable error: %s", err) - log.Printf(err.Error()) + log.Print(err.Error()) // Check if we timed out, otherwise we retry. It is safe to // retry since the only error case above is if the command diff --git a/provisioner/shell/provisioner.go b/provisioner/shell/provisioner.go index f8a646d66..30468aee1 100644 --- a/provisioner/shell/provisioner.go +++ b/provisioner/shell/provisioner.go @@ -348,7 +348,7 @@ func (p *Provisioner) retryable(f func() error) error { // Create an error and log it err = fmt.Errorf("Retryable error: %s", err) - log.Printf(err.Error()) + log.Print(err.Error()) // Check if we timed out, otherwise we retry. It is safe to // retry since the only error case above is if the command diff --git a/provisioner/windows-restart/provisioner.go b/provisioner/windows-restart/provisioner.go index 139acc19a..ae58dd9d1 100644 --- a/provisioner/windows-restart/provisioner.go +++ b/provisioner/windows-restart/provisioner.go @@ -217,7 +217,7 @@ func (p *Provisioner) retryable(f func() error) error { // Create an error and log it err = fmt.Errorf("Retryable error: %s", err) - log.Printf(err.Error()) + log.Print(err.Error()) // Check if we timed out, otherwise we retry. It is safe to // retry since the only error case above is if the command diff --git a/provisioner/windows-shell/provisioner.go b/provisioner/windows-shell/provisioner.go index e770d867a..b0d50998f 100644 --- a/provisioner/windows-shell/provisioner.go +++ b/provisioner/windows-shell/provisioner.go @@ -270,7 +270,7 @@ func (p *Provisioner) retryable(f func() error) error { // Create an error and log it err = fmt.Errorf("Retryable error: %s", err) - log.Printf(err.Error()) + log.Print(err.Error()) // Check if we timed out, otherwise we retry. It is safe to // retry since the only error case above is if the command