[Metricbeat] Make Oracle module GA (elastic#16833)

(cherry picked from commit 8a2ed76) # Conflicts: # Jenkinsfile
sayden · Mar 24, 2020 · 6238cf2 · 6238cf2
1 parent 4d0f8ca
commit 6238cf2
Show file tree

Hide file tree

Showing 14 changed files with 24 additions and 21 deletions.
diff --git a/CHANGELOG.next.asciidoc b/CHANGELOG.next.asciidoc
@@ -340,6 +340,7 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...master[Check the HEAD d
 - Add support for CouchDB v2 {issue}16352[16352] {pull}16455[16455]
 - Release Zookeeper/connection module as GA. {issue}14281[14281] {pull}17043[17043]
 - Replace vpc metricset into vpn, transitgateway and natgateway metricsets. {pull}16892[16892]
+- Release Oracle module as GA. {issue}14279[14279] {pull}16833[16833]
 
 *Packetbeat*
 

diff --git a/metricbeat/docs/modules/oracle.asciidoc b/metricbeat/docs/modules/oracle.asciidoc
@@ -6,8 +6,6 @@ This file is generated! See scripts/mage/docs_collector.go
 [role="xpack"]
 == Oracle module
 
-beta[]
-
 This is the https://www.oracle.com[Oracle] module for Metricbeat. It is under active development with feedback from the community. A single Metricset for Tablespace monitoring is added so the community can start gathering metrics from their nodes and contributing to the module.
 
 [float]

diff --git a/metricbeat/docs/modules/oracle/performance.asciidoc b/metricbeat/docs/modules/oracle/performance.asciidoc
@@ -5,8 +5,6 @@ This file is generated! See scripts/mage/docs_collector.go
 [[metricbeat-metricset-oracle-performance]]
 === Oracle performance metricset
 
-beta[]
-
 include::../../../../x-pack/metricbeat/module/oracle/performance/_meta/docs.asciidoc[]
 
 

diff --git a/metricbeat/docs/modules/oracle/tablespace.asciidoc b/metricbeat/docs/modules/oracle/tablespace.asciidoc
@@ -5,8 +5,6 @@ This file is generated! See scripts/mage/docs_collector.go
 [[metricbeat-metricset-oracle-tablespace]]
 === Oracle tablespace metricset
 
-beta[]
-
 include::../../../../x-pack/metricbeat/module/oracle/tablespace/_meta/docs.asciidoc[]
 
 

diff --git a/metricbeat/docs/modules_list.asciidoc b/metricbeat/docs/modules_list.asciidoc
@@ -194,9 +194,9 @@ This file is generated! See scripts/mage/docs_collector.go
 .1+| .1+|  |<<metricbeat-metricset-nginx-stubstatus,stubstatus>>   
 |<<metricbeat-module-openmetrics,Openmetrics>>  beta[]   |image:./images/icon-no.png[No prebuilt dashboards]    |  
 .1+| .1+|  |<<metricbeat-metricset-openmetrics-collector,collector>> beta[]  
-|<<metricbeat-module-oracle,Oracle>>  beta[]   |image:./images/icon-yes.png[Prebuilt dashboards are available]    |  
-.2+| .2+|  |<<metricbeat-metricset-oracle-performance,performance>> beta[]  
-|<<metricbeat-metricset-oracle-tablespace,tablespace>> beta[]  
+|<<metricbeat-module-oracle,Oracle>>     |image:./images/icon-yes.png[Prebuilt dashboards are available]    |  
+.2+| .2+|  |<<metricbeat-metricset-oracle-performance,performance>>   
+|<<metricbeat-metricset-oracle-tablespace,tablespace>>   
 |<<metricbeat-module-php_fpm,PHP_FPM>>     |image:./images/icon-no.png[No prebuilt dashboards]    |  
 .2+| .2+|  |<<metricbeat-metricset-php_fpm-pool,pool>>   
 |<<metricbeat-metricset-php_fpm-process,process>>   

diff --git a/x-pack/metricbeat/module/oracle/_meta/Dockerfile b/x-pack/metricbeat/module/oracle/_meta/Dockerfile
@@ -1,3 +1,3 @@
-FROM store/oracle/database-enterprise:12.2.0.1
+FROM docker.elastic.co/observability-ci/database-enterprise:12.2.0.1
 
-HEALTHCHECK --interval=1s --retries=90 CMD /usr/bin/echo 'select 1' | sqlplus sys/Oradoc_db1@localhost:1521/ORCLPDB1.localdomain AS SYSDBA
+HEALTHCHECK --interval=1s --retries=90 CMD /usr/bin/echo 'select 1' | /u01/app/oracle/product/12.2.0/dbhome_1/bin/sqlplus sys/Oradoc_db1@localhost:1521/ORCLPDB1.localdomain AS SYSDBA
diff --git a/x-pack/metricbeat/module/oracle/_meta/fields.yml b/x-pack/metricbeat/module/oracle/_meta/fields.yml
@@ -2,7 +2,7 @@
   title: "Oracle"
   description: Oracle database module
   short_config: false
-  release: beta
+  release: ga
   fields:
     - name: oracle
       type: group

diff --git a/x-pack/metricbeat/module/oracle/_meta/supported-versions.yml b/x-pack/metricbeat/module/oracle/_meta/supported-versions.yml
@@ -0,0 +1,2 @@
+variants:
+  - ORACLE_VERSION: 12.2.0.1
diff --git a/x-pack/metricbeat/module/oracle/docker-compose.yml b/x-pack/metricbeat/module/oracle/docker-compose.yml
@@ -0,0 +1,12 @@
+version: '2.3'
+
+services:
+  oracle:
+    image: docker.elastic.co/observability-ci/database-enterprise:${ORACLE_VERSION:-12.2.0.1}-1
+    build:
+      context: ./_meta
+      args:
+        ORACLE_VERSION: ${ORACLE_VERSION:-12.2.0.1}
+    ports:
+      - 1521
+      - 5500
diff --git a/x-pack/metricbeat/module/oracle/fields.go b/x-pack/metricbeat/module/oracle/fields.go
diff --git a/x-pack/metricbeat/module/oracle/performance/_meta/fields.yml b/x-pack/metricbeat/module/oracle/performance/_meta/fields.yml
@@ -1,7 +1,7 @@
 - name: performance
   type: group
   description: Performance related metrics on a single database instance
-  release: beta
+  release: ga
   fields:
     - name: machine
       type: keyword

diff --git a/x-pack/metricbeat/module/oracle/performance/metricset.go b/x-pack/metricbeat/module/oracle/performance/metricset.go
@@ -9,7 +9,6 @@ import (
 
 	"github.com/pkg/errors"
 
-	"github.com/elastic/beats/v7/libbeat/common/cfgwarn"
 	"github.com/elastic/beats/v7/metricbeat/mb"
 	"github.com/elastic/beats/v7/x-pack/metricbeat/module/oracle"
 )
@@ -35,8 +34,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.Beta("The oracle 'performance' metricset is beta.")
-
 	config := oracle.ConnectionDetails{}
 	if err := base.Module().UnpackConfig(&config); err != nil {
 		return nil, errors.Wrap(err, "error parsing config file")

diff --git a/x-pack/metricbeat/module/oracle/tablespace/_meta/fields.yml b/x-pack/metricbeat/module/oracle/tablespace/_meta/fields.yml
@@ -1,7 +1,7 @@
 - name: tablespace
   type: group
   description: tablespace
-  release: beta
+  release: ga
   fields:
     - name: name
       type: keyword

diff --git a/x-pack/metricbeat/module/oracle/tablespace/metricset.go b/x-pack/metricbeat/module/oracle/tablespace/metricset.go
@@ -9,7 +9,6 @@ import (
 
 	"github.com/pkg/errors"
 
-	"github.com/elastic/beats/v7/libbeat/common/cfgwarn"
 	"github.com/elastic/beats/v7/metricbeat/mb"
 	"github.com/elastic/beats/v7/x-pack/metricbeat/module/oracle"
 )
@@ -35,8 +34,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.Beta("The oracle 'tablespace' metricset is beta.")
-
 	config := oracle.ConnectionDetails{}
 	if err := base.Module().UnpackConfig(&config); err != nil {
 		return nil, errors.Wrap(err, "error parsing config file")