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] add-ldap-server-option.patch
There is a Customer ID set so I set the flag "Enterprise Customer affected".
I applied the patch Felix made. Successful build Package: univention-base-files Version: 8.0.0-3A~4.4.0.201903141103 Branch: ucs_4.4-0 Scope: errata4.4-0 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.
Verified: * Code * Function * Advisory
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.
<http://errata.software-univention.de/ucs/4.4/44.html>