Skip to content
This repository has been archived by the owner on Sep 26, 2021. It is now read-only.

add packet host driver #1236

Closed
wants to merge 1 commit into from
Closed

add packet host driver #1236

wants to merge 1 commit into from

Conversation

crunchywelch
Copy link

Signed-off-by: Aaron Welch welch@packet.net

time.Sleep(10 * time.Second)
}

log.Infof("Provision time: %v.\n", time.Since(t0))
Copy link
Contributor

Choose a reason for hiding this comment

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

This should probably be debug.

@ehazlett
Copy link
Contributor

ehazlett commented Jun 3, 2015

Overall this looks pretty good. I have to test it. You will also need to update the integration tests -- we merged a test refactor (should make it easier) yesterday.

Thanks!

Signed-off-by: Aaron Welch <welch@packet.net>
@crunchywelch
Copy link
Author

Ok great, I updated the log statements to debug and removed the package updates. Will you let me know what I need to do with the integration tests, or will it be obvious when I merge?

Thanks!

@crunchywelch
Copy link
Author

hey @ehazlett just wanted to do a bump on this, are there changes I need to make to have this work with the next release? lmk what I can do to help get this in. Thanks!

@ehazlett
Copy link
Contributor

Hi , thanks for your efforts and persistence in submitting this driver. We are extremely excited that there is so much interest in Docker Machine and we really appreciate your interest. However, at this time it is proving to be extremely difficult for us to keep up with reviewing and testing each of these drivers for inclusion in the Machine core. We really want to switch to a more pluggable model, as well as polish up a few things about the driver model which need to be changed to ensure a smooth and sustainable future.

Therefore, we will be moving to a plugin model for 0.5 and would love to have you involved in the design and development process. We are closing the outstanding driver PRs at this time, but please keep the code. We will stick closely to the current driver interface and you should be able to re-use a lot (if not all) of the existing driver along with the new plugin model. We will be moving all of the drivers which are merged directly into Machine today to the plugin model when it is available, so there will be no special treatment of those, and there will be documentation outlining the process of developing and using a Docker Machine driver plugin.

With all of that being said, we want to apologize for the lack of feedback on your pull request. As contributors ourselves, we understand that being left in limbo is no fun. We would have liked to address this sooner, and in the future we will be more responsive around these kinds of issues.

Once again, we thank you for the contribution and the tremendous support. Keep hacking strong!

If you want to contribute to the design of the plugin model, we'd love to get your input on this issue where we will be planning it:

#1626

@ehazlett & @nathanleclaire

@ehazlett ehazlett closed this Jul 31, 2015
tomeon pushed a commit to tomeon/machine that referenced this pull request May 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants