diff options
| author | Shireesh Anjal <shireesh@gluster.com> | 2011-08-22 20:44:39 +0530 |
|---|---|---|
| committer | Shireesh Anjal <shireesh@gluster.com> | 2011-08-22 20:45:24 +0530 |
| commit | 847bb5d1ee2e64427ba8a34275de826270ea12af (patch) | |
| tree | 9d09f8f1de43496764693674fcd972249e27e3eb | |
| parent | adbf8e51dd5d2eadd98823d42f8842d9bd6910af (diff) | |
Modified to fetch network statistics of a server only if it is online
| -rw-r--r-- | src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/utils/NetworkStatsFactory.java | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/utils/NetworkStatsFactory.java b/src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/utils/NetworkStatsFactory.java index efac10fb..fdc4b1d2 100644 --- a/src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/utils/NetworkStatsFactory.java +++ b/src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/utils/NetworkStatsFactory.java @@ -54,6 +54,13 @@ public class NetworkStatsFactory extends AbstractStatsFactory { String serverName = serverNames.get(i); Server server = new Server(serverName); serverUtil.fetchServerDetails(server); + if(!server.isOnline()) { + if(removeServerOnError) { + // server is offline. no point in trying to fetch it's details. + serverNames.remove(serverName); + } + continue; + } try { for(NetworkInterface networkInterface : server.getNetworkInterfaces()) { ServerStats stats = fetchStats(serverName, period, networkInterface.getName()); |
