Bug 54365 - [4.4] 61_notifier_protocol_version fails on non-primary even on fresh installations
[4.4] 61_notifier_protocol_version fails on non-primary even on fresh install...
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - System diagnostic
UCS 4.4
Other Linux
: P5 normal (vote)
: UCS 4.4-8-errata
Assigned To: Maximilian Janßen
Florian Best
https://git.knut.univention.de/univen...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2022-01-21 18:40 CET by Florian Best
Modified: 2022-01-26 16:57 CET (History)
3 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?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.034
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): bitesize
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2022-01-21 18:40:24 CET
We should backport the bug to UCS 4.4-8-errata.

+++ This bug was initially created as a clone of Bug #54336 +++

The ucr variable notifier/protocol/version is checked in a new diagnostic module since Bug #50733. 
This check fails even on fresh UCS5 installations if the diagnostic module is installed on e.g replica- or memberservers
since the ucr variable is only set when the univention-directory-notifier is installed.

Maybe the check should be skipped if the notifier is not installed to avoid confusion.
Comment 1 Maximilian Janßen univentionstaff 2022-01-26 15:30:45 CET
The diagnostic-module test 61_notifier_protocol_version will now be skipped on instances without the univention-directory-notifier installed.
The skip for the diagnostic-module test 61_notifier_protocol_version was removed from the test-module.

univention-management-console-module-diagnostic.yaml
76093d74f360 | Bug #54336: yaml
a22f415fc6c2 | Bug #54336: skip 61_notifier_protocol_version test on memberserver

univention-management-console-module-diagnostic (5.0.1-56)
a22f415fc6c2 | Bug #54336: skip 61_notifier_protocol_version test on memberserver

Package: univention-management-console-module-diagnostic
Version: 5.0.1-56A~4.4.0.202201212018
Branch: 4.4-8
Comment 2 Florian Best univentionstaff 2022-01-26 15:35:27 CET
OK: 61_notifier_protocol_version is skipped on non-master/backup
OK: test is not skipped anymore
OK: advisory/yaml