... which are hosted by 64-core D05 machine.
Remove explicit parallelism settings and rely on ninja
to use correct parallelism values for 8-core APMs and
64-core D05s.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
buildbot/osuosl/master/config/builders.py | ||
---|---|---|
308 ↗ | (On Diff #124758) | After a bit more testing, it appears that lit testsuite won't be parallelized without --threads=N and --build-threads=N settings. I'll rework this patch and resubmit. |
In this version jobs= settings are removed (so that they are picked up from slave setting). testsuite_flags are left with 8-thread parallelism for now, and we are going to switch them to 64 once APM hardware is decommissioned.
The overall plan is to
- Continue running APM hardware on the normal buildmaster.
- Connect D05 slaves to silent buildmaster and monitor them.
- Swap hardware: connect D05 slaves to normal buildmaster and connect APM slaves to silent buildmaster.
- Switch to 64-thread parallelism for the testsuite and remove APM slaves.
- Decommission APMs.
This looks fine, but I think it would be better if the slave names weren't tied to what is currently running on them, but rather to the machine configurations. E.g. if there's no difference in the environment of linaro-d05-01-quick and linaro-d05-global-isel, they should just be linaro-d05-01 and linaro-d05-02. But for instance if the machines running the libcxx bots need something special, then they can be linaro-d05-libcxx-01 and linaro-d05-libcxx-02. That way, if we want to repurpose a slave to run a different builder, it can keep its name without looking awkward, and it's also easy for us to tell which machines have different environments. What do you think?
@rovka , slaves linaro-d05-01-* will run in different containers on the same machine (linaro-d05-01). In this context there is no need to repurpose a slave.
I plan to add a second machine linaro-d05-02 that would host the same set of slaves/builders for redundancy.
True, but that's an implementation detail on our side, not something that is visible in zorg.
Anyway, that's just a nitpick, LGTM.