diff --git a/CHANGELOG-developer.next.asciidoc b/CHANGELOG-developer.next.asciidoc index dbe28b1bf503..4d20151d6cbd 100644 --- a/CHANGELOG-developer.next.asciidoc +++ b/CHANGELOG-developer.next.asciidoc @@ -70,6 +70,7 @@ The list below covers the major changes between 7.0.0-rc2 and main only. - Errors should be thrown as errors. Metricsets inside Metricbeat will now throw errors as the `error` log level. {pull}27804[27804] - Avoid panicking in `add_fields` processor when input event.Fields is a nil map. {pull}28219[28219] - Drop event batch when get HTTP status 413 from Elasticsearch to avoid infinite loop {issue}14350[14350] {pull}29368[29368] +- Allow to use metricbeat for named mssql instances. {issue}24076[24076] {pull}30859[30859] ==== Added diff --git a/x-pack/metricbeat/module/mssql/performance/_meta/data.json b/x-pack/metricbeat/module/mssql/performance/_meta/data.json index 656c0b525fdd..6ae827b42bce 100644 --- a/x-pack/metricbeat/module/mssql/performance/_meta/data.json +++ b/x-pack/metricbeat/module/mssql/performance/_meta/data.json @@ -12,25 +12,25 @@ "mssql": { "performance": { "active_temp_tables": 0, - "batch_requests_per_sec": 3355, + "batch_requests_per_sec": 7453, "buffer": { "cache_hit": { - "pct": 0.22 + "pct": 0.55 }, - "checkpoint_pages_per_sec": 295, - "database_pages": 2152, + "checkpoint_pages_per_sec": 124, + "database_pages": 2191, "page_life_expectancy": { - "sec": 1400 + "sec": 2721 }, - "target_pages": 3178496 + "target_pages": 1589248 }, - "compilations_per_sec": 1151, - "connections_reset_per_sec": 88, - "lock_waits_per_sec": 6, - "logins_per_sec": 1105, - "logouts_per_sec": 1103, - "page_splits_per_sec": 14, - "recompilations_per_sec": 2, + "compilations_per_sec": 2503, + "connections_reset_per_sec": 61, + "lock_waits_per_sec": 4, + "logins_per_sec": 2448, + "logouts_per_sec": 2446, + "page_splits_per_sec": 15, + "recompilations_per_sec": 0, "transactions": 0, "user_connections": 2 } diff --git a/x-pack/metricbeat/module/mssql/performance/performance.go b/x-pack/metricbeat/module/mssql/performance/performance.go index fb9c4b4c888c..da15da02302e 100644 --- a/x-pack/metricbeat/module/mssql/performance/performance.go +++ b/x-pack/metricbeat/module/mssql/performance/performance.go @@ -11,8 +11,6 @@ import ( "github.com/elastic/beats/v7/libbeat/common" - "github.com/pkg/errors" - "github.com/elastic/beats/v7/libbeat/logp" "github.com/elastic/beats/v7/metricbeat/mb" "github.com/elastic/beats/v7/x-pack/metricbeat/module/mssql" @@ -52,7 +50,7 @@ func New(base mb.BaseMetricSet) (mb.MetricSet, error) { db, err := mssql.NewConnection(base.HostData().URI) if err != nil { - return nil, errors.Wrap(err, "could not create connection to db") + return nil, fmt.Errorf("could not create connection to db %w", err) } return &MetricSet{ @@ -77,34 +75,23 @@ WHERE counter_name = 'SQL Compilations/sec' OR counter_name = 'SQL Re-Compilations/sec' OR counter_name = 'User Connections' OR counter_name = 'Page splits/sec' - OR ( counter_name = 'Lock Waits/sec' - AND instance_name = '_Total' ) OR counter_name = 'Page splits/sec' - OR ( object_name = 'SQLServer:Buffer Manager' - AND counter_name = 'Page life expectancy' ) OR counter_name = 'Batch Requests/sec' - OR ( counter_name = 'Buffer cache hit ratio' - AND object_name = 'SQLServer:Buffer Manager' ) - OR ( counter_name = 'Target pages' - AND object_name = 'SQLServer:Buffer Manager' ) - OR ( counter_name = 'Database pages' - AND object_name = 'SQLServer:Buffer Manager' ) - OR ( counter_name = 'Checkpoint pages/sec' - AND object_name = 'SQLServer:Buffer Manager' ) OR ( counter_name = 'Lock Waits/sec' AND instance_name = '_Total' ) - OR ( counter_name = 'Transactions' - AND object_name = 'SQLServer:General Statistics' ) - OR ( counter_name = 'Logins/sec' - AND object_name = 'SQLServer:General Statistics' ) - OR ( counter_name = 'Logouts/sec' - AND object_name = 'SQLServer:General Statistics' ) - OR ( counter_name = 'Connection Reset/sec' - AND object_name = 'SQLServer:General Statistics' ) - OR ( counter_name = 'Active Temp Tables' - AND object_name = 'SQLServer:General Statistics' )`) + OR ( counter_name IN ( 'Page life expectancy', + 'Buffer cache hit ratio', + 'Target pages', 'Database pages', + 'Checkpoint pages/sec' ) + AND object_name LIKE '%:Buffer Manager%' ) + OR ( counter_name IN ( 'Transactions', + 'Logins/sec', + 'Logouts/sec', + 'Connection Reset/sec', + 'Active Temp Tables' ) + AND object_name LIKE '%:General Statistics%' )`) if err != nil { - reporter.Error(errors.Wrapf(err, "error closing rows")) + reporter.Error(fmt.Errorf("error closing rows %w", err)) return } defer func() { @@ -117,7 +104,7 @@ WHERE counter_name = 'SQL Compilations/sec' for rows.Next() { var row performanceCounter if err = rows.Scan(&row.objectName, &row.counterName, &row.instanceName, &row.counterValue); err != nil { - reporter.Error(errors.Wrap(err, "error scanning rows")) + reporter.Error(fmt.Errorf("error scanning rows %w", err)) continue } @@ -135,7 +122,7 @@ WHERE counter_name = 'SQL Compilations/sec' res, err := schema.Apply(mapStr) if err != nil { - m.log.Error(errors.Wrap(err, "error applying schema")) + m.log.Error(fmt.Errorf("error applying schema %w", err)) return }