Commit 211c2b3
authored
Bump CPU limit on GHA runners (#3485)
[ci]
Another attempt at improving on DACH-NY/cn-test-failures#6681
See the last few comments there.
I thought about picking a bigger runner just for the ones that failed. But:
- it seems like arbitrary large ones could fail atm
- we don't need more memory, just more CPU
- I don't see a significant downside in bumping limits; I'm keeping requests the same, so shouldn't impact costs, and it seems like we get those CPU usage spikes rarely enough to not interfere with other workloads... will monitor the situation ofc
Open to reverting this again if we do find a better theory / solution... Specifically I still have no idea why we sometimes need more CPU.
Maybe we're getting CPU throttled for some weird k8s reason (that should improve by raising limits). Maybe we should configure more verbose logging on the runners and that will tell us. But honestly if this bump fixes it I'm happy enough to not dig deeper...
Signed-off-by: Martin Florian <martin.florian@digitalasset.com>1 parent 5c9c6f0 commit 211c2b3
1 file changed
+5
-5
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
56 | 56 | | |
57 | 57 | | |
58 | 58 | | |
59 | | - | |
| 59 | + | |
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
| |||
71 | 71 | | |
72 | 72 | | |
73 | 73 | | |
74 | | - | |
| 74 | + | |
75 | 75 | | |
76 | 76 | | |
77 | 77 | | |
| |||
86 | 86 | | |
87 | 87 | | |
88 | 88 | | |
89 | | - | |
| 89 | + | |
90 | 90 | | |
91 | 91 | | |
92 | 92 | | |
| |||
101 | 101 | | |
102 | 102 | | |
103 | 103 | | |
104 | | - | |
| 104 | + | |
105 | 105 | | |
106 | 106 | | |
107 | 107 | | |
| |||
116 | 116 | | |
117 | 117 | | |
118 | 118 | | |
119 | | - | |
| 119 | + | |
120 | 120 | | |
121 | 121 | | |
122 | 122 | | |
| |||
0 commit comments