Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong path added to profile.d #28

Open
cmontemuino opened this issue Nov 2, 2017 · 3 comments
Open

Wrong path added to profile.d #28

cmontemuino opened this issue Nov 2, 2017 · 3 comments

Comments

@cmontemuino
Copy link

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
Copy link
Owner

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

A PR with a fix would be welcome and appreciated!

@kknd22
Copy link
Contributor

kknd22 commented Dec 14, 2017

I encounter this as well - will create PR shortly

kknd22 added a commit to kknd22/ansible-kafka that referenced this issue Dec 14, 2017
jaytaylor#28  - Wrong path added to profile.d
jaytaylor pushed a commit that referenced this issue Dec 18, 2017
#28  - Wrong path added to profile.d
@lhoss
Copy link
Contributor

lhoss commented Feb 2, 2018

PR merged, case closed 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants