Merge pull request #5283 from hashicorp/do_4039

set default for ExpectDisconnect to false

provisioner/shell/provisioner.go

@@ -105,7 +105,7 @@ func (p *Provisioner) Prepare(raws ...interface{}) error {
 	}
 
 	if p.config.ExpectDisconnect == nil {
-		t := true
+		t := false
 		p.config.ExpectDisconnect = &t
 	}
 

provisioner/shell/provisioner_test.go

@@ -32,8 +32,8 @@ func TestProvisionerPrepare_Defaults(t *testing.T) {
 		t.Fatalf("err: %s", err)
 	}
 
-	if *p.config.ExpectDisconnect != true {
-		t.Errorf("expected ExpectDisconnect to be true")
+	if *p.config.ExpectDisconnect != false {
+		t.Errorf("expected ExpectDisconnect to default to false")
 	}
 
 	if p.config.RemotePath == "" {