[Freeipa-devel] [PATCH] 591 improve error reporting when adding/removing members

[Rob Crittenden]

Return reason for failure when updating group membership fails.

We used to return a list of dns that failed to be added. We now return a 
list of tuples instead. The tuple looks like (dn, reason) where reason 
is the exception that was returned.

Also made the label we use for failures to be singular instead of plural 
since we now print them out individually instead of as comma-separated.

https://fedorahosted.org/freeipa/ticket/270

$ ipa group-add-member --users=tuser9,tuser1 --groups=g1 g1
    Group name: g1
    Description: g1
    GID: 1332445043
    Member users: tuser1
    Failed members:
      user: tuser9: no such entry
      user: tuser1: This entry is already a member of the group
      group: g1: A group may not be added as a member of itself
-------------------------
Number of members added 0
-------------------------

rob

-------------- next part --------------
A non-text attachment was scrubbed...
Name: rcrit-freeipa-591-fail.patch
Type: application/mbox
Size: 15035 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20101028/cb463781/attachment.mbox>