Don't we need this line because we use it in our server.pp for defaults? Looking at those, I think we should look at moving them. puppet_home and puppet_basedir sounds more like puppet than foreman to me.