From 473700481e4fed58eda196df888f13543b66d113 Mon Sep 17 00:00:00 2001
From: Kelven Yang <kelven@cloud.com>
Date: Mon, 3 Jan 2011 14:38:17 -0800
Subject: [PATCH] Bug 7662: if capacity.check.period is given 0 or negative
 value, fall back to default value

---
 server/src/com/cloud/alert/AlertManagerImpl.java | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/server/src/com/cloud/alert/AlertManagerImpl.java b/server/src/com/cloud/alert/AlertManagerImpl.java
index 15ca0fda51f..7c412f57881 100644
--- a/server/src/com/cloud/alert/AlertManagerImpl.java
+++ b/server/src/com/cloud/alert/AlertManagerImpl.java
@@ -44,6 +44,7 @@ import org.apache.log4j.Logger;
 import com.cloud.alert.dao.AlertDao;
 import com.cloud.capacity.CapacityVO;
 import com.cloud.capacity.dao.CapacityDao;
+import com.cloud.configuration.Config;
 import com.cloud.configuration.dao.ConfigurationDao;
 import com.cloud.dc.DataCenterVO;
 import com.cloud.dc.HostPodVO;
@@ -175,7 +176,9 @@ public class AlertManagerImpl implements AlertManager {
 
         String capacityCheckPeriodStr = configs.get("capacity.check.period");
         if (capacityCheckPeriodStr != null) {
-            _capacityCheckPeriod = Long.parseLong(capacityCheckPeriodStr);
+            _capacityCheckPeriod = Long.parseLong(capacityCheckPeriodStr);
+            if(_capacityCheckPeriod <= 0)
+            	_capacityCheckPeriod = Long.parseLong(Config.CapacityCheckPeriod.getDefaultValue());
         }
         
         String cpuOverProvisioningFactorStr = configs.get("cpu.overprovisioning.factor");