Fix non-windows fields on system/filesystem (#21758)

* fix windows fields on system/filesystem * add changelog * fix tests
elastic · Oct 14, 2020 · 4a9b08a · 4a9b08a
1 parent 3f57f0e
commit 4a9b08a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.next.asciidoc b/CHANGELOG.next.asciidoc
@@ -371,6 +371,7 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...master[Check the HEAD d
 - Add a switch to the driver definition on SQL module to use pretty names {pull}17378[17378]
 - Fix retrieving resources by ID for the azure module. {pull}21711[21711] {issue}21707[21707]
 - Use timestamp from CloudWatch API when creating events. {pull}21498[21498]
+- Report the correct windows events for system/filesystem {pull}21758[21758]
 
 *Packetbeat*
 

diff --git a/metricbeat/module/system/filesystem/helper.go b/metricbeat/module/system/filesystem/helper.go
@@ -151,14 +151,14 @@ func GetFilesystemEvent(fsStat *FSStat) common.MapStr {
  "mount_point": fsStat.Mount,
  "total": fsStat.Total,
  "available": fsStat.Avail,
- "files": fsStat.Files,
+ "free":  fsStat.Free,
  "used": common.MapStr{
  "pct": fsStat.UsedPercent,
  "bytes": fsStat.Used,
  },
  }
  if runtime.GOOS != "windows" {
- evt.Put("free", fsStat.Free)
+ evt.Put("files", fsStat.Files)
  evt.Put("free_files", fsStat.FreeFiles)
  }
  return evt

diff --git a/metricbeat/module/system/test_system.py b/metricbeat/module/system/test_system.py
@@ -47,7 +47,7 @@
  "free_files", "mount_point", "total", "used.bytes",
  "used.pct"]
 
-SYSTEM_FILESYSTEM_FIELDS_WINDOWS = ["available", "device_name", "type", "files",
+SYSTEM_FILESYSTEM_FIELDS_WINDOWS = ["available", "device_name", "type", "free",
  "mount_point", "total", "used.bytes",
  "used.pct"]