You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by wi...@apache.org on 2014/09/24 10:01:24 UTC
git commit: updated refs/heads/statscollector-graphite to f7de57d
Repository: cloudstack
Updated Branches:
refs/heads/statscollector-graphite aa78e5709 -> f7de57d92 (forced update)
CLOUDSTACK-7583: Send VmStats to Graphite host when configured
This allows external processing of VmStats information without using
the usage server of CloudStack
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/f7de57d9
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/f7de57d9
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/f7de57d9
Branch: refs/heads/statscollector-graphite
Commit: f7de57d92aadc01f605873ccb4652eeea15ebba6
Parents: 6687727
Author: Wido den Hollander <wi...@widodh.nl>
Authored: Fri Sep 19 16:58:00 2014 +0200
Committer: Wido den Hollander <wi...@42on.com>
Committed: Wed Sep 24 10:01:11 2014 +0200
----------------------------------------------------------------------
server/src/com/cloud/configuration/Config.java | 7 +-
server/src/com/cloud/server/StatsCollector.java | 42 ++++++
setup/db/db/schema-441to450.sql | 3 +
.../utils/graphite/GraphiteClient.java | 129 +++++++++++++++++++
.../utils/graphite/GraphiteException.java | 31 +++++
5 files changed, 211 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/f7de57d9/server/src/com/cloud/configuration/Config.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/configuration/Config.java b/server/src/com/cloud/configuration/Config.java
index 9309e3a..1aa8d3b 100755
--- a/server/src/com/cloud/configuration/Config.java
+++ b/server/src/com/cloud/configuration/Config.java
@@ -2057,7 +2057,12 @@ public enum Config {
PublishAlertEvent("Advanced", ManagementServer.class, Boolean.class, "publish.alert.events", "true", "enable or disable publishing of alert events on the event bus", null),
PublishResourceStateEvent("Advanced", ManagementServer.class, Boolean.class, "publish.resource.state.events", "true", "enable or disable publishing of alert events on the event bus", null),
PublishUsageEvent("Advanced", ManagementServer.class, Boolean.class, "publish.usage.events", "true", "enable or disable publishing of usage events on the event bus", null),
- PublishAsynJobEvent("Advanced", ManagementServer.class, Boolean.class, "publish.async.job.events", "true", "enable or disable publishing of usage events on the event bus", null);
+ PublishAsynJobEvent("Advanced", ManagementServer.class, Boolean.class, "publish.async.job.events", "true", "enable or disable publishing of usage events on the event bus", null),
+
+ // StatsCollector
+ StatsOutPutGraphiteHost("Advanced", ManagementServer.class, String.class, "stats.output.graphite.host", "", "Graphite host to send statistics to", null),
+ StatsOutPutGraphitePort("Advanced", ManagementServer.class, Integer.class, "stats.output.graphite.port", "2003", "The port of the graphite host to send statistics to", null),
+ StatsOutPutGraphitePrefix("Advanced", ManagementServer.class, String.class, "stats.output.graphite.prefix", "", "Prefix to append to statistics going to Graphite", null);
private final String _category;
private final Class<?> _componentClass;
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/f7de57d9/server/src/com/cloud/server/StatsCollector.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/server/StatsCollector.java b/server/src/com/cloud/server/StatsCollector.java
index 9f4c14e..8de94d4 100755
--- a/server/src/com/cloud/server/StatsCollector.java
+++ b/server/src/com/cloud/server/StatsCollector.java
@@ -43,6 +43,8 @@ import org.apache.cloudstack.managed.context.ManagedContextRunnable;
import org.apache.cloudstack.storage.datastore.db.ImageStoreDao;
import org.apache.cloudstack.storage.datastore.db.PrimaryDataStoreDao;
import org.apache.cloudstack.storage.datastore.db.StoragePoolVO;
+import org.apache.cloudstack.graphite.GraphiteClient;
+import org.apache.cloudstack.graphite.GraphiteException;
import com.cloud.agent.AgentManager;
import com.cloud.agent.api.Answer;
@@ -194,6 +196,10 @@ public class StatsCollector extends ManagerBase implements ComponentMethodInterc
int vmDiskStatsInterval = 0;
List<Long> hostIds = null;
+ String graphiteHost = null;
+ int graphitePort = -1;
+ String graphitePrefix = null;
+
private ScheduledExecutorService _diskStatsUpdateExecutor;
private int _usageAggregationRange = 1440;
private String _usageTimeZone = "GMT";
@@ -233,6 +239,14 @@ public class StatsCollector extends ManagerBase implements ComponentMethodInterc
autoScaleStatsInterval = NumbersUtil.parseLong(configs.get("autoscale.stats.interval"), 60000L);
vmDiskStatsInterval = NumbersUtil.parseInt(configs.get("vm.disk.stats.interval"), 0);
+ graphiteHost = configs.get("stats.output.graphite.host");
+ graphitePort = NumbersUtil.parseInt(configs.get("stats.output.graphite.port"), 2003);
+ graphitePrefix = configs.get("stats.output.graphite.prefix");
+
+ if (!graphitePrefix.equals("")) {
+ graphitePrefix += ".";
+ }
+
if (hostStatsInterval > 0) {
_executor.scheduleWithFixedDelay(new HostCollector(), 15000L, hostStatsInterval, TimeUnit.MILLISECONDS);
}
@@ -372,6 +386,10 @@ public class StatsCollector extends ManagerBase implements ComponentMethodInterc
sc.addAnd("type", SearchCriteria.Op.NEQ, Host.Type.SecondaryStorageVM.toString());
List<HostVO> hosts = _hostDao.search(sc, null);
+ if (!graphiteHost.equals("")) {
+ s_logger.debug("Sending VmStats to Graphite host " + graphiteHost + ":" + graphitePort);
+ }
+
for (HostVO host : hosts) {
List<UserVmVO> vms = _userVmDao.listRunningByHostId(host.getId());
List<Long> vmIds = new ArrayList<Long>();
@@ -407,6 +425,30 @@ public class StatsCollector extends ManagerBase implements ComponentMethodInterc
_VmStats.put(vmId, statsInMemory);
}
+
+ if (!graphiteHost.equals("")) {
+ VMInstanceVO vmVO = _vmInstance.findById(vmId);
+ String vmName = vmVO.getInstanceName();
+
+ HashMap metrics = new HashMap<String, Integer>();
+
+ metrics.put(graphitePrefix + "cloudstack.stats.instances." + vmName + ".cpu.num", statsForCurrentIteration.getNumCPUs());
+ metrics.put(graphitePrefix + "cloudstack.stats.instances." + vmName + ".cpu.utilization", statsForCurrentIteration.getCPUUtilization());
+ metrics.put(graphitePrefix + "cloudstack.stats.instances." + vmName + ".network.read_kbs", statsForCurrentIteration.getNetworkReadKBs());
+ metrics.put(graphitePrefix + "cloudstack.stats.instances." + vmName + ".network.write_kbs", statsForCurrentIteration.getNetworkWriteKBs());
+ metrics.put(graphitePrefix + "cloudstack.stats.instances." + vmName + ".disk.write_kbs", statsForCurrentIteration.getDiskWriteKBs());
+ metrics.put(graphitePrefix + "cloudstack.stats.instances." + vmName + ".disk.read_kbs", statsForCurrentIteration.getDiskReadKBs());
+ metrics.put(graphitePrefix + "cloudstack.stats.instances." + vmName + ".disk.write_iops", statsForCurrentIteration.getDiskWriteIOs());
+ metrics.put(graphitePrefix + "cloudstack.stats.instances." + vmName + ".disk.read_iops", statsForCurrentIteration.getDiskReadIOs());
+
+ try {
+ GraphiteClient g = new GraphiteClient(graphiteHost, graphitePort);
+ g.sendMetrics(metrics);
+ } catch (GraphiteException e) {
+ s_logger.debug("Failed sending VmStats of " + vmName + " to Graphite host " + graphiteHost + ":" + graphitePort + ": " + e.getMessage());
+ }
+ }
+
}
}
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/f7de57d9/setup/db/db/schema-441to450.sql
----------------------------------------------------------------------
diff --git a/setup/db/db/schema-441to450.sql b/setup/db/db/schema-441to450.sql
index 39ac88c..7e8516e 100644
--- a/setup/db/db/schema-441to450.sql
+++ b/setup/db/db/schema-441to450.sql
@@ -739,3 +739,6 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid,hypervisor_type, hypervis
INSERT IGNORE INTO `cloud`.`hypervisor_capabilities`(uuid, hypervisor_type, hypervisor_version, max_guests_limit, security_group_enabled, max_data_volumes_limit, storage_motion_supported) VALUES (UUID(), 'XenServer', '6.5.0', 100, 1, 13, 1);
+INSERT IGNORE INTO `cloud`.`configuration` VALUES ("Advanced", 'DEFAULT', 'management-server', "stats.output.graphite.host", "", "Graphite host to send statistics to", "", NULL, NULL, 0);
+INSERT IGNORE INTO `cloud`.`configuration` VALUES ("Advanced", 'DEFAULT', 'management-server', "stats.output.graphite.port", "2003", "The port of the graphite host to send statistics to", "2003", NULL, NULL, 0);
+INSERT IGNORE INTO `cloud`.`configuration` VALUES ("Advanced", 'DEFAULT', 'management-server', "stats.output.graphite.prefix", "", "Prefix to append to statistics going to Graphite", "", NULL, NULL, 0);
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/f7de57d9/utils/src/org/apache/cloudstack/utils/graphite/GraphiteClient.java
----------------------------------------------------------------------
diff --git a/utils/src/org/apache/cloudstack/utils/graphite/GraphiteClient.java b/utils/src/org/apache/cloudstack/utils/graphite/GraphiteClient.java
new file mode 100644
index 0000000..8db434c
--- /dev/null
+++ b/utils/src/org/apache/cloudstack/utils/graphite/GraphiteClient.java
@@ -0,0 +1,129 @@
+//
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+//
+
+package org.apache.cloudstack.graphite;
+
+import java.io.IOException;
+import java.io.DataOutputStream;
+import java.net.Socket;
+import java.net.UnknownHostException;
+import java.util.HashMap;
+import java.util.Map;
+
+public class GraphiteClient {
+
+ private String graphiteHost;
+ private int graphitePort;
+
+ /**
+ * Create a new Graphite client
+ *
+ * @param graphiteHost Hostname of the Graphite host
+ * @param graphitePort TCP port of the Graphite host
+ */
+ public GraphiteClient(String graphiteHost, int graphitePort) {
+ this.graphiteHost = graphiteHost;
+ this.graphitePort = graphitePort;
+ }
+
+ /**
+ * Create a new Graphite client
+ *
+ * @param graphiteHost Hostname of the Graphite host. Will default to port 2003
+ */
+ public GraphiteClient(String graphiteHost) {
+ this.graphiteHost = graphiteHost;
+ this.graphitePort = 2003;
+ }
+
+ /**
+ * Get the current system timestamp to pass to Graphite
+ *
+ * @return Seconds passed since epoch (01-01-1970)
+ */
+ protected long getCurrentSystemTime() {
+ return System.currentTimeMillis() / 1000;
+ }
+
+ /**
+ * Connect to the Graphite host over TCP
+ *
+ * @return A Socket to the Graphite host
+ */
+ protected Socket connect() throws UnknownHostException, IOException {
+ return new Socket(this.graphiteHost, this.graphitePort);
+ }
+
+ /**
+ * Send a array of metrics to graphite.
+ *
+ * @param metrics the metrics as key-value-pairs
+ */
+ public void sendMetrics(Map<String, Integer> metrics) {
+ sendMetrics(metrics, this.getCurrentSystemTime());
+ }
+
+ /**
+ * Send a array of metrics with a given timestamp to graphite.
+ *
+ * @param metrics the metrics as key-value-pairs
+ * @param timeStamp the timestamp
+ */
+ public void sendMetrics(Map<String, Integer> metrics, long timeStamp) {
+ try {
+ Socket conn = connect();
+ DataOutputStream dos = new DataOutputStream(conn.getOutputStream());
+ for (Map.Entry<String, Integer> metric: metrics.entrySet()) {
+ dos.writeBytes(metric.getKey() + " " + metric.getValue() + " " + timeStamp + "\n");
+ }
+ conn.close();
+ } catch (UnknownHostException e) {
+ throw new GraphiteException("Unknown host: " + graphiteHost);
+ } catch (IOException e) {
+ throw new GraphiteException("Error while writing to graphite: " + e.getMessage(), e);
+ }
+ }
+
+ /**
+ * Send a single metric with the current time as timestamp to graphite.
+ *
+ * @param key The metric key
+ * @param value the metric value
+ *
+ * @throws GraphiteException if sending data to graphite failed
+ */
+ public void sendMetric(String key, int value) {
+ sendMetric(key, value, this.getCurrentSystemTime());
+ }
+
+ /**
+ * Send a single metric with a given timestamp to graphite.
+ *
+ * @param key The metric key
+ * @param value The metric value
+ * @param timeStamp the timestamp to use
+ *
+ * @throws GraphiteException if sending data to graphite failed
+ */
+ public void sendMetric(final String key, final int value, long timeStamp) {
+ HashMap metrics = new HashMap<String, Integer>();
+ metrics.put(key, value);
+ sendMetrics(metrics, timeStamp);
+ }
+}
\ No newline at end of file
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/f7de57d9/utils/src/org/apache/cloudstack/utils/graphite/GraphiteException.java
----------------------------------------------------------------------
diff --git a/utils/src/org/apache/cloudstack/utils/graphite/GraphiteException.java b/utils/src/org/apache/cloudstack/utils/graphite/GraphiteException.java
new file mode 100644
index 0000000..b11532e
--- /dev/null
+++ b/utils/src/org/apache/cloudstack/utils/graphite/GraphiteException.java
@@ -0,0 +1,31 @@
+//
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+// distributed with this work for additional information
+// regarding copyright ownership. The ASF licenses this file
+// to you under the Apache License, Version 2.0 (the
+// "License"); you may not use this file except in compliance
+// with the License. You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied. See the License for the
+// specific language governing permissions and limitations
+// under the License.
+//
+
+package org.apache.cloudstack.graphite;
+
+public class GraphiteException extends RuntimeException {
+
+ public GraphiteException(String message) {
+ super(message);
+ }
+
+ public GraphiteException(String message, Throwable cause) {
+ super(message, cause);
+ }
+}
\ No newline at end of file