Issue crc-org#838 Remove virtualbox driver support

gbraad-redhat · Nov 27, 2019 · 5d5ffc8 · 5d5ffc8
1 parent d6a16d6
commit 5d5ffc8
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 57 deletions.
diff --git a/pkg/crc/machine/driver_darwin.go b/pkg/crc/machine/driver_darwin.go
@@ -8,7 +8,6 @@ import (
 
 	"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 +18,8 @@ func init() {
 		DriverPath: constants.CrcBinDir,
 	}
 
-	VirtualBoxMacOSDriver := MachineDriver{
-		Name:     "VirtualBox",
-		Platform: crcos.DARWIN,
-		Driver:   "virtualbox",
-	}
-
 	SupportedDrivers = []MachineDriver{
 		HyperkitDriver,
-		VirtualBoxMacOSDriver,
 	}
 
 	DefaultDriver = HyperkitDriver
@@ -39,9 +31,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
@@ -7,7 +7,6 @@ import (
 
 	"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 +16,8 @@ func init() {
 		Driver:   "hyperv",
 	}
 
-	VirtualBoxWindowsDriver := MachineDriver{
-		Name:     "VirtualBox",
-		Platform: crcos.WINDOWS,
-		Driver:   "virtualbox",
-	}
-
 	SupportedDrivers = []MachineDriver{
 		HyperVDriver,
-		VirtualBoxWindowsDriver,
 	}
 
 	DefaultDriver = HyperVDriver
@@ -37,9 +29,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)