[Metricbeat][Redis] Add the Valkey keys_with_volatile_items keyspace field#52181
Open
aglowienka wants to merge 2 commits into
Open
[Metricbeat][Redis] Add the Valkey keys_with_volatile_items keyspace field#52181aglowienka wants to merge 2 commits into
aglowienka wants to merge 2 commits into
Conversation
…etricset Valkey >= 9.0 reports a per-database keys_with_volatile_items counter in INFO keyspace. Collect it as redis.keyspace.keys_with_volatile_items, following the same pattern as the Redis 7.4 subexpiry field: added to the schema and defaulted to 0 when the engine does not report it. Relates to elastic#52179 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
This pull request doesn't have a |
Contributor
🤖 GitHub commentsJust comment with:
|
Contributor
|
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
Ran `mage fields` and `mage fieldsDocs` to regenerate the generated assets for the new redis.keyspace.keys_with_volatile_items field. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
aglowienka
marked this pull request as ready for review
July 22, 2026 09:00
bmorelli25
approved these changes
Jul 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed commit message
Valkey ≥ 9.0 reports a per-database
keys_with_volatile_itemscounter inINFO keyspacethat theredis.keyspacemetricset did not collect. This addsredis.keyspace.keys_with_volatile_items,following the same pattern as the Redis 7.4
subexpiryfield (added in #47971): it is added to theschema and defaulted to
0when the engine does not report it, so behavior is unchanged on Redisand older Valkey.
Generated assets (
metricbeat/module/redis/fields.goanddocs/reference/metricbeat/exported-fields-redis.md) were regenerated withmage fields/mage fieldsDocs.Relates to #52179.
Checklist
[ ] I have made corresponding change to the default configuration files(n/a)./changelog/fragmentsusing the changelog toolDisruptive User Impact
None. Adds one new keyspace field; it defaults to
0when the engine does not report it.How to test this PR locally
TestParseKeyspaceStatsnow covers adbline that reportskeys_with_volatile_items(asserting itis captured) and the default-
0behavior when it is absent.Note on the sample event
metricbeat/module/redis/keyspace/_meta/data.json(the generated sample event) is produced by theintegration test against a live Redis/Valkey and is not regenerated by
mage update. It will pick upthe new field the next time the keyspace integration golden is regenerated; leaving it untouched here
keeps
mage checkgreen.Related
Companion to #52180 (parser robustness fix). On
mainthe parser already accepts the 4-field Valkeyline, so this change is purely additive.