Bug 57513 - ComputerMonitoringInterval should be renamed to ComputerMonitoringUpdateInterval
Summary: ComputerMonitoringInterval should be renamed to ComputerMonitoringUpdateInterval
Status: ASSIGNED
Alias: None
Product: UCS@school
Classification: Unclassified
Component: Documentation
Version: UCS@school 5.0
Hardware: Other Linux
: P5 normal
Target Milestone: ---
Assignee: Alexander Steffen
QA Contact:
URL: https://docs.software-univention.de/u...
Keywords:
Depends on:
Blocks:
 
Reported: 2024-08-14 10:33 CEST by Finn David
Modified: 2024-08-15 07:25 CEST (History)
2 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 2: Will only affect a few installed domains
How will those affected feel about the bug?: 1: Nuisance – not a big deal but noticeable
User Pain: 0.034
Enterprise Customer affected?:
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2024081221000111
Bug group (optional):
Customer ID: 32722
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Finn David univentionstaff 2024-08-14 10:33:53 CEST
The option veyon/Master/ComputerMonitoringInterval is non-existent and is in fact called veyon/Master/ComputerMonitoringUpdateInterval. This might lead to confusion.


root@Schule01:~# univention-app shell ucsschool-veyon-proxy /usr/bin/veyon-cli config list | grep -i computermonitoring
Master/ComputerMonitoringUpdateInterval=1000

root@Schule01:~# ucr dump | grep -i computermonitoring
veyon/Master/ComputerMonitoringUpdateInterval: 1000
Comment 1 Jan-Luca Kiok univentionstaff 2024-08-14 11:05:31 CEST
Hey, the UCRV you mentioned should not be used, this is in fact an app setting, which is named correctly. See also: https://forge.univention.org/bugzilla/show_bug.cgi?id=57451

I will quote the last comment from there:

> The mentioned settings are in fact not UCR variables, but app settings of the UCS@school Veyon Proxy - I totally get why this is misleading, the UCRvs are just present for technical reasons... The real interface is documented, therefore closing this bug:
> 
> root@mufflon:~# univention-app configure --list ucsschool-veyon-proxy
> veyon/WebAPI/ConnectionLimit: 5000 (Maximum number of connections for the veyon WebAPI)
> veyon/WebAPI/ConnectionIdleTimeout: 60 (Sets timeout for idling connections, in seconds. Reducing this value can help to avoid unnecessary VNC connections between the proxy and the Windows clients.)
> veyon/Core/ComputerStatePollingInterval: 1000 (Polling interval for state changes in milliseconds)
> veyon/Master/ComputerMonitoringUpdateInterval: 1000 (Update interval for the created thumbnail in milliseconds.)
> veyon/Master/ComputerMonitoringImageQuality: 2 (Sets the image quality value 0 (highest) to 4 (lowest).)
Comment 2 Finn David univentionstaff 2024-08-14 11:10:23 CEST
I'm aware of the app setting and that it should be used instead of the UCR-variable. But that wasn't my point, this is a docu-bug:

The documentation names it ComputerMonitoringInterval
The app setting is called  ComputerMonitoringUpdateInterval
Comment 3 Jan-Luca Kiok univentionstaff 2024-08-14 11:22:29 CEST
Oops, you are right - Thanks for pointing that out, I will see that we adapt this.