Repository navigation
Conversation
Signed-off-by: Jordi-14 <j.albareda.fortuny@gmail.com>
|
Preparing a ZIP file with standard source tarball and another tarball with pre-built docs for commit a5a66e0 ... |
|
❌ Build nut 2.8.5.5907-master failed (commit cdd77429b7 by @Jordi-14) |
Author
|
I checked the AppVeyor failure. powerpanel.exe compiled successfully, and both new PowerPanel regression tests passed. The log stops during NIT daemon cleanup after repeated “fork: Resource temporarily unavailable” errors, and the build ended at the one-hour mark. Could this AppVeyor job please be rerun to check whether the failure is transient? |
Member
|
Thanks for the heads-up (and the PR), Appveyor rebuild queued. |
|
✅ Build nut 2.8.5.5908-master completed (commit cdd77429b7 by @Jordi-14)
|
|
✅ Build nut 2.8.5.5908-master completed (commit cdd77429b7 by @Jordi-14) |
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
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.
Problem
On the tested CyberPower OLS2000ERT2UA, standard HID monitoring does not expose the actual output watts, apparent power or output current available through PowerPanel.
USB captures show that these readings use a HID-framed version of the existing PowerPanel text protocol.
Changes
Extend the existing
powerpaneldriver with an optional libusb-1.0 transport and reuse its text subdriver.Decode the additional status fields:
Q:output.current, in amperes.J:ups.realpower, in watts.E:ups.power, in VA.Serial operation remains the default. USB can be selected explicitly with
transport = usb, or throughport = autowhen no transport is specified.Example
ups.confsection:The initial implementation requires exactly one device with USB ID
0764:0601and checks that its product string identifies it as OLS2000ERT2UA. This USB ID is shared with other models, so the ID alone is insufficient to select this transport.The patch also updates the manual, compatibility list, release notes and USB-information generator, and adds parser and mocked USB transport regression tests.
Protocol evidence
0x02: 64-byte frames containing report0x29, a payload length, CR-terminated text and zero padding.0x81: report0x28, a payload length and response fragments.0x25and0x27perform the observed communication setup.C*commands are excluded from the transport's query allowlist.Four captured status replies are included as regression cases without device identifiers. The 149 W / 191 VA / 0.7 A case matched the vendor application's simultaneous readings.
Validation
Tested on Linux with a CyberPower OLS2000ERT2UA, firmware OS02RV15.
make distcheck-light: passed, including 17 regression tests and the NUT-Monitor application test.The hardware test used a bounded, isolated runtime with approximately 5.5 MiB peak container memory and no OOM events. Original TrueNAS UPS monitoring was restored afterwards with its configuration unchanged.
Distribution validation used:
Limitations
This is an initial query-only implementation.
This patch adds the verified transport and measurements. It does not claim complete telemetry parity with the existing HID driver and PowerPanel.
Development assistance
OpenAI Codex assisted with protocol analysis.
Contribution checklist
data/driver.list.in.Makefile.amfiles for new sources and tests.NEWS.adocand the driver manual.docs/nut.dict.make distcheck-light.make distcheck.The USB ID is already covered by the existing UPower hardware database. No new UPower device ID is introduced.
A separate NUT DDL contribution and additional hardware/platform validation can follow this initial implementation.