Bug 27158 - Direktes Setzen der primären Gruppe
Direktes Setzen der primären Gruppe
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: Samba
UCS@school 3.0
Other Linux
: P5 enhancement (vote)
: UCS@school 3.0 MS2
Assigned To: Stefan Gohmann
Arvid Requate
:
Depends on: 27157
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-18 08:20 CEST by Stefan Gohmann
Modified: 2012-06-27 08:34 CEST (History)
0 users

See Also:
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:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Gohmann univentionstaff 2012-05-18 08:20:08 CEST
Der Patch sollte auch nach UCS@school übernommen werden.

+++ This bug was initially created as a clone of Bug #27157 +++

Aus Bug #26806:

(In reply to comment #3)
> (In reply to comment #1)
> > Im Connector Debug ist zu sehen, dass für das Ändern der primären Gruppe in S4
> > ca. 6 Sekunden benötigt werden:
> > 
> > 30.04.2012 07:07:43,895 LDAP        (INFO   ): primary_group_sync_from_ucs:
> > primary Group needed change of membership in S4
> > 30.04.2012 07:07:49,758 LDAP        (INFO   ): primary_group_sync_from_ucs:
> > changed primary Group in S4
> > 
> > Zwischen den beiden Meldungen ist lediglich der folgende Aufruf:
> > self.lo_s4.lo.modify_s(compatible_modstring(object['dn']),[(ldap.MOD_REPLACE,
> > 'primaryGroupID', rid)])
> 
> Es gibt eine Möglichkeit direkt beim Anlegen die primäre Gruppe zu setzen:
> root@master322:~# ldbadd -H /var/lib/samba/private/sam.ldb  s3.ldif 
> ERR: Unwilling to perform : "The primary group isn't settable on add
> operations!" on DN CN=s3,CN=schueler,CN=users,OU=school,DC=deadlock32,DC=local
> Added 0 records with 1 failures
> root@master322:~# ldbadd -H /var/lib/samba/private/sam.ldb --controls="relax:0"
> s3.ldif 
> Added 1 records with 0 failures
> root@master322:~#
Comment 1 Stefan Gohmann univentionstaff 2012-05-18 09:47:37 CEST
Wurde umgesetzt.
Comment 2 Arvid Requate univentionstaff 2012-05-23 15:00:37 CEST
Verified:
 * Funktion ist OK.
 * Changelog OK.

Ein Vorschlag:

* Im Moment werden die per con_create_extenstions geänderte Controls auch auf die modify-Operation für post_attributes angewendet.

Ich denke es wäre konsistenter, wenn sich die con_create_extenstions Anpassungen sich nur auf die add-Operation auswirken. Optional wäre eine Umbenennung in con_create_add_extensions zu erwägen. Alternativ könnte man an con_create_extenstions zusätzlich zur addlist aus die modlist übergeben sowie zur Differenzierung ein ctrls_add und ctrls_mod.

Comment?
Comment 3 Arvid Requate univentionstaff 2012-05-23 15:37:40 CEST
Wird ggf. in UCS 3.0-2 per Bug #27157 noch angepasst. Für UCS@School 3.0 ist die aktuelle Implementierung ok.
Comment 4 Stefan Gohmann univentionstaff 2012-06-11 06:29:56 CEST
UCS@school 3.0 wurde veröffentlicht. Sollte der hier beschriebene Bug mit einer  neueren Version von UCS@school erneut auftreten, so sollte dieser Bug dupliziert werden: "Clone This Bug"