[NA] [CI] fix: only cancel-in-progress for PR runs to prevent main branch CI cancellation#5924
[NA] [CI] fix: only cancel-in-progress for PR runs to prevent main branch CI cancellation#5924
Conversation
Python SDK Unit Tests Results (Python 3.13)2 386 tests 2 386 ✅ 57s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 12224 tests 221 ✅ 2m 26s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 11201 tests 201 ✅ 3m 52s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 5139 tests 139 ✅ 2m 49s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 13560 tests 560 ✅ 5m 37s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 7 13 files ±0 13 suites ±0 5m 59s ⏱️ -10s For more details on these errors, see this check. Results for commit 0d35bc4. ± Comparison against base commit 3b5bee4. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 6253 tests 253 ✅ 2m 51s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 8 24 files 24 suites 3m 56s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 41 485 tests 1 485 ✅ 9m 27s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 2266 tests 266 ✅ 20m 42s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 3324 tests 324 ✅ 9m 56s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 15255 tests 255 ✅ 6m 13s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 16 22 files 22 suites 7m 27s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 10230 tests 227 ✅ 8m 55s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 14260 tests 260 ✅ 7m 29s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 9401 tests 398 ✅ 10m 9s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Backend Tests - Integration Group 1 23 files 23 suites 3m 5s ⏱️ Results for commit 7de85aa. ♻️ This comment has been updated with latest results. |
Details
Workflows with both
pull_requestandpushtriggers shared the same concurrency group on main (<workflow>-refs/heads/main). Withcancel-in-progress: true, back-to-back merges would cancel each other's main branch CI runs, causing developers to see cancelled/lost CI results.cancel-in-progressfromtrueto${{ github.event_name == 'pull_request' }}in 19 workflows so only PR runs cancel previous runs on the same PR, while main branch runs queue instead.opik-optimizer-e2e-tests.yamlconcurrency group to includepull_request.numberfor consistency.Change checklist
Issues
AI-WATERMARK
AI-WATERMARK: yes
Testing
cancel-in-progress: ${{ github.event_name == 'pull_request' }}viagrepcancel-in-progressvalueDocumentation
N/A — internal CI configuration change, no user-facing documentation needed.