test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH] tests: create the vm2 DUT with vm2 in sriov kvm TestSuite.
@ 2017-01-07 20:51 Rami Rosen
  2017-01-11  8:34 ` Liu, Yong
  0 siblings, 1 reply; 2+ messages in thread
From: Rami Rosen @ 2017-01-07 20:51 UTC (permalink / raw)
  To: yong.liu; +Cc: dts, Rami Rosen

This patch fixes the setup_4vm_4vf_env() method in sriov kvm TestSuite 
to use vm2 for creating vm2 DUT, instead of wrongly using vm1 for this.

Signed-off-by: Rami Rosen <rami.rosen@intel.com>
---
 tests/TestSuite_sriov_kvm.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/TestSuite_sriov_kvm.py b/tests/TestSuite_sriov_kvm.py
index 41af4dd..43d5ccf 100644
--- a/tests/TestSuite_sriov_kvm.py
+++ b/tests/TestSuite_sriov_kvm.py
@@ -441,7 +441,7 @@ class TestSriovKvm(TestCase):
 
             self.vm2 = QEMUKvm(self.dut, 'vm2', 'sriov_kvm')
             self.vm2.set_vm_device(driver='pci-assign', **vf2_prop)
-            self.vm_dut_2 = self.vm1.start()
+            self.vm_dut_2 = self.vm2.start()
             if self.vm_dut_2 is None:
                 raise Exception("Set up VM2 ENV failed!")
 
-- 
1.9.1

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dts] [PATCH] tests: create the vm2 DUT with vm2 in sriov kvm TestSuite.
  2017-01-07 20:51 [dts] [PATCH] tests: create the vm2 DUT with vm2 in sriov kvm TestSuite Rami Rosen
@ 2017-01-11  8:34 ` Liu, Yong
  0 siblings, 0 replies; 2+ messages in thread
From: Liu, Yong @ 2017-01-11  8:34 UTC (permalink / raw)
  To: Rami Rosen; +Cc: dts

Thanks Rami. Applied into master branch.

On 01/08/2017 04:51 AM, Rami Rosen wrote:
> This patch fixes the setup_4vm_4vf_env() method in sriov kvm TestSuite
> to use vm2 for creating vm2 DUT, instead of wrongly using vm1 for this.
>
> Signed-off-by: Rami Rosen <rami.rosen@intel.com>
> ---
>   tests/TestSuite_sriov_kvm.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/TestSuite_sriov_kvm.py b/tests/TestSuite_sriov_kvm.py
> index 41af4dd..43d5ccf 100644
> --- a/tests/TestSuite_sriov_kvm.py
> +++ b/tests/TestSuite_sriov_kvm.py
> @@ -441,7 +441,7 @@ class TestSriovKvm(TestCase):
>   
>               self.vm2 = QEMUKvm(self.dut, 'vm2', 'sriov_kvm')
>               self.vm2.set_vm_device(driver='pci-assign', **vf2_prop)
> -            self.vm_dut_2 = self.vm1.start()
> +            self.vm_dut_2 = self.vm2.start()
>               if self.vm_dut_2 is None:
>                   raise Exception("Set up VM2 ENV failed!")
>   

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-01-11  8:39 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-07 20:51 [dts] [PATCH] tests: create the vm2 DUT with vm2 in sriov kvm TestSuite Rami Rosen
2017-01-11  8:34 ` Liu, Yong

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).