[bitnami/odoo] Add pre-install/pre-upgrade job for module installation. #25124
Conversation
samaroon
force-pushed
the
main
branch
2 times, most recently
from
27dd75e
to
b88e9b9
Compare
Signed-off-by: Rahma Ahmed <rahma.ahmed@oneacrefund.org>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
carrodher
added
verify
Signed-off-by: Rahma Ahmed <rahma.ahmed@oneacrefund.org>
samaroon
force-pushed
the
main
branch
8 times, most recently
from
c4a4263
to
621ebdf
Compare
Signed-off-by: Rahma Ahmed <rahma.ahmed@oneacrefund.org>
| ## @param odooSkipModulesUpdate Skip Odoo update wizard | ||
| ## | ||
| odooSkipModulesUpdate: true |
There was a problem hiding this comment.
| ## @param odooSkipModulesUpdate Skip Odoo update wizard | |
| ## | |
| odooSkipModulesUpdate: true | |
| ## @param odooSkipModulesUpdate Skip update of modules during the deployment | |
| ## | |
| odooSkipModulesUpdate: false |
This would change the default behavior. I would not skip this by default
There was a problem hiding this comment.
@jotamartos This should be used as part of the ternary operator with a value of true or false.
There was a problem hiding this comment.
Sorry, my fault. In that case, use "false" instead of "no" but the idea is to follow the same approach we have in the container.
| {{- if .Values.upgradeJob.command }} | ||
| command: | ||
| {{- include "common.tplvalues.render" (dict "value" .Values.command "context" $) | nindent 12 }} | ||
| {{- end }} |
There was a problem hiding this comment.
If you do not provide a command, it'll basically run another pod with Odoo, will not simply update the plugins.
| ## @param odooSkipModulesUpdate Skip Odoo update wizard | ||
| ## | ||
| odooSkipModulesUpdate: true |
There was a problem hiding this comment.
Sorry, my fault. In that case, use "false" instead of "no" but the idea is to follow the same approach we have in the container.
|
This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution. |
|
Due to the lack of activity in the last 5 days since it was marked as "stale", we proceed to close this Pull Request. Do not hesitate to reopen it later if necessary. |
Description of the change
This PR introduces a new Kubernetes job defined in the Odoo Helm chart, which is triggered during the pre-install and pre-upgrade phases. The job, named
odoo-addons-install, utilizes arguments (args) and environment variables (env) from thevaluesfile to configure and run the Odoo modules installation. This setup ensures that all necessary modules are installed and configured before any system updates or upgrades are applied, improving stability and consistency across deployments.Benefits
This job enhances the stability and reliability of the Odoo system by ensuring all necessary modules are pre-installed, which prevents conflicts during upgrades.
Checklist
Chart.yamlaccording to semver. This is not necessary when the changes only affect README.md files.README.mdusing [readme-generator-for-helm] (https://github.com/bitnami/readme-generator-for-helm)