Douglas Schilling Landgraf has posted comments on this change.
Change subject: oVirt Node Upgrade: Support N configuration
......................................................................
Patch Set 20:
(2 comments)
Working to rebase the code.
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetoVirtISOsQuery.java
Line 85:
Line 86: boolean shouldAdd = false;
Line 87: RpmVersion isoVersion = new
RpmVersion(file.getName());
Line 88:
Line 89: String rpmParts[] =
RpmVersionUtils.splitRpmToParts(file.getName());
Done
Line 90: if (isoData.getVdsmCompatibilityVersion() !=
null) {
Line 91: shouldAdd =
isIsoCompatibleForUpgradeByClusterVersion(isoData);
Line 92: } else if (vdsOsVersion != null) {
Line 93: if
(VdsHandler.isIsoVersionCompatibleForUpgrade(vdsOsVersion, isoVersion)) {
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/OVirtNodeInfo.java
Line 31: return instance;
Line 32: }
Line 33:
Line 34: private OVirtNodeInfo() {
Line 35: final String delimiter = ":";
I think we talked that in previous reviews, it's more *nix like, use ":"
$ echo $PATH
/usr/lib64/qt-3.3/bin:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/home/douglas/.local/bin:/home/douglas/bin
Moved the delimeter to be a private static final field
Line 36: final String[] path =
Config.resolveOVirtISOsRepositoryPath().split(delimiter);
Line 37: final String minimumVersion[] = Config.<String>
GetValue(ConfigValues.OvirtInitialSupportedIsoVersion).split(delimiter);
Line 38:
Line 39: // Node prefix is part of regex to list ISOs (OvirtIsoPrefix)
--
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: 20
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Michael Burns <[email protected]>
Gerrit-Reviewer: Ravi Nori <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
