Skip to content
This repository has been archived by the owner on Nov 26, 2017. It is now read-only.

Add pause button to disable running monitors #382

Open
hubdotcom opened this issue Jan 3, 2016 · 3 comments
Open

Add pause button to disable running monitors #382

hubdotcom opened this issue Jan 3, 2016 · 3 comments

Comments

@hubdotcom
Copy link

Otherwise they have to be deleted!

@madflojo
Copy link
Contributor

madflojo commented Jan 3, 2016

👍

This should be do able by sending a delete message to bridge.py but not actually deleting it from the database.

@hubdotcom
Copy link
Author

Yes, perhaps it get's flagged with a date base "deleted" column then a cron fully removes any still flagged after 60 minutes

@madflojo
Copy link
Contributor

madflojo commented Jan 3, 2016

We can actually use a status key to show the Runbook as Paused. The main thing with the bridge.py process getting a delete message is that will trigger it to remove the monitor from Redis as if it was deleted. But from the UI it would still just show as Paused.

Once someone clicked resume or edited the monitor in any way it would then resync and start again. Something to think about is in this scenario it would be best to not resume the monitor after being edited, only after resume.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants