Update docker-compose.yml

[spanishkangaroo]

Using the provided docker-compose.yml and nginx.conf files, the stack fails by throwing the following error:

FATAL: data directory “/var/lib/postgresql/data/pgdata” has wrong ownership
HINT: The server must be started by the user that owns the data directory.

The issue is easily solved by replacing the volumen part in the postgres definition with:

volumes: - /var/lib/postgresql/data

Regards,

[coveralls]

Coverage decreased (-0.5%) to 80.231% when pulling a295052 on spanishkangaroo:patch-1 into 3dec49d on Wirecloud:develop.

[aarranz]

Hi @spanishkangaroo,

First, thanks for your interest on WireCloud. However, this change does not make sense for me. Can you provide more details? What operating system are you using?

I guess you are using Windows and you are suffering one of the integration problems that raises when using this operating system. In this case, I think is better to use a named volume as suggested in this thread as this will provide better persistence management (volume persistence between container recreations, for example).

[spanishkangaroo]

Hello, Yes, I've reviewed your info and you're completely right. I am using Windows and my solution was a quick and dirty patch to make. your docker-compose recipe work. Would you mind to add a fix for Windows users then? Regards,

…

On Tue, Dec 3, 2019, 12:55 Álvaro Arranz ***@***.***> wrote: Hi @spanishkangaroo <https://github.com/spanishkangaroo>, First, thanks for your interest on WireCloud. However, this change does not make sense for me. Can you provide more details? What operating system are you using? I guess you are using Windows and you are suffering one of the integration problems that raises when using this operating system. In this case, I think is better to use a named volume as suggested in this thread <docker-library/postgres#558 (comment)> as this will provide better persistence management (volume persistence between container recreations, for example). — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#450?email_source=notifications&email_token=AABDW3OVM2GVYATJLRVKH3TQWZCJTA5CNFSM4JSYP3M2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFZD5DY#issuecomment-561135247>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABDW3N3ALUNT6KHZLDOGKDQWZCJTANCNFSM4JSYP3MQ> .

[aarranz]

Hi @spanishkangaroo,

Sorry for the late response, I've updated docker-compose example files to use named volumes, this should make them work on any operating system, including windows 😄.

Best regards.