Add check when retrieving the worker process id using performance cou…

…nters (#23647) * fix for test * add check * changelog
elastic · Jan 26, 2021 · 3220d46 · 3220d46
1 parent d4e193d
commit 3220d46
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.next.asciidoc b/CHANGELOG.next.asciidoc
@@ -495,6 +495,7 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...master[Check the HEAD d
 - Update config in `windows.yml` file. {issue}23027[23027]{pull}23327[23327]
 - Add stack monitoring section to elasticsearch module documentation {pull}#23286[23286]
 - Fix metric grouping for windows/perfmon module {issue}23489[23489] {pull}23505[23505]
+- Add check for iis/application_pool metricset for nil worker process id values. {issue}23605[23605] {pull}23647[23647]
 
 *Packetbeat*
 

diff --git a/x-pack/metricbeat/module/iis/application_pool/reader.go b/x-pack/metricbeat/module/iis/application_pool/reader.go
@@ -263,7 +263,7 @@ func getw3wpProceses() (map[int]string, error) {
 func getProcessIds(counterValues map[string][]pdh.CounterValue) []WorkerProcess {
 	var workers []WorkerProcess
 	for key, values := range counterValues {
-		if strings.Contains(key, "\\ID Process") {
+		if strings.Contains(key, "\\ID Process") && values[0].Measurement != nil {
 			workers = append(workers, WorkerProcess{instanceName: values[0].Instance, processId: int(values[0].Measurement.(float64))})
 		}
 	}