Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Added generic DB validations before the upgrade ......................................................................
packaging: Added generic DB validations before the upgrade * This patch adds a simple validatedb.sh script that will run required validation scripts. * When a new validation script is added, it should be added to the validatedb runner script as well. * During the upgrade, this script is executed and handles the validations control to corresponding scripts. * If user decides not to clear DB issues or there's any other error, the exception is raised. * Patch also introduces some code cleanups Change-Id: Ief0e65231bb42b569734cdd2fa6491b645491f98 Bug-Url: https://bugzilla.redhat.com/893459 Signed-off-by: Alex Lourie <[email protected]> --- M Makefile M backend/manager/tools/dbutils/fkvalidator.sh M backend/manager/tools/dbutils/fkvalidator_sp.sql A backend/manager/tools/dbutils/validatedb.sh M packaging/fedora/setup/basedefs.py M packaging/fedora/setup/engine-upgrade.py M packaging/fedora/spec/ovirt-engine.spec.in 7 files changed, 200 insertions(+), 25 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12512 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief0e65231bb42b569734cdd2fa6491b645491f98 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alex Lourie <[email protected]> Gerrit-Reviewer: Alex Lourie <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Moran Goldboim <[email protected]> Gerrit-Reviewer: Ofer Schreiber <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
