diff --git a/plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java b/plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java index 9313e5673f3..e3c3943d223 100644 --- a/plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java +++ b/plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java @@ -4440,16 +4440,13 @@ public abstract class CitrixResourceBase implements ServerResource, HypervisorRe cmd.setCluster(_cluster); cmd.setPoolSync(false); - Pool pool; try { - pool = Pool.getByUuid(conn, _host.pool); + Pool pool = Pool.getByUuid(conn, _host.pool); Pool.Record poolr = pool.getRecord(conn); - - Host.Record hostr = poolr.master.getRecord(conn); + poolr.master.getRecord(conn); } catch (Throwable e) { s_logger.warn("Check for master failed, failing the FULL Cluster sync command"); } - StartupStorageCommand sscmd = initializeLocalSR(conn); if (sscmd != null) { return new StartupCommand[] {cmd, sscmd}; diff --git a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java index df6ce454335..eca90805b8b 100755 --- a/server/src/com/cloud/configuration/ConfigurationManagerImpl.java +++ b/server/src/com/cloud/configuration/ConfigurationManagerImpl.java @@ -3964,8 +3964,9 @@ public class ConfigurationManagerImpl extends ManagerBase implements Configurati + " capabilities can be sepcified for source nat service"); } - for (Capability capability : sourceNatServiceCapabilityMap.keySet()) { - String value = sourceNatServiceCapabilityMap.get(capability); + for (Map.Entry srcNatPair : sourceNatServiceCapabilityMap.entrySet()) { + Capability capability = srcNatPair.getKey(); + String value = srcNatPair.getValue(); if (capability == Capability.SupportedSourceNatTypes) { boolean perAccount = value.contains("peraccount"); boolean perZone = value.contains("perzone"); @@ -4284,7 +4285,7 @@ public class ConfigurationManagerImpl extends ManagerBase implements Configurati @Override public Pair, Integer> searchForNetworkOfferings(ListNetworkOfferingsCmd cmd) { Boolean isAscending = Boolean.parseBoolean(_configDao.getValue("sortkey.algorithm")); - isAscending = (isAscending == null ? true : isAscending); + isAscending = (isAscending == null ? Boolean.TRUE : isAscending); Filter searchFilter = new Filter(NetworkOfferingVO.class, "sortKey", isAscending, null, null); Account caller = CallContext.current().getCallingAccount(); SearchCriteria sc = _networkOfferingDao.createSearchCriteria();