Remove remote wmi logic from wmi.go. This can be reactivated by need

elastic · Mar 4, 2025 · 28ed9a7 · 28ed9a7
1 parent a779965
commit 28ed9a7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/metricbeat/module/windows/wmi/_meta/data.json b/metricbeat/module/windows/wmi/_meta/data.json
@@ -20,7 +20,6 @@
             "LocalDateTime": "2024-12-12T15:46:39.62Z",
             "NumberOfUsers": 1,
             "class": "Win32_OperatingSystem",
-            "host": "localhost",
             "namespace": "root\\cimv2"
         }
     }

diff --git a/metricbeat/module/windows/wmi/wmi.go b/metricbeat/module/windows/wmi/wmi.go
@@ -150,13 +150,15 @@ func (m *MetricSet) Fetch(report mb.ReporterV2) error {
 					MetricSetFields: mapstr.M{
 						"class":     queryConfig.Class,
 						"namespace": namespace,
-						"host":      m.config.Host,
+						// Remote WMI is intentionally hidden, this will always be localhost
+						// "host":      m.config.Host,
 					},
 				}
 
-				if m.config.Domain != "" {
-					event.MetricSetFields.Put("domain", m.config.Domain)
-				}
+				// Remote WMI is intentionally hidden, this will always be the empty string
+				// if m.config.Domain != "" {
+				// 	event.MetricSetFields.Put("domain", m.config.Domain)
+				// }
 
 				if m.config.IncludeQueries {
 					event.MetricSetFields.Put("query", query)