Add INDI UPS Driver using NUT for monitoring UPS status#2233
Merged
Conversation
knro
reviewed
May 3, 2025
knro
reviewed
May 3, 2025
knro
reviewed
May 3, 2025
- Removed socket-based connection handling and replaced it with a TCP connection object. - Updated connection initialization and handshake process to establish a TCP connection. - Simplified property management by removing unused server address properties. - Enhanced logging for connection status and command responses. - Improved error handling during communication with the NUT server.
Contributor
|
Please fix the CI issues (unused parameters) |
Contributor
|
@knro is the Fedora issue regarding ciso64 a global issue that needs investigation? |
knro
pushed a commit
to knro/indi
that referenced
this pull request
Aug 25, 2025
* Add INDI UPS Driver using NUT for monitoring UPS status * Refactor UPS driver to use TCP connection for NUT server communication - Removed socket-based connection handling and replaced it with a TCP connection object. - Updated connection initialization and handshake process to establish a TCP connection. - Simplified property management by removing unused server address properties. - Enhanced logging for connection status and command responses. - Improved error handling during communication with the NUT server. * Correct CI --------- Co-authored-by: Jérémie Klein <jeremie.klein@octo.com>
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
3 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 commit introduces a new driver that integrates with Network UPS Tools (NUT) to monitor UPS status and expose it through INDI's weather interface. The driver includes properties for server address, UPS name, connection settings, and simulation controls, along with methods for querying UPS status and updating weather parameters accordingly.