fake images: windows hyperv

this pr is a follow on to #27493.  it adds support for hyperv "fake"
images and suggests a benefit in terms of test speed.  for hyperv, we
create a generic 4MB vhdx and stick it into the temp dir.  this saves us
from any image copy or compression.

i also followed up on a few comments Paul made about using windows|unix
instead of each platform.

Signed-off-by: Brent Baude <[email protected]>

Author: baude

pkg/machine/e2e/config_darwin_test.go

@@ -1,14 +1,3 @@
 package e2e_test
 
-import "os"
-
 const podmanBinary = "../../../bin/darwin/podman"
-
-var (
-	fakeImagePath string = os.DevNull
-)
-
-func (i *initMachine) withFakeImage(_ *machineTestBuilder) *initMachine {
-	i.image = fakeImagePath
-	return i
-}

pkg/machine/e2e/config_freebsd_test.go

@@ -1,14 +1,3 @@
 package e2e_test
 
-import "os"
-
 const podmanBinary = "../../../bin/podman-remote"
-
-var (
-	fakeImagePath string = os.DevNull
-)
-
-func (i *initMachine) withFakeImage(_ *machineTestBuilder) *initMachine {
-	i.image = fakeImagePath
-	return i
-}

pkg/machine/e2e/config_linux_test.go

@@ -1,14 +1,3 @@
 package e2e_test
 
-import "os"
-
 const podmanBinary = "../../../bin/podman-remote"
-
-var (
-	fakeImagePath string = os.DevNull
-)
-
-func (i *initMachine) withFakeImage(_ *machineTestBuilder) *initMachine {
-	i.image = fakeImagePath
-	return i
-}

pkg/machine/e2e/config_unix_test.go

@@ -3,10 +3,25 @@
 package e2e_test
 
 import (
+	"os"
 	"os/exec"
 )
 
+var (
+	fakeImagePath string = os.DevNull
+)
+
 func pgrep(_ string) (string, error) {
 	out, err := exec.Command("pgrep", "gvproxy").Output()
 	return string(out), err
 }
+
+func initPlatform() {}
+
+// withFakeImage should be used in tests where the machine is
+// initialized (or not) but never started.  It is intended
+// to speed up CI by not processing our large machine files.
+func (i *initMachine) withFakeImage(_ *machineTestBuilder) *initMachine {
+	i.image = fakeImagePath
+	return i
+}

pkg/machine/e2e/config_windows_test.go

@@ -3,14 +3,39 @@ package e2e_test
 import (
 	"fmt"
 	"os/exec"
+	"path/filepath"
 	"strings"
 
+	"github.com/containers/libhvee/pkg/hypervctl"
+	"github.com/containers/podman/v6/pkg/machine/define"
 	. "github.com/onsi/ginkgo/v2"
 	. "github.com/onsi/gomega/gexec"
 )
 
 const podmanBinary = "../../../bin/windows/podman.exe"
 
+var (
+	// fakeImagePath string = "C:\\Users\\Baude\\fakedisk.vhdx"
+	fakeImagePath string = ""
+)
+
+func initPlatform() {
+	switch testProvider.VMType().String() {
+	case define.HyperVVirt.String():
+		vmm := hypervctl.NewVirtualMachineManager()
+		fullFileName := filepath.Join(tmpDir, randomString()) + ".vhdx"
+		if err := vmm.CreateVhdxFile(fullFileName, 15*1024*1024); err != nil {
+			Fail(fmt.Sprintf("Failed to create file %s %q", fullFileName, err))
+		}
+		fakeImagePath = fullFileName
+		fmt.Println("Created fake disk image: " + fakeImagePath)
+	case define.WSLVirt.String():
+		fmt.Println("Nothing yet, tbd")
+	default:
+		Fail(fmt.Sprintf("unknown Windows provider: %q", testProvider.VMType().String()))
+	}
+}
+
 // pgrep emulates the pgrep linux command
 func pgrep(n string) (string, error) {
 	// add filter to find the process and do no display a header
@@ -41,7 +66,17 @@ func runWslCommand(cmdArgs []string) (*machineSession, error) {
 	return &ms, nil
 }
 
-func (i *initMachine) withFakeImage(_ *machineTestBuilder) *initMachine {
-	i.image = mb.imagePath
+// withFakeImage should be used in tests where the machine is
+// initialized (or not) but never started.  It is intended
+// to speed up CI by not processing our large machine files.
+func (i *initMachine) withFakeImage(mb *machineTestBuilder) *initMachine {
+	switch testProvider.VMType() {
+	case define.HyperVVirt:
+		i.image = fakeImagePath
+	case define.WSLVirt:
+		i.image = mb.imagePath
+	default:
+		Fail(fmt.Sprintf("unknown Windows provider: %q", testProvider.VMType().String()))
+	}
 	return i
 }

pkg/machine/e2e/machine_test.go

@@ -72,6 +72,9 @@ var _ = BeforeSuite(func() {
 	if pullError != nil {
 		Fail(fmt.Sprintf("failed to pull disk: %q", pullError))
 	}
+
+	fmt.Println("Running platform specific set-up")
+	initPlatform()
 })
 
 type timing struct {