Roy Golan has posted comments on this change.

Change subject: core: adding support for version 3.4
......................................................................


Patch Set 2:

(4 comments)

....................................................
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VersionStorageFormatUtil.java
Line 56:                     put(Version.v3_0, new V2FormatTypeMapper());
Line 57:                     put(Version.v3_1, new 
ConstantStorageFormatTypeMapper(StorageFormatType.V3));
Line 58:                     put(Version.v3_2, new 
ConstantStorageFormatTypeMapper(StorageFormatType.V3));
Line 59:                     put(Version.v3_3, new 
ConstantStorageFormatTypeMapper(StorageFormatType.V3));
Line 60:                     put(Version.v3_4, new 
ConstantStorageFormatTypeMapper(StorageFormatType.V3));
VErsion needs to be an enum. will send a patch above yours
Line 61:                 }
Line 62:             };
Line 63: 
Line 64:     public static StorageFormatType getPreferredForVersion(Version v, 
StorageType type) {


....................................................
File packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
Line 71: select 
fn_db_add_config_value('ClusterEmulatedMachines','rhel6.2.0,pc-1.0','3.0');
Line 72: select 
fn_db_add_config_value('ClusterEmulatedMachines','rhel6.3.0,pc-1.0','3.1');
Line 73: select 
fn_db_add_config_value('ClusterEmulatedMachines','rhel6.4.0,pc-1.0','3.2');
Line 74: select 
fn_db_add_config_value('ClusterEmulatedMachines','rhel6.4.0,pc-1.0,pseries','3.3');
Line 75: select 
fn_db_add_config_value('ClusterEmulatedMachines','rhel6.4.0,pc-1.0,pseries','3.4');
this should be rhel6.5.0 - need to verify RHEL 6.5 will really see light with 
3.4
Line 76: select 
fn_db_add_config_value('CpuOverCommitDurationMinutes','2','general');
Line 77: --Handling Data directory for ENGINE
Line 78: select fn_db_add_config_value('DataDir','/usr/share/engine','general');
Line 79: select fn_db_add_config_value('DBEngine','Postgres','general');


Line 89: select 
fn_db_add_config_value('DesktopAudioDeviceType','WindowsXP,ac97,RHEL4,ac97,RHEL3,ac97,Windows2003x64,ac97,RHEL4x64,ac97,RHEL3x64,ac97,OtherLinux,ac97,Other,ac97,default,ich6','3.0');
Line 90: select 
fn_db_add_config_value('DesktopAudioDeviceType','WindowsXP,ac97,RHEL4,ac97,RHEL3,ac97,Windows2003x64,ac97,RHEL4x64,ac97,RHEL3x64,ac97,OtherLinux,ac97,Other,ac97,default,ich6','3.1');
Line 91: select 
fn_db_add_config_value('DesktopAudioDeviceType','WindowsXP,ac97,RHEL4,ac97,RHEL3,ac97,Windows2003x64,ac97,RHEL4x64,ac97,RHEL3x64,ac97,OtherLinux,ac97,Other,ac97,default,ich6','3.2');
Line 92: select 
fn_db_add_config_value('DesktopAudioDeviceType','WindowsXP,ac97,RHEL4,ac97,RHEL3,ac97,Windows2003x64,ac97,RHEL4x64,ac97,RHEL3x64,ac97,OtherLinux,ac97,Other,ac97,default,ich6','3.3');
Line 93: select 
fn_db_add_config_value('DesktopAudioDeviceType','WindowsXP,ac97,RHEL4,ac97,RHEL3,ac97,Windows2003x64,ac97,RHEL4x64,ac97,RHEL3x64,ac97,OtherLinux,ac97,Other,ac97,default,ich6','3.4');
this can be ommited. OSinfo is now talking care of all of that/.

actually all this config value can go down the sink hole... :)
Line 94: select 
fn_db_add_config_value('DisableFenceAtStartupInSec','300','general');
Line 95: select fn_db_add_config_value('DirectLUNDiskEnabled','false','3.0');
Line 96: --Handling NetBIOS Domain Name
Line 97: select fn_db_add_config_value('DomainName','example.com','general');


Line 519: select 
fn_db_add_config_value('VM64BitMaxMemorySizeInMB','524288','3.0');
Line 520: select 
fn_db_add_config_value('VM64BitMaxMemorySizeInMB','2097152','3.1');
Line 521: select 
fn_db_add_config_value('VM64BitMaxMemorySizeInMB','2097152','3.2');
Line 522: select 
fn_db_add_config_value('VM64BitMaxMemorySizeInMB','2097152','3.3');
Line 523: select 
fn_db_add_config_value('VM64BitMaxMemorySizeInMB','2097152','3.4');
also moved to osinfo. pls ommit
Line 524: select fn_db_add_config_value('VmGracefulShutdownMessage','System 
Administrator has initiated shutdown of this Virtual Machine. Virtual Machine 
is shutting down.','general');
Line 525: select 
fn_db_add_config_value('VmGracefulShutdownTimeout','30','general');
Line 526: select fn_db_add_config_value('VMMinMemorySizeInMB','256','general');
Line 527: --Number of subsequent failures in VM creation in a pool before 
giving up and stop creating new VMs


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7e26d21f0c3696017f3e70ba10079a06872a8442
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com>
Gerrit-Reviewer: Martin Peřina <mper...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Roy Golan <rgo...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to