-
Notifications
You must be signed in to change notification settings - Fork 2k
Generic Driver #406
Generic Driver #406
Conversation
|
Travis is failing |
|
I'll check thx |
|
Fixed tests. |
426aa62 to
b7c0ee4
Compare
b7c0ee4 to
0d6d02b
Compare
|
Perhaps the README.md in your branch should elaborate on the generic-driver under the 'Drivers' heading. |
|
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? |
drivers/generic/generic.go
Outdated
There was a problem hiding this comment.
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...
0d6d02b to
dfd8466
Compare
|
Updated to latest driver spec. |
|
maybe it's just me but when i pull this locally and run "go build" i get a failure: |
|
@andrewwatson you need to build with If it still errors then we may need to check the version in Godep |
247446d to
f421390
Compare
|
So... what remains for this to get merged? It'd be awesome if this could be part of 0.3.0 :) |
|
FYI I've been using this driver a lot the past week or so and it seems to On Thu, Apr 30, 2015, 7:11 AM Dave Henderson [email protected]
|
f421390 to
f207f39
Compare
|
@andrewwatson thx for the feedback! |
|
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! |
95b4f68 to
44450f4
Compare
|
@nathanleclaire PTAL |
|
Will do! |
|
I'm wondering if we might want to trim down the timeout on |
|
Sounds good to me. On Tue, May 5, 2015, 12:28 PM Nathan LeClaire [email protected]
|
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
|
@nathanleclaire rebased and reduced GetState to 1s -- PTAL |
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
|
@nathanleclaire ok i think i hit all of your comments. PTAL. thanks! |
|
LGTM |
|
@nathanleclaire LMAO |
|
@nathanleclaire "Generic is coming..." |
…sk ready (Closes docker-archive-public#406) - trying 10 for now
…_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

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.