Skip to content

[CrashRecover] Disable failing step based on failed boot count#1740

Merged
TD-er merged 1 commit intoletscontrolit:megafrom
TD-er:bugfix/disable_failing_plugin
Sep 13, 2018
Merged

[CrashRecover] Disable failing step based on failed boot count#1740
TD-er merged 1 commit intoletscontrolit:megafrom
TD-er:bugfix/disable_failing_plugin

Conversation

@TD-er
Copy link
Copy Markdown
Member

@TD-er TD-er commented Sep 13, 2018

When the node gets locked in a boot loop, the only recovery is to erase it and start all over.
This fix will count the number of failed boots and above a certain threshold it will start disabling plugin, then controller and then notification.

Only one will be disabled at once, so the user can see which one is giving issues.
The disabled state will not be saved, so make sure to save the failing part when it is accessible again.

When the node gets locked in a boot loop, the only recovery is to erase it and start all over.
This fix will count the number of failed boots and above a certain threshold it will start disabling plugin, then controller and then notification.

Only one will be disabled at once, so the user can see which one is giving issues.
The disabled state will not be saved, so make sure to save the failing part when it is accessible again.
@TD-er TD-er added Category: UX Related to operating ESP-easy Category: Stabiliy Things that work, but not as long as desired labels Sep 13, 2018
@TD-er
Copy link
Copy Markdown
Member Author

TD-er commented Sep 13, 2018

This is also addressing #1616

@TD-er TD-er merged commit 75a8378 into letscontrolit:mega Sep 13, 2018
@TD-er TD-er deleted the bugfix/disable_failing_plugin branch September 14, 2018 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category: Stabiliy Things that work, but not as long as desired Category: UX Related to operating ESP-easy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant