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

[FIX] PG env use the DB_ vars like intended in the container #33

Open
wants to merge 2 commits into
base: 14.0
Choose a base branch
from

Conversation

PierrickBrun
Copy link
Member

No description provided.

@PierrickBrun
Copy link
Member Author

@hparfr Je viens de reprendre cette PR avec le dernier déploiement

@hparfr
Copy link
Member

hparfr commented Apr 29, 2022

Pour moi c'est ok, j'ai juste un doute, il me semble que définir a la fois PG_DATABASE et ODOO_DB évite les erreurs (les chemins au run et exec ne sont pas les mêmes)

@PierrickBrun
Copy link
Member Author

PierrickBrun commented Apr 29, 2022

Je ne suis pas sûr qu'on parle de la même chose car ODOO_DB ne me dit rien

Mais si tu veux parler de PGDATABASE et DB_DATABASE, en principe il DB_* est exporté dans PG* au lancement du container:
https://github.com/camptocamp/docker-odoo-project/blob/master/bin/docker-entrypoint.sh#L12

Du coup psql a besoin de PG* et le container a besoin de DB_*, qui est utilisé dans la config.

Pour moi bin/docker-entrypoint.sh est executé dans les 2 cas mais je me trompe peut-être

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