rhel7.3 guest report error message likes
"Tray of device ... is not open" when 'change' command
be executed in first time.
Signed-off-by: NXueqiang Wei <xuwei@redhat.com>

Signed-off-by: NCong Li <coli@redhat.com>

replace '%' with '%s' and remove unused variable.
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

Add scenerios for virstor/vioscsi test
e.g run iozone in system disk/data disk before/during/after sub tests (shutdown, reboot)

Correct the CmdError exception to get stderr, which is usefull for the caller
to parse and search specific messages.
Signed-off-by: NPing Li <pingl@redhat.com>

s/fail_falg/fail_flag
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

In serial sessions, frequently the kernel might print debug or
error messages that make read_up_to_prompt to timeout. Let's try
to be a little more robust and send a carriage return, to see if
we can get to the prompt when safe=True.
Signed-off-by: NCong Li <coli@redhat.com>

Signed-off-by: NLin Ma <lma@suse.com>

1. force create image for block testing.
2. ignore errors in get_installation_logs in finally part,
   which will blocks real failed reason.
3. no need compare current driver version as expected
   if driver is uninstalled.
Signed-off-by: NCong Li <coli@redhat.com>

For example,
'session.sendline(params.get("shutdown_command"))' was discarded,
"Shutdown command is sent, ..." is inapproprivate.
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

Signed-off-by: NCong Li <coli@redhat.com>

utils_test.wait_for_login had been replaced with vm.wait_for_login
You can refer to virt-test/feea4fd.
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

suppress_exception always be True currently
Signed-off-by: NSuqin Huang <shuang@redhat.com>

Signed-off-by: NSuqin Huang <shuang@redhat.com>

Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

The dbd70861(2. add qtree info when test failed) introduced
NameError about 'vm'.
The reason is that no vm is passed to get_match_cdrom(),

And the order of (qtree_info, serial_num) is wrong.
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

1) correct the comment and code format
2) remove unused variable
3) avoid call params.get() repeatly.
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

1. Remove scenarios "downgrade" and "uninstall".
This case won't upgrade / install driver after
 downgrade / uninstall, which is not acceptable.
Will write new case to cover those scenarios.
2. rename variant "uninstall".
3. remove some unreasonable cfg.
Signed-off-by: NCong Li <coli@redhat.com>

Signed-off-by: NXu Tian <xutian@redhat.com>

Signed-off-by: NXu Tian <xutian@redhat.com>

1. fix a syntax error.
The end double quote of variant "guest_alias" will be
analyzed as a string \" in raw string.
2. Update the path of driver virtio serial.
Signed-off-by: NCong Li <coli@redhat.com>

fix omission of 82f52f85Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

Signed-off-by: NCong Li <coli@redhat.com>

No need to call get_ballooned_memory() again after
utils_misc.wait_for(lambda: compare_mem ==
                    self.get_ballooned_memory(),...)
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

Signed-off-by: NCong Li <coli@redhat.com>

1. re-get session after re-enabling the primary link.
2. fix a typo of session.
Signed-off-by: NCong Li <coli@redhat.com>

source image and dest image are different before execute
block-job-reopen, that is expect. If recorrect test step
it will be duplicate with block-job-complete test. so
drop this case.
Signed-off-by: NXu Tian <xutian@redhat.com>

Signed-off-by: NXu Han <xuhan@redhat.com>

Enclose multiple exception(in one line) in parentheses.
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

"/tmp/test.qcow2" is extremely common and used by many cases.
In order to avoid the risk of destroying old image file,
It's better to rename the image.

In view of QemuIOConfig.__init__ had used test.tmpdir,
we adopt the form "test.tmpdir" + "test.qcow2".

besides above,
remove unused variable and adjust the position of logging.info.
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

logging.info() is behind error.TestFail(),
which leads to logging.info will never be executed.
Signed-off-by: NWei Jiangang <weijg.fnst@cn.fujitsu.com>

Signed-off-by: NXu Tian <xutian@redhat.com>

If memory file in hugepage, both build-in memory and hotpluged
memory allocate from hugepages, so page number should be double.

Fix build-in memory size to avoid memory overwrite during local
migration test.

And add 1G hugepage variant but run basic test on 1G hugepage.
Signed-off-by: NXu Tian <xutian@redhat.com>

Signed-off-by: NXu Tian <xutian@redhat.com>

Signed-off-by: NSuqin Huang <shuang@redhat.com>