{"action":"created","issue":{"url":"https://api.github.com/repos/deric/puppet-zookeeper/issues/9","labels_url":"https://api.github.com/repos/deric/puppet-zookeeper/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/deric/puppet-zookeeper/issues/9/comments","events_url":"https://api.github.com/repos/deric/puppet-zookeeper/issues/9/events","html_url":"https://github.com/deric/puppet-zookeeper/pull/9","id":54607834,"number":9,"title":"Java requirement","user":{"login":"coreone","id":739341,"avatar_url":"https://avatars.githubusercontent.com/u/739341?v=3","gravatar_id":"","url":"https://api.github.com/users/coreone","html_url":"https://github.com/coreone","followers_url":"https://api.github.com/users/coreone/followers","following_url":"https://api.github.com/users/coreone/following{/other_user}","gists_url":"https://api.github.com/users/coreone/gists{/gist_id}","starred_url":"https://api.github.com/users/coreone/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coreone/subscriptions","organizations_url":"https://api.github.com/users/coreone/orgs","repos_url":"https://api.github.com/users/coreone/repos","events_url":"https://api.github.com/users/coreone/events{/privacy}","received_events_url":"https://api.github.com/users/coreone/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-01-16T18:25:02Z","updated_at":"2015-01-20T16:34:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/deric/puppet-zookeeper/pulls/9","html_url":"https://github.com/deric/puppet-zookeeper/pull/9","diff_url":"https://github.com/deric/puppet-zookeeper/pull/9.diff","patch_url":"https://github.com/deric/puppet-zookeeper/pull/9.patch"},"body":"I found when installing zookeeper that some systems did not have Java installed already, so I added the ability to set a flag to require a Java package be installed, along with a variable to set the name of the Java package.  This was only done for RHEL systems so far since I don't have access to a Debian one right now to test on, but it should be easy enough to potentially just copy the same code from redhat.pp to debian.pp to get it working."},"comment":{"url":"https://api.github.com/repos/deric/puppet-zookeeper/issues/comments/70684796","html_url":"https://github.com/deric/puppet-zookeeper/pull/9#issuecomment-70684796","issue_url":"https://api.github.com/repos/deric/puppet-zookeeper/issues/9","id":70684796,"user":{"login":"coreone","id":739341,"avatar_url":"https://avatars.githubusercontent.com/u/739341?v=3","gravatar_id":"","url":"https://api.github.com/users/coreone","html_url":"https://github.com/coreone","followers_url":"https://api.github.com/users/coreone/followers","following_url":"https://api.github.com/users/coreone/following{/other_user}","gists_url":"https://api.github.com/users/coreone/gists{/gist_id}","starred_url":"https://api.github.com/users/coreone/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coreone/subscriptions","organizations_url":"https://api.github.com/users/coreone/orgs","repos_url":"https://api.github.com/users/coreone/repos","events_url":"https://api.github.com/users/coreone/events{/privacy}","received_events_url":"https://api.github.com/users/coreone/received_events","type":"User","site_admin":false},"created_at":"2015-01-20T16:34:20Z","updated_at":"2015-01-20T16:34:20Z","body":"I tried another pass at this and it doesn't seem to work correctly either.  I also talked to my coworker who runs our Puppet service here, and he suggested that I just use a Java module to require the Java package on the system instead of trying to build a Java installation into a separate module.  So, do you think we should just abandon this and not try to do a Java installation inside the Zookeeper module?  I'm up for just using an outside module if that makes more sense to you, so feel free to delete this pull request if that sounds reasonable.\r\n\r\nThanks for all the anchor help, though!"}}