-
Notifications
You must be signed in to change notification settings - Fork 38
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 #8871: missing rudder_expected_reports.csv.res when starting the agent for the first time after an update #423
Conversation
"expected_reports_source" string => "${sys.workdir}/inputs/rudder_expected_reports.csv"; | ||
"expected_reports_temp" string => "${expected_reports_source}.tmp"; | ||
"expected_reports_file" string => "${expected_reports_source}.res"; | ||
# We add the PID to the file name to be able to handle cuncurrent agent runs with separate expected reports | ||
"expected_reports_file" string => "${sys.workdir}/state/rudder_expected_reports.${this.promiser_pid}.csv.res"; |
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.
Could you also add a comment here as to why we're placing the file in a different directory? (to avoid purging during update)
Commit modified |
7c7ba41
to
478bf82
Compare
Commit modified |
478bf82
to
9f55b99
Compare
bundle agent _clean_old_expected_reports_file(days) { | ||
|
||
files: | ||
"${sys.workdir}/state/rudder_expected_reports.*.csv.res" |
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.
This should also have .tmp
This needs be moved into log_rudder in master. |
Commit modified |
9f55b99
to
d4f4913
Compare
OK, looks good to me. Let's get some feedback on this before merging. |
…e agent for the first time after an update
d4f4913
to
b42b0e6
Compare
Commit modified |
This PR is not mergeable to upper versions. |
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/8871