search

smebberson / docker-alpine

Docker containers running Alpine Linux and s6 for process management. Solid, reliable containers.
MIT License
596 stars 186 forks source link

Fixed directory permissions issue for alpine-consul-ui #1

Closed yvann closed 9 years ago

yvann commented 9 years ago

Hi, thank you for all your images, I just found a small issue for consul-ui.

consului_1 | ==> Starting Consul agent...
consului_1 | ==> Error starting agent: Failed to start Consul client: Failed to start lan serf: mkdir /data/serf: permission denied
smebberson commented 9 years ago

@yvann, thanks for your PR!

We actually found that issue today too. I did include a commit (https://github.com/smebberson/docker-alpine/commit/19302c7c609288798795d0bb5f4c6ec3d799800b), only 6 hours ago which includes a working fix for this. Is it possible you tested this, prior to my last commit, but your fork actually included the commit?

Because we tested the latest commit and it works... perhaps you could rebuild the image from scratch and double check?

Thank you!

yvann commented 9 years ago

Ho yes sure ! I am going to test that and come back to you

yvann commented 9 years ago

OK, you were right ! I just tested and it works like a charm.

You can ignore this PR, thank you.

smebberson commented 9 years ago

@yvann, no worries. I'm glad you found it useful!

yvann commented 9 years ago

Your alpine-consul-base became my new base image ! Very convenient :)