Merge pull request #11792 from teddylear/feat-pause_after-powershell

feat: Add pause_after config to powershell provisioner like shell

Author: lbajolet-hashicorp

provisioner/powershell/provisioner.go

@@ -86,6 +86,9 @@ type Config struct {
 	//    ```
 	DebugMode int `mapstructure:"debug_mode"`
 
+	// A duration of how long to pause after the provisioner
+	PauseAfter time.Duration `mapstructure:"pause_after"`
+
 	ctx interpolate.Context
 }
 
@@ -349,6 +352,11 @@ func (p *Provisioner) Provision(ctx context.Context, ui packersdk.Ui, comm packe
 		log.Printf("remote cleanup script failed to upload; skipping the removal of temporary files: %s; ", strings.Join(uploadedScripts, ","))
 	}
 
+	if p.config.PauseAfter != 0 {
+		ui.Say(fmt.Sprintf("Pausing %s after this provisioner...", p.config.PauseAfter))
+		time.Sleep(p.config.PauseAfter)
+	}
+
 	return nil
 }
 

provisioner/powershell/provisioner.hcl2spec.go

@@ -381,6 +381,42 @@ func TestProvisionerProvision_ValidExitCodes(t *testing.T) {
 	}
 }
 
+func TestProvisionerProvision_PauseAfter(t *testing.T) {
+	config := testConfig()
+	delete(config, "inline")
+
+	// Defaults provided by Packer
+	config["remote_path"] = "c:/Windows/Temp/inlineScript.ps1"
+	config["inline"] = []string{"whoami"}
+	ui := testUi()
+	p := new(Provisioner)
+
+	// Defaults provided by Packer
+	p.config.PackerBuildName = "vmware"
+	p.config.PackerBuilderType = "iso"
+	p.config.ValidExitCodes = []int{0, 200}
+	pause_amount := time.Second
+	p.config.PauseAfter = pause_amount
+	comm := new(packersdk.MockCommunicator)
+	comm.StartExitStatus = 200
+	err := p.Prepare(config)
+	if err != nil {
+		t.Fatalf("Prepar failed: %s", err)
+	}
+
+	start_time := time.Now()
+	err = p.Provision(context.Background(), ui, comm, generatedData())
+	end_time := time.Now()
+
+	if err != nil {
+		t.Fatal("should not have error")
+	}
+
+	if end_time.Sub(start_time) < pause_amount {
+		t.Fatal("Didn't wait pause_amount")
+	}
+}
+
 func TestProvisionerProvision_InvalidExitCodes(t *testing.T) {
 	config := testConfig()
 	delete(config, "inline")

provisioner/powershell/provisioner_test.go

@@ -244,6 +244,9 @@ provisioner "powershell" {
   exists in order to deal with times when SSH may restart, such as a system
   reboot. Set this to a higher value if reboots take a longer amount of time.
 
+- `pause_after` (string) - Wait the amount of time after provisioning a powershell
+  script, this pause be taken if all previous steps were successful.
+
 @include 'provisioners/common-config.mdx'
 
 ## Default Environmental Variables