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

Telegraf post remove script doesn't work on AMZN AMIs #1379

Closed
chebrolus opened this issue Jun 16, 2016 · 2 comments
Closed

Telegraf post remove script doesn't work on AMZN AMIs #1379

chebrolus opened this issue Jun 16, 2016 · 2 comments

Comments

@chebrolus
Copy link
Contributor

chebrolus commented Jun 16, 2016

Directions

GitHub Issues are reserved for actionable bug reports and feature requests.
General questions should be sent to the InfluxDB mailing list.

Before opening an issue, search for similar bug reports or feature requests on GitHub Issues.
If no similar issue can be found, fill out either the "Bug Report" or the "Feature Request" section below.
Erase the other section and everything on and above this line.

Please note, the quickest way to fix a bug is to open a Pull Request.

Bug report

When you run yum remove telegraf on Amazon Linux AMI it doesn't remove init script for telegraf.

Relevant telegraf.conf:

System info:

[Include Telegraf version, operating system name, and other relevant details]
Amazon Linux

Steps to reproduce:

  1. Uninstall telegraf on Amazon Linux AMI
  2. You will notice /etc/init.d/telegraf still exist and same is the case with chkconfig

Expected behavior:

Expect all telegraf related scripts removed

Actual behavior:

Still leave telegraf init script in /etc/init.d

Additional info:

[Include gist of relevant config, logs, etc.]

Feature Request

Opening a feature request kicks off a discussion.

Proposal:

Install script does support Amazon Linux for these scripts and can follow similar conditional approach and remove telegraf related stuff in post remove

Current behavior:

Desired behavior:

Use case: [Why is this important (helps with prioritizing requests)]

chebrolus pushed a commit to chebrolus/telegraf that referenced this issue Jun 16, 2016
chebrolus pushed a commit to chebrolus/telegraf that referenced this issue Jun 17, 2016
@chebrolus
Copy link
Contributor Author

PR #1381

chebrolus pushed a commit to chebrolus/telegraf that referenced this issue Jun 24, 2016
@sjwang90
Copy link
Contributor

fixed

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

2 participants