On 01/18/2016 11:30 PM, Michal Židek wrote:
Hi!
see attached simple patch for ticket:
https://fedorahosted.org/sssd/ticket/2912
The first patch is for master and 1.13
the second for 1.12.
It seems like the decision to ignore cache
validity when background refresh is enabled
was not a good one and it was not possible
to fetch new netgroup entry even if the
old was invalidated with sss_cache.
Michal
Nack.
The decision was good, but the patch was wrong. The point was to disable
midpoint refresh if background refresh is on. The check should be moved
to this area in check_cache():
if (ret == EAGAIN) {
/* No callback required
* This was an out-of-band update. We'll return EOK
* so the calling function can return the cached entry
* immediately.
*/
something like: if (ret == EAGAIN && is_refreshed_on_bg()) then ret =
EOK (just return)
Also please extend is_refreshed_on_bg() to return true for users and
groups since it is already supported.