Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Conversation

@ehazlett
Copy link
Contributor

This adds support for a "generic" driver. This is very basic at the moment. Basically you just need an Ubuntu server with SSH. From there it provisions as normal.

@thaJeztah
Copy link
Contributor

Travis is failing

@ehazlett
Copy link
Contributor Author

I'll check thx

@ehazlett
Copy link
Contributor Author

Fixed tests.

@hansbogert
Copy link

Perhaps the README.md in your branch should elaborate on the generic-driver under the 'Drivers' heading.

@andrewwatson
Copy link
Contributor

Should this driver fail if you try to use it as root? Should it try one thing first that uses sudo to see if it works and then kindly inform the user that all hope is lost if it fails?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this seems like a number of things could go wrong and it would be hard to know what it was... the curl could fail or you might not have sudo permissions...

@ehazlett
Copy link
Contributor Author

Updated to latest driver spec.

@andrewwatson
Copy link
Contributor

maybe it's just me but when i pull this locally and run "go build" i get a failure:

drivers/openstack/client.go:423: provider.HTTPClient undefined (type *gophercloud.ProviderClient has no field or method HTTPClient)

@ehazlett
Copy link
Contributor Author

@andrewwatson you need to build with godep -- preferably using script/build

If it still errors then we may need to check the version in Godep

@hairyhenderson
Copy link
Contributor

So... what remains for this to get merged? It'd be awesome if this could be part of 0.3.0 :)

@andrewwatson
Copy link
Contributor

FYI I've been using this driver a lot the past week or so and it seems to
work really well. I've been driving machine from a local-exec provisioner
in terraform and loving it.

On Thu, Apr 30, 2015, 7:11 AM Dave Henderson [email protected]
wrote:

So... what remains for this to get merged? It'd be awesome if this could
be part of 0.3.0 :)


Reply to this email directly or view it on GitHub
#406 (comment).

@ehazlett
Copy link
Contributor Author

@andrewwatson thx for the feedback!

@andrewwatson
Copy link
Contributor

I just submitted a PR to ehazlett/machine for a change to use DialTimeout() but other than that I think this is ready to ship!

@ehazlett
Copy link
Contributor Author

@nathanleclaire PTAL

@nathanleclaire
Copy link
Contributor

Will do!

@nathanleclaire
Copy link
Contributor

I'm wondering if we might want to trim down the timeout on GetState to ~1 second. Generally for me ls returns in under a second, and with this if you have even one generic host which isn't around any more, your ls runs will take a minimum of 5 seconds. That's pretty long time to just hang at the CLI.

@andrewwatson
Copy link
Contributor

Sounds good to me.

On Tue, May 5, 2015, 12:28 PM Nathan LeClaire [email protected]
wrote:

I'm wondering if we might want to trim down the timeout on GetState to ~1
second. Generally for me ls returns in under a second, and with this if
you have even one generic host which isn't around any more, your ls runs
will take a minimum of 5 seconds. That's pretty long time to just hang at
the CLI.


Reply to this email directly or view it on GitHub
#406 (comment).

@ehazlett
Copy link
Contributor Author

ehazlett commented May 5, 2015

@nathanleclaire rebased and reduced GetState to 1s -- PTAL

@ehazlett
Copy link
Contributor Author

ehazlett commented May 5, 2015

@nathanleclaire ok i think i hit all of your comments. PTAL. thanks!

@nathanleclaire
Copy link
Contributor

LGTM

@nathanleclaire
Copy link
Contributor

Image

@ehazlett
Copy link
Contributor Author

ehazlett commented May 5, 2015

@nathanleclaire LMAO

@ehazlett
Copy link
Contributor Author

ehazlett commented May 5, 2015

@nathanleclaire "Generic is coming..."

ehazlett added a commit that referenced this pull request May 5, 2015
@ehazlett ehazlett merged commit a7d6316 into docker-archive-public:master May 5, 2015
@ehazlett ehazlett deleted the driver-generic branch May 5, 2015 20:47
@ghost ghost mentioned this pull request May 10, 2015
tomeon pushed a commit to tomeon/machine that referenced this pull request May 9, 2018
tomeon pushed a commit to tomeon/machine that referenced this pull request May 9, 2018
…_it_seems_that_3_seconds_is_not_enough_time_for_hyper_v_to_have_the_disk_ready_closes_406_trying_10_for_now

it seems that 3 seconds is not enough time for Hyper-V to have the disk ready (Closes docker-archive-public#406) - trying 10 for now
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants