Skip to content

Comments

deployment: add graphite ci-optimizations for hybrid_system_test#12767

Open
idan-starkware wants to merge 1 commit intomain-v0.14.2from
idan/main-v0.14.2/graphite-ci-optimizations-test-1
Open

deployment: add graphite ci-optimizations for hybrid_system_test#12767
idan-starkware wants to merge 1 commit intomain-v0.14.2from
idan/main-v0.14.2/graphite-ci-optimizations-test-1

Conversation

@idan-starkware
Copy link
Contributor

@idan-starkware idan-starkware commented Feb 22, 2026

Note

Low Risk
Workflow-only change that gates system-test execution based on an external CI-optimizer decision; risk is limited to potentially skipping/triggering tests incorrectly if the action or secret is misconfigured.

Overview
Adds a new optimize_ci job to .github/workflows/hybrid_system_test.yaml using withgraphite/graphite-ci-action and a GRAPHITE_CI_OPTIMIZER_TOKEN secret to determine when the workflow can be skipped.

Updates check-system-test-trigger to depend on optimize_ci and to short-circuit when Graphite indicates the run should be skipped, reducing unnecessary hybrid system test CI executions on PRs.

Written by Cursor Bugbot for commit b4636aa. This will update automatically on new commits. Configure here.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor Author

idan-starkware commented Feb 22, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@idan-starkware idan-starkware marked this pull request as ready for review February 22, 2026 12:08
Copy link

@cursor cursor bot left a 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.

@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-test-1 branch from cddc32a to b4636aa Compare February 22, 2026 13:31
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.

2 participants