Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add softlayer installation instructions #4727

Merged
merged 2 commits into from
Mar 19, 2014
Merged

Conversation

andreaturli
Copy link
Contributor

This simple PR adds the instructions to install Docker on IBM SoftLayer.

Docker-DCO-1.1-Signed-off-by: Andrea Turli andrea.turli@gmail.com (github: andreaturli)

Docker-DCO-1.1-Signed-off-by: Andrea Turli <andrea.turli@gmail.com> (github: andreaturli)
Docker-DCO-1.1-Signed-off-by: Andrea Turli <andrea.turli@gmail.com> (github: andreaturli)
@tianon
Copy link
Member

tianon commented Mar 18, 2014

I'm not a docs maintainer, just a casual observer, but I'm a little puzzled here. Is it just selecting the right SoftLayer Ubuntu image that makes this difficult with just the standard Ubuntu instructions?

@andreaturli
Copy link
Contributor Author

@tianon I'm not sure I got your comment, sorry.

@SvenDowideit
Copy link
Contributor

@andreaturli We're in the process of rearranging our documentation, and I'm having trouble working out where this document fits in - it seems to me to be showing how hard it is for a user to set up an Ubuntu virtual machine on SoftLayer, and has nothing Docker specific in it.

@andreaturli
Copy link
Contributor Author

@tianon if you look at the instructions for AWS there is nothing docker-specific in there too. My little guide is totally similar to that and it suggests an easy path to have docker running on SoftLayer without issues

@SvenDowideit
Copy link
Contributor

@andreaturli exactly : #4765

@SvenDowideit
Copy link
Contributor

@shykes this too?

jamtur01 added a commit that referenced this pull request Mar 19, 2014
add softlayer installation instructions
@jamtur01 jamtur01 merged commit ccddb33 into moby:master Mar 19, 2014
@jamtur01
Copy link
Contributor

Merged as per discussion.

@shykes
Copy link
Contributor

shykes commented Mar 19, 2014

Yes.

First not everyone who wants to use Docker on softlayer might be an Ubuntu user. So it's not necessarily obvious for everyone.

Second there might be softlayer-specific tweaks and configuration in the future. See GCE MTU config for inspiration, or imagine some custom storage-related configuration for example.

Third we are pointing to a default OS and configuration which we know works and we are free to change to match best practices. If we removed that default recommendation, then people might assume their favorite distro is automatically supported out of the box without SL-specific and docker-specific tweaks. Maybe in their search they will stumble upon incomplete or out-of-date instructions by their distro vendor or hosting provider or both. The resulting mess will be our responsibility to support and fix. I would much rather   maintain a single, authoritative set of instructions (and later turn it into an installer, or official Softlayer image, or plugin).

On Tue, Mar 18, 2014 at 11:41 PM, Sven Dowideit notifications@github.com
wrote:

@shykes this too?

Reply to this email directly or view it on GitHub:
#4727 (comment)

@tianon
Copy link
Member

tianon commented Mar 19, 2014

Ah, makes sense, hadn't thought about it like that :)

@andreaturli
Copy link
Contributor Author

thanks!

@andreaturli
Copy link
Contributor Author

@jamtur01 when is it supposed to be visible this merged content?

@jamtur01
Copy link
Contributor

It'll be updated at the next release.

@andreaturli
Copy link
Contributor Author

thanks @jamtur01 for your answer. Is there any way to see that change in http://docs.docker.io/en/master/installation/ (which is supposed to be unstable, right?) so that I can reference it on a blog post I'm about to publish.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants