[PATCH] D40607: [zorg] Add linaro-d05-* builders

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 1 02:45:14 PST 2017


rovka added a comment.

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?


https://reviews.llvm.org/D40607





More information about the llvm-commits mailing list