Only log error (don't also index it) if xpack is enabled. (#12265)

* Only log error (don't also index it) if xpack is enabled. Indexing it would index it into metricbeat-* indices instead of .monitoring-* indices, which are currently not equipped to handle error documents. * Adding CHANGELOG entry
elastic · May 28, 2019 · 0900e01 · 0900e01
1 parent b733788
commit 0900e01
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/CHANGELOG.next.asciidoc b/CHANGELOG.next.asciidoc
@@ -131,6 +131,7 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...master[Check the HEAD d
 - Fix direction of incoming IPv6 sockets. {pull}12248[12248]
 - Validate that kibana/status metricset cannot be used when xpack is enabled. {pull}12264[12264]
 - Ignore prometheus metrics when their values are NaN or Inf. {pull}12084[12084] {issue}10849[10849]
+- In the kibana/stats metricset, only log error (don't also index it) if xpack is enabled. {pull}12265[12265]
 
 *Packetbeat*
 

diff --git a/metricbeat/module/kibana/stats/stats.go b/metricbeat/module/kibana/stats/stats.go
@@ -132,6 +132,10 @@ func (m *MetricSet) Fetch(r mb.ReporterV2) error {
 
 	err := m.fetchStats(r, now)
 	if err != nil {
+		if m.XPackEnabled {
+			m.Logger().Error(err)
+			return nil
+		}
 		return err
 	}