Martin Betak has uploaded a new change for review.

Change subject: core: Drop legacy SysPrep paths from config
......................................................................

core: Drop legacy SysPrep paths from config

Change-Id: I675340ae90aa724b935b0cf0a25544c06b0177eb
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1086768
Signed-off-by: Martin Betak <mbe...@redhat.com>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
M packaging/etc/engine-config/engine-config.properties
3 files changed, 10 insertions(+), 61 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/31/34831/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
index d938b9b..bd59128 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
@@ -277,34 +277,6 @@
     @TypeConverterAttribute(Integer.class)
     @DefaultValueAttribute("3")
     MaxRerunVmOnVdsCount,
-    @Reloadable
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
-    SysPrepXPPath,
-    @Reloadable
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
-    SysPrep2K3Path,
-    @Reloadable
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
-    SysPrep2K8Path,
-    @Reloadable
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
-    SysPrep2K8x64Path,
-    @Reloadable
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
-    SysPrep2K8R2Path,
-    @Reloadable
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
-    SysPrepWindows7Path,
-    @Reloadable
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
-    SysPrepWindows7x64Path,
     @TypeConverterAttribute(Integer.class)
     @DefaultValueAttribute("1000")
     MaxVmsInPool,
@@ -1291,23 +1263,11 @@
 
     @TypeConverterAttribute(String.class)
     @DefaultValueAttribute("")
-    SysPrepWindows8Path,
-
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
     ProductKeyWindows8,
 
     @TypeConverterAttribute(String.class)
     @DefaultValueAttribute("")
-    SysPrepWindows8x64Path,
-
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
     ProductKeyWindows8x64,
-
-    @TypeConverterAttribute(String.class)
-    @DefaultValueAttribute("")
-    SysPrepWindows2012x64Path,
 
     @TypeConverterAttribute(String.class)
     @DefaultValueAttribute("")
diff --git a/packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 
b/packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
index 7719bbb..447c8de 100644
--- a/packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
+++ b/packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
@@ -632,19 +632,8 @@
 select fn_db_add_config_value('SupportForceCreateVG','false','3.0');
 select fn_db_add_config_value('SupportForceExtendVG','false','3.0');
 select fn_db_add_config_value('SupportForceExtendVG','false','3.1');
-select 
fn_db_add_config_value('SysPrep2K3Path','/etc/ovirt-engine/sysprep/sysprep.2k3','general');
-select 
fn_db_add_config_value('SysPrep2K8Path','/etc/ovirt-engine/sysprep/sysprep.2k8x86','general');
-select 
fn_db_add_config_value('SysPrep2K8R2Path','/etc/ovirt-engine/sysprep/sysprep.2k8','general');
-select 
fn_db_add_config_value('SysPrep2K8x64Path','/etc/ovirt-engine/sysprep/sysprep.2k8','general');
 select fn_db_add_config_value('SysPrepDefaultPassword','','general');
 select fn_db_add_config_value('SysPrepDefaultUser','','general');
-select 
fn_db_add_config_value('SysPrepWindows7Path','/etc/ovirt-engine/sysprep/sysprep.w7','general');
-select 
fn_db_add_config_value('SysPrepWindows7x64Path','/etc/ovirt-engine/sysprep/sysprep.w7x64','general');
-select 
fn_db_add_config_value('SysPrepWindows8Path','/etc/ovirt-engine/sysprep/sysprep.w8','general');
-select 
fn_db_add_config_value('SysPrepWindows8x64Path','/etc/ovirt-engine/sysprep/sysprep.w8x64','general');
-select 
fn_db_add_config_value('SysPrepWindows2012x64Path','/etc/ovirt-engine/sysprep/sysprep.2k12x64','general');
---Handling Path to an XP machine Sys-Prep file.
-select 
fn_db_add_config_value('SysPrepXPPath','/etc/ovirt-engine/sysprep/sysprep.xp','general');
 select 
fn_db_add_config_value('ThrottlerMaxWaitForVdsUpdateInMillis','10000','general');
 select fn_db_add_config_value('TimeoutToResetVdsInSeconds','60','general');
 select fn_db_add_config_value('DelayResetForSpmInSeconds','20','general');
@@ -1161,6 +1150,16 @@
 select fn_db_delete_config_value('GlusterAysncTasksSupport','3.0,3.1,3.2,3.3');
 select fn_db_delete_config_value('NormalizedMgmgNetworkEnabled','3.0,3.1,3.2');
 select 
fn_db_delete_config_value('HotPlugCpuEnabled','3.0,3.1,3.2,3.3,3.4,3.5');
+select fn_db_delete_config_value('SysPrepXPPath','general');
+select fn_db_delete_config_value('SysPrep2K3Path','general');
+select fn_db_delete_config_value('SysPrep2K8Path','general');
+select fn_db_delete_config_value('SysPrep2K8R2Path','general');
+select fn_db_delete_config_value('SysPrep2K8x64Path','general');
+select fn_db_delete_config_value('SysPrepWindows7Path','general');
+select fn_db_delete_config_value('SysPrepWindows7x64Path','general');
+select fn_db_delete_config_value('SysPrepWindows8Path','general');
+select fn_db_delete_config_value('SysPrepWindows8x64Path','general');
+select fn_db_delete_config_value('SysPrepWindows2012x64Path','general');
 
------------------------------------------------------------------------------------
 --                  Split config section
 -- The purpose of this section is to treat config option that was once
diff --git a/packaging/etc/engine-config/engine-config.properties 
b/packaging/etc/engine-config/engine-config.properties
index 6e94968..78fe73f 100644
--- a/packaging/etc/engine-config/engine-config.properties
+++ b/packaging/etc/engine-config/engine-config.properties
@@ -153,16 +153,6 @@
 StorageDomainFailureTimeoutInMinutes.type=Integer
 StoragePoolRefreshTimeInSeconds.description="Storage Pool Manager Polling Rate 
(in seconds)"
 StoragePoolRefreshTimeInSeconds.type=Integer
-SysPrep2K3Path.description="Path to a Windows 2003 machine Sys-Prep file."
-SysPrep2K8Path.description="Path to a Windows 2008 machine Sys-Prep file."
-SysPrep2K8R2Path.description="Path to a Windows 2008 R2 machine Sys-Prep file."
-SysPrep2K8x64Path.description="Path to a Windows 2008 x64 machine Sys-Prep 
file."
-SysPrepWindows7Path.description="Path to a Windows 7 machine Sys-Prep file."
-SysPrepWindows7x64Path.description="Path to a Windows 7 x64 machine Sys-Prep 
file."
-SysPrepWindows8Path.description= "Path to a Windows 8 machine Sys-Prep file."
-SysPrepWindows8x64Path.description= "Path to a Windows 8 x64 machine Sys-Prep 
file."
-SysPrepWindows2012x64Path.description="Path to a Windows 2012 x64 machine 
Sys-Prep file."
-SysPrepXPPath.description="Path to a Windows XP machine Sys-Prep file."
 TimeoutToResetVdsInSeconds.description="Base Communication timeout in seconds 
before trying to reset a host"
 TimeoutToResetVdsInSeconds.type=Integer
 DelayResetForSpmInSeconds.description="Additional delay before reset due to a 
communication issue, when host is SPM"


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I675340ae90aa724b935b0cf0a25544c06b0177eb
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbe...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to