From 19566176409704337ba3da84de4528b165bb221a Mon Sep 17 00:00:00 2001
From: Ryan Johnson <ryan.johnson@broadcom.com>
Date: Thu, 16 May 2024 00:21:38 -0400
Subject: [PATCH] chore: test debt

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
---
 GNUmakefile                                   | 35 -----------
 builder/vsphere/common/testing/utility.go     | 29 ++++++---
 builder/vsphere/driver/driver_test.go         | 38 +++++++-----
 builder/vsphere/driver/host_acc_test.go       |  6 +-
 builder/vsphere/driver/vm_clone_acc_test.go   |  6 +-
 builder/vsphere/driver/vm_create_acc_test.go  |  6 +-
 builder/vsphere/iso/builder_acc_test.go       | 61 +++++++++++--------
 .../builder/vsphere-clone/alpine/alpine.json  |  4 +-
 .../vsphere-clone/alpine/alpine.pkr.hcl       |  4 +-
 .../vsphere-iso/alpine/alpine-3.8.json        |  8 +--
 .../vsphere-iso/alpine/alpine-3.8.pkr.hcl     |  8 +--
 .../builder/vsphere-iso/ubuntu/preseed.cfg    |  8 +--
 .../vsphere-iso/ubuntu/ubuntu-16.04.json      |  8 +--
 .../vsphere-iso/ubuntu/ubuntu-16.04.pkr.hcl   |  8 +--
 .../vsphere-iso/windows/windows-10.json       |  8 +--
 .../vsphere-iso/windows/windows-10.pkr.hcl    |  8 +--
 examples/driver/main.go                       | 28 +++++++--
 17 files changed, 142 insertions(+), 131 deletions(-)
 delete mode 100644 GNUmakefile

diff --git a/GNUmakefile b/GNUmakefile
deleted file mode 100644
index 52d70ce05..000000000
--- a/GNUmakefile
+++ /dev/null
@@ -1,35 +0,0 @@
-NAME=vsphere
-BINARY=packer-plugin-${NAME}
-PLUGIN_FQN="$(shell grep -E '^module' <go.mod | sed -E 's/module *//')"
-
-COUNT?=1
-TEST?=$(shell go list ./...)
-HASHICORP_PACKER_PLUGIN_SDK_VERSION?=$(shell go list -m github.com/hashicorp/packer-plugin-sdk | cut -d " " -f2)
-
-.PHONY: dev
-
-build:
-	@go build -o ${BINARY}
-
-dev:
-	@go build -ldflags="-X '${PLUGIN_FQN}/version.VersionPrerelease=dev'" -o '${BINARY}'
-	packer plugins install --path ${BINARY} "$(shell echo "${PLUGIN_FQN}" | sed 's/packer-plugin-//')"
-
-test:
-	@go test -race -count $(COUNT) $(TEST) -timeout=3m
-
-install-packer-sdc: ## Install packer sofware development command
-	@go install github.com/hashicorp/packer-plugin-sdk/cmd/packer-sdc@${HASHICORP_PACKER_PLUGIN_SDK_VERSION}
-
-plugin-check: install-packer-sdc build
-	@packer-sdc plugin-check ${BINARY}
-
-testacc: dev
-	@PACKER_ACC=1 go test -count $(COUNT) -v $(TEST) -timeout=120m
-
-generate: install-packer-sdc
-	@go generate ./...
-	@rm -rf .docs
-	@packer-sdc renderdocs -src "docs" -partials docs-partials/ -dst ".docs/"
-	@./.web-docs/scripts/compile-to-webdocs.sh "." ".docs" ".web-docs" "hashicorp"
-	@rm -r ".docs"
diff --git a/builder/vsphere/common/testing/utility.go b/builder/vsphere/common/testing/utility.go
index c403e3884..b1849c49d 100644
--- a/builder/vsphere/common/testing/utility.go
+++ b/builder/vsphere/common/testing/utility.go
@@ -13,17 +13,21 @@ import (
 	"github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/driver"
 )
 
