We manage buildbot masters with puppet. We don't, however, manage the buildbot.tac file for the master. An instance where this would be great is when setting up a Tegra master. They need to have a special value for MAX_BROKER_REFS. The bug this would have been useful for, and the solution needs to not reverse behaviour from, is bug 708115.
any reason buildbot masters cannot also call into slavealloc and get their tac file from it? going to assign this to me as part of briar patch work
-0 on using slavealloc for this. masters are static end points, there's not much dynamic about them. we also use puppet to manage almost everything else about them. +0 or +1 if you want to integrate with production-masters.json somehow, be it with slavealloc or puppet
going to say this should be done with puppet rather than slavealloc.
Looks like we do this now: https://github.com/mozilla/build-puppet/blob/master/modules/buildmaster/manifests/buildbot_master/mozilla.pp#L49