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

allow to increase the db_maxconn of the connection pool using an environment variable #74018

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pedominguezbr
Copy link

@pedominguezbr pedominguezbr commented Oct 28, 2024

Update the line with; db_maxconn = 64, to something like: db_maxconn = {{ODOO_DATABASE_MAXCONN}} to allow the db_maxconn to be assigned via environment variable, since the parameter does not exist in the odoo cmdline

…ronment variable

Update the line with; db_maxconn = 64, to something like: db_maxconn = {{ODOO_DATABASE_MAXCONN}} to allow the db_maxconn to be assigned via environment variable, since the parameter does not exist in the odoo cmdline.

Signed-off-by: pedominguezbr <pe.dominguez.br@gmail.com>
@github-actions github-actions bot added odoo triage Triage is needed labels Oct 28, 2024
@carrodher carrodher changed the title allow to increase the db_maxconn of the connection pool using an envi… allow to increase the db_maxconn of the connection pool using an environment variable Oct 29, 2024
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Oct 29, 2024
@github-actions github-actions bot removed the triage Triage is needed label Oct 29, 2024
@github-actions github-actions bot removed the request for review from javsalgar October 29, 2024 06:55
Copy link
Member

@dgomezleon dgomezleon left a comment

Choose a reason for hiding this comment

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

Hi @pedominguezbr,

Thanks for contributing.

You would need to apply this for all the odoo versions. Also you will need to add this to README.md and odoo-env.sh as other environment variables do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-progress odoo verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants