这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@mrluc
Copy link

@mrluc mrluc commented Sep 11, 2015

This is more of a thought experiment.

But adding || true to the line that attempts to install linux-image-extra-... would mean that dokku could run places with custom kernels, like Linode, by falling back to DeviceMapper. People who really want AUFS could use pv-grub and the StackScript (once a documentation tweak is made).

Is that just a crazy idea? I know that DeviceMapper has worse performance for starting/stopping containers, but as AUFS isn't upstream, it seems like it might work

@josegonzalez
Copy link
Member

Here is where docker requires it in their install.sh.

Might be worth us just installing their install into our install:

curl -sSL https://get.docker.com/ | sh

@mrluc
Copy link
Author

mrluc commented Sep 11, 2015

I don't know if that would cause any other problems, but yeah -- the underlying concern being "why should dokku require aufs if docker doesn't any more?" I'm not sure which is the right way to go about it, but custom-kernel options e.g. for Linode could still be mentioned in the hosting-provider-specific docs and linked to from the install page, as it's something that some users would really want.

josegonzalez added a commit that referenced this pull request Sep 15, 2015
Also simplify deb-based installation everywhere.

Closes #1295
Closes #1461
@josegonzalez
Copy link
Member

Closing in favor of 1467. @mrluc can you check if the bootstrap.sh from that branch can be run directly on linode?

josegonzalez added a commit that referenced this pull request Sep 15, 2015
Also simplify deb-based installation everywhere.

Closes #1295
Closes #1461
josegonzalez added a commit that referenced this pull request Sep 15, 2015
Also simplify deb-based installation everywhere.

Closes #1295
Closes #1461
josegonzalez added a commit that referenced this pull request Sep 15, 2015
Also simplify deb-based installation everywhere.

Closes #1295
Closes #1461
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.

2 participants