Comments (7)
@dpipemazo Thank you for confirming, I am checking with the RedisTimeSeries team what will be the expected behavior in this case.
I agree that extra intervals can be clipped in the plug-in, but I expect RedisTimeSeries to return the correct values.
from grafana-redis-datasource.
Sure, thanks for helping to bring it to their attention.
from grafana-redis-datasource.
@dpipemazo Thank you for submitting another issue.
Have you tried the same command from redis-cli
and see the kworker returning 0 for the outside interval? How about remove aggregation?
I will take a look and get back to you.
from grafana-redis-datasource.
@mikhailredis Thanks for taking a look. I should have confirmed in redis-cli
, apologies for not doing that originally. I reproduced a similar situation to the graphs above through running:
ts.mrange 1598405335639 1598410333697 AGGREGATION avg 1000 WITHLABELS FILTER element=monitor type=process subtype1=cpu_user
The full resultant text file is attached and is telling -- the source of the issue is indeed coming from Redis Time Series as we can see some example entries from the query:
1484) 1) "monitor:process:99:kworker/7:1:cpu_user"
2) 1) 1) "element"
2) "monitor"
2) 1) "type"
2) "process"
3) 1) "container"
2) "x86_64"
4) 1) "device"
2) "default"
5) 1) "language"
2) "Python"
6) 1) "version"
2) "1.7.2"
7) 1) "level"
2) "INFO"
8) 1) "subtype0"
2) "99:kworker/7:1"
9) 1) "subtype1"
2) "cpu_user"
3) 1) 1) (integer) 1598456570000
2) 0
and
1482) 1) "monitor:process:9866:kworker/3:0:cpu_user"
2) 1) 1) "element"
2) "monitor"
2) 1) "type"
2) "process"
3) 1) "container"
2) "x86_64"
4) 1) "device"
2) "default"
5) 1) "language"
2) "Python"
6) 1) "version"
2) "1.7.2"
7) 1) "level"
2) "INFO"
8) 1) "subtype0"
2) "9866:kworker/3:0"
9) 1) "subtype1"
2) "cpu_user"
3) 1) 1) (integer) 1598404695000
2) 0
Where we can see for data existing on either side of the time series we are getting a 0 entry back with a timestamp outside of the original range.
I can (and will) file an issue on the Redis Time Series repo about this however I'm not sure it's as straightforward as asking them to do it. This is a debatably reasonable response on their end and I can see use cases where we'd want to know there was something that existed outside of the time range query. It's likely a flag we ask for from them and likely would take a while to get in if approved?
I do think given how this "Data outside of time range" cannot be turned off in grafana and materially impacts the usability of the graphs it would be nice to clip any time values returned from redis to (min, max) on the time series queried. That would solve this pretty easily I think without too much effort.
Full output of the mrange query attached below.
from grafana-redis-datasource.
I did go ahead and file the issue on the Redis Time Series repo as well: RedisTimeSeries/RedisTimeSeries#506
from grafana-redis-datasource.
@dpipemazo Thank you, let's wait for the RedisTimeSeries team response.
from grafana-redis-datasource.
@dpipemazo RedisTimeSeries team accepted the issue and will work on it.
Do you mind to close this issue and follow RedisTimeSeries/RedisTimeSeries#506?
from grafana-redis-datasource.
Related Issues (20)
- Redis and Grafan: JSONPath not returning JSON object correctly HOT 4
- Simple way to query all streams and retrieve length of stream HOT 2
- Add support for FT.SEARCH and FT.AGGREGATE HOT 1
- How to configure add Redis Datasource in values.yaml
- Unable to connect to Redis Datasource via Grafana Cloud HOT 3
- Unable to hide Redis query data HOT 2
- Is this still maintained? HOT 23
- CLIENT INFO fails HOT 5
- TS.MRANGE sometimes doesn't display key in the panel HOT 4
- TS.MRANGE works intermittently HOT 2
- Add FILTER_BY_VALUE support to TS.RANGE and TS.MRANGE
- How to query a value of key without bracket ? HOT 4
- Is it possible to return multiple options when querying an array of json, instead of a single stringified array, in the Grafana dashboard? HOT 3
- Provided dashboards broken in Grafana v10
- Grafana Live: Prevent panels from going blank during dashboard refresh
- docker image Ubuntu basic import redis-datasource-2.2.0
- High and Mediuma CVEs present in latest image (2.2.0)
- TS.MRANGE LATEST HOT 1
- Building
- OSS Redis Cluster: getInstance error: cannot decode redis array into string HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from grafana-redis-datasource.