+const (
+	testDefaultUsername    = "administrator@vsphere.local"
+	testDefaultPassword    = "VMw@re1!"
+	testDefaultApiEndpoint = "vcenter.example.com"
+)
+
 func NewVMName() string {
-	rand.Seed(time.Now().UnixNano())
-	return fmt.Sprintf("test-%v", rand.Intn(1000))
+	r := rand.New(rand.NewSource(time.Now().UnixNano()))
+	return fmt.Sprintf("test-%v", r.Intn(1000))
 }
 
 func RenderConfig(builderType string, config map[string]interface{}) string {
 	t := map[string][]map[string]interface{}{
 		"builders": {
-			map[string]interface{}{
-				"type": builderType,
-			},
+			{"type": builderType},
 		},
 	}
 	for k, v := range config {
@@ -37,21 +41,27 @@ func RenderConfig(builderType string, config map[string]interface{}) string {
 func TestConn() (driver.Driver, error) {
 	username := os.Getenv("VSPHERE_USERNAME")
 	if username == "" {
-		username = "root"
+		username = testDefaultUsername
 	}
+
 	password := os.Getenv("VSPHERE_PASSWORD")
 	if password == "" {
-		password = "jetbrains"
+		password = testDefaultPassword
+	}
+
+	vcenter := os.Getenv("VSPHERE_VCENTER_SERVER")
+	if vcenter == "" {
+		vcenter = testDefaultApiEndpoint
 	}
 
 	d, err := driver.NewDriver(&driver.ConnectConfig{
-		VCenterServer:      "vcenter.example.com",
+		VCenterServer:      vcenter,
 		Username:           username,
 		Password:           password,
 		InsecureConnection: true,
 	})
 	if err != nil {
-		return nil, fmt.Errorf("error connecting to vCenter Server instance: %v", err)
+		return nil, fmt.Errorf("error connecting to endpoint: %v", err)
 	}
 	return d, nil
 }
@@ -61,7 +71,6 @@ func GetVM(d driver.Driver, name string) (driver.VirtualMachine, error) {
 	if err != nil {
 		return nil, fmt.Errorf("error finding virtual machine: %v", err)
 	}
-
 	return vm, nil
 }
 
diff --git a/builder/vsphere/driver/driver_test.go b/builder/vsphere/driver/driver_test.go
index 3004557bf..a8cf5dfb2 100644
--- a/builder/vsphere/driver/driver_test.go
+++ b/builder/vsphere/driver/driver_test.go
@@ -23,21 +23,27 @@ import (
 	"github.com/vmware/govmomi/vim25/soap"
 )
 
-// Defines whether acceptance tests should be run
-const TestHostName = "esxi-01.example.com"
+const (
+	testDefaultUsername    = "administrator@vsphere.local"
+	testDefaultPassword    = "VMw@re1!"
+	testDefaultApiEndpoint = "vcenter.example.com"
+	testDefaultHost        = "esxi-01.example.com"
+)
 
-func newTestDriver(t *testing.T) Driver {
-	username := os.Getenv("VSPHERE_USERNAME")
-	if username == "" {
-		username = "root"
-	}
-	password := os.Getenv("VSPHERE_PASSWORD")
-	if password == "" {
-		password = "jetbrains"
+func getEnv(key, fallback string) string {
+	if value := os.Getenv(key); value != "" {
+		return value
 	}
+	return fallback
+}
+
+func newTestDriver(t *testing.T) Driver {
+	username := getEnv("VSPHERE_USERNAME", testDefaultUsername)
+	password := getEnv("VSPHERE_PASSWORD", testDefaultPassword)
+	vcenter := getEnv("VSPHERE_VCENTER_SERVER", testDefaultApiEndpoint)
 
 	d, err := NewDriver(&ConnectConfig{
-		VCenterServer:      "vcenter.example.com",
+		VCenterServer:      vcenter,
 		Username:           username,
 		Password:           password,
 		InsecureConnection: true,
@@ -49,8 +55,8 @@ func newTestDriver(t *testing.T) Driver {
 }
 
 func newVMName() string {
-	rand.Seed(time.Now().UTC().UnixNano())
-	return fmt.Sprintf("test-%v", rand.Intn(1000))
+	r := rand.New(rand.NewSource(time.Now().UTC().UnixNano()))
+	return fmt.Sprintf("test-%v", r.Intn(1000))
 }
 
 type VCenterSimulator struct {
@@ -94,7 +100,7 @@ func (s *VCenterSimulator) Close() {
 	}
 }
 
-// Simulator shortcut to choose any pre created VM.
+// Simulator shortcut to choose any pre-created virtual machine.
 func (s *VCenterSimulator) ChooseSimulatorPreCreatedVM() (VirtualMachine, *simulator.VirtualMachine) {
 	machine := simulator.Map.Any("VirtualMachine").(*simulator.VirtualMachine)
 	ref := machine.Reference()
@@ -102,7 +108,7 @@ func (s *VCenterSimulator) ChooseSimulatorPreCreatedVM() (VirtualMachine, *simul
 	return vm, machine
 }
 
-// Simulator shortcut to choose any pre created Datastore.
+// Simulator shortcut to choose any pre-created datastore.
 func (s *VCenterSimulator) ChooseSimulatorPreCreatedDatastore() (Datastore, *simulator.Datastore) {
 	ds := simulator.Map.Any("Datastore").(*simulator.Datastore)
 	ref := ds.Reference()
@@ -110,7 +116,7 @@ func (s *VCenterSimulator) ChooseSimulatorPreCreatedDatastore() (Datastore, *sim
 	return datastore, ds
 }
 
-// Simulator shortcut to choose any pre created Host.
+// Simulator shortcut to choose any pre-created EXSi host.
 func (s *VCenterSimulator) ChooseSimulatorPreCreatedHost() (*Host, *simulator.HostSystem) {
 	h := simulator.Map.Any("HostSystem").(*simulator.HostSystem)
 	ref := h.Reference()
diff --git a/builder/vsphere/driver/host_acc_test.go b/builder/vsphere/driver/host_acc_test.go
index ee2040f11..c55d4cbae 100644
--- a/builder/vsphere/driver/host_acc_test.go
+++ b/builder/vsphere/driver/host_acc_test.go
@@ -10,7 +10,7 @@ import (
 func TestHostAcc(t *testing.T) {
 	t.Skip("Acceptance tests not configured yet.")
 	d := newTestDriver(t)
-	host, err := d.FindHost(TestHostName)
+	host, err := d.FindHost(testDefaultHost)
 	if err != nil {
 		t.Fatalf("unexpected error: '%s'", err)
 	}
@@ -19,7 +19,7 @@ func TestHostAcc(t *testing.T) {
 	if err != nil {
 		t.Fatalf("unexpected error: '%s'", err)
 	}
-	if info.Name != TestHostName {
-		t.Errorf("unexpected result: expected '%s', but returned '%s'", TestHostName, info.Name)
+	if info.Name != testDefaultHost {
+		t.Errorf("unexpected result: expected '%s', but returned '%s'", testDefaultHost, info.Name)
 	}
 }
diff --git a/builder/vsphere/driver/vm_clone_acc_test.go b/builder/vsphere/driver/vm_clone_acc_test.go
index 5aa8923b8..65f80154a 100644
--- a/builder/vsphere/driver/vm_clone_acc_test.go
+++ b/builder/vsphere/driver/vm_clone_acc_test.go
@@ -32,7 +32,7 @@ func TestVMAcc_clone(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			tc.config.Host = TestHostName
+			tc.config.Host = testDefaultHost
 			tc.config.Name = newVMName()
 
 			templateName := "alpine"
@@ -88,8 +88,8 @@ func cloneDefaultCheck(t *testing.T, vm VirtualMachine, config *CloneConfig) {
 	if err != nil {
 		t.Fatal("Cannot read host properties: ", err)
 	}
-	if hostInfo.Name != TestHostName {
-		t.Errorf("Invalid host name: expected '%v', got '%v'", TestHostName, hostInfo.Name)
+	if hostInfo.Name != testDefaultHost {
+		t.Errorf("Invalid host name: expected '%v', got '%v'", testDefaultHost, hostInfo.Name)
 	}
 
 	p := d.NewResourcePool(vmInfo.ResourcePool)
diff --git a/builder/vsphere/driver/vm_create_acc_test.go b/builder/vsphere/driver/vm_create_acc_test.go
index e0c581911..76c1ca749 100644
--- a/builder/vsphere/driver/vm_create_acc_test.go
+++ b/builder/vsphere/driver/vm_create_acc_test.go
@@ -20,7 +20,7 @@ func TestVMAcc_create(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			tc.config.Host = TestHostName
+			tc.config.Host = testDefaultHost
 			tc.config.Name = newVMName()
 
 			d := newTestDriver(t)
@@ -71,8 +71,8 @@ func createDefaultCheck(t *testing.T, vm VirtualMachine, config *CreateConfig) {
 	if err != nil {
 		t.Fatal("Cannot read host properties: ", err)
 	}
-	if hostInfo.Name != TestHostName {
-		t.Errorf("Invalid host name: expected '%v', got '%v'", TestHostName, hostInfo.Name)
+	if hostInfo.Name != testDefaultHost {
+		t.Errorf("Invalid host name: expected '%v', got '%v'", testDefaultHost, hostInfo.Name)
 	}
 
 	p := d.NewResourcePool(vmInfo.ResourcePool)
diff --git a/builder/vsphere/iso/builder_acc_test.go b/builder/vsphere/iso/builder_acc_test.go
index 8f80d662f..56f55c037 100644
--- a/builder/vsphere/iso/builder_acc_test.go
+++ b/builder/vsphere/iso/builder_acc_test.go
@@ -14,6 +14,13 @@ import (
 	"github.com/vmware/govmomi/vim25/types"
 )
 
+const (
+	defaultUsername    = "administrator@vsphere.local"
+	defaultPassword    = "VMw@re1!"
+	defaultApiEndpoint = "vcenter.example.com"
+	defaultHost        = "esxi-01.example.com"
+)
+
 func TestAccISOBuilderAcc_default(t *testing.T) {
 	config := defaultConfig()
 	testCase := &acctest.PluginTestCase{
@@ -38,25 +45,18 @@ func TestAccISOBuilderAcc_default(t *testing.T) {
 	acctest.TestPlugin(t, testCase)
 }
 
-func defaultConfig() map[string]interface{} {
-	username := os.Getenv("VSPHERE_USERNAME")
-	if username == "" {
-		username = "root"
-	}
-	password := os.Getenv("VSPHERE_PASSWORD")
-	if password == "" {
-		password = "jetbrains"
-	}
-
-	vcenter := os.Getenv("VSPHERE_VCENTER_SERVER")
-	if vcenter == "" {
-		vcenter = "vcenter.example.com"
+func getEnv(key, fallback string) string {
+	if value := os.Getenv(key); value != "" {
+		return value
 	}
+	return fallback
+}
 
-	host := os.Getenv("VSPHERE_HOST")
-	if host == "" {
-		host = "esxi-01.example.com"
-	}
+func defaultConfig() map[string]interface{} {
+	username := getEnv("VSPHERE_USERNAME", defaultUsername)
+	password := getEnv("VSPHERE_PASSWORD", defaultPassword)
+	vcenter := getEnv("VSPHERE_VCENTER_SERVER", defaultApiEndpoint)
+	host := getEnv("VSPHERE_HOST", defaultHost)
 
 	config := map[string]interface{}{
 		"vcenter_server":      vcenter,
@@ -65,8 +65,8 @@ func defaultConfig() map[string]interface{} {
 		"host":                host,
 		"insecure_connection": true,
 
-		"ssh_username": "root",
-		"ssh_password": "jetbrains",
+		"ssh_username": "packer",
+		"ssh_password": "VMw@re1!",
 
 		"vm_name": commonT.NewVMName(),
 		"storage": map[string]interface{}{
@@ -546,21 +546,32 @@ func TestAccISOBuilderAcc_full(t *testing.T) {
 func fullConfig() map[string]interface{} {
 	username := os.Getenv("VSPHERE_USERNAME")
 	if username == "" {
-		username = "root"
+		username = "administrator@vsphere.local"
 	}
+
 	password := os.Getenv("VSPHERE_PASSWORD")
 	if password == "" {
-		password = "jetbrains"
+		password = "VMw@re1!"
+	}
+
+	vcenter := os.Getenv("VSPHERE_VCENTER_SERVER")
+	if vcenter == "" {
+		vcenter = "vcenter.example.com"
+	}
+
+	host := os.Getenv("VSPHERE_HOST")
+	if host == "" {
+		host = "esxi-01.example.com"
 	}
 
 	config := map[string]interface{}{
-		"vcenter_server":      "vcenter.example.com",
+		"vcenter_server":      vcenter,
 		"username":            username,
 		"password":            password,
+		"host":                host,
 		"insecure_connection": true,
 
 		"vm_name": commonT.NewVMName(),
-		"host":    "esxi-01.example.com",
 
 		"RAM": 512,
 		"disk_controller_type": []string{
@@ -602,8 +613,8 @@ func fullConfig() map[string]interface{} {
 			"/media/floppy/SETUP.SH<enter>",
 		},
 
-		"ssh_username": "root",
-		"ssh_password": "jetbrains",
+		"ssh_username": "packer",
+		"ssh_password": "VMw@re1!",
 	}
 
 	return config
diff --git a/examples/builder/vsphere-clone/alpine/alpine.json b/examples/builder/vsphere-clone/alpine/alpine.json
index 885183db8..847315a61 100644
--- a/examples/builder/vsphere-clone/alpine/alpine.json
+++ b/examples/builder/vsphere-clone/alpine/alpine.json
@@ -4,8 +4,8 @@
       "type": "vsphere-clone",
 
       "vcenter_server": "vcenter.example.com",
-      "username":            "root",
-      "password":            "jetbrains",
+      "username": "administrator@vsphere.local",
+      "password": "VMw@re1!",
       "insecure_connection": "true",
 
       "template": "alpine",
diff --git a/examples/builder/vsphere-clone/alpine/alpine.pkr.hcl b/examples/builder/vsphere-clone/alpine/alpine.pkr.hcl
index 4df5415d6..809ba859a 100644
--- a/examples/builder/vsphere-clone/alpine/alpine.pkr.hcl
+++ b/examples/builder/vsphere-clone/alpine/alpine.pkr.hcl
@@ -12,9 +12,9 @@ source "vsphere-clone" "example_clone" {
   communicator        = "none"
   host                = "esxi-01.example.com"
   insecure_connection = "true"
-  password            = "jetbrains"
+  password            = "VMw@re1!"
   template            = "alpine"
-  username            = "root"
+  username            = "administrator@vsphere.local"
   vcenter_server      = "vcenter.example.com"
   vm_name             = "alpine-clone-${local.timestamp}"
 }
diff --git a/examples/builder/vsphere-iso/alpine/alpine-3.8.json b/examples/builder/vsphere-iso/alpine/alpine-3.8.json
index 88f566e4b..8c15628bf 100644
--- a/examples/builder/vsphere-iso/alpine/alpine-3.8.json
+++ b/examples/builder/vsphere-iso/alpine/alpine-3.8.json
@@ -39,16 +39,16 @@
           "network_card": "vmxnet3"
         }
       ],
-      "password": "jetbrains",
-      "ssh_password": "jetbrains",
-      "ssh_username": "root",
+      "password": "VMw@re1!",
+      "ssh_password": "VMw@re1!",
+      "ssh_username": "packer",
       "storage": [
         {
           "disk_size": 1024,
           "disk_thin_provisioned": true
         }
       ],
-      "username": "root",
+      "username": "administrator@vsphere.local",
       "vcenter_server": "vcenter.example.com",
       "vm_name": "alpine-{{timestamp}}"
     }
diff --git a/examples/builder/vsphere-iso/alpine/alpine-3.8.pkr.hcl b/examples/builder/vsphere-iso/alpine/alpine-3.8.pkr.hcl
index c9d8e61c1..f55a0833f 100644
--- a/examples/builder/vsphere-iso/alpine/alpine-3.8.pkr.hcl
+++ b/examples/builder/vsphere-iso/alpine/alpine-3.8.pkr.hcl
@@ -23,14 +23,14 @@ source "vsphere-iso" "autogenerated_1" {
   network_adapters {
     network_card = "vmxnet3"
   }
-  password     = "jetbrains"
-  ssh_password = "jetbrains"
-  ssh_username = "root"
+  password     = "VMw@re1!"
+  ssh_password = "VMw@re1!"
+  ssh_username = "packer"
   storage {
     disk_size             = 1024
     disk_thin_provisioned = true
   }
-  username       = "root"
+  username       = "administrator@vsphere.local"
   vcenter_server = "vcenter.example.com"
   vm_name        = "alpine-${local.timestamp}"
 }
diff --git a/examples/builder/vsphere-iso/ubuntu/preseed.cfg b/examples/builder/vsphere-iso/ubuntu/preseed.cfg
index ec963b6b2..68deeb303 100644
--- a/examples/builder/vsphere-iso/ubuntu/preseed.cfg
+++ b/examples/builder/vsphere-iso/ubuntu/preseed.cfg
@@ -1,7 +1,7 @@
-d-i passwd/user-fullname string jetbrains
-d-i passwd/username string jetbrains
-d-i passwd/user-password password jetbrains
-d-i passwd/user-password-again password jetbrains
+d-i passwd/user-fullname string packer
+d-i passwd/username string packer
+d-i passwd/user-password password VMw@re1!
+d-i passwd/user-password-again password VMw@re1!
 d-i user-setup/allow-password-weak boolean true
 
 d-i partman-auto/disk string /dev/sda
diff --git a/examples/builder/vsphere-iso/ubuntu/ubuntu-16.04.json b/examples/builder/vsphere-iso/ubuntu/ubuntu-16.04.json
index 5abd2fab4..a9020d1ed 100644
--- a/examples/builder/vsphere-iso/ubuntu/ubuntu-16.04.json
+++ b/examples/builder/vsphere-iso/ubuntu/ubuntu-16.04.json
@@ -38,16 +38,16 @@
           "network_card": "vmxnet3"
         }
       ],
-      "password": "jetbrains",
-      "ssh_password": "jetbrains",
-      "ssh_username": "jetbrains",
+      "password": "VMw@re1!",
+      "ssh_password": "VMw@re1!",
+      "ssh_username": "packer",
       "storage": [
         {
           "disk_size": 32768,
           "disk_thin_provisioned": true
         }
       ],
-      "username": "root",
+      "username": "administrator@vsphere.local",
       "vcenter_server": "vcenter.example.com",
       "vm_name": "example-ubuntu"
     }
diff --git a/examples/builder/vsphere-iso/ubuntu/ubuntu-16.04.pkr.hcl b/examples/builder/vsphere-iso/ubuntu/ubuntu-16.04.pkr.hcl
index e39dc3397..a0f8eeca3 100644
--- a/examples/builder/vsphere-iso/ubuntu/ubuntu-16.04.pkr.hcl
+++ b/examples/builder/vsphere-iso/ubuntu/ubuntu-16.04.pkr.hcl
@@ -31,14 +31,14 @@ source "vsphere-iso" "example" {
   network_adapters {
     network_card = "vmxnet3"
   }
-  password     = "jetbrains"
-  ssh_password = "jetbrains"
-  ssh_username = "jetbrains"
+  password     = "VMw@re1!"
+  ssh_password = "VMw@re1!"
+  ssh_username = "packer"
   storage {
     disk_size             = 32768
     disk_thin_provisioned = true
   }
-  username       = "root"
+  username       = "administrator@vsphere.local"
   vcenter_server = "vcenter.example.com"
   vm_name        = "example-ubuntu"
 }
diff --git a/examples/builder/vsphere-iso/windows/windows-10.json b/examples/builder/vsphere-iso/windows/windows-10.json
index 482382196..22568282c 100644
--- a/examples/builder/vsphere-iso/windows/windows-10.json
+++ b/examples/builder/vsphere-iso/windows/windows-10.json
@@ -23,18 +23,18 @@
           "network_card": "vmxnet3"
         }
       ],
-      "password": "jetbrains",
+      "password": "VMw@re1!",
       "storage": [
         {
           "disk_size": 32768,
           "disk_thin_provisioned": true
         }
       ],
-      "username": "root",
+      "username": "administrator@vsphere.local",
       "vcenter_server": "vcenter.example.com",
       "vm_name": "example-windows",
-      "winrm_password": "jetbrains",
-      "winrm_username": "jetbrains"
+      "winrm_password": "packer",
+      "winrm_username": "VMw@re1!"
     }
   ],
   "provisioners": [
diff --git a/examples/builder/vsphere-iso/windows/windows-10.pkr.hcl b/examples/builder/vsphere-iso/windows/windows-10.pkr.hcl
index c56f427ea..3bf876810 100644
--- a/examples/builder/vsphere-iso/windows/windows-10.pkr.hcl
+++ b/examples/builder/vsphere-iso/windows/windows-10.pkr.hcl
@@ -20,16 +20,16 @@ source "vsphere-iso" "example_windows" {
   network_adapters {
     network_card = "vmxnet3"
   }
-  password = "jetbrains"
+  password = "VMw@re1!"
   storage {
     disk_size             = 32768
     disk_thin_provisioned = true
   }
-  username       = "root"
+  username       = "administrator@vsphere.local"
   vcenter_server = "vcenter.example.com"
   vm_name        = "example-windows"
-  winrm_password = "jetbrains"
-  winrm_username = "jetbrains"
+  winrm_password = "VMw@re1!"
+  winrm_username = "packer"
 }
 
 # a build block invokes sources and runs provisioning steps on them. The
diff --git a/examples/driver/main.go b/examples/driver/main.go
index 46d65d384..f7cc22dd1 100644
--- a/examples/driver/main.go
+++ b/examples/driver/main.go
@@ -5,22 +5,42 @@ package main
 
 import (
 	"fmt"
+	"os"
 
 	"github.com/hashicorp/packer-plugin-vsphere/builder/vsphere/driver"
 )
 
+const (
+	defaultUsername    = "administrator@vsphere.local"
+	defaultPassword    = "VMw@re1!"
+	defaultApiEndpoint = "vcenter.example.com"
+	defaultHost        = "esxi-01.example.com"
+)
+
+func getEnv(key, fallback string) string {
+	if value := os.Getenv(key); value != "" {
+		return value
+	}
+	return fallback
+}
+
 func main() {
+	username := getEnv("VSPHERE_USERNAME", defaultUsername)
+	password := getEnv("VSPHERE_PASSWORD", defaultPassword)
+	vcenter := getEnv("VSPHERE_VCENTER_SERVER", defaultApiEndpoint)
+	host := getEnv("VSPHERE_HOST", defaultHost)
+
 	d, err := driver.NewDriver(&driver.ConnectConfig{
-		VCenterServer:      "vcenter.example.com",
-		Username:           "root",
-		Password:           "jetbrains",
+		VCenterServer:      vcenter,
+		Username:           username,
+		Password:           password,
 		InsecureConnection: true,
 	})
 	if err != nil {
 		panic(err)
 	}
 
-	ds, err := d.FindDatastore("", "esxi-01.example.com")
+	ds, err := d.FindDatastore("", host)
 	if err != nil {
 		panic(err)
 	}