JAVA_HOME shouldn't be hardcoded here.  We should change tomcat::instance to have a "java_home" parameter that you can pass in to specify it. That "java_home" param probably should have a default value of a "typical" Java 7 home...but folks can pass in a different value if they wish.