On (07/07/14 13:41), 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?
It is well known bug in codegen script
https://fedorahosted.org/sssd/ticket/2344
Just do not include this file into new version of patch :-)
LS