On 10/01/2014 03:19 PM, Pavel Reichl
Priority of the ticket this patch was written for was bumped to
please see attached patch.
This patch is updated version of patch which was introduced in now
stalled thread: NSS: disable midpoint refresh for netgroups if
ptask refresh is enabled
I have picked some (from my POV) important comments from the
previous thread and pasted them here.
On Mon, 2014-04-07 at 19:07 +0200, Jakub Hrozek wrote:
What I meant is that this addition:
+ if (req_type == SSS_DP_NETGR &&
+ dctx->domain->refresh_expired_interval != 0)
+ ret = EOK;
Makes sense as well to me, because this way neither midpoint
expired record request would be scheduled. What I'm not sure
about is --
would we expand this condition in the future when we add more
Jakub Hrozek wrote:
>>> So how about checking that:
>>> refresh_expired_interval <
>> Are you proposing to abort startup in this case? That
sounds like a bit
>> too heavy solution..
> No, I would rather log loudly about this problem and then I
> refresh_expired_interval to entry_cache_timeout * 3/4 as is
> man pages. And again I would log this action loudly.
That would work for me. I don't think we should over-engineer
solution, after all, the goal was to make it clear to the admin
they need to tune the config themselves..
sssd-devel mailing list
Please consider reviewing it soon!