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 #8909: missing rudder_expected_reports.csv.res when starting the agent for the first time after an update - technique changes #1013

Conversation

amousset
Copy link
Member

…e agent for the first time after an update - technique changes
@amousset amousset force-pushed the bug_8909/missing_rudder_expected_reports_csv_res_when_starting_the_agent_for_the_first_time_after_an_update_technique_changes branch from bce2f5d to bdedf6d Compare August 29, 2016 15:11
@amousset
Copy link
Member Author

Commit modified


methods:

# Directly call the cleanup bundle from ncf's logger_rudder
Copy link
Member

Choose a reason for hiding this comment

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

You may as well call this log_rudder, because that's it's future name

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, it should be done for 3.2+

@jooooooon
Copy link
Member

This depends on Normation/ncf#423, don't merge yet

@Normation-Quality-Assistant
Copy link
Contributor

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-techniques/pull/1013 --run-tests
-- Your faithful QA

@amousset
Copy link
Member Author

amousset commented Sep 6, 2016

OK, merging this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants