Merge pull request #7917 from hashicorp/cleanup_unused_cancel_funcs

These cancel functions have been obsolete since March 2019

provisioner/ansible/provisioner.go

@@ -343,16 +343,6 @@ func (p *Provisioner) Provision(ctx context.Context, ui packer.Ui, comm packer.C
 	return nil
 }
 
-func (p *Provisioner) Cancel() {
-	if p.done != nil {
-		close(p.done)
-	}
-	if p.adapter != nil {
-		p.adapter.Shutdown()
-	}
-	os.Exit(0)
-}
-
 func (p *Provisioner) executeAnsible(ui packer.Ui, comm packer.Communicator, privKeyFile string) error {
 	playbook, _ := filepath.Abs(p.config.PlaybookFile)
 	inventory := p.config.InventoryFile

provisioner/breakpoint/provisioner.go

@@ -3,7 +3,6 @@ package breakpoint
 import (
 	"context"
 	"fmt"
-	"os"
 
 	"golang.org/x/sync/errgroup"
 
@@ -79,8 +78,3 @@ func (p *Provisioner) Provision(ctx context.Context, ui packer.Ui, comm packer.C
 	}
 	return nil
 }
-
-func (p *Provisioner) Cancel() {
-	// Just hard quit.
-	os.Exit(0)
-}

provisioner/converge/provisioner.go

@@ -234,9 +234,3 @@ func (p *Provisioner) applyModules(ui packer.Ui, comm packer.Communicator) error
 
 	return nil
 }
-
-// Cancel the provisioning process
-func (p *Provisioner) Cancel() {
-	// there's not an awful lot we can do to cancel Converge at the moment.
-	// The default semantics are fine.
-}

provisioner/inspec/provisioner.go

@@ -317,15 +317,6 @@ func (p *Provisioner) Provision(ctx context.Context, ui packer.Ui, comm packer.C
 
 	return nil
 }
-func (p *Provisioner) Cancel() {
-	if p.done != nil {
-		close(p.done)
-	}
-	if p.adapter != nil {
-		p.adapter.Shutdown()
-	}
-	os.Exit(0)
-}
 
 func (p *Provisioner) executeInspec(ui packer.Ui, comm packer.Communicator, privKeyFile string) error {
 	var envvars []string

provisioner/powershell/provisioner.go

@@ -305,12 +305,6 @@ func (p *Provisioner) Provision(ctx context.Context, ui packer.Ui, comm packer.C
 	return nil
 }
 
-func (p *Provisioner) Cancel() {
-	// Just hard quit. It isn't a big deal if what we're doing keeps running
-	// on the other side.
-	os.Exit(0)
-}
-
 // Environment variables required within the remote environment are uploaded
 // within a PS script and then enabled by 'dot sourcing' the script
 // immediately prior to execution of the main command

provisioner/shell-local/provisioner.go

@@ -30,7 +30,3 @@ func (p *Provisioner) Provision(ctx context.Context, ui packer.Ui, _ packer.Comm
 
 	return retErr
 }
-
-func (p *Provisioner) Cancel() {
-	// Just do nothing. When the process ends, so will our provisioner
-}

provisioner/sleep/provisioner.go

@@ -26,5 +26,3 @@ func (p *Provisioner) Provision(ctx context.Context, _ packer.Ui, _ packer.Commu
 		return nil
 	}
 }
-
-func (p *Provisioner) Cancel() {}