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

Remove virtualbox support from VM scenarios #5904

Closed
conorsch opened this issue Apr 16, 2021 · 2 comments · Fixed by #5922
Closed

Remove virtualbox support from VM scenarios #5904

conorsch opened this issue Apr 16, 2021 · 2 comments · Fixed by #5922

Comments

@conorsch
Copy link
Contributor

Description

Our Molecule-based VM scenarios for developing server config support both libvirt & virtualbox. Some, such as the upgrade scenario, only support libvirt. We originally added virtualbox support for macos support, but that was before we had a containerized dev env for developing app code, that works well on all platforms.

At sprint planning yesterday, a quick straw poll showed that none of the team members are using Virtualbox. Let's remove it from the config and the docs. Doing so will reduce the amount of code we're maintaining for testing, and slightly reduce the amount of code we need to address as part of removing Xenial codepaths.

@eloquence
Copy link
Member

eloquence commented Apr 16, 2021

Noting that we may provide macOS support for running SecureDrop VMs in future, depending on developer initerest, in which case we should evaluate again what the options are, which may include libvirt (which is cross-platform, but KVM is Linux-specific) or a cloud-based solution. However, we do not have the bandwidth to maintain three (!) different approaches for managing VMs (libvirt, Qubes, VirtualBox).

@conorsch
Copy link
Contributor Author

conorsch commented May 4, 2021

Started on these changes in #5922. As mentioned there, we'll need a separate docs PR, which I don't have in draft state. Worth pointing out that #5911 removed a few Virtualbox scenarios already, and #5883 will handle the rest.

@eloquence eloquence removed this from To Do (Kanban Period) in SecureDrop Team Board May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants