Douglas Schilling Landgraf has posted comments on this change.

Change subject: oVirt Node Upgrade: Support N configuration
......................................................................


Patch Set 16:

(5 comments)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetoVirtISOsQuery.java
Line 44: 
Line 45:         RpmVersion vdsOsVersion = 
VdsHandler.getOvirtHostOsVersion(vds);
Line 46:         String nodeOS = vds.getHostOs();
Line 47:         String nodeOsSupported = null;
Line 48:         Matcher matcher;
To avoid the error: cannot find symbol
Line 49: 
Line 50:         for (OVirtNodeInfo.Entry info : 
OVirtNodeInfo.getInstance().get()) {
Line 51: 
Line 52:             try {


Line 54:                 log.infoFormat("info.nodePattern [{0}]", 
info.nodePattern);
Line 55:                 matcher = 
info.nodePattern.matcher(nodeOS.toLowerCase());
Line 56:                 matcher.find();
Line 57:                 nodeOsSupported = matcher.group(0);
Line 58:                 log.infoFormat("nodeOS [{0}] supported node OS [{1}]", 
nodeOS.toLowerCase(), nodeOsSupported);
Done
Line 59:             } catch (RuntimeException e) {
Line 60:                 log.infoFormat("Cannot match nodeOS [{0}]]", 
nodeOS.toLowerCase());
Line 61:                 continue;
Line 62:             }


....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/OVirtNodeInfo.java
Line 15:     public static class Entry {
Line 16:         public Pattern isoPattern;
Line 17:         public File path;
Line 18:         public String minimumVersion;
Line 19:         public Pattern nodePattern;
Done
Line 20:     }
Line 21: 
Line 22:     private static final Log log = 
LogFactory.getLog(OVirtNodeInfo.class);
Line 23:     private static volatile OVirtNodeInfo instance;


....................................................
File 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
Line 1461:     @DefaultValueAttribute("3")
Line 1462:     IterationsWithBalloonProblem(525),
Line 1463: 
Line 1464:     @TypeConverterAttribute(String.class)
Line 1465:     @DefaultValueAttribute("^ovirt.*:^rhev.*")
Looks like my rebase failed, Done
Line 1466:     oVirtNode(525),
Line 1467: 
Line 1468:     Invalid(65535);
Line 1469: 


....................................................
File packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
Line 369: select 
fn_db_add_config_value('OvfVirtualSystemType','ENGINE','general');
Line 370: --Handling The ovirt-node installation files path
Line 371: select 
fn_db_add_config_value('OvirtInitialSupportedIsoVersion','2.5.5:5.8','general');
Line 372: select 
fn_db_add_config_value('OvirtIsoPrefix','^ovirt-node-iso-(.*)\.*\.iso$:^rhevh-(.*)\.*\.iso$','general');
Line 373: select 
fn_db_add_config_value('oVirtNode','^ovirt.*:^rhev.*','general');
Done
Line 374: select 
fn_db_add_config_value('oVirtISOsRepositoryPath','/usr/share/ovirt-node-iso:/usr/share/rhev-hypervisor','general');
Line 375: select 
fn_db_add_config_value('oVirtUpgradeScriptName','/usr/share/vdsm-reg/vdsm-upgrade','general');
Line 376: select 
fn_db_add_config_value('oVirtUploadPath','/data/updates/ovirt-node-image.iso','general');
Line 377: select 
fn_db_add_config_value('OvfUpdateIntervalInMinutes','60','general');


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibfb9dc5d0dc8780b519107acbe0ae866831f782c
Gerrit-PatchSet: 16
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsl...@redhat.com>
Gerrit-Reviewer: Michael Burns <mbu...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@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