Skip to content

Commit 27f2ec4

Browse files
[CI] Change Windows Machine Shape to 32 CPU
This patch changes the windows machine shape from 64 core machines to 32 core machines but bumps the maximum number of nodes/runners to 16 (for a total of 512 cores allocated to Windows testing). We are currently running into some queueing delays on Windows that are higher than desired. Lowering the core count but increasing the number of nodes will allow higher throughput as there are quite a few serial steps in the build process so for the vast majority of some builds, parallelism isn't even being used (like some of the leaf projects) due to the majority of the time being taken up setting up caches, CMake configures, and cloning the repository.
1 parent ce2709f commit 27f2ec4

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

premerge/main.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,13 +103,13 @@ resource "google_container_node_pool" "llvm_premerge_windows" {
103103

104104
autoscaling {
105105
total_min_node_count = 0
106-
total_max_node_count = 8
106+
total_max_node_count = 16
107107
}
108108

109109
# We do not set a taint for the windows nodes as kubernetes by default sets
110110
# a node.kubernetes.io/os taint for windows nodes.
111111
node_config {
112-
machine_type = "n2-standard-64"
112+
machine_type = "n2-standard-32"
113113
labels = {
114114
"premerge-platform" : "windows"
115115
}

premerge/windows_runner_values.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ githubConfigUrl: "https://github.com/llvm"
22
githubConfigSecret: "github-token"
33

44
minRunners: 0
5-
maxRunners: 8
5+
maxRunners: 16
66

77
template:
88
metadata:
@@ -24,7 +24,7 @@ template:
2424
- name: runner
2525
resources:
2626
requests:
27-
cpu: 55
27+
cpu: 28
2828
image: ghcr.io/llvm/ci-windows-2019:latest
2929
command: ["run.cmd"]
3030
args: ["--jitconfig", "$(ACTIONS_RUNNER_INPUT_JITCONFIG)"]

0 commit comments

Comments
 (0)