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 #15223: Execute a single directive on the agent #247

Conversation

peckpeck
Copy link
Member

@@ -0,0 +1,121 @@
#!/bin/bash
# @description List directives that can be run on this agent
Copy link
Member

Choose a reason for hiding this comment

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

Run a specific list of directives

fi
if [ ! -f "${BASE_PATH}/rudder-directives.csv" ]
then
echo "Your server didn't generate rudder-directives.csv, maybe you are not up to date"
Copy link
Member

Choose a reason for hiding this comment

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

or node isn't accepted yet within Rudder ?

[ "${mode}" = "audit" ] && AUDIT="true"
fi

BS="${BS},set_dry_run_mode_${DRY_RUN}"
Copy link
Member

Choose a reason for hiding this comment

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

how is DRY_RUN defined ?

@peckpeck
Copy link
Member Author

PR updated with a new commit

@ncharles
Copy link
Member

you have conflict in your PR

}

# Check that a bootstrap is necessary
bootstrap_check() {
Copy link
Member

Choose a reason for hiding this comment

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

this is already in 5.1


BS="${BS},rudder_end"

echo rudder agent run ${OPTS} -b "${BS}"
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 want the echo there

@peckpeck
Copy link
Member Author

PR rebased

@peckpeck peckpeck force-pushed the ust_15223/execute_a_single_directive_on_the_agent branch from 2830d28 to 8a81a9f Compare August 26, 2019 10:04
@peckpeck
Copy link
Member Author

PR updated with a new commit

@peckpeck peckpeck force-pushed the ust_15223/execute_a_single_directive_on_the_agent branch from 4658a46 to 27b959d Compare August 26, 2019 10:08
@peckpeck
Copy link
Member Author

Commit modified

@Normation-Quality-Assistant

This PR is not mergeable to upper versions.
Since it is "Ready for merge" you must merge it by yourself using the following command:
rudder-dev merge https://github.com/Normation/rudder-agent/pull/247
-- Your faithful QA
Kant merge: "Morality is not the doctrine of how we may make ourselves happy, but how we may make ourselves worthy of happiness."
(https://ci.normation.com/jenkins/job/merge-accepted-pr/14096/console)

@peckpeck
Copy link
Member Author

OK, squash merging this PR

@peckpeck peckpeck force-pushed the ust_15223/execute_a_single_directive_on_the_agent branch from 27b959d to 2a1bc3e Compare August 28, 2019 10:20
@peckpeck peckpeck merged commit 2a1bc3e into Normation:master Aug 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants