{"action":"created","issue":{"url":"https://api.github.com/repos/puppet-community/puppet-kafka/issues/9","labels_url":"https://api.github.com/repos/puppet-community/puppet-kafka/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/puppet-community/puppet-kafka/issues/9/comments","events_url":"https://api.github.com/repos/puppet-community/puppet-kafka/issues/9/events","html_url":"https://github.com/puppet-community/puppet-kafka/issues/9","id":74987029,"number":9,"title":"Both \"version\" and \"scala_version\" are ignored within \"kafka::broker\"","user":{"login":"loki2302","id":906606,"avatar_url":"https://avatars.githubusercontent.com/u/906606?v=3","gravatar_id":"","url":"https://api.github.com/users/loki2302","html_url":"https://github.com/loki2302","followers_url":"https://api.github.com/users/loki2302/followers","following_url":"https://api.github.com/users/loki2302/following{/other_user}","gists_url":"https://api.github.com/users/loki2302/gists{/gist_id}","starred_url":"https://api.github.com/users/loki2302/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/loki2302/subscriptions","organizations_url":"https://api.github.com/users/loki2302/orgs","repos_url":"https://api.github.com/users/loki2302/repos","events_url":"https://api.github.com/users/loki2302/events{/privacy}","received_events_url":"https://api.github.com/users/loki2302/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-10T19:39:57Z","updated_at":"2015-05-12T01:32:39Z","closed_at":null,"body":"```puppet\r\nclass { 'kafka::broker':\r\n  version => '0.8.2.1',\r\n  scala_version => '2.10',\r\n  install_java => true\r\n}\r\n```\r\nInstead of 2.10 there's 2.8.0 and instead of 0.8.2.1 there's 0.8.1.1:\r\n```\r\n...\r\nNotice: /Stage[main]/Kafka/File[/var/log/kafka]/ensure: created\r\nNotice: /Stage[main]/Kafka/File[/opt/kafka-2.8.0-0.8.1.1]/ensure: created\r\nNotice: /Stage[main]/Kafka/File[/opt/kafka]/ensure: created\r\n...\r\n```"},"comment":{"url":"https://api.github.com/repos/puppet-community/puppet-kafka/issues/comments/101091424","html_url":"https://github.com/puppet-community/puppet-kafka/issues/9#issuecomment-101091424","issue_url":"https://api.github.com/repos/puppet-community/puppet-kafka/issues/9","id":101091424,"user":{"login":"dominics","id":97427,"avatar_url":"https://avatars.githubusercontent.com/u/97427?v=3","gravatar_id":"","url":"https://api.github.com/users/dominics","html_url":"https://github.com/dominics","followers_url":"https://api.github.com/users/dominics/followers","following_url":"https://api.github.com/users/dominics/following{/other_user}","gists_url":"https://api.github.com/users/dominics/gists{/gist_id}","starred_url":"https://api.github.com/users/dominics/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dominics/subscriptions","organizations_url":"https://api.github.com/users/dominics/orgs","repos_url":"https://api.github.com/users/dominics/repos","events_url":"https://api.github.com/users/dominics/events{/privacy}","received_events_url":"https://api.github.com/users/dominics/received_events","type":"User","site_admin":false},"created_at":"2015-05-12T01:32:39Z","updated_at":"2015-05-12T01:32:39Z","body":"This issue is not caused by the version being *ignored*. The declaration given above correctly passes the versions to the `::kafka` init class. (If you put a `notify` in there, you'll see it received the correct values.)\r\n\r\nWhat's actually happening is the `$install_dir`/`$install_directory` is incorrectly handled. If you don't specify it to `kafka::broker`, then it uses `$kafka::params::install_dir` (which is always a static default value of `/opt/kafka-2.8.0-0.8.1.1` - built in `params.pp`). This value is eventually passed to `::kafka`, which means the symlink isn't changed, and the downloaded package is never untar'd.\r\n\r\nBasically [this line right here](https://github.com/puppet-community/puppet-kafka/blob/master/manifests/broker.pp#L51) makes no sense at all. \r\n\r\n(Yaaaaay, [another case](https://www.devco.net/archives/2013/12/09/the-problem-with-params-pp.php) of a \"params class\" causing confusion and breaking things.)\r\n\r\n"}}