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

Restore script /etc/init.d/zabbix-agent from defaults #265

Closed
wants to merge 1 commit into from
Closed

Restore script /etc/init.d/zabbix-agent from defaults #265

wants to merge 1 commit into from

Conversation

caiohasouza
Copy link

No description provided.

@bastelfreak
Copy link
Member

This is one of the reasons why I didn't want to manage the whole file with puppet :(
I'm afraid that we've to check the init script for RHEL/Debian and Zabbix2/3 and find the differences...

@caiohasouza
Copy link
Author

Right, i think that better not manage this file too.

@bastelfreak
Copy link
Member

We need more debugging here. I guess the upstream init scripts for zabbix2 and 3 differ, and probably also the ones for Debian vs RHEL. I'm not sure if I find the time to debug it in the next few days. Maybe someone else can.

@bastelfreak
Copy link
Member

@XDexter On which RHEL version are you and which zabbix version?

@caiohasouza
Copy link
Author

Hello @bastelfreak.

Centos 6.8
Zabbix 2.4.8

@HT43-bqxFqB
Copy link

HT43-bqxFqB commented Oct 10, 2016

On CentOS 6.8 the binary file for the agent is under /usr/sbin/ not under /usr/local/sbin/. Currently there is no way to configure this in the init script. Better option would be to just let the package maintainer handle that and leave the init scripts/unit files how they were on installation. Or at least provide a manage_init_file parameter to control the creation of systemd/init-v scripts

@bastelfreak
Copy link
Member

Hi @XDexter and @HT43-bqxFqB, we recently introduced this:
https://github.com/voxpupuli/puppet-zabbix/blob/master/templates/zabbix-agent-redhat.init.erb#L21

Does it fix your issues?

@caiohasouza
Copy link
Author

Hello @bastelfreak

Yes, thanks!

Att;

@HT43-bqxFqB
Copy link

Fixed, thanks!

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

Successfully merging this pull request may close these issues.

3 participants