Eli Mesika has submitted this change and it was merged.

Change subject: core: rename upgrade functions in db upgrade scripts
......................................................................


core: rename upgrade functions in db upgrade scripts

Removing the version suffix from function names in upgrade scripts.
The reasons are
1) Each time that you have to rename your upgrade script after sync with master 
you have to rename the function as well
2) As a result, may function are pushed with a wrong suffix (for example 
upgrade 03_01_0950 had a function name suffix of 08200)

Since the reason for that numbering was in the first place for avoiding 
dropping real installed functions, now you will have just to
put in a temporary function created in an upgrade script just the prefix 
"__temp_"

Hope that this will make handling upgrade scripts more accurate and less 
painful.

Change-Id: I73db2cf50776488b603d62afcee4033573c6092a
Signed-off-by: Eli Mesika <[email protected]>
---
M backend/manager/dbscripts/upgrade/03_00_0000_add_schema_version.sql
M backend/manager/dbscripts/upgrade/03_00_0300_add_local_admin_user.sql
M backend/manager/dbscripts/upgrade/03_01_0050_add_disks_table.sql
M backend/manager/dbscripts/upgrade/03_01_0070_populate_disks_table.sql
M 
backend/manager/dbscripts/upgrade/03_01_0100_migrate_templates_to_vm_static.sql
M backend/manager/dbscripts/upgrade/03_01_0170_add_quota_tables.sql
M backend/manager/dbscripts/upgrade/03_01_0200_add_vm_device_table.sql
M backend/manager/dbscripts/upgrade/03_01_0220_vm_device_upgrade_data.sql
M backend/manager/dbscripts/upgrade/03_01_0260_add_job_table.sql
M backend/manager/dbscripts/upgrade/03_01_0270_add_job_subject_entity_table.sql
M backend/manager/dbscripts/upgrade/03_01_0280_add_step_table.sql
M backend/manager/dbscripts/upgrade/03_01_0370_vm_device_upgrade_data.sql
M backend/manager/dbscripts/upgrade/03_01_0390_add_snapshots_table.sql
M backend/manager/dbscripts/upgrade/03_01_0430_add_image_domain_map_table.sql
M backend/manager/dbscripts/upgrade/03_01_0440_vm_device_upgrade_data.sql
M backend/manager/dbscripts/upgrade/03_01_0520_quota_upgrade_script.sql
M backend/manager/dbscripts/upgrade/03_01_0560_vm_device_upgrade_data.sql
M backend/manager/dbscripts/upgrade/03_01_0600_migrate_images_to_snapshots.sql
M backend/manager/dbscripts/upgrade/03_01_0690_add_gluster_volume_tables.sql
M backend/manager/dbscripts/upgrade/03_01_0730_add_empty_cd_to_vm.sql
M backend/manager/dbscripts/upgrade/03_01_0740_add_vm_device_soundcard.sql
M backend/manager/dbscripts/upgrade/03_01_0850_add_gluster_transport_types.sql
M backend/manager/dbscripts/upgrade/03_01_0930_convert_spec_param_to_json.sql
M backend/manager/dbscripts/upgrade/03_01_0940_add_disks_roles.sql
M 
backend/manager/dbscripts/upgrade/03_01_0950_add_permissions_on_storage_to_users.sql
M backend/manager/dbscripts/upgrade/03_01_1000_gluster_role_maps.sql
26 files changed, 242 insertions(+), 242 deletions(-)

Approvals:
  Eli Mesika: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I73db2cf50776488b603d62afcee4033573c6092a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to