{"action":"created","issue":{"url":"https://api.github.com/repos/electrical/puppet-lib-file_concat/issues/8","labels_url":"https://api.github.com/repos/electrical/puppet-lib-file_concat/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/electrical/puppet-lib-file_concat/issues/8/comments","events_url":"https://api.github.com/repos/electrical/puppet-lib-file_concat/issues/8/events","html_url":"https://github.com/electrical/puppet-lib-file_concat/pull/8","id":59277329,"number":8,"title":"(MAINT) Remove unnecessary require","user":{"login":"cprice404","id":1341071,"avatar_url":"https://avatars.githubusercontent.com/u/1341071?v=3","gravatar_id":"","url":"https://api.github.com/users/cprice404","html_url":"https://github.com/cprice404","followers_url":"https://api.github.com/users/cprice404/followers","following_url":"https://api.github.com/users/cprice404/following{/other_user}","gists_url":"https://api.github.com/users/cprice404/gists{/gist_id}","starred_url":"https://api.github.com/users/cprice404/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cprice404/subscriptions","organizations_url":"https://api.github.com/users/cprice404/orgs","repos_url":"https://api.github.com/users/cprice404/repos","events_url":"https://api.github.com/users/cprice404/events{/privacy}","received_events_url":"https://api.github.com/users/cprice404/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-02-27T18:47:24Z","updated_at":"2015-03-10T21:42:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/electrical/puppet-lib-file_concat/pulls/8","html_url":"https://github.com/electrical/puppet-lib-file_concat/pull/8","diff_url":"https://github.com/electrical/puppet-lib-file_concat/pull/8.diff","patch_url":"https://github.com/electrical/puppet-lib-file_concat/pull/8.patch"},"body":"Due to some issues in the way that Puppet loads code, and some\r\ndifferences in how that works in JRuby vs MRI Ruby, doing a\r\n`require` on the `puppet/type/file` code multiple times in a\r\nsingle JRuby interpreter will cause problems.  Since this code\r\nautomatically gets loaded by Puppet before any custom\r\ntype/provider is loaded, the `require` statement is unnecessary.\r\n\r\nThis is related to the following Puppet Jira tickets:\r\n\r\nhttps://tickets.puppetlabs.com/browse/SERVER-64\r\nhttps://tickets.puppetlabs.com/browse/SERVER-40\r\n\r\nand also to the following puppet-logstash PR:\r\n\r\nhttps://github.com/elasticsearch/puppet-logstash/pull/196\r\n\r\nHopefully there will be a more permanent fix in JRuby at some point:\r\n\r\nhttps://github.com/jruby/jruby/issues/2477\r\n\r\nBut in the meantime, this simple workaround makes `file_concat`\r\ncompatible with Puppet Server."},"comment":{"url":"https://api.github.com/repos/electrical/puppet-lib-file_concat/issues/comments/78154718","html_url":"https://github.com/electrical/puppet-lib-file_concat/pull/8#issuecomment-78154718","issue_url":"https://api.github.com/repos/electrical/puppet-lib-file_concat/issues/8","id":78154718,"user":{"login":"Sharpie","id":54181,"avatar_url":"https://avatars.githubusercontent.com/u/54181?v=3","gravatar_id":"","url":"https://api.github.com/users/Sharpie","html_url":"https://github.com/Sharpie","followers_url":"https://api.github.com/users/Sharpie/followers","following_url":"https://api.github.com/users/Sharpie/following{/other_user}","gists_url":"https://api.github.com/users/Sharpie/gists{/gist_id}","starred_url":"https://api.github.com/users/Sharpie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sharpie/subscriptions","organizations_url":"https://api.github.com/users/Sharpie/orgs","repos_url":"https://api.github.com/users/Sharpie/repos","events_url":"https://api.github.com/users/Sharpie/events{/privacy}","received_events_url":"https://api.github.com/users/Sharpie/received_events","type":"User","site_admin":false},"created_at":"2015-03-10T21:42:19Z","updated_at":"2015-03-10T21:42:19Z","body":":+1: Just ran into this when deploying ELK on Puppet Server. Will have to point r10k at Chris's fork until this patch lands."}}