On Mon, 2014-07-07 at 13:41 +0200, Jan Cholasta wrote:
On 29.6.2014 13:14, William wrote:
> On Sun, 2014-06-29 at 20:22 +0930, William wrote:
>> On Tue, 2014-06-24 at 10:21 +0200, Jan Cholasta wrote:
>>> Hi,
>>>
>>> I'm terribly sorry for the late reply.
>>>
>>
>> That's okay, I understand how it can be.
>>
>>> Could you add a sssd.conf option for the cache timeout?
>>
>> Done. The option exists, and I updated the man page (?), and the default
>> option for this.
Thanks. Could you please rename the option to
"entry_cache_ssh_host_timeout", so that it's consistent with the rest of
the cache timeout options?
> However, I can't quite work out how to access confdb
>> inside of ipa_hostid.c when it calls sysdb_store_ssh_host.
I guess you can store the value in sss_domain_info, like the rest of the
cache timeouts. See confdb_get_domain_internal.
>>
>>>
>>> Besides the above, there are some whitespace errors:
>>>
>>> 0001-Allow-sss_cache-to-expire-sshKnownHosts.patch:60: trailing whitespace.
>>> errno_t
>>> 0001-Allow-sss_cache-to-expire-sshKnownHosts.patch:121: trailing
whitespace.
>>> int
>>> 0001-Allow-sss_cache-to-expire-sshKnownHosts.patch:220: trailing
whitespace.
>>>
>>> 0001-Allow-sss_cache-to-expire-sshKnownHosts.patch:296: trailing
whitespace.
>>>
>>
>> All 4 fixed, and I (hope) haven't added any more.
>>
>> Thanks for your help.
>
> Helps if I attach the patch.
It certainly does :)
Why are there changes to DBus code in the patch?
I guess it's because of
https://fedorahosted.org/sssd/ticket/2344
One must be careful what he adds to the commit before the ticket is
fixed.