windows MK_WRONG_BINARY_WSL even when not in powershell

[trixnz]

The offending function below is incorrectly assuming that if WSLENV has a non-empty value, that the shell is an WSL shell. According to this post, this environment variable is used for sharing variables into new WSL sessions, and is set within Windows too.

minikube/pkg/minikube/driver/driver.go

Lines 174 to 176 in 51ec098

	func IsMicrosoftWSL() bool {
	return os.Getenv("WSL_DISTRO_NAME") != "" || os.Getenv("WSLPATH") != "" || os.Getenv("WSLENV") != ""
	}

Steps to reproduce the issue:

1. Ensure the WSLENV environment variable has a non-empty value. Mine is WT_SESSION::WT_PROFILE_ID
2. Run minikube from a regular Windows cmd.exe shell
3. Observe the incorrect output

> minikube docker-env

❌  Exiting due to MK_WRONG_BINARY_WSL: You are trying to run windows .exe binary inside WSL, for better integration please use Linux binary instead (Download at https://minikube.sigs.k8s.io/docs/start/.). Otherwise if you still want to do this, you can do it using --force

[medyagh]

@trixnz are u trying to run the Linux Binary ? or the .exe ?

we do NOT reocmmend running the linux binary on windows. do u mind trying with minkube.exe Binary and see if u get the same error ?

[trixnz]

This is being run with the Windows minikube binary acquired via chocolatey. I'm not interacting with WSL in any capacity.

[medyagh]

thats interesting ! this is our logic for WSL detection

func IsMicrosoftWSL() bool {
	return os.Getenv("WSL_DISTRO_NAME") != "" || os.Getenv("WSLPATH") != "" || os.Getenv("WSLENV") != ""
}

in our integraiton tests on windows our tests never failed, I wonder why your machine has "WSLENV" env set ? is that something specific to your windows setup or many other users have same problem? (and a question for us is why our integraiton windows machines are not like that ?)

@trixnz meanwhile I wonder does --force override the error for you ?
minikube docker-env --force . ?

[medyagh]

another question @trixnz , does this work fine in Powershell ? (not cmd) ? (our integration tests runs in powershell)

in-fact I think you might have discovered a minikube bug that I hope u help us get more information to fix it

[trixnz]

After looking at the envs being preserved through WSLENV, I realised this is exclusive to cmd.exe sessions spawned by Windows Terminal (WT_SESSION and WT_PROFILE_ID are env vars provided by Windows Terminal). It actually behaves correctly when run from a regular cmd.exe (and powershell) session.

I think it's improper behaviour to check for WSLENV being set, as this is not indicative of the shell being run under WSL, but rather which environment variables from Windows should be carried over into new WSL sessions.

@trixnz meanwhile I wonder does --force override the error for you ?
minikube docker-env --force . ?

No, it errors that --force is an unknown flag.

> minikube status --force
Error: unknown flag: --force
See 'minikube status --help' for usage.

[lewis52ravens]

@trixnz is right, it is definitely an issue with the WSLENV environment variable. I also noticed that this behavior only happens when attempting to start minikube from a PowerShell session launched from Windows Terminal, not when launched from a regular PowerShell session launched from powershell.exe.

However even in a regular PowerShell session, if you set the env var WSLENV to any value:
$env:wslenv = "Blahblah"
Then subsequent calls to minikube start in that session will also result in the MK_WRONG_BINARY_WSL error.

[kadel]

@trixnz is right, it is definitely an issue with the WSLENV environment variable. I also noticed that this behavior only happens when attempting to start minikube from a PowerShell session launched from Windows Terminal, not when launched from a regular PowerShell session launched from powershell.exe.

I can confirm that. The same thing is happening to me. Executing from the normal "Windows PowerShell" terminal is ok. When using PowerShell from Windows Terminal I get the error.

[mrehanabbasi]

I'm getting the same error. Have to set WSLENV to empty every time to make it work.