coopdevs / odoo-role

Ansible role provisioning Odoo.
GNU General Public License v3.0
13 stars 20 forks source link

odoo_role_db_port not defined #161

Closed oyale closed 6 months ago

oyale commented 6 months ago

In odoo.conf.j2 there is a conditional on odoo_role_db_port variable.

https://github.com/coopdevs/odoo-role/blob/fecda00a3f8a61151283211bab8e98cdd95f69bd/templates/odoo.conf.j2#L34

However, if not defined, Ansible will crash. Tested with devenv.

TASK [odoo-role : Add Odoo config] *****************************************************************************************************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: ansible.errors.AnsibleUndefinedVariable: 'odoo_role_db_port' is undefined
fatal: [odoo-role-test.local]: FAILED! => {"changed": false, "msg": "AnsibleUndefinedVariable: 'odoo_role_db_port' is undefined"}
### Tasks
- [ ] https://github.com/coopdevs/odoo-role/pull/162
oyale commented 6 months ago

@danypr92 could you take a look? I've noticed that the previous change on that line was yours doing the opposite :octopus: