Merge pull request #3842 from afbjorklund/wsl2-hostname

Add fallback for busybox hostname with wsl2

Author: jandubois

pkg/store/instance_windows.go

@@ -4,7 +4,9 @@
 package store
 
 import (
+	"context"
 	"fmt"
+	"os/exec"
 	"regexp"
 	"strings"
 
@@ -115,6 +117,34 @@ func GetWslStatus(instName string) (string, error) {
 	return instState, nil
 }
 
-func GetSSHAddress(_ string) (string, error) {
-	return "127.0.0.1", nil
+// GetSSHAddress runs a hostname command to get the IP from inside of a wsl2 VM.
+//
+// Expected output (whitespace preserved, [] for optional):
+// PS > wsl -d <distroName> bash -c hostname -I | cut -d' ' -f1
+// 168.1.1.1 [10.0.0.1]
+// But busybox hostname does not implement --all-ip-addresses:
+// hostname: unrecognized option: I
+func GetSSHAddress(instName string) (string, error) {
+	ctx := context.TODO()
+	distroName := "lima-" + instName
+	// Ubuntu
+	cmd := exec.CommandContext(ctx, "wsl.exe", "-d", distroName, "bash", "-c", `hostname -I | cut -d ' ' -f1`)
+	out, err := cmd.CombinedOutput()
+	if err == nil {
+		return strings.TrimSpace(string(out)), nil
+	}
+	// Alpine
+	cmd = exec.CommandContext(ctx, "wsl.exe", "-d", distroName, "sh", "-c", `ip route get 1 | awk '{gsub("^.*src ",""); print $1; exit}'`)
+	out, err = cmd.CombinedOutput()
+	if err == nil {
+		return strings.TrimSpace(string(out)), nil
+	}
+	// fallback
+	cmd = exec.CommandContext(ctx, "wsl.exe", "-d", distroName, "hostname", "-i")
+	out, err = cmd.CombinedOutput()
+	if err != nil || strings.HasPrefix(string(out), "127.") {
+		return "", fmt.Errorf("failed to get hostname for instance %q, err: %w (out=%q)", instName, err, string(out))
+	}
+
+	return strings.TrimSpace(string(out)), nil
 }