[Freeipa-devel] [PATCH] 129 migrate process cannot handle multivalued pkey attribute
Martin Kosek
mkosek at redhat.com
Mon Oct 3 07:54:49 UTC 2011
On Sat, 2011-10-01 at 19:45 +0200, Jan Cholasta wrote:
> On 29.9.2011 12:01, Martin Kosek wrote:
> > When group/user is migrated, the attribute used for RDN may be
> > multivalued. Make sure that we pick the value used in the RDN
> > which should be the unique one and not just the first one.
> >
> > https://fedorahosted.org/freeipa/ticket/1892
> >
>
> Every time you do "import *", god kills a kitten. Also, it pollutes the
> module namespace with unnecessary symbols and decreases code readability.
World is not just black and white. In this case I think its OK since
ipalib/dn.py has a nice maintained __all__ list with all 3 DN related
classes. Thus. I see no namespace pollution.
>
> I'm a bit puzzled why do you do this:
> + try:
> + pkey = dn[ldap_obj.primary_key.name].lower()
> + except KeyError:
> + failed[ldap_obj_name][str(dn)] =
> unicode(_rdn_err_msg)
> + continue
>
> and not just this:
> + pkey = ava.value.lower()
Good point. Updated patch attached.
Martin
>
> Besides that, the issue seems to be fixed.
>
> Honza
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mkosek-129-2-migrate-process-cannot-handle-multivalued-pkey-attri.patch
Type: text/x-patch
Size: 2517 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20111003/402f561d/attachment.bin>
More information about the Freeipa-devel
mailing list