Skip to content

ci: Ignore concurrency groups for main#324

Merged
hk21702 merged 1 commit intomainfrom
ci-concurrency-cancel
Feb 19, 2026
Merged

ci: Ignore concurrency groups for main#324
hk21702 merged 1 commit intomainfrom
ci-concurrency-cancel

Conversation

@hk21702
Copy link
Copy Markdown
Collaborator

@hk21702 hk21702 commented Feb 17, 2026

The "cancel-in-progress" concurrency option has proved to be problematic at times. Besides the bad interactions with TiCS, sometimes workflows get cancelled, but they never actually get replaced with a fresh one.

Though the TiCS interaction has been roughly resolved previously with a workaround, it appears to be better to just not cancel in-progress runs on main. Since this is an open repository, and we're not using any special GitHub runners, we shouldn't get charged by GitHub for runner usage.

However, simply disabling cancel in progress with the existing setup would mean it could potentially take a very long time to get relevant results. Thus, instead, here we define the concurrency groups such that they should be irrelevant when the ref is main. To ensure that TiCS doesn't break, we add a strict, non-cancelling job-level concurrency group to the "Process coverage reports" job in QA.

@hk21702 hk21702 self-assigned this Feb 17, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.89%. Comparing base (0704bc1) to head (c8c6ca7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #324   +/-   ##
=======================================
  Coverage   88.89%   88.89%           
=======================================
  Files          56       56           
  Lines        5279     5279           
=======================================
  Hits         4693     4693           
  Misses        586      586           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@CarlosNihelton CarlosNihelton left a comment

Choose a reason for hiding this comment

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

LGTM!

@hk21702 hk21702 merged commit 630a23f into main Feb 19, 2026
38 checks passed
@hk21702 hk21702 deleted the ci-concurrency-cancel branch February 19, 2026 14:15
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.

3 participants