Deprecated VirtualBox with warning #21631

divysinghvi · 2025-09-24T17:55:24Z

This PR adds a deprecation warning for the VirtualBox driver, similar to the existing hyperkit deprecation warning. The VirtualBox driver will display a clear warning message to users indicating it will be removed in a future release and suggesting alternative drivers.
Before

Before

😄 minikube v1.37.0 on Darwin 15.6.1 (arm64)
✨ Using the virtualbox driver based on user configuration
👍 Starting "minikube" primary control-plane node in "minikube" cluster

After

divysinghvi@Divys-MacBook-Air minikube % ./out/minikube start --driver=virtualbox
😄  minikube v1.37.0 on Darwin 15.6.1 (arm64)
✨  Using the virtualbox driver based on user configuration
❗  The 'virtualbox' driver is deprecated and will be removed in a future release.
    You can use alternative drivers such as 'vfkit', 'qemu', or 'docker'.
    https://minikube.sigs.k8s.io/docs/drivers/vfkit/
    https://minikube.sigs.k8s.io/docs/drivers/qemu/
    https://minikube.sigs.k8s.io/docs/drivers/docker/

After suggested change

✨  Using the virtualbox driver based on user configuration
❗  The 'virtualbox' driver is deprecated and will be removed in a future release.
    You can use alternative drivers: qemu2, vfkit, krunkit, parallels, docker, podman, ssh.

Fixes #21608

k8s-ci-robot · 2025-09-24T17:55:34Z

Hi @divysinghvi. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

minikube-bot · 2025-09-24T17:57:03Z

Can one of the admins verify this patch?

divysinghvi · 2025-09-24T17:58:28Z

@nirs https://minikube.sigs.k8s.io/docs/drivers/ i was thinking since we say stuff like preferred , experimental on this docs page shouldn't we mention the deprecated one's also this could be useful for people who are new so that then focus on right drivers .

medyagh · 2025-09-24T18:14:47Z

cmd/minikube/cmd/start.go

 	}
-	out.WarningT(`Due to changes in macOS 13+ minikube doesn't currently support VirtualBox. You can use alternative drivers such as 'vfkit', 'qemu', or 'docker'.
+	out.WarningT(`The 'virtualbox' driver is deprecated and will be removed in a future release.
+    You can use alternative drivers such as 'vfkit', 'qemu', or 'docker'.


we should list the drivers available Per Platform,
I believe regstiry package might have a func that gives you Available Drviers

I have done the change this is how it looks now

✨ Using the virtualbox driver based on user configuration ❗ The 'virtualbox' driver is deprecated and will be removed in a future release. You can use alternative drivers: qemu2, vfkit, krunkit, parallels, docker, podman, ssh.

You can go back and make a similar fix to hyperkit warning (in a new PR), to make them look more similar:

Deprecate HyperKit driver with warning #21024

Yes i have to change the hyperkit one in a new pr i was waiting for this one to be confirmed if this looks good i can make the pr for the hyperkit one also thanks

…nstead

k8s-ci-robot · 2025-10-01T17:22:01Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: divysinghvi, medyagh

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

~~OWNERS~~ [medyagh]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

medyagh · 2025-10-01T17:23:15Z

thank you @divysinghvi that looks better, yes you can create a follow up PR to make hyperkit and others do the same

I suggest the follow up Pr have a general func called

warnDriverDepricated

and that can have for loop that goes ever every Depricatred driverd and checks if that matches the selected driver.
and only differen thing there would be the message if needs to be different

medyagh · 2025-10-01T17:23:46Z

/ok-to-test

divysinghvi · 2025-10-01T17:24:33Z

yes that could be really useful in future cause since we have a lot of upcoming deprecated drivers as nirs and abjorklund were discussing like parallals etc actually i did make the hyperkit message change also but it's better to have a common function for this task will make change in that too

k8s-ci-robot · 2025-10-01T17:24:47Z

@divysinghvi: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
pull-minikube-integration	`b128d3d`	link	true	`/test pull-minikube-integration`

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

divysinghvi · 2025-10-01T17:26:43Z

@medyagh why is integration test failing on this pr any clue??

minikube-pr-bot · 2025-10-01T18:16:16Z

kvm2 driver with docker runtime

┌────────────────┬──────────┬────────────────────────┐
│    COMMAND     │ MINIKUBE │ MINIKUBE  ( PR 21631 ) │
├────────────────┼──────────┼────────────────────────┤
│ minikube start │ 45.4s    │ 44.3s                  │
│ enable ingress │ 17.0s    │ 15.9s                  │
└────────────────┴──────────┴────────────────────────┘

Times for minikube (PR 21631) ingress: 15.8s 15.8s 16.3s 15.8s 15.8s
Times for minikube ingress: 15.8s 19.8s 16.3s 16.8s 16.3s

Times for minikube start: 45.0s 44.5s 45.7s 45.7s 45.8s
Times for minikube (PR 21631) start: 46.0s 41.4s 44.3s 45.3s 44.4s

docker driver with docker runtime

┌────────────────┬──────────┬────────────────────────┐
│    COMMAND     │ MINIKUBE │ MINIKUBE  ( PR 21631 ) │
├────────────────┼──────────┼────────────────────────┤
│ minikube start │ 21.5s    │ 21.5s                  │
│ enable ingress │ 12.6s    │ 11.0s                  │
└────────────────┴──────────┴────────────────────────┘

Times for minikube start: 22.2s 21.5s 20.9s 23.0s 19.9s
Times for minikube (PR 21631) start: 21.6s 20.9s 21.4s 22.4s 21.4s

Times for minikube ingress: 10.6s 15.6s 10.6s 13.6s 12.6s
Times for minikube (PR 21631) ingress: 11.6s 11.6s 10.6s 10.7s 10.6s

docker driver with containerd runtime

┌────────────────┬──────────┬────────────────────────┐
│    COMMAND     │ MINIKUBE │ MINIKUBE  ( PR 21631 ) │
├────────────────┼──────────┼────────────────────────┤
│ minikube start │ 19.8s    │ 19.9s                  │
│ enable ingress │ 26.5s    │ 23.5s                  │
└────────────────┴──────────┴────────────────────────┘

Times for minikube ingress: 24.1s 23.1s 39.1s 23.1s 23.1s
Times for minikube (PR 21631) ingress: 23.1s 23.1s 24.1s 23.1s 24.1s

Times for minikube (PR 21631) start: 19.9s 20.5s 19.5s 19.5s 20.2s
Times for minikube start: 20.5s 19.9s 20.1s 18.8s 19.5s

minikube-pr-bot · 2025-10-01T19:53:55Z

Here are the number of top 10 failed tests in each environments with lowest flake rate.

Environment	Test Name	Flake Rate
Docker_Linux_crio (6 failed)	TestFunctional/parallel/ServiceCmdConnect(gopogh)	0.00% (chart)
Docker_Linux_crio (6 failed)	TestFunctional/parallel/ServiceCmd/DeployApp(gopogh)	0.00% (chart)
Docker_Linux_crio (6 failed)	TestFunctional/parallel/ServiceCmd/HTTPS(gopogh)	0.00% (chart)
Docker_Linux_crio (6 failed)	TestFunctional/parallel/ServiceCmd/Format(gopogh)	0.00% (chart)
Docker_Linux_crio (6 failed)	TestFunctional/parallel/ServiceCmd/URL(gopogh)	0.00% (chart)
KVM_Linux_crio (4 failed)	TestCertExpiration(gopogh)	6.38% (chart)
Docker_Linux_crio_arm64 (6 failed)	TestFunctional/parallel/ServiceCmdConnect(gopogh)	0.00% (chart)
Docker_Linux_crio_arm64 (6 failed)	TestFunctional/parallel/ServiceCmd/DeployApp(gopogh)	0.00% (chart)
Docker_Linux_crio_arm64 (6 failed)	TestFunctional/parallel/ServiceCmd/HTTPS(gopogh)	0.00% (chart)
Docker_Linux_crio_arm64 (6 failed)	TestFunctional/parallel/ServiceCmd/Format(gopogh)	0.00% (chart)
Docker_Linux_crio_arm64 (6 failed)	TestFunctional/parallel/ServiceCmd/URL(gopogh)	0.00% (chart)

To see the flake rates of all tests by environment, click here.

nirs · 2025-10-03T18:00:23Z

cmd/minikube/cmd/start.go

-    https://minikube.sigs.k8s.io/docs/drivers/qemu/
-    https://minikube.sigs.k8s.io/docs/drivers/docker/
-	`)
+    You can use alternative drivers: {{.drivers}}.`, out.V{"drivers": strings.Join(driver.SupportedDrivers(), ", ")})


