Skip to content

Instantly share code, notes, and snippets.

@dzomaya
Last active December 27, 2021 22:19
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save dzomaya/876d4f8f14db0c7fc04cfec220ac9a36 to your computer and use it in GitHub Desktop.
Save dzomaya/876d4f8f14db0c7fc04cfec220ac9a36 to your computer and use it in GitHub Desktop.
Review of Uninterruptible Power Supply (UPS) Management Protocol -- Commands and Responses RFC. See: https://datatracker.ietf.org/doc/draft-rprice-ups-management-protocol/

Would publication be a good/bad idea?

Good idea overall.

Is it clear enough to be published and implemented?

Yes. Comments and follow-up questions intended to help improve it are below, but even as-is it is useful.

  • Section 2.8

    • "Data lead" may be confusing. Suggest replacing "the system to which the data lead is connected is known as the primary" with something more explicit such as "the system which communicates directly with the UPS unit (e.g. using a USB, RS232, or network connection) is known as the primary"
  • Section 3

    • In Figure 4: UPS protects multiple systems "but if the UPS had status "OB" the Secondary shuts down." Should there be additional text that the OB shutdown depends on the configuration? i.e. while the default case may be to shut down, it is a configurable setting.
  • Section 4.2.1

    • Suggest replacing "trio" with "3" or "three" for simplicity
  • Section 4.2.3

    • Suggest dropping "high-level" from " In current practice, commands such as "FSD" are made available only to a high-level administrative user (2.2)" to match other references to the administrative user
  • Section 4.2.7

    • Suggest changing "then go off and wait for the response" to "wait"
  • Section 4.2.3

    • Can we add what "FSD" is an acronynm for (Forced ShutDown)? E.g. reword 1st line to "A Management Daemon (2.6) which is Primary (2.8) and has the required authority, uses this command to set status symbol "FSD" (forced shutdown) in the Attachment Daemon (2.1)."
      • Related: FSD is spelled out as "Forced ShutDown" in 6.5.1 and "Forced Shut Down" in 5.1. Should be consistent throughout
  • Section 4.3.2. Error Responses

  • Section 5.1

    • The term "public supply" is used four times. I understand this means utility/wall power/where the UPS is plugged in. I would suggest something like "input power source", "input" "utility", "utility power", or "wall power" as more common/easy to understand. For reference, the HID PDC spec (https://www.usb.org/sites/default/files/pdcv11.pdf) uses "input source power", "Input", and "utility" (probably best to pick just 1).
      • Note that "wall power" is used elsewhere in the RFC draft and we should change/leave it alone as needed based on any changes here
    • Regarding: " Note: "OB" does not imply "DISCHRG" if the battery is floating." Is there a specific UPS feature or scenario driving this note? Generally, if the UPS is operating off the battery, it is discharging it.
    • Suggest removing "is offline," from the OB definition to avoid confusion between "offline" and the various definitions of OFF from different UPS topologies and models
  • Section 5.2

    • Suggest changing "deduces" to "detects"
  • Appendix A

    • Suggest replacing "domestic" with "consumer-grade" to avoid confusion around the term "domestic" being used as the opposite of "international".
  • Appendix B

    • In Step 7, is the description of the outlets only shutting off correct in the common case? Many UPSes do not offer outlet control and if they are on battery, a shutdown command will eventually lead to them shutting off.
@dzomaya
Copy link
Author

dzomaya commented Dec 27, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment