Disable registration and tests instead of excluding from build.

elastic · Dec 12, 2018 · 415a0ac · 415a0ac
1 parent 2cc3f44
commit 415a0ac
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 13 deletions.
diff --git a/x-pack/auditbeat/module/system/packages/config.go b/x-pack/auditbeat/module/system/packages/config.go
@@ -2,8 +2,6 @@
 // or more contributor license agreements. Licensed under the Elastic License;
 // you may not use this file except in compliance with the Elastic License.
 
-// +build ignore
-
 package packages
 
 // Config defines the host metricset's configuration options.

diff --git a/x-pack/auditbeat/module/system/packages/packages.go b/x-pack/auditbeat/module/system/packages/packages.go
@@ -2,8 +2,6 @@
 // or more contributor license agreements. Licensed under the Elastic License;
 // you may not use this file except in compliance with the Elastic License.
 
-// +build ignore
-
 package packages
 
 import (
@@ -40,9 +38,9 @@ const (
 )
 
 func init() {
-	mb.Registry.MustAddMetricSet(moduleName, metricsetName, New,
+	/*mb.Registry.MustAddMetricSet(moduleName, metricsetName, New,
 		mb.DefaultMetricSet(),
-	)
+	)*/
 }
 
 // MetricSet collects data about the host.

diff --git a/x-pack/auditbeat/module/system/packages/packages_dummy.go b/x-pack/auditbeat/module/system/packages/packages_dummy.go
diff --git a/x-pack/auditbeat/module/system/packages/packages_test.go b/x-pack/auditbeat/module/system/packages/packages_test.go
@@ -2,8 +2,6 @@
 // or more contributor license agreements. Licensed under the Elastic License;
 // you may not use this file except in compliance with the Elastic License.
 
-// +build ignore
-
 package packages
 
 import (
@@ -13,6 +11,8 @@ import (
 )
 
 func TestData(t *testing.T) {
+	t.Skip("Packages metricset is disabled")
+
 	f := mbtest.NewReportingMetricSetV2(t, getConfig())
 	err := mbtest.WriteEventsReporterV2(f, t, "")
 	if err != nil {