[SSSD] [PATCH] Configuration parsing modifications

Jan Zelený jzeleny at redhat.com
Thu Apr 21 09:23:19 UTC 2011


> On Fri, 2011-04-01 at 16:46 +0200, Jan Zelený wrote:
> > These changes are all related to the following ticket:
> > https://fedorahosted.org/sssd/ticket/763
> > 
> > Changes in SSSDConfig.py merge old and new domain record instead of just
> > deleting the old and inserting the new one. The old approach let to loss
> > of some information like comments and blank lines in the config file.
> > 
> > Changes in API config were performed so our Python scripts (like
> > sss_obfuscate) don't add extra config options to the config file.
> 
> Nack.
> 
> As we discussed on IRC, the 'services' option in [sssd] must be left as
> mandatory.
> 
> Similarly, do not make krb5_realm optional. It must exist if using the
> kerberos provider.
>        krb5_realm (string)
>            The name of the Kerberos realm. This option is required
>            and must be specified.
> 
> Also, the SSSDConfigTest.py tests are failing due to these changes.
> Please fix this.

Sending corrected patch.

Jan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jzeleny-012-2-python-config-updates.patch
Type: text/x-patch
Size: 7771 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110421/1265bc13/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110421/1265bc13/attachment.sig>


More information about the sssd-devel mailing list