Skip to content

feat(configure): generate config/server.env with default and override env vars#45

Merged
nilshamerlinck merged 3 commits into
trobz:mainfrom
dzungtran89:T68041
Jun 29, 2026
Merged

feat(configure): generate config/server.env with default and override env vars#45
nilshamerlinck merged 3 commits into
trobz:mainfrom
dzungtran89:T68041

Conversation

@dzungtran89

Copy link
Copy Markdown
Contributor

What

Generate a config/server.env for odoo instances during configure, so the
systemd unit's EnvironmentFile is populated.

Contents

  • Default thread limits are written for every odoo instance.
  • Values from the deploy config's env key override those defaults.
  • The file is created when missing and regenerated on --recreate, keeping a
    backup of the previous one.

Forge-ID: 68041

… env vars

New 5.2 sub-step writes config/server.env: unconditional thread-limit
defaults (OMP/OPENBLAS/NUMEXPR/MKL) enriched by the deploy.yml `env` dict
(user values win). Written when missing; --recreate backs up and
regenerates, mirroring odoo.conf.

Forge-ID: 68041
Add the odoo.conf `config` overrides and the server.env `env` overrides
to the schema example and options table.

Forge-ID: 68041
Comment thread trobz_deploy/command/configure.py Outdated
fg="yellow",
)

# 5.2: config/server.env — default thread limits, enriched by deploy.yml `env`.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you please isolate it in its own step? env

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks, @nilshamerlinck . The PR has been updated.

Move server.env out of the `config` step into a dedicated `env` step so
it can be run or skipped independently.

Forge-ID: 68041
@nilshamerlinck nilshamerlinck merged commit 8d73f3c into trobz:main Jun 29, 2026
7 checks passed
@dzungtran89 dzungtran89 deleted the T68041 branch June 29, 2026 09:51
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