RedisCluster and full Redis support

[b-viguier]

Summary

• NR extension wraps calls to Redis extension, but some functions are missing (mget in particular)
• What about RedisCluster? The exposed interface is quite similar, is it possible to also wrap it?

Desired Behavior

• Redis::mget function is monitored by NR
• RedisCluster::* calls are monitored by NR

Possible Solution

As far as I understand the code, the magic should happen here 😅

Additional context

• Redis::mget is used by the Redis cache adapter provided in Symfony applications. Then in our application we see set calls, but no mget
• Symfony also provides some adapters for RedisCluster, but it's hard to bypass the framework to manually corresponding database records 😕

🙇‍♂️

[joshuabenuck]

@b-viguier Thanks for submitting this enhancement request! This is definitely something we are interested in pursuing. We don't currently have a timeline for when we will be able to get to it, but are adding it to our backlog for consideration. We are also open to PRs if you are able to help out in the meantime!

[ostrolucky]

In case you use snc redis bundle, I've made an extension for it that records redis queries to NR

[duartegarin]

Hi NR team!
I would like to bump this.
We just moved to cluster mode and were disappointed to see our redis metrics evaporated.
Running single node Redis only gets you so far and most installations in bigger applications will need sharding in Redis, so this becomes a must have pretty quickly.
Do you have any updates on this? Are there any workarounds?
Thanks!

[samuelnogueira]

@duartegarin as a workaround, you can use newrelic_record_datastore_segment function to record RedisCluster metrics.
We're using these PSR cache decorators to do just that: https://github.com/samuelnogueira/cache-datastore-newrelic, but we see this as a stopgap measure, until New Relic PHP agent supports this.

[duartegarin]

Hi @samuelnogueira ,
Thank you very much! I'll need to figure out how to do this as I'm working with Drupal+PHPRedis.
The decorators is a good idea.
Cheers