-
Notifications
You must be signed in to change notification settings - Fork 120
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
--excluded-times-of-day ignored the seconds, which causes unexpected killing #178
Comments
If I remember correctly the flag supports wrapping around. You could try |
Thanks @linki --- I cannot remove 23:59:xx execution anyway, and this could be one issue to everyone who wants to run certain time during the day. |
@tonyairwallex I understand. If we assume that seconds default to We could either fail when users specify seconds specifically or somehow parse and support it: We would need to default it to |
@linki Exactly! I like the solution you proposed ~ |
I set the excluded time as:
0:00-13:59,16:00-23:59
and triggered the execution.
Expected result:
Run during 14:00:00 to 15:59:59
(Should exclude 16:00:00 to 23:59:59 rather than 23:59:00)
Actual result:
Killing executed at:13:59:27 and 23:59:27, which are UNEXPECTED.
More importantly, I cannot work out how to remove the potential 23:59:xx execution.
seconds doesn't work in exclude:
--excluded-times-of-day 16:00:00-23:59:59
It failed to run as the parameter only support minutes input.
trying with exclude: 0:00-14:00,16:00-23:59, 23:59-0:00
-- but less optimistic with the third part working. '23:59-0:00'
The text was updated successfully, but these errors were encountered: