Commit 1d4c656
fbcode_builder: emit Show runner info step and timeout-minutes per generated job
Summary:
Adds two diagnostics to the GitHub Actions workflows generated by `getdeps.py generate-github-actions`:
1. **`Show runner info` step** as the first step in every job, dumping CPU core count, CPU model, and total RAM in OS-appropriate form: `nproc` / `/proc/cpuinfo` / `free -h` on Linux, `sysctl` on macOS, `Get-CimInstance` in pwsh on Windows. Surfaces the runner shape in CI logs so failures with thread-count or memory dependencies are easier to diagnose without manual workflow edits.
2. **`timeout-minutes`** on the build job. Defaults to 60 minutes (vs the 6-hour GitHub default), and can be overridden per-project via `timeout_minutes = N` in the `[github.actions]` section of the project manifest. The `timeout_minutes` field is added to the `[github.actions]` schema in `getdeps/manifest.py`.
Rebalancer's downstream OSS branch (richard/fixes) hand-edits both of these into its three generated workflow YAMLs to compensate for the gap. Centralizing them in the generator means rebalancer (and any other downstream that does the same) stops fighting regenerations. None of the 30 `generated` `getdeps_*.yml` files currently checked into fbsource have these additions, so this diff is a strict superset of the existing CI behavior — projects that don't want the diagnostics simply get them at no behavioral cost; projects that already wanted them no longer need to maintain hand-edits.
Reviewed By: sahilsd
Differential Revision: D101588403
fbshipit-source-id: f67a50755897e6809ea25a6940ca6679e9913c581 parent 7e0576c commit 1d4c656
2 files changed
Lines changed: 31 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1091 | 1091 | | |
1092 | 1092 | | |
1093 | 1093 | | |
| 1094 | + | |
| 1095 | + | |
| 1096 | + | |
1094 | 1097 | | |
1095 | 1098 | | |
1096 | 1099 | | |
| |||
1177 | 1180 | | |
1178 | 1181 | | |
1179 | 1182 | | |
| 1183 | + | |
1180 | 1184 | | |
1181 | 1185 | | |
1182 | 1186 | | |
1183 | 1187 | | |
1184 | 1188 | | |
| 1189 | + | |
| 1190 | + | |
| 1191 | + | |
| 1192 | + | |
| 1193 | + | |
| 1194 | + | |
| 1195 | + | |
| 1196 | + | |
| 1197 | + | |
| 1198 | + | |
| 1199 | + | |
| 1200 | + | |
| 1201 | + | |
| 1202 | + | |
| 1203 | + | |
| 1204 | + | |
| 1205 | + | |
| 1206 | + | |
| 1207 | + | |
| 1208 | + | |
| 1209 | + | |
| 1210 | + | |
| 1211 | + | |
| 1212 | + | |
| 1213 | + | |
| 1214 | + | |
1185 | 1215 | | |
1186 | 1216 | | |
1187 | 1217 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
111 | 111 | | |
112 | 112 | | |
113 | 113 | | |
| 114 | + | |
114 | 115 | | |
115 | 116 | | |
116 | 117 | | |
| |||
0 commit comments