Skip to content

Comments

deployment: add graphite ci-optimizations upload_artifacts_workflow#12777

Open
idan-starkware wants to merge 2 commits intoidan/main-v0.14.2/graphite-ci-optimizations-sequencer_cdk8s-testfrom
idan/main-v0.14.2/graphite-ci-optimizations-upload_artifacts_workflow
Open

deployment: add graphite ci-optimizations upload_artifacts_workflow#12777
idan-starkware wants to merge 2 commits intoidan/main-v0.14.2/graphite-ci-optimizations-sequencer_cdk8s-testfrom
idan/main-v0.14.2/graphite-ci-optimizations-upload_artifacts_workflow

Conversation

@idan-starkware
Copy link
Contributor

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

Note

Low Risk
Workflow-only change that may skip artifact uploads on PRs; main risk is inadvertently skipping needed builds if the optimizer is misconfigured or the secret/token is missing.

Overview
Integrates Graphite’s CI optimizer into upload_artifacts_workflow.yml by adding a new optimize_ci job.

Makes native-blockifier-artifacts-push depend on optimize_ci and conditionally run on PRs only when the optimizer returns skip == 'false' (while continuing to run for non-PR events).

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

@reviewable-StarkWare
Copy link

This change is Reviewable

@idan-starkware idan-starkware marked this pull request as ready for review February 22, 2026 12:30
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-upload_artifacts_workflow branch from 224e1dc to f755483 Compare February 22, 2026 12:39
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-upload_artifacts_workflow branch from f755483 to f772a8c Compare February 22, 2026 12:43
@idan-starkware idan-starkware changed the title deployment: add graphite ci-optimizations deployment: add graphite ci-optimizations upload_artifacts_workflow Feb 22, 2026
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-sequencer_cdk8s-test branch from cdae0bc to d898daa Compare February 22, 2026 13:08
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-upload_artifacts_workflow branch from ca2652b to c909f18 Compare February 22, 2026 13:08
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-sequencer_cdk8s-test branch from d898daa to b6905d4 Compare February 22, 2026 13:13
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-upload_artifacts_workflow branch from c909f18 to 3b2cd65 Compare February 22, 2026 13:13
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-sequencer_cdk8s-test branch from b6905d4 to cf6e3bb Compare February 22, 2026 13:16
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-upload_artifacts_workflow branch from 3b2cd65 to 81e235a Compare February 22, 2026 13:16
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-upload_artifacts_workflow branch from 81e235a to 23133d8 Compare February 22, 2026 13:31
@idan-starkware idan-starkware force-pushed the idan/main-v0.14.2/graphite-ci-optimizations-sequencer_cdk8s-test branch from cf6e3bb to 20cd862 Compare February 22, 2026 13:31
@github-actions
Copy link

Artifacts upload workflows:

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.

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