Skip to content

Conversation

@gewenyu99
Copy link
Contributor

Because ingestion takes time and the query apis are cached, e2e runs every night, but checks if yesterdays' events were properly captured.

We'll set up alerting. knowing 1 day later is better than never. We could also just run these ~2 hours after integration nightly on the same day, but is flakier.

how these work

integration generates dated ID -> identifies -> captures events -> 24 hours later -> e2e test queries yesterday's captures to make sure prev. events made it through ingestion.

@daniloc
Copy link
Collaborator

daniloc commented Nov 12, 2025

Dumb question:

Sometimes ingest delay is NOT two hours, right. Sometimes it's a few minutes (does this vary by account?)

could we implement something akin to retry logic that backs off each time? 5 minutes > 20 minutes > 2 hours > 8 hours

Maybe that would be fighting the tools too much, but I think it could work if we could cancel a scheduled job upon receiving our desired outcome.

@gewenyu99
Copy link
Contributor Author

@daniloc I'm down to try this. I'm (probably) gonna ship this for now with a 24 hour delay, and do a separate job with this back off later.

I think if it's not overly complicated, this is the right way to approach it, but weary of how much we'd benefit from knowing an example is broken ~20 hours earlier. For now, I don't think it's worth it if doing back off retry proves difficult.

Can't know until I try!

Copy link
Collaborator

@daniloc daniloc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do eeeeet

@daniloc
Copy link
Collaborator

daniloc commented Nov 13, 2025

fair enough!

@gewenyu99
Copy link
Contributor Author

Aight this :should: work, did a bunch of tests. But we have alerts, so we'll find out tonight

@gewenyu99 gewenyu99 merged commit 05f827d into main Nov 13, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants