etan39
/
cloudstack
mirror of https://github.com/apache/cloudstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

bug 11938: Remove thread.sleep() from HostCapacityChecker. cpu and memory capacities can be null for host in maintenance so check for that, wrap the host capacity checker thread with try and catch to make it more robust. Add more logging for entry and exit of thread. 

status 11938: resolved fixed
reviewed-by: Abhi
This commit is contained in:
root 2011-11-16 20:07:49 +05:30
parent e790059a34
commit 1db4e8b3f5
1 changed files with 21 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
server/src/com/cloud/capacity/CapacityManagerImpl.java
View File

@ -108,8 +108,8 @@ public class CapacityManagerImpl implements CapacityManager, StateListener<State
}
@Override
public boolean start() {
_executor.schedule(new HostCapacityCollector(), _hostCapacityCheckerDelay, TimeUnit.SECONDS);
public boolean start() {
_executor.scheduleWithFixedDelay(new HostCapacityCollector(), _hostCapacityCheckerDelay, _hostCapacityCheckerInterval, TimeUnit.SECONDS);
return true;
}
@ -406,13 +406,10 @@ public class CapacityManagerImpl implements CapacityManager, StateListener<State
@Override
public void run() {
while (!_stopped) {
try {
Thread.sleep(_hostCapacityCheckerInterval * 1000);
} catch (InterruptedException e1) {
}
// get all hosts...even if they are not in 'UP' state
try{
s_logger.debug("Starting host capacity checker .....");
// get all hosts...even if they are not in 'UP' state
List<HostVO> hosts = _hostDao.listByType(Host.Type.Routing);
// prep the service offerings
@ -455,6 +452,11 @@ public class CapacityManagerImpl implements CapacityManager, StateListener<State
CapacityVO cpuCap = _capacityDao.findByHostIdType(host.getId(), CapacityVO.CAPACITY_TYPE_CPU);
CapacityVO memCap = _capacityDao.findByHostIdType(host.getId(), CapacityVO.CAPACITY_TYPE_MEMORY);
if(cpuCap == null || memCap == null){
s_logger.debug("Didnt find cpu/memory capacity for host " + host.getId() + ", skipping it");
continue;
}
if (cpuCap.getUsedCapacity() == usedCpu && cpuCap.getReservedCapacity() == reservedCpu) {
s_logger.debug("No need to calibrate cpu capacity, host:" + host.getId() + " usedCpu: " + cpuCap.getUsedCapacity()
@ -486,16 +488,16 @@ public class CapacityManagerImpl implements CapacityManager, StateListener<State
memCap.setUsedCapacity(usedMemory);
}
try {
_capacityDao.update(cpuCap.getId(), cpuCap);
_capacityDao.update(memCap.getId(), memCap);
} catch (Exception e) {
}
}
}
}
_capacityDao.update(cpuCap.getId(), cpuCap);
_capacityDao.update(memCap.getId(), memCap);
}
s_logger.debug("Done running host capacity checker ....");
}catch (Throwable t) {
s_logger.error("Caught an exception while running host capacity checker", t);
}
}
}
@Override