-
Notifications
You must be signed in to change notification settings - Fork 760
Streamline CHC/Buildkite CI #5958
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
Conversation
|
/merge-queue |
|
π Merge queue workflow triggered! View the run: https://github.com/tensorzero/tensorzero/actions/runs/21460544258 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π‘ Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 71a4ec0a8e
βΉοΈ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with π.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
|
/merge-queue |
|
π Merge queue workflow triggered! View the run: https://github.com/tensorzero/tensorzero/actions/runs/21461375665 |
|
/merge-queue |
|
π Merge queue workflow triggered! View the run: https://github.com/tensorzero/tensorzero/actions/runs/21462602161 |
|
/merge-queue |
|
π Merge queue workflow triggered! View the run: https://github.com/tensorzero/tensorzero/actions/runs/21463752139 |
Note
Medium Risk
Moderate risk because it changes CI orchestration and ClickHouse database cleanup/truncation logic, which could cause flaky runs or unintended DB deletions if patterns/time windows are wrong.
Overview
Speeds up and hardens ClickHouse Cloud e2e CI by enabling shallow clones, starting an early background "wake" query for idle ClickHouse instances, and compiling ClickHouse tests in parallel with fixture loading.
Improves reliability by adding retries to tool installer downloads and tuning ClickHouse HTTP
curlretry/timeouts.Broadens and delays stale DB cleanup (
tensorzero%older than 3h), standardizes the migration-manager DB name format (tensorzero_e2e_tests_migration_manager_*), and updates fixture loading to dynamically truncate all non-view tables (excluding internal/migration/config tables) instead of truncating a hardcoded list.Written by Cursor Bugbot for commit 3287122. This will update automatically on new commits. Configure here.