Code review comment for lp://staging/~camptocamp/anybox.buildbot.openerp/trunk-fix_994523-afe

Revision history for this message
Georges Racinet (gracinet) wrote :

I agree with the usage of psql in that context. I'm doing exactly the same to manage running instances : not depend on psycopg2 for such simple requests to avoid any trouble.

Did you however check that all the relevant environmental variables (PGPORT notably) are passed correcly in that step ? We're using lots of builds on slaves with multiple PostgreSQL versions.

About 'openerp-addons = all', all it does is transmit to the main server script (in the disguise of bin/test_openerp) within the '-i' option, so what you're seeing is a mere reflection of the behaviour of OpenERP itself.

« Back to merge proposal