Eli Mesika has posted comments on this change.

Change subject: core: Support for noTSX and Broadwell processors
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/41592/1/packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
File packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql:

Line 902: select 
fn_db_update_default_config_value('VdsmSSLProtocol','SSLv3','TLSv1','general', 
false);
Line 903: select 
fn_db_update_default_config_value('ExternalCommunicationProtocol','SSLv3','TLSv1','general',
 false);
Line 904: 
Line 905: -- Adding no-TSX Intel processors
Line 906: select fn_db_update_default_config_value('ServerCPUList', '3:Intel 
Conroe Family:vmx,nx,model_Conroe:Conroe:x86_64; 4:Intel Penryn 
Family:vmx,nx,model_Penryn:Penryn:x86_64; 5:Intel Nehalem 
Family:vmx,nx,model_Nehalem:Nehalem:x86_64; 6:Intel Westmere 
Family:aes,vmx,nx,model_Westmere:Westmere:x86_64; 7:Intel SandyBridge 
Family:vmx,nx,model_SandyBridge:SandyBridge:x86_64; 8:Intel Haswell 
Family:vmx,nx,model_Haswell:Haswell:x86_64; 2:AMD Opteron 
G1:svm,nx,model_Opteron_G1:Opteron_G1:x86_64; 3:AMD Opteron 
G2:svm,nx,model_Opteron_G2:Opteron_G2:x86_64; 4:AMD Opteron 
G3:svm,nx,model_Opteron_G3:Opteron_G3:x86_64; 5:AMD Opteron 
G4:svm,nx,model_Opteron_G4:Opteron_G4:x86_64; 6:AMD Opteron 
G5:svm,nx,model_Opteron_G5:Opteron_G5:x86_64; 3:IBM POWER 
8:powernv,model_power8:power8:ppc64;', '3:Intel Conroe 
Family:vmx,nx,model_Conroe:Conroe:x86_64; 4:Intel Penryn 
Family:vmx,nx,model_Penryn:Penryn:x86_64; 5:Intel Nehalem 
Family:vmx,nx,model_Nehalem:Nehalem:x86_64; 6:Intel Westmere Fami!
 ly:aes,vmx,nx,model_Westmere:Westmere:x86_64; 7:Intel SandyBridge 
Family:vmx,nx,model_SandyBridge:SandyBridge:x86_64; 8:Intel Haswell-noTSX 
Family:vmx,nx,model_Haswell-noTSX:Haswell-noTSX:x86_64; 9:Intel Haswell 
Family:vmx,nx,model_Haswell:Haswell:x86_64; 10:Intel Broadwell-noTSX 
Family:vmx,nx,model_Broadwell-noTSX:Broadwell-noTSX:x86_64; 11:Intel Broadwell 
Family:vmx,nx,model_Broadwell:Broadwell:x86_64; 2:AMD Opteron 
G1:svm,nx,model_Opteron_G1:Opteron_G1:x86_64; 3:AMD Opteron 
G2:svm,nx,model_Opteron_G2:Opteron_G2:x86_64; 4:AMD Opteron 
G3:svm,nx,model_Opteron_G3:Opteron_G3:x86_64; 5:AMD Opteron 
G4:svm,nx,model_Opteron_G4:Opteron_G4:x86_64; 6:AMD Opteron 
G5:svm,nx,model_Opteron_G5:Opteron_G5:x86_64; 3:IBM POWER 
8:powernv,model_power8:power8:ppc64;', '3.5', FALSE);
You are creating the new value for 3.5 and 3.6 but updating only to 3.5 
that means that dev environments with existing 3.6 settings will not be 
affected, are you going to communicate that fact or you assume that nobody from 
dev will have an issue with  that 

Just want to be sure :-)
Line 907: 
Line 908: 
Line 909: 
------------------------------------------------------------------------------------
Line 910: --              Cleanup deprecated configuration values section


-- 
To view, visit https://gerrit.ovirt.org/41592
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I678e153f82e529d710f09cfad7029a75d36d2478
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jakub Niedermertl <jnied...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to