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

riello_ser driver does not support ondelay/offdelay #738

Open
Skarafaz opened this issue Nov 4, 2019 · 5 comments
Open

riello_ser driver does not support ondelay/offdelay #738

Skarafaz opened this issue Nov 4, 2019 · 5 comments
Labels
bug impacts-release-2.7.4 Issues reported against NUT release 2.7.4 (maybe vanilla or with minor packaging tweaks) Riello Riello UPS devices (serial, usb), usually with a variant of Qx protocol serial port

Comments

@Skarafaz
Copy link

Skarafaz commented Nov 4, 2019

Hi!

I'm monitoring a Riello NDG1500 with nut 2.7.4 (debian package) using riello_ser driver.
When I try to set ondelay/offdelay variables in ups.conf I get the error "variable not supported"; is this hardware related? I really need to set an off delay because my nas (a slave) takes about 5 minutes to shutdown. Thanks.

@clepple
Copy link
Member

clepple commented Nov 4, 2019

Per the protocol documentation, this should be supported by the hardware (NDG1500 uses the GPSER protocol): https://networkupstools.org/ups-protocols.html#_riello

However, it does not look like the driver implements any variables: https://github.com/networkupstools/nut/blob/v2.7.4/drivers/riello_ser.c#L758

@clepple
Copy link
Member

clepple commented Nov 4, 2019

The authors of the driver seem to have been working on a patch, but it stalled:

@CentroricercheRPS do you have a version of the patch with setvar?

@clepple
Copy link
Member

clepple commented Nov 4, 2019

@CentroricercheRPS Since the ups.delay.shutdown variable is sent at shutdown, and is not maintained in non-volatile memory on the UPS, @zykh's 2nd comment applies:

  • if they don't set values in the device, but only in NUT, add (and
    document in riello_{ser,usb} man pages) corresponding options to be
    set in ups.conf (or using the -x var=value command line option),
    since the driver, when launched with the -k flag or through
    upsdrvctl shutdown, cannot be aware of any 'transient' change made
    previously.

@Skarafaz
Copy link
Author

Skarafaz commented Nov 5, 2019

Thanks for the support @clepple!

@Skarafaz
Copy link
Author

Skarafaz commented Nov 7, 2019

@clepple just in case... what is the right way to deal with slow slaves without offdelay? Should I increase FINALDELAY variable in upsmon.conf?

@jimklimov jimklimov added bug serial port Riello Riello UPS devices (serial, usb), usually with a variant of Qx protocol impacts-release-2.7.4 Issues reported against NUT release 2.7.4 (maybe vanilla or with minor packaging tweaks) labels Sep 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug impacts-release-2.7.4 Issues reported against NUT release 2.7.4 (maybe vanilla or with minor packaging tweaks) Riello Riello UPS devices (serial, usb), usually with a variant of Qx protocol serial port
Projects
None yet
Development

No branches or pull requests

3 participants