-
Notifications
You must be signed in to change notification settings - Fork 34
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
Incorrect schedule in certain timezone when using hash cron extention #67
Comments
Hello @phibid, what version of fugit is used by Gitlab? Thanks in advance. |
They're using fugit 1.2.1, https://gitlab.com/gitlab-org/gitlab/-/blob/master/Gemfile.lock#L436 |
@phibid thanks for reporting that. I'm seeing the issue with fugit 1.2.1 and the current 1.5.3 pre. Although it points at the fifth Tuesday of May for me, 2022-05-31 but not the 2022-05-24 you seem to be seeing. I will attempt to fix that. Give me a few days. |
Ah yes sorry, typo, last Tuesday of May being the "May 31, 2022 6:30pm GMT+0200", not the 24th Thanks ! |
@phibid Please upgrade your et-orbi gem to 1.2.7 and try again. Tell me if it fixes the issue for you. |
Thanks ! However I am not directly using Fugit, but through Gitlab. Let me add this info in the original Gitlab issue and see the progression on this. Thanks again, will keep you updated. |
@phibid OK, I saw your added info on Gitlab, I will complement it there. Closing the issue here. I am ready to re-open it if necessary. Thanks again! |
Opening an issue here too after opening it in Gitlab, https://gitlab.com/gitlab-org/gitlab/-/issues/354513, as it seems that Fugit is being used in Gitlab for the cron process.
The use of the cron hash extension don't give the correct schedule in some timezone.
For example, in UTC, 30 18 * * 2#5 will correctly be scheduled the last Tuesday of this month, the 29th.
However, in Paris timezone, it will be scheduled in 2 months, the last Tuesday of May, the 24th.
Not sure if this issue is coming from Fugit or from the implementation done by Gitlab, but if you think that you are not concerned by this bug, just feel free to close this issue.
The text was updated successfully, but these errors were encountered: