diff mbox series

idmapd: Turn down the verbosity in flush_inotify()

Message ID 20200805190054.39513-1-steved@redhat.com
State New
Headers show
Series idmapd: Turn down the verbosity in flush_inotify() | expand

Commit Message

Steve Dickson Aug. 5, 2020, 7 p.m. UTC
So idmapd does not flood the logs with messages
nobody will understand, only print the message
with verbose is set.

Signed-off-by: Steve Dickson <steved@redhat.com>
---
 utils/idmapd/idmapd.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Steve Dickson Aug. 7, 2020, 3:11 p.m. UTC | #1
On 8/5/20 3:00 PM, Steve Dickson wrote:
> So idmapd does not flood the logs with messages
> nobody will understand, only print the message
> with verbose is set.
> 
> Signed-off-by: Steve Dickson <steved@redhat.com>
Committed...

steved.
> ---
>  utils/idmapd/idmapd.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/utils/idmapd/idmapd.c b/utils/idmapd/idmapd.c
> index 8631414..7d1096d 100644
> --- a/utils/idmapd/idmapd.c
> +++ b/utils/idmapd/idmapd.c
> @@ -500,7 +500,8 @@ flush_inotify(int fd)
>  		     ptr += sizeof(struct inotify_event) + ev->len) {
>  
>  			ev = (const struct inotify_event *)ptr;
> -			xlog_warn("pipefs inotify: wd=%i, mask=0x%08x, len=%i, name=%s",
> +			if (verbose > 1)
> +				xlog_warn("pipefs inotify: wd=%i, mask=0x%08x, len=%i, name=%s",
>  				  ev->wd, ev->mask, ev->len, ev->len ? ev->name : "");
>  		}
>  	}
>
diff mbox series

Patch

diff --git a/utils/idmapd/idmapd.c b/utils/idmapd/idmapd.c
index 8631414..7d1096d 100644
--- a/utils/idmapd/idmapd.c
+++ b/utils/idmapd/idmapd.c
@@ -500,7 +500,8 @@  flush_inotify(int fd)
 		     ptr += sizeof(struct inotify_event) + ev->len) {
 
 			ev = (const struct inotify_event *)ptr;
-			xlog_warn("pipefs inotify: wd=%i, mask=0x%08x, len=%i, name=%s",
+			if (verbose > 1)
+				xlog_warn("pipefs inotify: wd=%i, mask=0x%08x, len=%i, name=%s",
 				  ev->wd, ev->mask, ev->len, ev->len ? ev->name : "");
 		}
 	}