Remove useless duplication on PreCreateCheck

Signed-off-by: Olivier Gambier <olivier@docker.com>

drivers/exoscale/exoscale.go

@@ -188,10 +188,6 @@ func (d *Driver) GetState() (state.State, error) {
 	return state.None, nil
 }
 
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 func (d *Driver) createDefaultSecurityGroup(client *egoscale.Client, group string) (string, error) {
 	rules := []egoscale.SecurityGroupRule{
 		{

drivers/fakedriver/fakedriver.go

@@ -57,10 +57,6 @@ func (d *Driver) GetState() (state.State, error) {
 	return d.MockState, nil
 }
 
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 func (d *Driver) Create() error {
 	return nil
 }

drivers/google/google.go

@@ -165,12 +165,6 @@ func (d *Driver) initApis() (*ComputeUtil, error) {
 	return newComputeUtil(d)
 }
 
-// PreCreateCheck allows for pre-create operations to make sure a driver is ready for creation
-// It's a noop on GCE.
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 // Create creates a GCE VM instance acting as a docker host.
 func (d *Driver) Create() error {
 	c, err := newComputeUtil(d)

drivers/hyperv/hyperv.go

@@ -101,10 +101,6 @@ func (d *Driver) DriverName() string {
 	return "hyperv"
 }
 
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 func (d *Driver) GetURL() (string, error) {
 	ip, err := d.GetIP()
 	if err != nil {

drivers/none/driver.go

@@ -82,10 +82,6 @@ func (d *Driver) Kill() error {
 	return fmt.Errorf("hosts without a driver cannot be killed")
 }
 
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 func (d *Driver) Remove() error {
 	return nil
 }

drivers/openstack/openstack.go

@@ -335,10 +335,6 @@ func (d *Driver) GetState() (state.State, error) {
 	return state.None, nil
 }
 
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 func (d *Driver) Create() error {
 	d.KeyPairName = fmt.Sprintf("%s-%s", d.MachineName, mcnutils.GenerateRandomID())
 

drivers/softlayer/driver.go

@@ -303,10 +303,6 @@ func (d *Driver) GetActiveTransaction() (string, error) {
 	return t, nil
 }
 
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 func (d *Driver) waitForStart() {
 	log.Infof("Waiting for host to become available")
 	for {

drivers/vmwarefusion/fusion.go

@@ -198,10 +198,6 @@ func (d *Driver) GetState() (state.State, error) {
 	return state.Stopped, nil
 }
 
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 func (d *Driver) Create() error {
 	b2dutils := mcnutils.NewB2dUtils("", "", d.StorePath)
 	if err := b2dutils.CopyIsoToMachineDir(d.Boot2DockerURL, d.MachineName); err != nil {

drivers/vmwarevcloudair/vcloudair.go

@@ -241,10 +241,6 @@ func (d *Driver) GetState() (state.State, error) {
 
 }
 
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 func (d *Driver) Create() error {
 
 	key, err := d.createSSHKey()

drivers/vmwarevsphere/vsphere.go

@@ -212,10 +212,6 @@ func (d *Driver) GetState() (state.State, error) {
 	return state.None, nil
 }
 
-func (d *Driver) PreCreateCheck() error {
-	return nil
-}
-
 // the current implementation does the following:
 // 1. check whether the docker directory contains the boot2docker ISO
 // 2. generate an SSH keypair and bundle it in a tar.

libmachine/drivers/base.go

@@ -52,3 +52,8 @@ func (d *BaseDriver) GetSSHUsername() string {
 
 	return d.SSHUser
 }
+
+// PreCreateCheck is called to enforce pre-creation steps
+func (d *BaseDriver) PreCreateCheck() error {
+	return nil
+}