Giter Site home page Giter Site logo

Comments (7)

mikhail-vl avatar mikhail-vl commented on June 20, 2024 1

@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.

dpipemazo avatar dpipemazo commented on June 20, 2024 1

Sure, thanks for helping to bring it to their attention.

from grafana-redis-datasource.

mikhail-vl avatar mikhail-vl commented on June 20, 2024

@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.

dpipemazo avatar dpipemazo commented on June 20, 2024

@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.

mrange.txt

from grafana-redis-datasource.

dpipemazo avatar dpipemazo commented on June 20, 2024

I did go ahead and file the issue on the Redis Time Series repo as well: RedisTimeSeries/RedisTimeSeries#506

from grafana-redis-datasource.

mikhail-vl avatar mikhail-vl commented on June 20, 2024

@dpipemazo Thank you, let's wait for the RedisTimeSeries team response.

from grafana-redis-datasource.

mikhail-vl avatar mikhail-vl commented on June 20, 2024

@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)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo 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.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.