search

g0v-it / data-budget

The Smart Data Management Platform to feed https://budget.g0v.it application
Other
34 stars 3 forks source link

default environment file for container ports #5

Closed rspisser closed 5 years ago

rspisser commented 5 years ago

you can overwrite these values by setting the environment variable before you launch docker-compose -f docker-compose-run.yml up

ecow commented 5 years ago

Non c'e un modo per evitare il file .env (ad esempio non si riesce ad usare il tag environment nel file doker-compose? oppure avere dei valori default? Per convenzione il file di solito .env non si committa sul repository (è inserito in .git ignore) perchè potrebbe contenere dati riservati (password, login). Non è questo il caso . Se non ci sono alternative, basterebbe chiamarlo env.dist o env.development ma occorre anche modificare il README.md altrimenti se uno segue le istruzioni che ci sono oggi non funziona.

rspisser commented 5 years ago

Non c'e un modo per evitare il file .env (ad esempio non si riesce ad usare il tag environment nel file doker-compose? oppure avere dei valori default? Per convenzione il file di solito .env non si committa sul repository (è inserito in .git ignore) perchè potrebbe contenere dati riservati (password, login). Non è questo il caso . Se non ci sono alternative, basterebbe chiamarlo env.dist o env.development ma occorre anche modificare il README.md altrimenti se uno segue le istruzioni che ci sono oggi non funziona.

OK chiaro. L'obiettivo della modifica era di avere delle porte di default (specificate nel file .env) ma che permetteva a runtime di sovrascrivere i valori. Devo trovare un modo diverso (e ovviamente documentarlo)

rspisser commented 5 years ago

il file docker compose (PR #6 ) contiene gia default per le porte e la possibilita' di specificare altri valori via environment al momento del lancio. Direi che il file .env lo possiamo tenere fuori dal repo

non capisco - nel file docker-compose nella pull request non ci sono le porte di default.

luca commented 5 years ago

il file docker compose (PR #6 ) contiene gia default per le porte e la possibilita' di specificare altri valori via environment al momento del lancio. Direi che il file .env lo possiamo tenere fuori dal repo

non capisco - nel file docker-compose nella pull request non ci sono le porte di default.

sorry, avevo letto male il file docker compose

ecow commented 5 years ago

OK chiaro. L'obiettivo della modifica era di avere delle porte di default (specificate nel file .env) ma che permetteva a runtime di sovrascrivere i valori. Devo trovare un modo diverso (e ovviamente documentarlo)

Se ti serve non vedo problemi a parte il nome del file. nel readme basta modificare docker-compose up -d in . env.development; docker-compose up -d