You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.
When a plugin panics while it is in use by a task, it emits a DeadAvailablePlugin event. This event should be handled by runner, and it should apply the same eligibility rules to the affected apPool. I.e. start a new available plugin, and insert it in the dead plugin's place.
The text was updated successfully, but these errors were encountered:
Should the amount of times this occurs be saved? And should that value disable a consistently panicking plugin with the end result that it is removed from the active list?
Yeah, there should be an upper bound on the number of times it restarts the plugin. However, if the task continues to fail because the plugin continues to fail, they will both stop when the task failure reaches its upper bound.
Not sure about the second thing. Rather than removed maybe loaded plugins should have states akin to Pulse's tasks, and when a plugin fails consistently we set its state to disabled.
Only worry I have is if the error is environmental and long tail. Lets say it fails once a month and restarts. To the user this is a minor disturbance because of something on the system. If we could make it record failures but have those failures age out that would be nice. Maybe a N failures per T time window.
And yes, I actually had a story a while back on a "Disabled" plugin state. Allow the user to enable or unload/load but prevent it being considered for Metric Catalog and remove all running APs.
When a plugin panics while it is in use by a task, it emits a
DeadAvailablePlugin
event. This event should be handled by runner, and it should apply the same eligibility rules to the affectedapPool
. I.e. start a new available plugin, and insert it in the dead plugin's place.The text was updated successfully, but these errors were encountered: