Bug 31437 - Make sysvol readable for members of the group "Enterprise Domain Controllers"
Make sysvol readable for members of the group "Enterprise Domain Controllers"
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: S4 Connector
UCS 3.1
Other Linux
: P5 normal (vote)
: UCS 3.1-1-errata
Assigned To: Arvid Requate
Stefan Gohmann
:
Depends on:
Blocks: 31438
  Show dependency treegraph
 
Reported: 2013-05-23 22:11 CEST by Arvid Requate
Modified: 2013-05-30 10:28 CEST (History)
3 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
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
concept for this change. (2.87 KB, text/plain)
2013-05-23 23:43 CEST, Arvid Requate
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Arvid Requate univentionstaff 2013-05-23 22:11:24 CEST
The changes of Bug 31271 demand that a different way for sysvol synchronization is created:

The group "Enterprise Domain Controllers" needs to be created with its propper builtin SID (S-1-5-9) and all currently registered samba4 DC need to be added. After waiting for samba4-idmap to write the updated mapping to idmap.ldb, samba-tool ntacl sysvolreset should be called to re-create the fACLs from the directory-NTACLs.
Comment 1 Arvid Requate univentionstaff 2013-05-23 23:43:33 CEST
Created attachment 5236 [details]
concept for this change.
Comment 2 Arvid Requate univentionstaff 2013-05-23 23:44:22 CEST
The attached concept is implemented, tests required.
Comment 3 Arvid Requate univentionstaff 2013-05-27 19:42:49 CEST
Advisories:
* 2013-05-07-univention-samba4.yaml
* 2013-05-27-univention-s4-controller.yaml
Comment 4 Alexander Kläser univentionstaff 2013-05-29 12:41:53 CEST
"Enterprise Domain Controllers" → As this group has the fixed SID of a builtin group (S-1-5-9) it would be good to mark it as builtin group via sambaGroupType=5:

http://pig.made-it.com/samba-accounts.html#22762
Comment 5 Arvid Requate univentionstaff 2013-05-29 13:03:28 CEST
Great idea, could have been mine ;-) this simplifies things quite a bit, as builtin groups are filtered by default in the S4 Connector mapping.
Fixed and advisories are updated.
Comment 6 Alexander Kläser univentionstaff 2013-05-29 13:48:59 CEST
(In reply to comment #5)
> Great idea, could have been mine ;-) this simplifies things quite a bit, as
> builtin groups are filtered by default in the S4 Connector mapping.
> Fixed and advisories are updated.

Wonderful! 2 minds, 1 idea :) !
Comment 7 Stefan Gohmann univentionstaff 2013-05-29 17:43:42 CEST
3.1-1 errata: OK, it works for the default scenarios. Only one issue: if the master is already updated and a slave is installed without the new erratum, then the slave is not member of the group: Bug #31572

YAML: OK

3.1-2: OK (Code comparison)

Changelog: OK
Comment 8 Arvid Requate univentionstaff 2013-05-29 19:05:05 CEST
On a plain ucs3.1-1 master updated to the latest (yet unreleased) errata3.1-1 I just faced the following traceback while installing univention-s4-connector for the first time. Looks like this occurrs because no credentials are given in this case:

============================================================
Starting univention-s4-connector daemon.
done.
Traceback (most recent call last):
  File "/usr/share/univention-samba4/scripts/create_group_Enterprise_Domain_Controllers.py", line 115, in <module>
    create_group_Enterprise_Domain_Controllers(lo)
  File "/usr/share/univention-samba4/scripts/create_group_Enterprise_Domain_Controllers.py", line 65, in create_group_Enterprise_Domain_Controllers
    uid = univention.admin.allocators.request(lo, position, 'groupName', value=groupName)
  File "/usr/lib/pymodules/python2.6/univention/admin/allocators.py", line 166, in request
    return acquireUnique(lo, position, type, value, _type2attr[type], scope = _type2scope[type])
  File "/usr/lib/pymodules/python2.6/univention/admin/allocators.py", line 144, in acquireUnique
    univention.admin.locking.lock(lo, position, type, value, scope=scope)
  File "/usr/lib/pymodules/python2.6/univention/admin/locking.py", line 71, in lock
    raise e
univention.admin.uexceptions.permissionDenied
Setting dns/backend
============================================================
Comment 9 Arvid Requate univentionstaff 2013-05-29 19:07:44 CEST
Rebuilt univention-samba4 (containing the script), advisory updated.
Comment 10 Stefan Gohmann univentionstaff 2013-05-30 07:09:37 CEST
OK, it works. A minor issue was split to Bug #31575.
Comment 11 Moritz Muehlenhoff univentionstaff 2013-05-30 10:28:17 CEST
http://errata.univention.de/ucs/3.1/115.html
Comment 12 Moritz Muehlenhoff univentionstaff 2013-05-30 10:28:52 CEST
http://errata.univention.de/ucs/3.1/114.html