Univention Bugzilla – Bug 35911
Traceback in proof_uniqueMembers
Last modified: 2023-03-25 06:52:00 CET
+++ This bug was initially created as a clone of Bug #33030 +++ Traceback (most recent call last): File "./proof_uniqueMembers", line 239, in <module> main() File "./proof_uniqueMembers", line 230, in main check_groups(conn, basedn) File "./proof_uniqueMembers", line 136, in check_groups count_changes += check_groups_by_dn(conn, group_dn, group) File "./proof_uniqueMembers", line 150, in check_groups_by_dn member_filter, base = member_dn.split(',', 1) ValueError: need more than 1 value to unpack
Cannot reproduce this specific problem, neither in my UCS 3.2-3, nor in UCS 4.0. Actually, there is a following construction now: try: member_filter, base = member_dn.split(',', 1) except ValueError: remmembers.add(member_dn) continue So ValueError won't be raised. The problem I've noticed is when I run the 00_base/52proofuniquemember --> the test passes although there are tracebacks in the output, this should not happen I believe. Also, if I eliminate the sources of the traces in my env, the test would fail.
Created attachment 6404 [details] Test 52proofuniquemember passes with tracebacks
Created attachment 6405 [details] Test 52proofuniquemember failure
(In reply to Dmitry Galkin from comment #1) > The problem I've noticed is when I run the 00_base/52proofuniquemember --> > the test passes although there are tracebacks in the output, this should not > happen I believe. Same case observed in Jenkins: http://jenkins.knut.univention.de:8080/job/UCS-4.0/job/UCS-4.0-0/job/Autotest%20MultiEnv/65/SambaVersion=s4,Systemrolle=master/testReport/00_base/52proofuniquemember/test/ Opened another Bug #36799
(In reply to Dmitry Galkin from comment #4) > Opened another Bug #36799 OK, I assigned the bug to you. For this bug, please add a new test case with an empty uniqueMember entry. This was the original cause of Bug #33030.
(In reply to Stefan Gohmann from comment #5) > For this bug, please add a new test case with an empty uniqueMember entry. > This was the original cause of Bug #33030. r56846: * 01_base/52proofuniquemember_empty_attribute: test how proof_uniqueMembers handles the empty 'uniqueMember' attribute (Bug #35911)
(In reply to Dmitry Galkin from comment #6) > r56846: > * 01_base/52proofuniquemember_empty_attribute: test how > proof_uniqueMembers handles the empty 'uniqueMember' attribute (Bug > #35911) Note: the "univention-s4-connector | univention-ad-connector" packages were added due to '/usr/share/ucs-test/lib/ldap_glue.py' which imports 'compatible_modstring'. Without those, the attribute will be deleted if set to "" or will cause a traceback if set to another value..
For this bug is no separate QA needed.