Bug 50503 - diagnostic module to check UCRV dns/backend on UCS@school DCs
diagnostic module to check UCRV dns/backend on UCS@school DCs
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - System diagnostic
UCS@school 4.4
Other Linux
: P5 normal (vote)
: UCS@school 4.4 v5-errata
Assigned To: Toni Röhmeyer
Tobias Wenzel
:
: 50499 50502 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2019-11-15 08:10 CET by Christian Völker
Modified: 2020-10-15 09:40 CEST (History)
6 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 4: Minor Usability: Impairs usability in secondary scenarios
Who will be affected by this bug?: 1: Will affect a very few installed domains
How will those affected feel about the bug?: 5: Blocking further progress on the daily work
User Pain: 0.114
Enterprise Customer affected?:
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2019110521001298, 2019111321001022
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 Christian Völker univentionstaff 2019-11-15 08:10:55 CET
In UCS we can switch between backends for DNS by UCRV dns/backend from samba4 to ldap. This is -more or less- equal functionality.

In ucs@school for the school-slaves this variable has mandantory to be set to "samba4"!

We should consider to make sure this variable is always set to samba4 when ucs@school is installed (an on a school-slave)

Might be possible by creating a policy to overwrite this setting for all OUs.

We should *NOT* allow customers to set it to something else than samba4.
Comment 1 Daniel Tröder univentionstaff 2019-11-15 08:20:41 CET
On DC master and DC backup system running Samba4, the DNS backend must also not be changed to "ldap".
Comment 2 Erik Damrose univentionstaff 2019-11-15 09:53:55 CET
*** Bug 50502 has been marked as a duplicate of this bug. ***
Comment 3 Sönke Schwardt-Krummrich univentionstaff 2019-11-15 10:27:21 CET
> Might be possible by creating a policy to overwrite this setting for all OUs.

I would suggest to set the UCR variable within the UCS@school metapackages via --force to "samba4". This makes sure, that the UCR variable is not mistakenly set e.g. on "central slaves", and only set if UCS@school is installed.
Comment 4 Michel Smidt 2020-03-06 16:30:06 CET
I set "Who will be affected by this bug?" to "Will affect a very few installed domains", because I think that only very few domains are really affected. Irrespective of the fact that it has really unpleasant impacts.
Feel free to discuss it.
Comment 5 Ingo Steuwer univentionstaff 2020-03-11 12:25:58 CET
(In reply to Sönke Schwardt-Krummrich from comment #3)
> > Might be possible by creating a policy to overwrite this setting for all OUs.
> 
> I would suggest to set the UCR variable within the UCS@school metapackages
> via --force to "samba4". This makes sure, that the UCR variable is not
> mistakenly set e.g. on "central slaves", and only set if UCS@school is
> installed.

Using "--force" has often unexpected side effects, for example the UCR Policy cron job will report this by mail daily for all instances.

I suggest to have an UCS@school specific UMC check for this which links to the documentation. For documentation we have Bugs #50499 and #50501
Comment 6 Ingo Steuwer univentionstaff 2020-04-17 14:27:21 CEST
*** Bug 50499 has been marked as a duplicate of this bug. ***
Comment 8 Daniel Tröder univentionstaff 2020-05-01 19:43:52 CEST
*** Bug 50499 has been marked as a duplicate of this bug. ***
Comment 9 Daniel Tröder univentionstaff 2020-05-11 09:50:06 CEST
Write a diagnostic test (add to package 'ucs-school-umc-diagnostic') that verifies that if the hosts role is a {master, backup, slave} domain controller and Samba4 ist installed, that the value of the UCR "dns/backend" is "samba4".
Comment 10 Toni Röhmeyer univentionstaff 2020-05-14 14:57:10 CEST
Added diagnostic check to ucs-school-umc-diagnostic with commit

commit 4a1bcb616680bd6857cd62820687edf8c20b77a5
Bug #50503: added diagnostic check

to branch troehmey/bug50503.
Comment 11 Tobias Wenzel univentionstaff 2020-05-28 15:57:07 CEST
QA

Description & Warning
 (...) and samba4 is installed

-> You do not check if samba4 is installed, only if the ucr-v is set correctly.
Please add the corresponding code & description.
Comment 12 Toni Röhmeyer univentionstaff 2020-05-31 11:56:25 CEST
The diagnostic module now checks if samba is actually installed and the version is 4.*


Solution pushed with commits

commit 63b7ac5579bc2aed16b9ebc621ecd14427807f28
Bug #50503: check samba version

commit 6ecb3410f339df6e705e1bdd1185cab3251a5196
Bug #50503: add check if samba is installed

on branch troehmey/bug50503
Comment 13 Tobias Wenzel univentionstaff 2020-06-02 15:29:00 CEST
Code -> looks fine, works as expected.

Description:

UCS@school: test that verifies that if the hosts role is a master, slave or backup DC and samba4 is installed

-> I think this is misleading: The check is run for master/slave & backup, other roles are ignored. I would suggest:

UCS@school: Test that checks if the host role is master, slave or backup DC, samba4 is installed.
Comment 14 Toni Röhmeyer univentionstaff 2020-06-03 15:31:17 CEST
Description has been improved with commit

commit 2c2732fa5acb342bdfaed50592d50870865551e4
Bug #50503: improved description
Comment 15 Tobias Wenzel univentionstaff 2020-06-03 15:33:48 CEST
QA -> all ok.
Reopen for build&merge

Please add merge, add changelog&yaml and build.
Comment 16 Toni Röhmeyer univentionstaff 2020-06-03 15:58:40 CEST
Merged with commits:

commit 61792655abe334a4d12a9dae1d7cff4d4a012437
Bug #50503: added yaml

commit 49c7dc7eced832ed1c8d7fa6ce4b18812bd98cc2
Bug #50503: added changelog entry

commit a639b862b89d4337e68b015fb7b504097a6a9448
Merge: 463f411d2 66720e02f
Bug #50503: Merge branch 'troehmey/bug50503' into 4.4

to 4.4


Successful build:

Package: ucs-school-umc-diagnostic
Version: 1.0.0-10A~4.4.0.202006031541
Branch: ucs_4.4-0
Scope: ucs-school-4.4
Comment 17 Tobias Wenzel univentionstaff 2020-06-24 14:21:24 CEST
QA -> all ok -> VERIFY

Changelog -> ok
YAML -> ok
Functionality -> works like before.
Comment 18 Tobias Wenzel univentionstaff 2020-06-25 10:27:16 CEST
UCS@school 4.5 v5 has been released (errata update to the release).

http://docs.software-univention.de/changelog-ucsschool-4.4v5-de.html

If this error occurs again, please clone this bug.
Comment 19 Tobias Wenzel univentionstaff 2020-07-30 13:14:33 CEST
Due to technical problems, the version had to be republished.

UCS@school 4.4 v5 has been released (errata update to the release).

http://docs.software-univention.de/changelog-ucsschool-4.4v5-de.html#changelog:ucsschool:2020-07-30

If this error occurs again, please clone this bug.