So I like the idea of using the same JAVA_BIN and INSTALLER_DIR from sysconfig, but JAVA_ARGS is questionable, mainly because it dictates things like memory usage etc. This command in particular is a client tool, and may have different tuning characteristics then the service itself. Same goes for puppetdb-export and puppetdb-import. I'm tempted to say this should stay with the defaults from template for now, but I'd love a way to tune on a per tool basis some day.

puppetdb-foreground doesn't have the same rules however, since its actually just an alternate way of starting the service I can't see a problem with it using JAVA_ARGS, and PDB-142 is asking for it.