Re: [libvirt] [PATCH 4/6] tests: qemumonitorjson: Raise the necessary debug level for QAPI schema checks

2018-07-13 Thread Ján Tomko 

On Thu, Jul 12, 2018 at 01:58:42PM +0200, Peter Krempa wrote:

The debug output of the schema validator on success is not so
interresting that it should be printed when basic debugging is enabled.



*interesting


Print it only when test debugging is set to 3 and more.

Signed-off-by: Peter Krempa 
---
tests/qemumonitorjsontest.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)


Jano


signature.asc
Description: Digital signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH 4/6] tests: qemumonitorjson: Raise the necessary debug level for QAPI schema checks

2018-07-12 Thread Peter Krempa 
The debug output of the schema validator on success is not so
interresting that it should be printed when basic debugging is enabled.

Print it only when test debugging is set to 3 and more.

Signed-off-by: Peter Krempa 
---
 tests/qemumonitorjsontest.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/qemumonitorjsontest.c b/tests/qemumonitorjsontest.c
index 9039cef423..fce2108932 100644
--- a/tests/qemumonitorjsontest.c
+++ b/tests/qemumonitorjsontest.c
@@ -2838,7 +2838,7 @@ testQAPISchema(const void *opaque)
 ret = 0;
 }

-if (virTestGetDebug() ||
+if (virTestGetDebug() >= 3 ||
 (ret < 0 && virTestGetVerbose())) {
 char *debugstr = virBufferContentAndReset();
 fprintf(stderr, "\n%s\n", debugstr);
-- 
2.16.2

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list