Change in ovirt-engine[master]: core: truncate white list upon upgrade

yzaslavs at redhat.com yzaslavs at redhat.com
Mon Oct 15 07:15:19 UTC 2012


Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: truncate white list upon upgrade
......................................................................


core: truncate white list upon upgrade

As white-list data is internal configuration
and is not populated due to user activity, it is
possible to truncate the object_column_white_list_table per
upgrade (as post upgrade step).
This fix was introduced due to a scneario in which sshFingerPrint
key was added vds_static (and vds view) and due to the reentrant nature
of add_object_colmn_white_list_table, it was not added to white list,
as the check of "if exists" was done on the whole table.
It is possible to perform "if exist" check per column, but it looks like
an overhead , as explained above

Change-Id: I94f3c474c0209a9049d1d32487d049e1e161214a
Signed-off-by: Yair Zaslavsky <yzaslavs at redhat.com>
---
M backend/manager/dbscripts/upgrade/post_upgrade/0010_add_object_column_white_list_table.sql
1 file changed, 3 insertions(+), 4 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/8272
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I94f3c474c0209a9049d1d32487d049e1e161214a
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list