Change in ovirt-engine[master]: core: db - prevent installing same script content twice

Eli Mesika has submitted this change and it was merged. Change subject: core: db - prevent installing same script content twice ...................................................................... core: db - prevent installing same script content twice This patch should resolve the following scenario: Example: database last installed script is 03010130 latest database version file in repository 03010250 There is a need to install 03010250 without the 03010140 - 03010240 scripts because 03010140 - 03010240 deals with new feature A and 03010250 deals with new feature B and we currently want to skip feature A changes from some reason (stability, etc.) The way to do that is to copy 03010250 file as 03010131 and run the upgrade regularly (you will not be able to run the original 03010250 directly without this change) However, later we want to take feature A and all latest scripts , we have again 03010250 Running it again may lead to an ERROR if we are lucky (since upgrade scrips are not reentrant) or to data corruption in the worst case. This patch ensure that 03010250 will be skipped and a comment will be inserted to the schema_verson table (in a new comment column added to that table in the pre_upgrade step) saying that 03010250 state is SKIPPED since it was already installed by 03010131. The patch is based on checking the file md5 and comparing it to the md5 of installed scripts inside the schema_version table. Change-Id: Id41a4b9406a9e60de783f779c9a51e5de7ce7676 Signed-off-by: Eli Mesika <emesika(a)redhat.com&gt; --- M backend/manager/dbscripts/dbfunctions.sh A backend/manager/dbscripts/upgrade/pre_upgrade/add_comment_col_to_schema_version.sql 2 files changed, 30 insertions(+), 11 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4169 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id41a4b9406a9e60de783f779c9a51e5de7ce7676 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com&gt; Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com&gt; Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com&gt; Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com&gt;