elastic · jsoriano · Dec 21, 2018 · Dec 20, 2018 · Dec 21, 2018 · Dec 21, 2018
diff --git a/CHANGELOG.asciidoc b/CHANGELOG.asciidoc
@@ -81,6 +81,7 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...master[Check the HEAD d
 *Metricbeat*
 
 - Add `key` metricset to the Redis module. {issue}9582[9582] {pull}9657[9657] {pull}9746[9746]
+- Add `socket_summary` metricset to system defaults, removing experimental tag and supporting Windows {pull}9709[9709]
 
 *Packetbeat*
 

diff --git a/metricbeat/docs/modules/system.asciidoc b/metricbeat/docs/modules/system.asciidoc
@@ -37,6 +37,7 @@ metricbeat.modules:
     - process         # Per process metrics
     - process_summary # Process summary
     - uptime          # System Uptime
+    - socket_summary  # Socket summary
     #- core           # Per CPU core usage
     #- diskio         # Disk IO
     #- filesystem     # File system usage for each mountpoint

diff --git a/metricbeat/docs/modules/system/socket_summary.asciidoc b/metricbeat/docs/modules/system/socket_summary.asciidoc
@@ -5,8 +5,6 @@ This file is generated! See scripts/docs_collector.py
 [[metricbeat-metricset-system-socket_summary]]
 === System socket_summary metricset
 
-experimental[]
-
 include::../../../module/system/socket_summary/_meta/docs.asciidoc[]
 
 

diff --git a/metricbeat/docs/modules_list.asciidoc b/metricbeat/docs/modules_list.asciidoc
@@ -135,7 +135,7 @@ This file is generated! See scripts/docs_collector.py
 |<<metricbeat-metricset-system-process_summary,process_summary>>   
 |<<metricbeat-metricset-system-raid,raid>>   
 |<<metricbeat-metricset-system-socket,socket>>   
-|<<metricbeat-metricset-system-socket_summary,socket_summary>> experimental[]  
+|<<metricbeat-metricset-system-socket_summary,socket_summary>>   
 |<<metricbeat-metricset-system-uptime,uptime>>   
 |<<metricbeat-module-traefik,traefik>>  beta[]   |image:./images/icon-no.png[No prebuilt dashboards]    |  
 .1+| .1+|  |<<metricbeat-metricset-traefik-health,health>> beta[]  

diff --git a/metricbeat/metricbeat.reference.yml b/metricbeat/metricbeat.reference.yml
@@ -57,6 +57,7 @@ metricbeat.modules:
     - process         # Per process metrics
     - process_summary # Process summary
     - uptime          # System Uptime
+    - socket_summary  # Socket summary
     #- core           # Per CPU core usage
     #- diskio         # Disk IO
     #- filesystem     # File system usage for each mountpoint

diff --git a/metricbeat/module/system/_meta/config.reference.yml b/metricbeat/module/system/_meta/config.reference.yml
@@ -7,6 +7,7 @@
     - process         # Per process metrics
     - process_summary # Process summary
     - uptime          # System Uptime
+    - socket_summary  # Socket summary
     #- core           # Per CPU core usage
     #- diskio         # Disk IO
     #- filesystem     # File system usage for each mountpoint

diff --git a/metricbeat/module/system/_meta/config.yml b/metricbeat/module/system/_meta/config.yml
@@ -7,6 +7,7 @@
     - network
     - process
     - process_summary
+    - socket_summary
     #- core
     #- diskio
     #- socket

diff --git a/metricbeat/module/system/fields.go b/metricbeat/module/system/fields.go
diff --git a/metricbeat/module/system/socket_summary/_meta/docs.asciidoc b/metricbeat/module/system/socket_summary/_meta/docs.asciidoc
@@ -9,6 +9,7 @@ This metricset is available on:
 - FreeBSD
 - Linux
 - macOS
+- Windows
 
 [float]
 === Configuration

diff --git a/metricbeat/module/system/socket_summary/_meta/fields.yml b/metricbeat/module/system/socket_summary/_meta/fields.yml
@@ -3,6 +3,7 @@
   type: group
   description: >
     Summary metrics of open sockets in the host system
+  release: ga
   fields:
     - name: all
       type: group

diff --git a/metricbeat/module/system/socket_summary/socket_summary.go b/metricbeat/module/system/socket_summary/socket_summary.go
@@ -18,17 +18,12 @@
 package socket_summary
 
 import (
-	"runtime"
 	"syscall"
 
-	"github.com/pkg/errors"
+	"github.com/shirou/gopsutil/net"
 
 	"github.com/elastic/beats/libbeat/common"
-
-	"github.com/elastic/beats/libbeat/common/cfgwarn"
 	"github.com/elastic/beats/metricbeat/mb"
-
-	"github.com/shirou/gopsutil/net"
 )
 
 // init registers the MetricSet with the central registry as soon as the program
@@ -38,6 +33,7 @@ import (
 func init() {
 	mb.Registry.MustAddMetricSet("system", "socket_summary", New,
 		mb.WithNamespace("system.socket.summary"),
+		mb.DefaultMetricSet(),
 	)
 }
 
@@ -52,11 +48,6 @@ type MetricSet struct {
 // New creates a new instance of the MetricSet. New is responsible for unpacking
 // any MetricSet specific configuration options if there are any.
 func New(base mb.BaseMetricSet) (mb.MetricSet, error) {
-	cfgwarn.Experimental("The socket_summary metricset is experimental.")
-	if runtime.GOOS == "windows" {
-		return nil, errors.New("socket_summary metricset is not supported in Windows")
-	}
-
 	return &MetricSet{
 		BaseMetricSet: base,
 	}, nil

diff --git a/metricbeat/modules.d/system.yml b/metricbeat/modules.d/system.yml
@@ -10,6 +10,7 @@
     - network
     - process
     - process_summary
+    - socket_summary
     #- core
     #- diskio
     #- socket

diff --git a/metricbeat/tests/system/test_system.py b/metricbeat/tests/system/test_system.py
@@ -448,7 +448,7 @@ def test_process_metricbeat(self):
         assert isinstance(output["system.process.cpu.start_time"], six.string_types)
         self.check_username(output["system.process.username"])
 
-    @unittest.skipUnless(re.match("(?i)linux|darwin|freebsd", sys.platform), "os")
+    @unittest.skipUnless(re.match("(?i)win|linux|darwin|freebsd", sys.platform), "os")
     def test_socket_summary(self):
         """
         Test system/socket_summary output.

diff --git a/x-pack/metricbeat/metricbeat.reference.yml b/x-pack/metricbeat/metricbeat.reference.yml
@@ -57,6 +57,7 @@ metricbeat.modules:
     - process         # Per process metrics
     - process_summary # Process summary
     - uptime          # System Uptime
+    - socket_summary  # Socket summary
     #- core           # Per CPU core usage
     #- diskio         # Disk IO
     #- filesystem     # File system usage for each mountpoint
-Original file line number
+Diff line change
@@ Expand Up / @@ -9,6 +9,7 @@ This metricset is available on: @@
     - FreeBSD
     - Linux
     - macOS
+    - Windows
     [float]
     === Configuration
@@ Expand Down @@