nscd: also invalidate netgroup cache on reload

Message ID mvm1txyshlh.fsf@hawking.suse.de
State Committed
Headers

Commit Message

Andreas Schwab March 19, 2014, 8:44 a.m. UTC
  * nscd/nscd.service: Also invalidate netgroup cache on reload.
---
 nscd/nscd.service | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Siddhesh Poyarekar March 19, 2014, 11:27 a.m. UTC | #1
Looks good.

Thanks,
Siddhesh

On Wed, Mar 19, 2014 at 09:44:26AM +0100, Andreas Schwab wrote:
> 	* nscd/nscd.service: Also invalidate netgroup cache on reload.
> ---
>  nscd/nscd.service | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/nscd/nscd.service b/nscd/nscd.service
> index b7428a3..ab38e8f 100644
> --- a/nscd/nscd.service
> +++ b/nscd/nscd.service
> @@ -11,6 +11,7 @@ ExecReload=/usr/sbin/nscd -i passwd
>  ExecReload=/usr/sbin/nscd -i group
>  ExecReload=/usr/sbin/nscd -i hosts
>  ExecReload=/usr/sbin/nscd -i services
> +ExecReload=/usr/sbin/nscd -i netgroup
>  Restart=always
>  PIDFile=/run/nscd/nscd.pid
>  
> -- 
> 1.9.0
> 
> -- 
> Andreas Schwab, SUSE Labs, schwab@suse.de
> GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE  1748 E4D4 88E3 0EEA B9D7
> "And now for something completely different."
  
Mike Frysinger March 19, 2014, 7:20 p.m. UTC | #2
On Wed 19 Mar 2014 09:44:26 Andreas Schwab wrote:
>  ExecReload=/usr/sbin/nscd -i group
>  ExecReload=/usr/sbin/nscd -i hosts
>  ExecReload=/usr/sbin/nscd -i services
> +ExecReload=/usr/sbin/nscd -i netgroup

seems like maybe we should extend the command line interface to accept 
multiple databases in one go ?  and even add a "all" keyword ?
-mike
  
Siddhesh Poyarekar March 20, 2014, 4:30 a.m. UTC | #3
On Wed, Mar 19, 2014 at 03:20:56PM -0400, Mike Frysinger wrote:
> seems like maybe we should extend the command line interface to accept 
> multiple databases in one go ?  and even add a "all" keyword ?

Sounds like a good idea to me.

Siddhesh
  
Mike Frysinger March 20, 2014, 7:12 p.m. UTC | #4
On Thu 20 Mar 2014 10:00:18 Siddhesh Poyarekar wrote:
> On Wed, Mar 19, 2014 at 03:20:56PM -0400, Mike Frysinger wrote:
> > seems like maybe we should extend the command line interface to accept
> > multiple databases in one go ?  and even add a "all" keyword ?
> 
> Sounds like a good idea to me.

moved to the TODO in the wiki.  i don't use nscd myself :).
-mike
  

Patch

diff --git a/nscd/nscd.service b/nscd/nscd.service
index b7428a3..ab38e8f 100644
--- a/nscd/nscd.service
+++ b/nscd/nscd.service
@@ -11,6 +11,7 @@  ExecReload=/usr/sbin/nscd -i passwd
 ExecReload=/usr/sbin/nscd -i group
 ExecReload=/usr/sbin/nscd -i hosts
 ExecReload=/usr/sbin/nscd -i services
+ExecReload=/usr/sbin/nscd -i netgroup
 Restart=always
 PIDFile=/run/nscd/nscd.pid