Merge pull request #237 from szook/fix-mssql-for-non-default-instances

fix mssql for non-default sql instances
2026-02-08 14:06:38 +00:00 · 2018-08-04 08:33:23 +02:00 · 2018-08-03 16:31:19 -04:00 · 2018-08-03 22:16:08 +02:00 · 2018-08-03 15:39:26 -04:00 · 2018-08-03 13:45:51 -04:00
3 changed files with 3555 additions and 3 deletions
--- a/README.md
+++ b/README.md
@@ -9,15 +9,16 @@ Prometheus exporter for Windows machines, using the WMI (Windows Management Inst

 Name     | Description | Enabled by default
 ---------|-------------|--------------------
-ad | [Win32_PerfRawData_DirectoryServices_DirectoryServices](https://msdn.microsoft.com/en-us/library/ms803980.aspx) Active Directory | 
+ad | [Win32_PerfRawData_DirectoryServices_DirectoryServices](https://msdn.microsoft.com/en-us/library/ms803980.aspx) Active Directory |
 cpu | [Win32_PerfRawData_PerfOS_Processor](https://msdn.microsoft.com/en-us/library/aa394317(v=vs.90).aspx) metrics (cpu usage) | &#10003;
 cs | [Win32_ComputerSystem](https://msdn.microsoft.com/en-us/library/aa394102) metrics (system properties, num cpus/total memory) | &#10003;
 dns | [Win32_PerfRawData_DNS_DNS](https://technet.microsoft.com/en-us/library/cc977686.aspx) metrics (DNS Server) |
-hyperv | Performance counters for Hyper-V hosts | 
+hyperv | Performance counters for Hyper-V hosts |
 iis | [Win32_PerfRawData_W3SVC_WebService](https://msdn.microsoft.com/en-us/library/aa394345) IIS metrics |
 logical_disk | [Win32_PerfRawData_PerfDisk_LogicalDisk](https://msdn.microsoft.com/en-us/windows/hardware/aa394307(v=vs.71)) metrics (disk I/O) | &#10003;
 net | [Win32_PerfRawData_Tcpip_NetworkInterface](https://technet.microsoft.com/en-us/security/aa394340(v=vs.80)) metrics (network interface I/O) | &#10003;
-msmq | [Win32_PerfRawData_MSMQ_MSMQQueue](http://wutils.com/wmi/root/cimv2/win32_perfrawdata_msmq_msmqqueue/) metrics (MSMQ/journal count) | 
+msmq | [Win32_PerfRawData_MSMQ_MSMQQueue](http://wutils.com/wmi/root/cimv2/win32_perfrawdata_msmq_msmqqueue/) metrics (MSMQ/journal count) |
+mssql | various [SQL Server Performance Objects](https://docs.microsoft.com/en-us/sql/relational-databases/performance-monitor/use-sql-server-objects#SQLServerPOs) metrics  |
 os | [Win32_OperatingSystem](https://msdn.microsoft.com/en-us/library/aa394239) metrics (memory, processes, users) | &#10003;
 process | [Win32_PerfRawData_PerfProc_Process](https://msdn.microsoft.com/en-us/library/aa394323(v=vs.85).aspx) metrics (per-process stats) |
 service | [Win32_Service](https://msdn.microsoft.com/en-us/library/aa394418(v=vs.85).aspx) metrics (service states) | &#10003;
--- a/collector/mssql.go
+++ b/collector/mssql.go
--- a/collector/process.go
+++ b/collector/process.go
@@ -177,6 +177,11 @@ type Win32_PerfRawData_PerfProc_Process struct {
 	WorkingSetPrivate       uint64
 }

+type WorkerProcess struct {
+	AppPoolName string
+	ProcessId   uint32
+}
+
 func (c *ProcessCollector) collect(ch chan<- prometheus.Metric) (*prometheus.Desc, error) {
 	var dst []Win32_PerfRawData_PerfProc_Process
 	q := queryAllWhere(&dst, c.queryWhereClause)
@@ -184,6 +189,10 @@ func (c *ProcessCollector) collect(ch chan<- prometheus.Metric) (*prometheus.Des
 		return nil, err
 	}

+	var dst_wp []WorkerProcess
+	q_wp := queryAll(&dst_wp)
+	wmi.QueryNamespace(q_wp, &dst_wp, "root\\WebAdministration")
+
 	for _, process := range dst {

 		if process.Name == "_Total" {
@@ -194,6 +203,13 @@ func (c *ProcessCollector) collect(ch chan<- prometheus.Metric) (*prometheus.Des
 		pid := strconv.FormatUint(uint64(process.IDProcess), 10)
 		cpid := strconv.FormatUint(uint64(process.CreatingProcessID), 10)

+		for _, wp := range dst_wp {
+			if wp.ProcessId == process.IDProcess {
+				processName = strings.Join([]string{processName, wp.AppPoolName}, "_")
+				break
+			}
+		}
+
 		ch <- prometheus.MustNewConstMetric(
 			c.StartTime,
 			prometheus.GaugeValue,
Author	SHA1	Message	Date
Calle Pettersson	bad1e7f7b0	Merge pull request #237 from szook/fix-mssql-for-non-default-instances fix mssql for non-default sql instances	2018-08-04 08:33:23 +02:00
Steve Zook	c868c00e89	fix mssql for non-default sql instances the only sql servers I have access to are using the default `MSSQLSERVER` instance names. I was contacted by someone using a different instance name and it was failing. Found 2 bugs in the code: 1) I was returning the default `MSSQLSERVER` as an instance even if it wasn't found in the registry 2) learned that the WMI class naming scheme for non-default instances was not what I had coded. Changed to incorproate new knowledge.	2018-08-03 16:31:19 -04:00
Calle Pettersson	5035e97369	Merge pull request #236 from szook/add-mssql-accessmethod-class add mssql accessmethod class	2018-08-03 22:16:08 +02:00
Steve Zook	144715e3d2	remove persec from variable names removed persec from variable names but when I did that, there was some variable name collisions, so I went through and name-spaced the class variables. 1	2018-08-03 15:39:26 -04:00
Steve Zook	a20cf1274a	add mssql accessmethod class in the process of trying to build a grafana dashboard with "useful" mssql metrics, I was looking around for what metrics might be useful and came across an article of [15 SQL Server Performace Counters to Monitor](https://blogs.sentryone.com/allenwhite/sql-server-performance-counters-to-monitor/) two of the suggested metrics are provided by the AccessMethod class, which I was not yet capturing. So I added metrics for the Win32_PerfRawData_MSSQLSERVER_SQLServerAccessMethods class.	2018-08-03 13:45:51 -04:00
Calle Pettersson	626a25cd00	Merge pull request #235 from szook/reorder-functions reorder functions	2018-08-03 16:42:32 +02:00
Calle Pettersson	96dd456bb1	Merge pull request #234 from szook/fix-mssql-metric-name-typo fix mssql metric name type	2018-08-03 16:27:40 +02:00
Steve Zook	af1b8bf4d0	fix mssql metric name type found that I had one metric with an extra `s` in `_genstatss_`, making it inconsistent with it's siblings.	2018-08-03 09:52:15 -04:00
Steve Zook	d83615a818	reorder functions so that the two functions that list available child collectors are next to each other - should make less likely to not miss editing one of those when adding additional wmi classes also removed the 'filter' from the `mssqlFilterAvailableClassCollectors` function as it's not filtering anyting.	2018-08-03 09:48:32 -04:00
szook	fe4c61a70e	Add mssql collector (#230 ) add mssql collector	2018-08-02 08:42:02 +02:00
Calle Pettersson	143705bbf6	Merge pull request #227 from szook/add-worker-process-name-to-w3wp-processes append IIS worker process name to corresponding w3wp processes	2018-07-23 16:20:07 +02:00
Steve Zook	e8ffb736d0	add worker process name to w3wp processes modify the `process` collector so that the IIS worker process name is appended to the corresponding `w3wp` process. before: ``` wmi_process_private_bytes{creating_process_id="2068",process="w3wp",process_id="12308"} 7.18204928e+08 ``` after: ``` wmi_process_private_bytes{creating_process_id="2068",process="w3wp_our.website.com",process_id="12308"} 7.18204928e+08 ``` reason: We have some IIS servers hosting many .NET applications. When there is resource contention on one of those servers, it's nice to know which IIS application pool is the culprit. Having only the process_id to differentiate between w3wp processes requires additional work to figure out which is which. Also it does not allow for historial trending as the process_id can change across restarts.	2018-07-23 07:53:50 -04:00