I like the general approach (apart from the app_host_name issue) but this is not the best variable name: as a user I'd expect that anything called `puppet_modules_location` would point to the puppet modules directory itself, not to the parent directory thereof.