To be a little more clear about what I propose: - pre- or post scripts may have an overall check whether they apply to the current state of the database. This makes a script robust against being called multiple times, so that it can simply exit while logging a warning.
- if rename_columns detects a database layout inconsistent with its arguments, this should raise a fatal error. -- You received this bug notification because you are a member of OpenUpgrade Committers, which is the registrant for OpenUpgrade Server. https://bugs.launchpad.net/bugs/953468 Title: rename_columns should have extra checks Status in OpenUpgrade Server: Confirmed Bug description: - check that table_exists - check that target column does not exist - check that source column exist and return: True if the rename is actually performed False otherwise Maybe some added verbose logging. To manage notifications about this bug go to: https://bugs.launchpad.net/openupgrade-server/+bug/953468/+subscriptions -- Mailing list: https://launchpad.net/~credativ Post to : [email protected] Unsubscribe : https://launchpad.net/~credativ More help : https://help.launchpad.net/ListHelp

