All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] virt: Repairs Migration_multi_host_fd test Pylint correction.
@ 2012-10-11 12:30 Jiří Župka
  2012-10-11 12:46 ` Lucas Meneghel Rodrigues
  0 siblings, 1 reply; 2+ messages in thread
From: Jiří Župka @ 2012-10-11 12:30 UTC (permalink / raw)
  To: autotest, kvm, kvm-autotest, lmr, ldoktor, jzupka

Pylint correction makes migration_multi_host_fd test dysfunctional.

pull-request: https://github.com/autotest/virt-test/pull/55

Signed-off-by: Jiří Župka <jzupka@redhat.com>
---
 kvm/tests/migration_multi_host_fd.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/kvm/tests/migration_multi_host_fd.py b/kvm/tests/migration_multi_host_fd.py
index ec9210a..7f08205 100644
--- a/kvm/tests/migration_multi_host_fd.py
+++ b/kvm/tests/migration_multi_host_fd.py
@@ -105,7 +105,7 @@ def run_migration_multi_host_fd(test, params, env):
                 s = self._create_server(mig_port)
                 try:
                     conn, _ = s.accept()
-                    fd = s.fileno()
+                    fd = conn.fileno()
                     logging.debug("File descrtiptor %d used for"
                                   " migration." % (fd))
 
-- 
1.7.7.6


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

* Re: [PATCH] virt: Repairs Migration_multi_host_fd test Pylint correction.
  2012-10-11 12:30 [PATCH] virt: Repairs Migration_multi_host_fd test Pylint correction Jiří Župka
@ 2012-10-11 12:46 ` Lucas Meneghel Rodrigues
  0 siblings, 0 replies; 2+ messages in thread
From: Lucas Meneghel Rodrigues @ 2012-10-11 12:46 UTC (permalink / raw)
  To: Jiří Župka; +Cc: autotest, kvm, kvm-autotest, ldoktor

On 10/11/2012 09:30 AM, Jiří Župka wrote:
> Pylint correction makes migration_multi_host_fd test dysfunctional.

Ops, I'm sorry about that, applied!

> pull-request: https://github.com/autotest/virt-test/pull/55
>
> Signed-off-by: Jiří Župka <jzupka@redhat.com>
> ---
>   kvm/tests/migration_multi_host_fd.py |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/kvm/tests/migration_multi_host_fd.py b/kvm/tests/migration_multi_host_fd.py
> index ec9210a..7f08205 100644
> --- a/kvm/tests/migration_multi_host_fd.py
> +++ b/kvm/tests/migration_multi_host_fd.py
> @@ -105,7 +105,7 @@ def run_migration_multi_host_fd(test, params, env):
>                   s = self._create_server(mig_port)
>                   try:
>                       conn, _ = s.accept()
> -                    fd = s.fileno()
> +                    fd = conn.fileno()
>                       logging.debug("File descrtiptor %d used for"
>                                     " migration." % (fd))
>
>


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

end of thread, other threads:[~2012-10-11 12:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-11 12:30 [PATCH] virt: Repairs Migration_multi_host_fd test Pylint correction Jiří Župka
2012-10-11 12:46 ` Lucas Meneghel Rodrigues

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.