Skip to content

Commit 133834c

Browse files
authored
Limit concurrency of ci_windows_x64_msvc. (#20338)
Progress on #18813. There's a lot that is fundamentally broken here (see notes on that issue). I want to see if an incremental approach lets us at run at some increased frequency, but otherwise I'll need to revert #20306 and switch this back to only running once a day. ci-exactly: windows_x64_msvc
1 parent 9e6b0a6 commit 133834c

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

.github/workflows/ci_windows_x64_msvc.yml

+10
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,13 @@ on:
1010
workflow_call:
1111
workflow_dispatch:
1212

13+
concurrency:
14+
# Limit to only one run at a time on our current self-hosted Windows runners.
15+
# The network mount and build dir technically functions with concurrent jobs
16+
# but performance degrades to the point of being unusably slow.
17+
group: ${{ github.workflow }}
18+
cancel-in-progress: false
19+
1320
jobs:
1421
windows_x64_msvc:
1522
runs-on: azure-windows-scale
@@ -24,6 +31,7 @@ jobs:
2431
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
2532
with:
2633
submodules: true
34+
# TODO(#18813): Remove this. Environment setup needs to happen on runners, not in workflows.
2735
- name: "Create build dir"
2836
run: |
2937
$currentTime = (Get-Date).ToString('HHmmss')
@@ -43,6 +51,8 @@ jobs:
4351
python3 -m venv .venv
4452
.venv/Scripts/activate.bat
4553
python3 -m pip install -r runtime/bindings/python/iree/runtime/build_requirements.txt
54+
# TODO(#18813): Fix to actually use either ccache or sccache.
55+
# Not need to install both, and neither is currently used.
4656
- name: "Installing requirements"
4757
run: choco install ccache --yes
4858
- name: "Configuring MSVC"

0 commit comments

Comments
 (0)