Skip to content

Commit

Permalink
Disallow using Docker Desktop 4.34.0 on macos (#19576)
Browse files Browse the repository at this point in the history
* Disallow using Docker Desktop 4.34.0 on macos

* Disallow using Docker Desktop 4.34.0 on macos

* remove unneeded changes

* remove unneeded changes

* update changelog
  • Loading branch information
medyagh authored Sep 9, 2024
1 parent fe41066 commit bd82a28
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 1 deletion.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

Breaking Changes:
* Bump minimum podman version to 4.9.0 [#19457](https://github.com/kubernetes/minikube/pull/19457)

* Disallow using Docker Desktop 4.34.0
Features:
* Bump default Kubernetes version to v1.31.0 [#19435](https://github.com/kubernetes/minikube/pull/19435)
* Add new driver for macOS: vfkit [#19423](https://github.com/kubernetes/minikube/pull/19423)
Expand Down
11 changes: 11 additions & 0 deletions pkg/minikube/registry/drvs/docker/docker.go
Original file line number Diff line number Diff line change
Expand Up @@ -289,6 +289,17 @@ func checkDockerDesktopVersion(version string) (s registry.State) {
Fix: "Update Docker Desktop to 4.16.1 or greater",
}
}

if runtime.GOOS == "darwin" && currSemver.EQ(semver.MustParse("4.34.0")) {
return registry.State{
Reason: "PROVIDER_DOCKER_DESKTOP_VERSION_BAD",
Running: true,
Error: errors.New("Docker Desktop 4.34.0 has a regression that prevents minikube from listing the containers"),
Installed: true,
Fix: "Use a different Docker desktop version, more info at https://github.com/docker/cli/issues/5412",
}
}

return s
}

Expand Down

0 comments on commit bd82a28

Please sign in to comment.