Univention Bugzilla – Full Text Bug Listing |
Summary: | sync from ucs: 'otherTelephone': no such attribute for delete | ||
---|---|---|---|
Product: | UCS | Reporter: | Arvid Requate <requate> |
Component: | S4 Connector | Assignee: | Stefan Gohmann <gohmann> |
Status: | CLOSED FIXED | QA Contact: | Arvid Requate <requate> |
Severity: | normal | ||
Priority: | P5 | CC: | botner, ebersbach, gohmann, petersen, thomas.manninger, walkenhorst |
Version: | UCS 3.1 | ||
Target Milestone: | UCS 3.2-3-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
See Also: | https://forge.univention.org/bugzilla/show_bug.cgi?id=36480 | ||
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: | |||
Bug Depends on: | |||
Bug Blocks: | 35903 | ||
Attachments: | con_other_attribute_DEL.patch |
Description
Arvid Requate
2013-04-24 17:00:56 CEST
Level 4 log in upload_haU3bS.bz2 attached to Ticket#: 2013041821001047. Unfortunately due to Bug 31133 the modlist is not logged. ldifs of the example can by found in the Ticket as well. I have the same problem. I can i resolve it? (In reply to Thomas Manninger from comment #2) > I have the same problem. I can i resolve it? after a univention-s4-connector restart, it works correctly. Also seen at 2014041721010635, Latest UCS@School at UCS 3.2-1 while initially syncing exam-users to s4. This causes a persistent reject as exam users will be removed after exam. (In reply to Arvid Requate from comment #1) > Level 4 log in upload_haU3bS.bz2 attached to Ticket#: 2013041821001047. > Unfortunately due to Bug 31133 the modlist is not logged. ldifs of the > example can by found in the Ticket as well. We have modlists at the referenced ticket - relevant snip: [(1, 'otherTelephone', None), (2, 'streetAddress', [u'F...and so on... Created attachment 5884 [details]
con_other_attribute_DEL.patch
Untestet patch proposal. No idea why this doesn't occur for the primary attribute "telephoneNumber".
Also reported at Ticket #2014062621000429. I've fixed this while writing a test case for Bug #35391. It doesn't make sense to add a MOD_DELTE to the modlist for a new object. UCS 3.2-3: r53315 UCS 4.0-0: r53316 YAML: r53318 Changing the phone number in in windows adds the old number to "otherTelephone". -> univention-ldapsearch cn=Administrator| grep -i phoneNu telephoneNumber: 126 -> univention-s4search cn=Administrator| grep -i phoneNu telephoneNumber: 126 Modified telephon number via windows RSAT to 127 Modified telephon number via windows RSAT to 128 -> univention-s4search cn=Administrator| grep -i phone telephoneNumber: 128 otherTelephone: 128 otherTelephone: 127 -> univention-ldapsearch cn=Administrator| grep -i phone telephoneNumber: 128 telephoneNumber: 127 (In reply to Felix Botner from comment #9) > Changing the phone number in in windows adds the old number to > "otherTelephone". Good catch. The problem is independent from my change. However, it is a bug and it should have been fixed with: UCS 3.2-3: r53462 UCS 4.0-0: r53463 YAML: r53464 This part of commits r53315 and r53316 looks strange: Index: modules/univention/s4connector/lockingdb.py =================================================================== --- modules/univention/s4connector/lockingdb.py (Revision 53314) +++ modules/univention/s4connector/lockingdb.py (Revision 53315) @@ -220,6 +220,10 @@ print 'E: uuid1 is locked for UCS' l.lock_ucs(uuid1) + l.lock_ucs(uuid1) + l.lock_ucs(uuid1) + l.lock_ucs(uuid1) + l.lock_ucs(uuid1) (In reply to Arvid Requate from comment #11) > This part of commits r53315 and r53316 looks strange: > > > Index: modules/univention/s4connector/lockingdb.py > =================================================================== > --- modules/univention/s4connector/lockingdb.py (Revision 53314) > +++ modules/univention/s4connector/lockingdb.py (Revision 53315) > @@ -220,6 +220,10 @@ > print 'E: uuid1 is locked for UCS' > > l.lock_ucs(uuid1) > + l.lock_ucs(uuid1) > + l.lock_ucs(uuid1) > + l.lock_ucs(uuid1) > + l.lock_ucs(uuid1) That's only test code. I tested if it is possible to lock a uuid multiple times. Ok, works. * Created a udm user with two phone numbers: sync ok * Created a udm user with one phone number: sync ok * Added another phone number via RSAT: sync ok * Removed the primary phone number in RSAT: sync ok, otherTelephoneNumber remains * Added another phone number via udm: sync ok, set as primary Advisory is ok as well. |