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

Dmitri Pal dpal at redhat.com
Tue Nov 19 19:49:08 UTC 2013


On 11/19/2013 12:08 PM, 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
>
> LS
I understand that part but do you want libini in future to return more
rich error codes than errno as it does now?

-- 
Thank you,
Dmitri Pal

Sr. Engineering Manager for IdM portfolio
Red Hat Inc.


-------------------------------
Looking to carve out IT costs?
www.redhat.com/carveoutcosts/






More information about the sssd-devel mailing list