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

[BT] Restarting the tasks is not enough in case of BLE stack stuck #1725

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

1technophile
Copy link
Owner

Description:

So restart the ESP instead

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

@1technophile 1technophile merged commit fac602e into development Aug 3, 2023
142 checks passed
@1technophile 1technophile deleted the ble-restart2 branch August 3, 2023 11:49
@eljojo
Copy link

eljojo commented Sep 8, 2023

@1technophile Hi Florian, would you mind cutting a new release? been observing times without data capture since the last update that I think could be fixed by this! (PS: great job on the web ui and gateway in general!)

@1technophile
Copy link
Owner Author

You can try the development version from here:
https://docs.openmqttgateway.com/dev/ and report if it solves your issue.

@eljojo
Copy link

eljojo commented Sep 9, 2023

I think it does!
image

before seeing your message it came to mind I could rebuild by forking the repo and pushing an empty commit, so I did that and extracted the upgrade image from the artifacts. Been running it for a few days and it seems to restart appropriately.

it's interesting to be between 60k-65k seconds

@eljojo
Copy link

eljojo commented Sep 9, 2023

I wonder why the restarts became necessary, I couldn't observe any other pattern that would indicate. It seemed to not happen before the web ui.

@1technophile
Copy link
Owner Author

The restart is necessary due to this issue
espressif/arduino-esp32#5860

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

Successfully merging this pull request may close these issues.

2 participants