-
Couldn't load subscription status.
- Fork 736
Remove libvirt driver from GUI #4018
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
Conversation
7a4efdd to
e69743f
Compare
e69743f to
e3a69b1
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4018 +/- ##
=======================================
Coverage 89.33% 89.33%
=======================================
Files 260 260
Lines 14735 14735
=======================================
Hits 13164 13164
Misses 1571 1571 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just tested with
sudo snap install multipass- setting the driver to libvirt in the gui
- confirming it can launch instances after installing libvirt and connecting multipass to libvirt with
snap connect multipass:libvirt
Then I installed this PR with sudo snap refresh multipass --channel edge/pr4018 and was able to select a different backend in the GUI and could not return to the libvirt backend as expected
e3a69b1 to
a7fd894
Compare
a7fd894 to
20b3f75
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was able to recreate on the CLI as well, the full error ends in |
|
Hey @Sploder12, @levkropp, libvirt is very restrictive by default and a locally built Multipass daemon usually fails to launch VMs with "Permission denied" errors (that's regardless of the new permissions). I am not aware of that affecting the snap, so there is probably something else going on in here, but I wanted to make sure you guys don't confuse the two problems. The permissions error we've always had on local builds can be overcome with something like:
|
|
Thanks @ricab! That fixed it for me, I almost went on a wild goose chase! |
|
Glad to be of service 🙂 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can confirm that this fixed it on my end as well. Thank you @ricab !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, all good 😄

This PR removes libvirt from the list of drivers in the GUI. If a user has libvirt they will still be on libvirt, however, once they switch to a different driver libvirt will no-longer be selectable.
MULTI-1889