-
Notifications
You must be signed in to change notification settings - Fork 105
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
[BUG] Comma separated cron time ranges causes fatal error in ES #71
Comments
Comment by thenom This also seems to happen with:
With the above monitor but not with:
|
Comment by Angrboda2 Confirmed, our entire production system was taken down when I tried an expression like this: Further experimentation showed expressions without commas worked fine. |
Comment by gmfmi-znk I get exactly the same issue using this CRON expression : |
Hmm, I suspected this might have been related to this issue but it looks like that was fixed in cron-utils version We did, however, upgrade to version 9.1.3 in ODFE 1.13.0.0. Is anyone still seeing this issue in v1.13.0.0? |
Closing with reference to - opendistro-for-elasticsearch/alerting#245 |
Issue opendistro-for-elasticsearch/alerting#245 was closed in favor of this one as it was migrated to OpenSearch Alerting. This issue should still remain open for tracking. |
It appears that the cron version upgrade has resolved this issue as it is no longer occurring in our newer versions. Closing. |
Issue by thenom
Thursday Aug 27, 2020 at 10:28 GMT
Originally opened as opendistro-for-elasticsearch/alerting#245
Describe the bug
Comma separated cron time ranges causes fatal error in ES
Other plugins installed
Stock OpenDistro 1.8.0
To Reproduce
Note the 00-06,20-23 for the hour in the cron expression:
https://opendistro.github.io/for-elasticsearch-docs/docs/alerting/cron/
Expected behavior
Create a monitor that runs between 0am to 6am and 8pm to 11pm.
Error generated
The text was updated successfully, but these errors were encountered: