-
-
Notifications
You must be signed in to change notification settings - Fork 229
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
Using $manage_service in zabbix::proxy #374
Conversation
manifests/proxy.pp
Outdated
Package["zabbix-proxy-${db}"], | ||
File[$include_dir], | ||
File[$proxy_configfile_path]], | ||
if $manage_service == true { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should be enough:
if $manage_service {
Thanks for the PR @Samgarr. Are you able to also add spec tests for this change? |
I'm not sure right now, Ruby is new for me, spec tests also. |
Don't worry. Feel free to join our IRC channel #voxpupuli on freenode. We're happy to help you with the tests. |
56207eb
to
8a9ee67
Compare
manifests/proxy.pp
Outdated
zabbix_proxy => $zabbix_proxy, | ||
zabbix_proxy_ip => $zabbix_proxy_ip, | ||
before => [ | ||
Service[$proxy_service_name], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this the only different line? We shouldt do code duplication here. Please write Service[$proxy_service_name] into a variable and use that in the before parameter.
Signed-off-by: Samgarr <honza@pechek.cz>
ce9ae8a
to
37e7173
Compare
Rebased, spec tests added. |
Thanks! |
This PR allowing use of $manage_service in zabbix::proxy class.