[build-tools] Add eas/posthog_wait_for_query workflow function#3949
Open
gwdp wants to merge 1 commit into
Open
[build-tools] Add eas/posthog_wait_for_query workflow function#3949gwdp wants to merge 1 commit into
gwdp wants to merge 1 commit into
Conversation
Contributor
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Jul 3, 2026
1dbcac7 to
eb42509
Compare
9ce8b59 to
aa30fd2
Compare
eb42509 to
cef86d5
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## gwdp/eng-21551-posthog-annotation #3949 +/- ##
=====================================================================
+ Coverage 59.62% 59.65% +0.04%
=====================================================================
Files 943 944 +1
Lines 41467 41503 +36
Branches 8729 8734 +5
=====================================================================
+ Hits 24719 24755 +36
Misses 16654 16654
Partials 94 94 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
cef86d5 to
dcf0c4b
Compare
aa30fd2 to
9c65fdb
Compare
dcf0c4b to
c8039b5
Compare
9c65fdb to
bccd6db
Compare
c8039b5 to
e9b2794
Compare
bccd6db to
a662e94
Compare
e9b2794 to
f4b72c3
Compare
a662e94 to
52dc644
Compare
f4b72c3 to
2c11c6e
Compare
8407075 to
d3e76bf
Compare
2c11c6e to
0327984
Compare
d3e76bf to
b52c51f
Compare
303bdc5 to
c78323e
Compare
b52c51f to
27b3af6
Compare
27b3af6 to
a437436
Compare
c78323e to
150dc15
Compare
|
✅ Thank you for adding the changelog entry! |
|
Subscribed to pull request
Generated by CodeMention Warning: The preamble and epilogue options in commentConfiguration are deprecated. Use template instead. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Why
Gate an EAS workflow on any PostHog condition you can express in HogQL, not just a numeric threshold. Wait until a query is true before continuing. Sibling to
wait_for_metricfor when the condition is a string match, a compound expression, or anything operator/threshold can't say.How
Adds
eas/posthog_wait_for_query. Polls a HogQLquerythat returns a single boolean, until it's true, everyinterval_seconds(default 30) for up totimeout_seconds(default 600). The query is the condition, so PostHog does the comparison instead of us reimplementing it.Same shape as the metric gate: no
ignore_error(timing out fails the step), network/429/5xx retry until the deadline, auth and query errors fail right away. Shares the query transport withwait_for_metric.Test Plan
CI passes. Ran it live against coin-flip:
query_truecontinues right away andquery_timeoutfails on timeout as intended. Test workflow below (true, timeout, and a string-condition job).Test workflow