Why did you remove this? please keep the links to the docs. This make it very easy to find more info by clicking on the link in the shell, and it is not required for this change.

Deprecated VirtualBox with warning

36bbf53

k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Sep 24, 2025

k8s-ci-robot requested review from medyagh and nirs September 24, 2025 17:55

k8s-ci-robot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Sep 24, 2025

k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 24, 2025

medyagh requested changes Sep 24, 2025

View reviewed changes

Removed Hardcoded Driver message and use SupportedDrivers()function i…

b128d3d

…nstead

medyagh approved these changes Oct 1, 2025

View reviewed changes

k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 1, 2025

k8s-ci-robot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Oct 1, 2025

divysinghvi mentioned this pull request Oct 3, 2025

Automated generic deprecation warning #21689

Open

nirs suggested changes Oct 3, 2025

View reviewed changes

Deprecated VirtualBox with warning #21631

Are you sure you want to change the base?

Deprecated VirtualBox with warning #21631

Conversation

divysinghvi commented Sep 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Before

After

Uh oh!

k8s-ci-robot commented Sep 24, 2025

Uh oh!

minikube-bot commented Sep 24, 2025

Uh oh!

divysinghvi commented Sep 24, 2025

Uh oh!

medyagh Sep 24, 2025

Choose a reason for hiding this comment

Uh oh!

divysinghvi Sep 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

afbjorklund Sep 25, 2025

Choose a reason for hiding this comment

Uh oh!

divysinghvi Sep 25, 2025

Choose a reason for hiding this comment

Uh oh!

k8s-ci-robot commented Oct 1, 2025

Uh oh!

medyagh commented Oct 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

medyagh commented Oct 1, 2025

Uh oh!

divysinghvi commented Oct 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

k8s-ci-robot commented Oct 1, 2025

Uh oh!

divysinghvi commented Oct 1, 2025

Uh oh!

minikube-pr-bot commented Oct 1, 2025

Uh oh!

minikube-pr-bot commented Oct 1, 2025

Uh oh!

nirs Oct 3, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

divysinghvi commented Sep 24, 2025 •

edited

Loading

divysinghvi Sep 25, 2025 •

edited

Loading

medyagh commented Oct 1, 2025 •

edited

Loading

divysinghvi commented Oct 1, 2025 •

edited

Loading