Re: [ovs-dev] [PATCH] ovs-lib: don't log service status operation

2014-02-27 Thread Flavio Leitner
On Wed, Feb 26, 2014 at 08:33:22PM -0800, Gurucharan Shetty wrote: > On Wed, Feb 26, 2014 at 8:28 PM, Ben Pfaff wrote: > > On Wed, Feb 26, 2014 at 11:19:15PM -0300, Flavio Leitner wrote: > >> It prevents non-root users to check service status. > >> > >> Signed-off-by: Flavio Leitner > > > > Guru,

Re: [ovs-dev] [PATCH] ovs-lib: don't log service status operation

2014-02-26 Thread Gurucharan Shetty
On Wed, Feb 26, 2014 at 8:28 PM, Ben Pfaff wrote: > On Wed, Feb 26, 2014 at 11:19:15PM -0300, Flavio Leitner wrote: >> It prevents non-root users to check service status. >> >> Signed-off-by: Flavio Leitner > > Guru, if you're OK with this change, it's fine by me. Otherwise, if > it's important

Re: [ovs-dev] [PATCH] ovs-lib: don't log service status operation

2014-02-26 Thread Ben Pfaff
On Wed, Feb 26, 2014 at 11:19:15PM -0300, Flavio Leitner wrote: > It prevents non-root users to check service status. > > Signed-off-by: Flavio Leitner Guru, if you're OK with this change, it's fine by me. Otherwise, if it's important to log, then we could compromise by using "tee" only if "tes

[ovs-dev] [PATCH] ovs-lib: don't log service status operation

2014-02-26 Thread Flavio Leitner
It prevents non-root users to check service status. Signed-off-by: Flavio Leitner --- utilities/ovs-lib.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/utilities/ovs-lib.in b/utilities/ovs-lib.in index 029ed3b..f14862d 100644 --- a/utilities/ovs-lib.in +++ b/utilities/