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 #16518: Missing report on "Synchronize files" on simple relays when shared-files are empty #1575

Conversation

amousset
Copy link
Member

@amousset amousset commented Jan 9, 2020

peckpeck and others added 30 commits December 13, 2018 17:39
Fdall and others added 23 commits November 13, 2019 14:37
…lso_remove_remove_limits_conf_from_rsyslog_config_pr' into branches/rudder/6.0
@@ -158,7 +158,8 @@ bundle agent propagatePromises

pass3.files_propagated.sync_method_rsync::
"any" usebundle => rudder_common_report("DistributePolicy", "result_success", "${distribute_policy_common.directiveId}", "Synchronize files", "None", "Shared files synchronized using rsync");
pass3.(files_propagated|(could_not_propagate_files.update_successful)).!sync_method_rsync::
# Cannot directly use seccess class as it is not defined if there are no files to copy
Copy link
Member

Choose a reason for hiding this comment

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

seccess => success

Copy link
Member Author

Choose a reason for hiding this comment

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

👁️

…e relays when shared-files are empty

Fixes #16518: Missing report on \"Synchronize files\" on simple relays when shared-files are empty
@amousset
Copy link
Member Author

amousset commented Jan 9, 2020

PR updated with a new commit

…n simple relays when shared-files are empty

Fixes #16518: Missing report on \"Synchronize files\" on simple relays when shared-files are empty
@amousset
Copy link
Member Author

amousset commented Jan 9, 2020

PR updated with a new commit

@amousset amousset changed the base branch from branches/rudder/6.0 to branches/rudder/5.0 January 9, 2020 16:56
@amousset amousset closed this Jan 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants