Bug 35251

Summary: connector/s4/mapping/group/syncmode=write & group membership
Product: UCS Reporter: Stefan Gohmann <gohmann>
Component: S4 ConnectorAssignee: Felix Botner <botner>
Status: CLOSED FIXED QA Contact: Stefan Gohmann <gohmann>
Severity: normal    
Priority: P5 CC: markus.daehlmann, schwardt, walkenhorst
Version: UCS 3.2   
Target Milestone: UCS 3.2-3-errata   
Hardware: Other   
OS: Linux   
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: 35905    

Description Stefan Gohmann univentionstaff 2014-07-03 20:37:27 CEST
Even if connector/s4/mapping/group/syncmode is set to write, group members are synced back from S4 to OpenLDAP when a user was changed. The post hook functions should consider the connector/s4/mapping/group/syncmode variable.
Comment 1 Felix Botner univentionstaff 2014-09-04 13:35:29 CEST
Added sync_mode check to post modify functions object_memberships_sync_from_ucs (return if group sync mode is read) and object_memberships_sync_to_ucs (return if group sync_mode is write).

YAML: 2014-09-03-univention-s4-connector.yaml
Comment 2 Stefan Gohmann univentionstaff 2014-09-09 08:03:24 CEST
Several other sync_mode checks consider the value 'none' too. For example:
  if self.property[property_type].sync_mode in ['write', 'none']:

Could add 'none'?

YAML: OK, small adaption with r53465
Comment 3 Felix Botner univentionstaff 2014-09-09 09:56:08 CEST
(In reply to Stefan Gohmann from comment #2)
> Several other sync_mode checks consider the value 'none' too. For example:
>   if self.property[property_type].sync_mode in ['write', 'none']:
> 
> Could add 'none'?
> 
> YAML: OK, small adaption with r53465

OK
Comment 4 Stefan Gohmann univentionstaff 2014-09-09 10:43:00 CEST
YAML: OK, small adaption with r53465

Code review: OK

UCS 4.0 merge: OK

ucs-test: OK
Comment 5 Janek Walkenhorst univentionstaff 2014-09-10 17:41:11 CEST
http://errata.univention.de/ucs/3.2/199.html