Skip to content
This repository was archived by the owner on Apr 13, 2020. It is now read-only.

[DCK] Test DB name will be "prod" by default instead of "test" #20

Merged
merged 1 commit into from
Jan 30, 2019

Conversation

yajo
Copy link
Contributor

@yajo yajo commented Jan 30, 2019

From this change onwards, test environments will have their default databases called "prod", instead of "test" by default.

This will ease the process of transfering data between environments, such as what happens when using the test environment for staging or migrating a production deployment.

In case you were using ONLY with PGDATABASE in addons.yaml to filter some addons, to still let you have a way to know the current environment, a new $DOODBA_ENVIRONMENT variable is added, with each environment name hardcoded in it. Example:

# Common addons
---
# Enable web ribbon in non-prod environments
ONLY:
  DOODBA_ENVIRONMENT:
  - devel
  - test
web:
- web_environment_ribbon

Read here the risk of this patch: Tecnativa/doodba#67 (comment)

@yajo yajo force-pushed the single-db-name-odoo branch from eec724c to 239eb70 Compare January 30, 2019 09:49
@yajo yajo added the enhancement New feature or request label Jan 30, 2019
@yajo yajo force-pushed the single-db-name-odoo branch from 239eb70 to c059fca Compare January 30, 2019 10:28
This will ease the process of transfering data between environments, such as what happens when using the test environment for staging or migrating a production deployment.

A new `$DOODBA_ENVIRONMENT` variable has been added, to allow the container to be conscient of its environment, just in case you need to use the `ONLY` filtering system in `addons.yaml`.
@yajo yajo force-pushed the single-db-name-odoo branch from c059fca to 97c814b Compare January 30, 2019 10:36
@yajo yajo changed the title [DCK] Homogeneous DB name "odoo" [DCK] Test DB name will be "prod" by default instead of "test" Jan 30, 2019
@yajo
Copy link
Contributor Author

yajo commented Jan 30, 2019

I updated the description to explain the new PR approach. The initial one was too dangerous.

@yajo yajo merged commit 45a0f65 into master Jan 30, 2019
@yajo yajo deleted the single-db-name-odoo branch January 30, 2019 10:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants