Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow turning off the VBox accelerate3d flag #19729

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

afbjorklund
Copy link
Collaborator

Oracle VirtualBox version 7.1 seems to have some issues with
the --accelerate3d that was used by default with VirtualBox.

minikube start --driver=virtualbox --no-accelerate3d-off

Closes #19639

Oracle VirtualBox version 7.1 seems to have some issues with
the --accelerate3d that was used by default with VirtualBox.
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: afbjorklund
Once this PR has been reviewed and has the lgtm label, please assign prezha for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files:

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

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 29, 2024
@afbjorklund
Copy link
Collaborator Author

afbjorklund commented Sep 29, 2024

But minikube started just fine (on Linux) without this fix, as well.

When using 7.1.2 from https://www.virtualbox.org/wiki/Downloads

So it might not be needed with all graphics card and on all distros.

Just with some, so make it an option (like --no-vtx-check for cpuid)

@afbjorklund
Copy link
Collaborator Author

Added PR for the failing functional_podman_ubuntu test in CI:

Copy link
Member

@spowelljr spowelljr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I completely agree with the idea of the PR, the flag being a double negative is a little confusing though, could we just rename it to accelerate3d?

ie. --accelerate3d=true vs --no-accelerate3d-off=true

@afbjorklund
Copy link
Collaborator Author

I completely agree with the idea of the PR, the flag being a double negative is a little confusing though, could we just rename it to accelerate3d?

ie. --accelerate3d=true vs --no-accelerate3d-off=true

Sure, but that is not what the flag does. It removes the parameter --accelerate3d=off from VBoxManage modifyvm.

Because in some builds, that flag is now throwing an error... But the 3D acceleration was never meant to be enabled.

@spowelljr
Copy link
Member

spowelljr commented Sep 30, 2024

I see, if the accelerate3d flag is not specified what does it default to? If it defaults to false we could just remove it for everyone, but I'm assuming that's not the case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't start Minikube: VBOX_E_NOT_SUPPORTED
3 participants