jaytaylor / ansible-kafka

Ansible Kafka role
https://galaxy.ansible.com/jaytaylor/kafka/
Other
67 stars 52 forks source link

Wrong path added to profile.d #28

Open cmontemuino opened 7 years ago

cmontemuino commented 7 years ago

In the ansible-kafka/tasks/system.yml, we have the following task:

- name: "Add kafka utilities to /etc/profile* so they'll be available in $PATH"
  lineinfile: dest=/etc/profile.d/Z99-kafka.sh regexp="^export PATH=.*$" line='export PATH="${PATH}:/usr/local/lib/kafka/bin"' state=present create=yes
sudo: yes

Please bear in mind that /usr/local/lib/kafka/bin is wrong. It should be /usr/local/kafka/bin instead.

jaytaylor commented 7 years ago

Hey @cmontemuino, thanks for the taking the time to file this issue.

A PR with a fix would be welcome and appreciated!

kknd22 commented 6 years ago

I encounter this as well - will create PR shortly

lhoss commented 6 years ago

PR merged, case closed 👍