[odoo] startup script for openerp-gevent
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenERP buildout recipe |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
Odoo has a new script for aynchronous long polling, called openerp-gevent.
The recipe needs to produce the analogous one for its context.
The openerp-gevent script is automatically called from server.py if the --workers option is passed.
The recipe must find a way to have it launch its own generated script instead.
Repeating what I wrote earlier on https:/
* it has an hardcoded shebang on /usr/bin/env python
* it can't have the sys.path initialization that is the core principle of zc.buildout.
Basically, that openerp-gevent should have been the one generated by
setup.py (it looks very much alike) and should not be versioned with the sources. Besides, the
launcher should not make any assumptions on its name or path. I'll try
and explain that to the odoo team.
In the meanwhile, a monkey patch from bin/start_openerp should do the trick
Related branches
- Georges Racinet: Approve
-
Diff: 29 lines (+6/-5)1 file modifiedanybox/recipe/openerp/server.py (+6/-5)
Changed in anybox.recipe.openerp: | |
status: | Fix Committed → Fix Released |
Done ! The gevent server seems to start smoothly, and I could also change its port from buildout.cfg
This proves that the config is forwarded properly (saw the addons paths be correct, too).
Besides that, I did not perform functional tests, hence can't really tell if it works as expected.
On a funny note, this benefits from efforts done last summer to support OpenERP 5.0 in that there was already a known-to-work way to monkey patch during startup.
I think these changes can go to the stable branch as well, since (again) everything is protected by major version checks, and would appreciate user feedback