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

Fixes #8813: Use rudder agent commands in system techniques #1004

Conversation

peckpeck
Copy link
Member

@peckpeck peckpeck commented Aug 4, 2016

# Add Rudder entry
"/var/spool/cron/crontabs/root"
create => "true",
perms => mog("600", "root", "cron"),
edit_line => insert_lines("0,5,10,15,20,25,30,35,40,45,50,55 * * * * /usr/bin/test -x /opt/rudder/bin/check-rudder-agent \&\& /opt/rudder/bin/check-rudder-agent >/dev/null"),
edit_line => insert_lines("0,5,10,15,20,25,30,35,40,45,50,55 * * * * /usr/bin/test -x /opt/rudder/bin/check-rudder-agent \&\& /opt/rudder/bin/rudder agent check >/dev/null"),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need the -x test on /opt/rudder/bin/check-rudder-agent if you're not going to call it :)

@peckpeck
Copy link
Member Author

peckpeck commented Aug 4, 2016

Commit modified

@peckpeck peckpeck force-pushed the arch_8813/use_rudder_agent_commands_in_system_techniques branch from 5b0507f to 4707c3d Compare August 4, 2016 14:00
@@ -2,11 +2,11 @@
#
# Will manually run cf-agent in case cf-execd is no longer running. cf-agent will fire up a new cf-execd.
#
# To temporarily avoid this behaviour, touch /opt/rudder/etc/disable-agent.
# To temporarily avoid this behaviour, run 'ruddre agent disable'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/ruddre/rudder/

@peckpeck
Copy link
Member Author

peckpeck commented Aug 4, 2016

Commit modified

@peckpeck peckpeck force-pushed the arch_8813/use_rudder_agent_commands_in_system_techniques branch from 4707c3d to c8f954b Compare August 4, 2016 14:50
@Normation-Quality-Assistant
Copy link
Contributor

OK, merging this PR

@Normation-Quality-Assistant Normation-Quality-Assistant merged commit c8f954b into Normation:master Aug 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants