Proxmox-Port/qemu
1
0
Fork 0
mirror of https://github.com/qemu/qemu.git synced 2025-08-07 06:13:49 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

migration-test: Only hide error if !QTEST_LOG

Browse Source 
The errors are very useful when debugging qtest failures, especially when
QTEST_LOG=1 is set.  Let's allow override MigrateStart.hide_stderr when
QTEST_LOG=1 is specified, because that means the user wants to be verbose.

Not very nice to introduce the first QTEST_LOG env access in migration-test.c,
however it should be handy.  Without this patch, I was hacking error_report()
when debugging such errors.  Let's make things easier.

Signed-off-by: Peter Xu <peterx@redhat.com>
Message-Id: <20201021212721.440373-7-peterx@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
This commit is contained in:
Peter Xu 2020-10-21 17:27:21 -04:00 committed by Dr. David Alan Gilbert
parent d246ea5039
commit a47295014d
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tests/qtest/migration-test.c
View File

@ -464,6 +464,10 @@ static void migrate_postcopy_start(QTestState *from, QTestState *to)
} }
typedef struct { typedef struct {
/*
* QTEST_LOG=1 may override this. When QTEST_LOG=1, we always dump errors
* unconditionally, because it means the user would like to be verbose.
*/
bool hide_stderr; bool hide_stderr;
bool use_shmem; bool use_shmem;
/* only launch the target process */ /* only launch the target process */
@ -557,7 +561,7 @@ static int test_migrate_start(QTestState **from, QTestState **to,
g_free(bootpath); g_free(bootpath);
if (args->hide_stderr) { if (!getenv("QTEST_LOG") && args->hide_stderr) {
ignore_stderr = "2>/dev/null"; ignore_stderr = "2>/dev/null";
} else { } else {
ignore_stderr = ""; ignore_stderr = "";