Procurement pre-migration fails when stock is not installed. about openupgrade HOT 8 CLOSED

BTW, Why does the procurement pre-migration is ran if it's not installed in my DB. Is this normal?

from openupgrade.

Well, I took a peek in the loading.py module to see why and now I know:

$ psql -d mercurio8 -c "SELECT name, state FROM ir_module_module WHERE state!='uninstalled'" | grep "\(pro\|sto\)"
 project                      | to upgrade
 procurement                  | to install
 project_issue                | to upgrade
 product                      | installed
 process                      | to upgrade
 crm_profiling                | to upgrade

Procurement, which is auto-installabled, now depends solely on base and product, and since their both installed, it is automatically marked to be installed. I removed the 8.0-procurement-fixes branch for the moment till I know for sure how to deal with this.

from openupgrade.

Strange thing is, that the migrate decorator from openerp/openupgrade/openupgrade.py checks for version and does not run the migration if it is not set. That should be the case for a new installation. Can you check what it gets at that point?

from openupgrade.

Hi @StefanRijnhart,

I see no check about this in the decorator. Currently the migrate decorator simply calls the decorated function. No questions asked.

from openupgrade.

I have reworked the branch which moves this past this issue: https://github.com/mvaled/OpenUpgrade/tree/8.0-procurement-fixes

from openupgrade.

Opps. I didn't get you exactly. It checks the version, not the state.... Let's try to inspect this.

from openupgrade.

Well, now it simply does not get executed: version is None. It seems I did something I can't recall now that let the DB in an unstable state. Closing this now.

from openupgrade.

Thanks for the follow-up!

from openupgrade.