[SSSD] [PATCHES] SSSDConfig fixes

Jakub Hrozek jhrozek at redhat.com
Thu Dec 10 16:17:51 UTC 2009


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 12/10/2009 04:46 PM, Stephen Gallagher wrote:
> On 12/10/2009 10:38 AM, Jakub Hrozek wrote:
>> On 12/10/2009 04:10 PM, Stephen Gallagher wrote:
>>> Patch 0001: I forgot to update the tests after adding a bunch of new
>>> options for IPA. This adds those.
> 
> 
>> testListProviders is failing now, it looks like
>> domain.list_providers().keys() contains only ['krb5', 'local', 'proxy',
>> 'ldap'], no 'ipa'.
> 
> This was fixed in another patch, but now realizing that it wasn't yet
> committed, so I'm merging them together. New version is attached.
> 

ACK

> 
> 
>>> Patch 0002: https://fedorahosted.org/sssd/ticket/320
>>> We were not processing Boolean values correctly (they were always
>>> getting implicitly cast to "True".
> 
> 
>> testBadBool throws:
>> IOError: [Errno 2] No such file or directory:
>> 'testconfigs/sssd-invalid-badbool.conf'
> 
>> Maybe you just forgot to git-add?
> 
> Exactly right. Updated patch attached.
> 

Tests pass now and both the code and the tests looks sane -> ACK

_______________________________________________
sssd-devel mailing list
sssd-devel at lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAkshHyoACgkQHsardTLnvCUtuQCfV/L3XvR5ilHPVnqaQdoaYWSu
feYAoMKmIQ3d09oTVS/8gB8OJaX1XAhJ
=T6lN
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list