-
Notifications
You must be signed in to change notification settings - Fork 489
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
Batch never joined #362
Comments
@AlexGrs I have looked over this several times now and do not see anything wrong. My thinking is that there is a bug in the join code. Could you create a recording of your data so I can try and reproduce this locally. Run this to record the batch data Grab the returned ID Then in Kapacitor data dir typically /var/lib/kapacitor or ~/.kapacitor there is a dir called |
Recording on-going. As soon as I have the file I'll send it to your email address. Done. You have received the records. |
Tell me if you need more infos to reproduce the problem 😄 I'm sure we will manage to solve this ! |
@AlexGrs What version of Kapacitor are you using?
|
kapacitord version |
hmmm, well it works for me. I have used both recordings you sent and I am using the same version and it is working for me. I get several alerts from the joined alert. I did add a Try this: |
I switched to log alert. It works when I replay it: It is the first time i can see those messages ( But when I do that in with 'real time' metrics, it doesn't work. I only see |
Ok i can confirm. I just register the data for 10 minutes. At the same time I was checking my alert log in real time. I only saw alerts coming from my two debug streams but nothing from the join. Then i used your command to replay it : and i also saw the join alerts. |
@AlexGrs Thanks, I am stumped on how it doesn't work for an enabled task but does during a replay. The purpose of the replay system is to be identical to the real thing to test tasks, they only vary but how data is ingested into the task, which should not effect how the join functions... I am digging further in and I'll keep you posted. |
I agree. I'm also stuck on this point. I tried again and see the same. Not working in live, but working with replay. There clearly is something i don't get here. Could it be linked with ingestion speed which is different from a real case ? |
@AlexGrs Good news, I can reproduce the issue. I'll update when I have a fix.... |
Good news !! I will be really happy to test the fix 👓 |
Fix #362, join+batches+tolerance did not work
@AlexGrs the fix has been merged, it was pretty obvious once I found it. Feel free to test using master, or the nightly builds tonight. Thanks |
Indeed it works perfectly :) |
I m still trying to get a working version of my alert for my tests success rates. Because there is a lot of delays in alert with stream, I was thinking about using a batch alert.
I can now successfully have in real time success and total numbers. But as soon as I try to join them to compute a success rate, nothing happens. I never have any alert, even if I print successfully the number of success and total tests before.
The success and total point seems to have the same timestamp (at least it should be match by the tolerance option)
Here is my definition :
The text was updated successfully, but these errors were encountered: