INSTCMD and SET VAR status tracking implementation - bis#660
Merged
zykh merged 7 commits intocmdset_statusfrom Feb 12, 2019
Merged
INSTCMD and SET VAR status tracking implementation - bis#660zykh merged 7 commits intocmdset_statusfrom
zykh merged 7 commits intocmdset_statusfrom
Conversation
This is just a big, big rename, no code changes.
Add a couple of functions to change in a predictable way the value of the general enablement of tracking and make it visible only inside upsd.c. Also, move the tracking type (tracking_t) and the list of items inside upsd.c.
Member
|
Same comments as on #659. I do think the I have only had time to test the client-side error handling - I was hoping to do more with this over the weekend, but I was rewiring the network, and went down a rabbit hole trying to get a physical UPS running for testing. I might have some time later tonight to build and test the changes on the server side. |
Rename the default timeout used in network operations by upsclient and nut-scanner to be more specific: from DEFAULT_TIMEOUT to DEFAULT_NETWORK_TIMEOUT. Plus, make the default timeout used when retrieving the result of an INSTCMD/SET VAR with TRACKING enabled a common #define'd value (DEFAULT_TRACKING_TIMEOUT), and use it also to publish the default value of the -t option in the help messages of upscmd and upsrw. As suggested by Charles Lepple.
Also, remove some nesting in tracking_del().
To keep things simple, at least for now, only generate manpages and not html pages (and, as such, don't even think of using our linkman AsciiDoc macro with those functions, to avoid dead links).
- addition of upscli_{read,send}line_timeout(),
- upscli_cleanup() -> upcli_cleanup(void)
Note: only increase 'current' and not 'age', because the upscli_cleanup() change could (potentially) make it not compatible with previous versions.
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.
Follow-up to #659 -- #659 (comment)