Skip to content
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] Monitors created from backend incorrectly display Schedule when editing Monitor #32

Closed
qreshi opened this issue Aug 3, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@qreshi
Copy link
Contributor

qreshi commented Aug 3, 2021

Describe the bug
If a Monitor is created (at least with cron expression, did not check interval behavior) using the backend REST API, when going to the Edit Monitor page in the frontend plugin, the Monitor Schedule section displays the default of 1 minute interval. The cron information is actually being stored in the formik object derived from the Monitor object but it isn't the selected default. If the user ends up saving the Monitor without doing anything, the saved Monitor schedule will become the default of 1 minute.

To Reproduce
Steps to reproduce the behavior:

  1. Create a Monitor with a cron schedule using the backend API
  2. Go to Monitors page in the Alerting console
  3. Select the Monitor to go to the Monitor details page
  4. Select Edit
  5. Scroll down to Monitor Schedule

Expected behavior
When creating a Monitor from the backend API, if the Monitor schedule is a cron expression, then default to the Custom cron expression display in the Edit Monitor page to ensure it can be shown regardless of the expression. May also need to test this with other schedule types to see if they behave correctly.

Screenshots
User reported this issue in a forum question which includes screenshots. The user was on the OpenDistro version of Alerting but this behavior has not been altered in OpenSearch and all new issues are being handled here.

@qreshi qreshi added bug Something isn't working untriaged labels Aug 3, 2021
@nareshachary
Copy link

Hi There,

This issue has been there since August, 2021. Can you please help prioritize this issue as any automation process of creating/updating monitors is blocked by this issue. Would appreciate your help!

Thanks,
Naresh

@AWSHurneyt
Copy link
Collaborator

This issue has been resolved with the above PR in the 2.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants