Issue crc-org#838 Remove virtualbox driver support

gbraad-redhat · Nov 29, 2019 · 185a017 · 185a017
1 parent d6a16d6
commit 185a017
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 67 deletions.
diff --git a/pkg/crc/machine/client/client_darwin.go b/pkg/crc/machine/client/client_darwin.go
@@ -5,8 +5,6 @@ import (
 	"os"
 
 	"github.com/code-ready/crc/pkg/crc/errors"
-	"github.com/code-ready/machine/drivers/virtualbox"
-	"github.com/code-ready/machine/libmachine/drivers/plugin"
 	"github.com/code-ready/machine/libmachine/drivers/plugin/localbinary"
 )
 
@@ -15,8 +13,6 @@ func StartDriver() {
 	if os.Getenv(localbinary.PluginEnvKey) == localbinary.PluginEnvVal {
 		driverName := os.Getenv(localbinary.PluginEnvDriverName)
 		switch driverName {
-		case "virtualbox":
-			plugin.RegisterDriver(virtualbox.NewDriver("", ""))
 		default:
 			errors.ExitWithMessage(1, fmt.Sprintf("Unregistered driver: %s\n", driverName))
 		}

diff --git a/pkg/crc/machine/client/client_windows.go b/pkg/crc/machine/client/client_windows.go
@@ -6,8 +6,6 @@ import (
 
 	"github.com/code-ready/crc/pkg/crc/errors"
 	"github.com/code-ready/machine/drivers/hyperv"
-	"github.com/code-ready/machine/drivers/virtualbox"
-	"github.com/code-ready/machine/libmachine/drivers/plugin"
 	"github.com/code-ready/machine/libmachine/drivers/plugin/localbinary"
 )
 
@@ -16,8 +14,6 @@ func StartDriver() {
 	if os.Getenv(localbinary.PluginEnvKey) == localbinary.PluginEnvVal {
 		driverName := os.Getenv(localbinary.PluginEnvDriverName)
 		switch driverName {
-		case "virtualbox":
-			plugin.RegisterDriver(virtualbox.NewDriver("", ""))
 		case "hyperv":
 			plugin.RegisterDriver(hyperv.NewDriver("", ""))
 		default:

diff --git a/pkg/crc/machine/driver_darwin.go b/pkg/crc/machine/driver_darwin.go
@@ -3,12 +3,10 @@ package machine
 import (
 	"github.com/code-ready/crc/pkg/crc/constants"
 	"github.com/code-ready/crc/pkg/crc/errors"
-	"github.com/code-ready/crc/pkg/crc/logging"
 	crcos "github.com/code-ready/crc/pkg/os"
 
 	"github.com/code-ready/crc/pkg/crc/machine/config"
 	"github.com/code-ready/crc/pkg/crc/machine/hyperkit"
-	"github.com/code-ready/crc/pkg/crc/machine/virtualbox"
 )
 
 func init() {
@@ -19,15 +17,8 @@ func init() {
 		DriverPath: constants.CrcBinDir,
 	}
 
-	VirtualBoxMacOSDriver := MachineDriver{
-		Name:     "VirtualBox",
-		Platform: crcos.DARWIN,
-		Driver:   "virtualbox",
-	}
-
 	SupportedDrivers = []MachineDriver{
 		HyperkitDriver,
-		VirtualBoxMacOSDriver,
 	}
 
 	DefaultDriver = HyperkitDriver
@@ -39,9 +30,6 @@ func getDriverOptions(machineConfig config.MachineConfig) interface{} {
 	// Supported drivers
 	switch machineConfig.VMDriver {
 
-	case "virtualbox":
-		logging.Warn("Virtualbox support is deprecated and will be removed in the next release.")
-		driver = virtualbox.CreateHost(machineConfig)
 	case "hyperkit":
 		driver = hyperkit.CreateHost(machineConfig)
 

diff --git a/pkg/crc/machine/driver_windows.go b/pkg/crc/machine/driver_windows.go
@@ -2,12 +2,10 @@ package machine
 
 import (
 	"github.com/code-ready/crc/pkg/crc/errors"
-	"github.com/code-ready/crc/pkg/crc/logging"
 	crcos "github.com/code-ready/crc/pkg/os"
 
 	"github.com/code-ready/crc/pkg/crc/machine/config"
 	"github.com/code-ready/crc/pkg/crc/machine/hyperv"
-	"github.com/code-ready/crc/pkg/crc/machine/virtualbox"
 )
 
 func init() {
@@ -17,15 +15,8 @@ func init() {
 		Driver:   "hyperv",
 	}
 
-	VirtualBoxWindowsDriver := MachineDriver{
-		Name:     "VirtualBox",
-		Platform: crcos.WINDOWS,
-		Driver:   "virtualbox",
-	}
-
 	SupportedDrivers = []MachineDriver{
 		HyperVDriver,
-		VirtualBoxWindowsDriver,
 	}
 
 	DefaultDriver = HyperVDriver
@@ -37,9 +28,6 @@ func getDriverOptions(machineConfig config.MachineConfig) interface{} {
 	// Supported drivers
 	switch machineConfig.VMDriver {
 
-	case "virtualbox":
-		logging.Warn("Virtualbox support is deprecated and will be removed in the next release.")
-		driver = virtualbox.CreateHost(machineConfig)
 	case "hyperv":
 		driver = hyperv.CreateHost(machineConfig)
 

diff --git a/pkg/crc/machine/virtualbox/driver.go b/pkg/crc/machine/virtualbox/driver.go
diff --git a/pkg/crc/services/dns/dns_windows.go b/pkg/crc/services/dns/dns_windows.go
@@ -20,12 +20,6 @@ const (
 )
 
 func runPostStartForOS(serviceConfig services.ServicePostStartConfig, result *services.ServicePostStartResult) (services.ServicePostStartResult, error) {
-	// bailout for Virtualbox
-	if serviceConfig.DriverName == "virtualbox" {
-		result.Success = true
-		return *result, nil
-	}
-
 	_, switchName := winnet.SelectSwitchByNameOrDefault(AlternativeNetwork)
 	networkInterface := fmt.Sprintf("vEthernet (%s)", switchName)