Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add redis key metricset #9657

Merged
merged 11 commits into from
Dec 21, 2018
1 change: 1 addition & 0 deletions CHANGELOG.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,7 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha1...master[Check the HEAD d
- Support GET requests in Jolokia module. {issue}8566[8566] {pull}9226[9226]
- Add freebsd support for the uptime metricset. {pull}9413[9413]
- Add `host.os.name` field to add_host_metadata processor. {issue}8948[8948] {pull}9405[9405]
- Add `key` metricset to the Redis module. {issue}9582[9582] {pull}9657[9657]

*Packetbeat*

Expand Down
57 changes: 57 additions & 0 deletions metricbeat/docs/fields.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -20340,6 +20340,63 @@ type: long
Count of slow operations


--

[float]
== key fields

`key` contains information about keys.



*`redis.key.name`*::
+
--
type: keyword

Key name.


--

*`redis.key.type`*::
+
--
type: keyword

Key type as shown by `TYPE` command.


--

*`redis.key.length`*::
+
--
type: long

Length of the key (Number of elements for lists, length for strings, cardinality for sets).


--

*`redis.key.value`*::
+
--
type: text

Value of the key for keys of type `string`.


--

*`redis.key.expire.ttl`*::
+
--
type: long

Seconds to expire.


--

[float]
Expand Down
4 changes: 4 additions & 0 deletions metricbeat/docs/modules/redis.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -81,9 +81,13 @@ The following metricsets are available:

* <<metricbeat-metricset-redis-info,info>>

* <<metricbeat-metricset-redis-key,key>>

* <<metricbeat-metricset-redis-keyspace,keyspace>>

include::redis/info.asciidoc[]

include::redis/key.asciidoc[]

include::redis/keyspace.asciidoc[]

23 changes: 23 additions & 0 deletions metricbeat/docs/modules/redis/key.asciidoc
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
////
This file is generated! See scripts/docs_collector.py
////

[[metricbeat-metricset-redis-key]]
=== Redis key metricset

beta[]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we go GA with this directly?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, let's go for GA, I forgot to add the beta warning in the metricset in any case 😄


include::../../../module/redis/key/_meta/docs.asciidoc[]


==== Fields

For a description of each field in the metricset, see the
<<exported-fields-redis,exported fields>> section.

Here is an example document generated by this metricset:

[source,json]
----
include::../../../module/redis/key/_meta/data.json[]
----
3 changes: 2 additions & 1 deletion metricbeat/docs/modules_list.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,8 @@ This file is generated! See scripts/docs_collector.py
|<<metricbeat-metricset-rabbitmq-node,node>> beta[]
|<<metricbeat-metricset-rabbitmq-queue,queue>> beta[]
|<<metricbeat-module-redis,Redis>> |image:./images/icon-yes.png[Prebuilt dashboards are available] |
.2+| .2+| |<<metricbeat-metricset-redis-info,info>>
.3+| .3+| |<<metricbeat-metricset-redis-info,info>>
|<<metricbeat-metricset-redis-key,key>> beta[]
|<<metricbeat-metricset-redis-keyspace,keyspace>>
|<<metricbeat-module-system,System>> |image:./images/icon-yes.png[Prebuilt dashboards are available] |
.14+| .14+| |<<metricbeat-metricset-system-core,core>>
Expand Down
1 change: 1 addition & 0 deletions metricbeat/include/list.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion metricbeat/module/redis/fields.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

33 changes: 33 additions & 0 deletions metricbeat/module/redis/key/_meta/data.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
{
"@timestamp": "2017-10-12T08:05:34.853Z",
"agent": {
"hostname": "host.example.com",
"name": "host.example.com"
},
"event": {
"dataset": "redis.key",
"duration": 115000,
"module": "redis"
},
"metricset": {
"name": "key"
},
"redis": {
"key": {
"expire": {
"ttl": 360
},
"length": 3,
"name": "foo",
"type": "string",
"value": "bar"
},
"keyspace": {
"id": "db0"
}
},
"service": {
"address": "192.168.16.2:6379",
"type": "redis"
}
}
25 changes: 25 additions & 0 deletions metricbeat/module/redis/key/_meta/docs.asciidoc
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
The Redis `key` metricset collects information about Redis keys.

For each key matching one of the configured patterns, an event is sent to
elasticsearch with information about this key, what includes the type, its
jsoriano marked this conversation as resolved.
Show resolved Hide resolved
length when available, the value when it is a string, and its ttl.

Patterns are configured as a list containing these fields:
* `pattern` (required): pattern for key names, as accepted by the Redis
`KEYS` or `SCAN` commands.
* `max` (optional): safeguard when using patterns with wildcards to avoid
collecting too many keys (Default: 0, no limit)
* `keyspace` (optional): Identifier of the database to use to look for the keys
(Default: 0)

For example the following configuration will collect information about all keys
whose name starts with `pipeline-*`, with a limit of 20 keys.

[source,yaml]
------------------------------------------------------------------------------
- module: redis
metricsets: ['key']
key.patterns:
- name: 'pipeline-*'
max: 20
jsoriano marked this conversation as resolved.
Show resolved Hide resolved
------------------------------------------------------------------------------
30 changes: 30 additions & 0 deletions metricbeat/module/redis/key/_meta/fields.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
- name: key
type: group
description: >
`key` contains information about keys.
release: beta
fields:
- name: name
type: keyword
description: >
Key name.

- name: type
type: keyword
description: >
Key type as shown by `TYPE` command.

- name: length
type: long
description: >
Length of the key (Number of elements for lists, length for strings, cardinality for sets).

- name: value
type: text
jsoriano marked this conversation as resolved.
Show resolved Hide resolved
description: >
Value of the key for keys of type `string`.

- name: expire.ttl
type: long
description: >
Seconds to expire.
36 changes: 36 additions & 0 deletions metricbeat/module/redis/key/data.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
// Licensed to Elasticsearch B.V. under one or more contributor
// license agreements. See the NOTICE file distributed with
// this work for additional information regarding copyright
// ownership. Elasticsearch B.V. licenses this file to you under
// the Apache License, Version 2.0 (the "License"); you may
// not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing,
// software distributed under the License is distributed on an
// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.

package key

import (
"fmt"

"github.com/elastic/beats/libbeat/common"
"github.com/elastic/beats/metricbeat/mb"
)

func eventMapping(r mb.ReporterV2, keyspace uint, info map[string]interface{}) {
r.Event(mb.Event{
MetricSetFields: info,
ModuleFields: common.MapStr{
"keyspace": common.MapStr{
"id": fmt.Sprintf("db%d", keyspace),
jsoriano marked this conversation as resolved.
Show resolved Hide resolved
},
},
})
}
115 changes: 115 additions & 0 deletions metricbeat/module/redis/key/key.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,115 @@
// Licensed to Elasticsearch B.V. under one or more contributor
// license agreements. See the NOTICE file distributed with
// this work for additional information regarding copyright
// ownership. Elasticsearch B.V. licenses this file to you under
// the Apache License, Version 2.0 (the "License"); you may
// not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing,
// software distributed under the License is distributed on an
// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
// KIND, either express or implied. See the License for the
// specific language governing permissions and limitations
// under the License.

package key

import (
"time"

"github.com/elastic/beats/libbeat/logp"
"github.com/elastic/beats/metricbeat/mb"
"github.com/elastic/beats/metricbeat/mb/parse"
"github.com/elastic/beats/metricbeat/module/redis"

rd "github.com/garyburd/redigo/redis"
)

var (
debugf = logp.MakeDebug("redis-key")
)

func init() {
mb.Registry.MustAddMetricSet("redis", "key", New,
mb.WithHostParser(parse.PassThruHostParser),
)
}

// MetricSet for fetching Redis server information and statistics.
type MetricSet struct {
mb.BaseMetricSet
pool *rd.Pool
patterns []KeyPattern
}

// KeyPattern contains the information required to query keys
type KeyPattern struct {
jsoriano marked this conversation as resolved.
Show resolved Hide resolved
Keyspace uint `config:"keyspace"`
Pattern string `config:"pattern" validate:"required"`
Max uint `config:"max"`
}

// New creates new instance of MetricSet
func New(base mb.BaseMetricSet) (mb.MetricSet, error) {
// Unpack additional configuration options.
config := struct {
IdleTimeout time.Duration `config:"idle_timeout"`
Network string `config:"network"`
MaxConn int `config:"maxconn" validate:"min=1"`
Password string `config:"password"`
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Config and pool initialization can be probably reused in other metricsets, to be fixed in a follow up.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Issue created for follow ups #9678

Patterns []KeyPattern `config:"key.patterns" validate:"nonzero,required"`
}{
Network: "tcp",
MaxConn: 10,
Password: "",
}
err := base.Module().UnpackConfig(&config)
if err != nil {
return nil, err
}

return &MetricSet{
BaseMetricSet: base,
pool: redis.CreatePool(base.Host(), config.Password, config.Network,
config.MaxConn, config.IdleTimeout, base.Module().Config().Timeout),
patterns: config.Patterns,
}, nil
}

// Fetch fetches information from Redis keys
func (m *MetricSet) Fetch(r mb.ReporterV2) {
conn := m.pool.Get()
for _, p := range m.patterns {
if err := redis.Select(conn, p.Keyspace); err != nil {
logp.Err("Failed to select keyspace %d: %s", p.Keyspace, err)
continue
}

keys, err := redis.FetchKeys(conn, p.Pattern, p.Max)
if err != nil {
logp.Err("Failed to fetch list of keys in keyspace %d with pattern '%s': %s", p.Keyspace, p.Pattern, err)
continue
}
if p.Max > 0 && len(keys) > int(p.Max) {
debugf("Collecting stats for %d keys, but there are more available for pattern '%s' in keyspace %d", p.Max)
keys = keys[:p.Max]
}

for _, key := range keys {
keyInfo, err := redis.FetchKeyInfo(conn, key)
if err != nil {
logp.Err("Failed to fetch key info for key %s in keyspace %d", key, p.Keyspace)
continue
}
eventMapping(r, p.Keyspace, keyInfo)
}
}
}

// Close connections
func (m *MetricSet) Close() error {
return m.pool.Close()
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this needs to be added in other metricsets too, to be added in the follow-up PR.

Loading