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

rpm should not be starting service on installation or upgrade #3015

Closed
blieberman opened this issue Jul 13, 2017 · 2 comments · Fixed by #3156
Closed

rpm should not be starting service on installation or upgrade #3015

blieberman opened this issue Jul 13, 2017 · 2 comments · Fixed by #3156
Labels
bug unexpected problem or unintended behavior
Milestone

Comments

@blieberman
Copy link

Bug report

Relevant telegraf.conf:

Conf not relevant.

System info:

Telegraf v1.3.4 (git: release-1.3 7bbd3da)
Amazon Linux latest

Steps to reproduce:

  1. Install or upgrade a telegraf rpm
  2. Wait

Expected behavior:

RPM to not automatically be switching on upon installation or upgrade, as it can break automated deploys or cases where the telegraf is being used under a non built in service manager i.e. supervisord.

Fedora suggest that you should not automatically switch on the service as this is insecure, instead you should rely on the init.d script via chkconfig to switch on the service. https://fedoraproject.org/wiki/EPEL:SysVInitScripts?rd=Packaging:SysVInitScript#Why_don.27t_we....

Actual behavior:

Service is restarted.

Additional info:

Problem appears to be in: https://github.com/influxdata/telegraf/blob/master/scripts/post-install.sh
I can submit a patch if you believe this is a legitimate bug.

@blieberman blieberman changed the title rpm should not be starting service on installation rpm should not be starting service on installation or upgrade Jul 13, 2017
@danielnelson
Copy link
Contributor

Yeah we should change this, but we should make sure that in debian it does start the service automatically.

As an aside, at some point I would like to have separate maintainer scripts for each platform.

@blieberman
Copy link
Author

Sounds good. Thanks for the quick response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants