[SSSD] [PATCH] Specific error message for missing sssd.conf

Jakub Hrozek jhrozek at redhat.com
Wed Nov 20 10:43:40 UTC 2013


On Tue, Nov 19, 2013 at 09:44:28PM +0100, Pavel Reichl wrote:
> On Tue, 2013-11-19 at 18:08 +0100, Lukas Slebodnik wrote:
> > On (19/11/13 11:16), Dmitri Pal wrote:
> > >On 11/19/2013 10:16 AM, Michal Židek wrote:
> > >> On 11/19/2013 03:46 PM, Jakub Hrozek wrote:
> > >>> On Tue, Nov 19, 2013 at 03:41:04PM +0100, Lukas Slebodnik wrote:
> > >>>> ENOENT means "No such file or directory", Please add more specific
> > >>>> error
> > >>>> message and syslog message (sss_log) should also contain information
> > >>>> about path
> > >>>> to config_file
> > >>>
> > >>> Would it make sense to extend load_configuration with new error codes
> > >>> instead of errno? Then we wouldn't have to guess in the monitor..
> > >>
> > >> +1
> > >>
> > >> We can safely mix errno with our specific error codes, so for now,
> > >> probaly adding just one specific error for missing conf file will be
> > >> enough.
> > >>
> > >> _______________________________________________
> > >> sssd-devel mailing list
> > >> sssd-devel at lists.fedorahosted.org
> > >> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> > >
> > >Do you expect this to trickle down to ding-libs?
> > >Eventually?
> > >
> > IITC, Intention was to transform ENOENT returned from libini_config to sssd
> > specific error code. Something like SSS_MISSING_CONF
> 
> Would adding ERR_MISSING_CONF to sssd_errors (util_errors.h) be a good
> idea?

That's what I proposed, yes.



More information about the sssd-devel mailing list