[SSSD] [PATCH] Add offline callback and fix info file handling

Stephen Gallagher sgallagh at redhat.com
Fri May 14 20:10:22 UTC 2010


On 05/14/2010 03:11 PM, Sumit Bose wrote:
> On Fri, May 14, 2010 at 08:58:39PM +0200, Sumit Bose wrote:
>> On Fri, May 14, 2010 at 11:55:01AM -0400, Stephen Gallagher wrote:
>>> On 05/14/2010 08:29 AM, Sumit Bose wrote:
>>>> Hi,
>>>>
>>>> this series of patches aims to fix the handling of the info file for the
>>>> locator plugin to allow a graceful fallback to the configuration from
>>>> krb5.conf is the sssd backend is offline.
>>>>
>>>> 0001-Revert-Create-kdcinfo-and-kpasswdinfo-file-at-startu.patch:
>>>>        Remove the old commit
>>>>
>>>
>>> Ack.
>>>
>>>> 0002-Refactor-data-provider-callbacks.patch
>>>>        Remove references to 'online' from the main callback code, move
>>>>        callback code to a separate file to make life easier for potential
>>>>        unit test.
>>>>
>>>
>>> Ack.
>>>
>>>> 0003-Add-offline-callbacks.patch
>>>>
>>>
>>> Nack. Please condense be_run_offline_cb() and be_run_online_cb() into a
>>> single function. They are nearly identical already.
>
> There was an error in the new function. Ne versions attached.
>
> bye.
> Sumit
>
>>
>> ok, I've put the common code into a new function. Most of the changes
>> are in 0002.
>>
>>>
>>>> 0004-Refactor-krb5_finalize.patch
>>>>        Create a new subroutine which only unlinks the files.
>>>>
>>>
>>> Ack.
>>>
>>>> 0005-Add-callback-to-remove-krb5-info-files-when-going-of.patch
>>>>
>>>
>>> Nack.
>>> Nitpick: you added an extra "ret = EOK" into ipa_service_init()
>>
>> done
>>
>> New versions attached.
>>
>> bye,
>> Sumit
>>


Nack.

The following two conditions don't work (tested using IPA):

1) Shutting down the SSSD with 'service sssd stop' (i.e. sending 
SIGTERM) does not remove kdcinfo files.

2) When starting up online, the kdcinfo file is generated correctly on 
the first request. It is removed properly when going offline (I turned 
off the dirsrv on the ipa server). However, when coming back online, the 
kdcinfo file is not regenerated.

-- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/



More information about the sssd-devel mailing list