Univention Bugzilla – Bug 35208
add ldap server option to univention-policy-update-config-registry.py
Last modified: 2019-04-10 14:19:04 CEST
We need an optional option -l|--ldap-server in univention-policy-update-config-registry.py in order to define the ldap server to look for policies (default ldap/server -> local ldap).
Created attachment 5976 [details]
There is a Customer ID set so I set the flag "Enterprise Customer affected".
I applied the patch Felix made.
7d48096 Bug #35208: yaml
d880a03 Bug #35208: changelog
6faa4b1 Bug #35208: added ldap server option to script univention-policy-update-config-registry.py
root@master10:~# /usr/lib/univention-directory-policy/univention-policy-update-config-registry -l something
could not open policy for cn=master10,cn=dc,cn=computers,dc=ar41i1,dc=qa
Looking at the code it might be difficult to add a better error message, what do you think? I would be ok with the code as it is.
There has been a better error message in the past, but it has been deactivated for some reason:
if proc.wait() != 0:
# no output: this script is called by cron
# print 'WARN: univention_policy_result failed - LDAP server may be down'
git blame doesn't give any meaningful result as to why that is as the code was refactored. I don't want to break anything, so it should stay as it is.
We also need a backport for 4.3-3
I've simply cloned the bug for 4.3, and will set this back to verified.