fix: Add proper cleanup for MQTT client connections#611
Open
cjb0001 wants to merge 66 commits intoMyElectricalData:mainfrom
Open
fix: Add proper cleanup for MQTT client connections#611cjb0001 wants to merge 66 commits intoMyElectricalData:mainfrom
cjb0001 wants to merge 66 commits intoMyElectricalData:mainfrom
Conversation
…uxdb-production-only fix export to influxdb for production
…work-database Feat/rework database
…work-database BREAKING CHANGE: database rework
BREAKING CHANGE: Release 1.0.0 with database rework - Upgrade FastAPI to latest (0.111.0)
…work-database feat: rework database
Automatically generated by python-semantic-release
…work-database fix: update requirement
Automatically generated by python-semantic-release
…ild-args Feat/build args
Automatically generated by python-semantic-release
…ild-args feat: add build arg on staging
Automatically generated by python-semantic-release
…ild-args fix: build arg + downgrade python to 3.11.9 (armv7 bug)
Automatically generated by python-semantic-release
…ild-args fix: requirements
Automatically generated by python-semantic-release
…ild-args Fix rust uninstall
…ild-args feat fix dep
…ild-args feat: generate release
Automatically generated by python-semantic-release
…ild-args update setuptools
…ild-args feat: rework
Automatically generated by python-semantic-release
…ild-args fix: influxdb method lower
Automatically generated by python-semantic-release
…ild-args fix: med plan upper
Automatically generated by python-semantic-release
…ild-args fix: opentel trash + bug
Automatically generated by python-semantic-release
…ild-args Feat/build args
Automatically generated by python-semantic-release
Fix export to influxdb config flag
- Add disconnect() method to properly stop background threads - Add __del__ destructor as safety net for cleanup - Add context manager support (__enter__/__exit__) - Update HomeAssistant class to cleanup MQTT on exit - Update ExportMqtt class to cleanup MQTT on exit This fixes a memory leak where MQTT client background threads were never stopped, causing thread and memory accumulation over time. Fixes: Memory leak in long-running deployments
Collaborator
|
Hello cjb0001, merci mais je suis en train de réécrire complètement le code de MyElectricalData. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This fixes a memory leak where MQTT client background threads were never stopped, causing thread and memory accumulation over time.
Fixes: Memory leak in long-running deployments