Bug 53942 - ucsschool-veyon-proxy should not be installed on primary
ucsschool-veyon-proxy should not be installed on primary
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: Veyon
UCS@school 5.0
Other Linux
: P5 normal (vote)
: UCS@school 5.0 v1
Assigned To: Ole Schwiegert
Johannes Keiser
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2021-10-18 15:24 CEST by Tobias Wenzel
Modified: 2024-03-11 15:35 CET (History)
3 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 2: Improvement: Would be a product improvement
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.023
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Wenzel univentionstaff 2021-10-18 15:24:50 CEST
When I installed a multi-server env, ucsschool-veyon-proxy was installed on my primary:


root@ucs-2731:~# univention-app info
UCS: 5.0-0 errata127
Installed: ucsschool=5.0 b3 4.4/ucsschool-veyon-proxy=1.1
Upgradable: 
root@ucs-2731:~# ucr get server/role 
domaincontroller_master


-> ucsschool-veyon-proxy should not be installed on primary
Comment 1 Ole Schwiegert univentionstaff 2021-11-04 10:43:30 CET
Implemented on oschwieg/53942

The problem was that the UMC schoolinstaller still installed the Veyon proxy app regardless of server roles or single/multiserver env.

The fix checks if the server role is ucsschool/singlemaster and only if that is True the app is installed by the umc installer. (The code is there for singlemaster env, which is not handled by prejoin hooks)

For the QA:
- Veyon not installed on multiserver primary node
- Veyon installed on singleserver primary node

--> Install app from appcenter and install package from branch before running the installer.
Comment 2 Johannes Keiser univentionstaff 2021-11-16 09:54:44 CET
OK:
before patch single
root@primary50:~# univention-app info 
UCS: 5.0-0 errata150
Installed: cups=2.2.1 samba4=4.13 squid=3.5 ucsschool=5.0 b3 4.4/ucsschool-veyon-proxy=1.1

before patch multi
root@primary50:~# univention-app info
UCS: 5.0-0 errata150
Installed: ucsschool=5.0 b3 4.4/ucsschool-veyon-proxy=1.1


with patch single
root@primary50:~/ucs-school-umc-installer# univention-app info
UCS: 5.0-0 errata152
Installed: cups=2.2.1 samba4=4.13 squid=3.5 ucsschool=5.0 b3 4.4/ucsschool-veyon-proxy=1.1

with patch multi
root@primary50:~# univention-app info
UCS: 5.0-0 errata152
Installed: ucsschool=5.0 b3

Can be merged
Comment 3 Ole Schwiegert univentionstaff 2021-11-16 12:32:58 CET
Build and merged:

Package: ucs-school-umc-installer
Version: 8.0.6A~5.0.0.202111161228
Branch: ucs_5.0-0
Scope: ucs-school-5.0

Release notes were added.
Comment 4 Tobias Wenzel univentionstaff 2024-03-11 15:35:50 CET
This package version has already been released as an errata update, closing this bug.

Please clone this bug if this happens again.