voxpupuli / puppet-zabbix

Puppet module for creating and maintaining zabbix components with puppet.
https://forge.puppet.com/puppet/zabbix
Apache License 2.0
80 stars 228 forks source link

Require startup script for the zabbix agent service #794

Closed root-expert closed 2 years ago

root-expert commented 2 years ago

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Fixes: #788

root-expert commented 2 years ago

@baron1405 Are you able to give this a try? Note that the master (and this) branch requires puppet-systemd module version 3.1.0.

baron1405 commented 2 years ago

@root-expert I tried this branch and am happy to report that it works! Specifically:

  1. Cloned this repo
  2. Checked out the startup branch
  3. Copied the module to my modules so it appeared as if I wrote another module called zabbix (tweaked the Gemfile to fit in my modules)
  4. Pointed my zabbix_agent module at the module from step 3
  5. Removed my systemd unit workaround and had zabbix manage the startup script once again
  6. Ran the acceptance test for my zabbix_agent module
  7. Acceptance test passed!

Thanks for your efforts to resolve this bug!

root-expert commented 2 years ago

Thanks for the confirmation!