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

Normalize database connection parameters #13

Closed
cecton opened this issue Mar 17, 2015 · 1 comment
Closed

Normalize database connection parameters #13

cecton opened this issue Mar 17, 2015 · 1 comment

Comments

@cecton
Copy link
Contributor

cecton commented Mar 17, 2015

Since Odoo 8.0 is a bit... complicated about the connection variables, I suggest we could "normalize" the connection parameter to set the --db_* argument AND PG* env variables to make sure all the situations are covered.

Current Priorities

  • --db_host, then PGHOST, ... (then localhost or /run/postgresql)
  • --db_port, then PGPORT, ... (5432)
  • --db_user, then LOGNAME, then die
  • --db_password, then PGPASSWORD

During a backup on Odoo 8.0, it's a bit different...

  • --db_host, then PGHOST, ... (then localhost or /run/postgresql)
  • --db_port, then PGPORT, ... (5432)
  • --db_user, then LOGNAME, then die
  • PGPASSWORD, then --db_password, then none
@cecton cecton mentioned this issue Apr 7, 2015
@sle-odoo
Copy link
Contributor

sle-odoo commented Nov 3, 2016

We've finally been bitten by this kind of things (see a3d207f). I'm closing this as we now only use the --db* arguments in the Docker file but we have opened a task to clarify and fix these issues upstream.

Regards

@sle-odoo sle-odoo closed this as completed Nov 3, 2016
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

No branches or pull requests

2 participants