diff mbox

virt: Repairs Migration_multi_host_fd test Pylint correction.

Message ID 1349958618-7854-1-git-send-email-jzupka@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jiri Zupka Oct. 11, 2012, 12:30 p.m. UTC
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(-)

Comments

Lucas Meneghel Rodrigues Oct. 11, 2012, 12:46 p.m. UTC | #1
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))
>
>

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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))