Skip to content

[IMP] server_environment: allow env variable conf - #223

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
dixmit:16.0-imp-variable-conf
Mar 4, 2025
Merged

[IMP] server_environment: allow env variable conf#223
OCA-git-bot merged 1 commit into
OCA:16.0from
dixmit:16.0-imp-variable-conf

Conversation

@etobella

@etobella etobella commented Mar 4, 2025

Copy link
Copy Markdown
Member

Backport of #189

@luisDIXMIT luisDIXMIT left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@kluna1998 kluna1998 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍🏽

@etobella

etobella commented Mar 4, 2025

Copy link
Copy Markdown
Member Author

@petrus-v @simahawk your comments would be awesome in this PR 🙏

@etobella

etobella commented Mar 4, 2025

Copy link
Copy Markdown
Member Author

/ocabot merge minor

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-223-by-etobella-bump-minor, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 44cdc0f into OCA:16.0 Mar 4, 2025
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 9446754. Thanks a lot for contributing to OCA. ❤️

@etobella
etobella deleted the 16.0-imp-variable-conf branch March 4, 2025 18:26
ivantodorovich added a commit to camptocamp/docker-odoo-project that referenced this pull request Jun 30, 2026
Originally, this was the only way to configure the running environment for the
server_environment module.

3 years ago, a PR improved the module in version 14.0, to read the running
environment from the environment variable `RUNNING_ENV` instead of the config
file, if set.

Original PR in 14.0:
- OCA/server-env#146

Later, this PR was forward ported to 16.0 and 17.0 (sadly no 15.0):
- 15.0: NOT PORTED
- 16.0: OCA/server-env#223
- 17.0: OCA/server-env#189

Finally, it was fully integrated into the migrations to 18.0 and 19.0.
So it's safe to assume that in 18.0 and 19.0, no matter the module version used,
the running environment will be read from the RUNNING_ENV variable.

---

In 19.0, Odoo started complaining about unknown config file options, so this
supresses the following warning:

```
WARNING ? odoo.tools.config: unknown option 'running_env' in the config file at
/etc/odoo.cfg, option stored as-is, without parsing
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants