Skip to content

chore(ci): reduce cache conflict and usage#7642

Open
erickguan wants to merge 10 commits into
mainfrom
reduce-cache-usage
Open

chore(ci): reduce cache conflict and usage#7642
erickguan wants to merge 10 commits into
mainfrom
reduce-cache-usage

Conversation

@erickguan
Copy link
Copy Markdown
Member

@erickguan erickguan commented May 30, 2026

Which issue does this PR close?

None.

Rationale for this change

Reduce cache conflict among jobs.

What changes are included in this PR?

  • Concurrency group for each binding
  • Binding test will build cache firstly and reuse them in individual tests.
  • Slightly reduce cache usage
  • Various fixes for looking up dependency files
  • Bump CI action versions

Are there any user-facing changes?

None

AI Usage Statement

No.

@erickguan erickguan requested review from Xuanwo and tisonkun as code owners May 30, 2026 03:01
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" labels May 30, 2026
Comment thread .github/workflows/ci_bindings_c.yml Outdated

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event_name }}
group: binding_c-${{ github.ref }}-${{ github.event_name }}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why this change? Isn't it identical?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I want to share the same concurrency group with .github/workflows/test_behavior_binding_c.yml.

Each binding shares the same concurrency group. If there are multiple workflows for a binding, these workflows will run in sequence.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Rollbacked. Tweaking concurrency groups brought more troubles than its worth.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/ci The PR modifies CI-related content or has a title that begins with "ci" size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants