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
Fixes #5845: Add portable init services handling in the Techniques #574
Fixes #5845: Add portable init services handling in the Techniques #574
Conversation
95a756e
to
35c2b39
Compare
@@ -128,6 +121,14 @@ bundle agent check_openvpn_client_configuration | |||
|
|||
methods: | |||
|
|||
openvpn_bin_present.ovpn_client_autostart:: |
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.
ovpn_client_autostart was not there when using commands, see below
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.
It is defined by:
# Autostart ?
"ovpn_client_autostart" expression => strcmp("&OVPN_CLIENT_AUTOSTART&","true");
Actually, I do want to only bootstart ovpn if this class is here :)
Cool, a lot of better code ! |
35c2b39
to
d186dcf
Compare
Updated ! |
|
||
"restart_apache" usebundle => service_restart("httpd"); | ||
|
||
!redhat.apache_bin_present.apache_unbootstarted:: |
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.
The apache_unbootstarted class was defined by some code that you removed above. You can't use it, therefore. But you shuoldn't have too, the point of the method you're calling is to ensure it's set to start on boot, so you can just call it unconditionally!
d186dcf
to
41a8c36
Compare
Updated ! |
41a8c36
to
cbc4e27
Compare
…_support_to_techniques Fixes #5845: Add portable init services handling in the Techniques
…_not_use_non_posix_shell_bits Fixes #6032: Prevent rudder-inventory-ldap post installation script to o...
Ticket: http://www.rudder-project.org/redmine/issues/5845
To be merged in 3.0