[Freeipa-devel] [PATCHES] 506-507 Fix ticket #4085

[Simo Sorce]

On Wed, 2014-01-15 at 15:31 -0500, Rob Crittenden wrote:
> Simo Sorce wrote:
> > Uniform fallback for global_policy and stop explicitly adding the
> > default policy to all user accounts.
> >
> > Tested.
> >
> > Simo.
> 
> ACK but you need this change in order for the unit tests to pass:
> 
> diff --git a/ipatests/test_xmlrpc/test_user_plugin.py 
> b/ipatests/test_xmlrpc/
> test_user_plugin.py
> index 14a4b50..9b17775 100644
> --- a/ipatests/test_xmlrpc/test_user_plugin.py
> +++ b/ipatests/test_xmlrpc/test_user_plugin.py
> @@ -99,10 +99,6 @@ def get_user_result(uid, givenname, sn, operation='sh
> ow', omit=[],
>               mepmanagedentry=[get_group_dn(uid)],
>               objectclass=add_oc(objectclasses.user, u'ipantuserattrs'),
>               krbprincipalname=[u'%s@%s' % (uid, api.env.realm)],
> -            krbpwdpolicyreference=[DN(('cn', 'global_policy'),
> -                                      ('cn', api.env.realm),
> -                                      ('cn', 'kerberos'),
> -                                      api.env.basedn)],
>           )
>       if operation in ('show', 'show-all', 'find', 'mod'):
>           result.update(
> 

Attached  both patches again, the first is the same as before, the
second is updated with the above snippet.

Thanks for the review.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-simo-507-2-Stop-adding-a-default-password-policy-reference.patch
Type: text/x-patch
Size: 2713 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140115/4f12399c/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-simo-506-Harmonize-policy-discovery-to-kdb-driver.patch
Type: text/x-patch
Size: 5442 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20140115/4f12399c/attachment-0001.bin>