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 #4840: Job Scheduler Technique will miss some jobs when using a non-default agent execution frequency #1370
Fixes #4840: Job Scheduler Technique will miss some jobs when using a non-default agent execution frequency #1370
Conversation
@@ -56,6 +56,29 @@ body classes rudder_common_classes_persist(prefix, persist) | |||
persist_time => ${persist}; | |||
} | |||
|
|||
|
|||
body classes rudder_common_classes_exclusive_persist_codes(prefix, persist, keptcodes, repairedcodes, errorcodes) |
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.
Ideally, this should go into ncf lib.
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.
well, rudder-stdlib is full of deprectated stuff
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.
i wasn't sure about the ncf part - it's a rudder_ something, and that's not commont to have rudder_ stuff there
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.
You can rename it to remove rudder in the name, it is a generic body.
Commit modified |
c81baf7
to
4407315
Compare
Commit modified |
pass2:: | ||
# check if the job should run | ||
"schedule_${iterator}" usebundle => | ||
schedule_simple("${job_scheduler_canon_name[${iterator}]}_RudderUniqueID", "${g.agent_run_interval}", 0, "${max_execution_delay_hours_${iterator}}", "0", "${job_scheduler_time_min[${iterator}]}", "0", "0", "24", "0", "nodups"); |
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.
We should have a fallback on 5 mintues in case system techniques are not up to date.
4407315
to
a9130b8
Compare
Commit modified |
… non-default agent execution frequency
a9130b8
to
aa1ce5b
Compare
Commit modified |
to be merged with Normation/ncf#884 |
OK, merging this PR |
https://issues.rudder.io/issues/4840