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

usbhid-ups: allow to throttle or suppress logged messages about OL+DISCHRG situation #2216

Merged
merged 3 commits into from
Dec 10, 2023

Conversation

jimklimov
Copy link
Member

Closes: #1970
Closes: #2212
Closes: #2213
Closes: #2214
Closes: #2215

Introduces a number of new configurable toggles with hopefully reasonable defaults, which should help to reduce log noise on systems where an UPS reports "online" and "discharging" simultaneously (so far some CyberPower and APC models were reported to behave so, with different physical situations).

…onlinedischarge" in favor of "onlinedischarge_onbattery" option name [networkupstools#2213]

Signed-off-by: Jim Klimov <[email protected]>
…onlinedischarge_log_throttle_sec" setting and/or throttling by changes of battery.charge [networkupstools#2214]

Signed-off-by: Jim Klimov <[email protected]>
…onlinedischarge_log_throttle_hovercharge" setting [networkupstools#2215]

Signed-off-by: Jim Klimov <[email protected]>
@jimklimov jimklimov added this to the 2.8.2 milestone Dec 5, 2023
@jimklimov jimklimov added the ready / code review Author (and CI) consider the PR worthy of human rewievers' time label Dec 6, 2023
@jimklimov jimklimov merged commit ec3df0e into networkupstools:master Dec 10, 2023
6 checks passed
@jimklimov jimklimov deleted the issue-1970-throttle branch December 10, 2023 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment