From: Cleber Rosa <crosa@redhat.com>
To: Eduardo Habkost <ehabkost@redhat.com>
Cc: "Aleksandar Markovic" <amarkovic@wavecomp.com>,
"Aleksandar Rikalo" <arikalo@wavecomp.com>,
qemu-devel@nongnu.org,
"Wainer dos Santos Moschetta" <wainersm@redhat.com>,
"Aleksandar Markovic" <aleksandar.markovic@rt-rk.com>,
"Willian Rampazzo" <wrampazz@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@redhat.com>,
"Aurelien Jarno" <aurelien@aurel32.net>,
"David Gibson" <david@gibson.dropbear.id.au>
Subject: Re: [PATCH] Acceptance tests: use avocado.utils.ssh for SSH interaction
Date: Fri, 20 Sep 2019 16:27:32 -0400 [thread overview]
Message-ID: <20190920202732.GA7009@localhost.localdomain> (raw)
In-Reply-To: <20190920191654.GF5035@habkost.net>
On Fri, Sep 20, 2019 at 04:16:54PM -0300, Eduardo Habkost wrote:
> On Thu, Sep 19, 2019 at 06:59:05PM -0400, Cleber Rosa wrote:
> > This replaces paramiko with avocado.utils.ssh module, which is based
> > on a (open)ssh binary, supposedly more ubiquitous.
> >
> > Signed-off-by: Cleber Rosa <crosa@redhat.com>
>
> I just noticed that I have some patches on python-next that never
> went to a pull request (sorry for the oversight!).
>
> I assume I should remove
> https://lore.kernel.org/qemu-devel/20190830184033.7630-1-philmd@redhat.com/
> from the queue?
>
Yep, I believe so (assuming this one gets reviewed before the next
pull request).
- Cleber.
> > ---
> > tests/acceptance/linux_ssh_mips_malta.py | 74 ++++++++----------------
> > tests/requirements.txt | 3 +-
> > 2 files changed, 25 insertions(+), 52 deletions(-)
> >
> > diff --git a/tests/acceptance/linux_ssh_mips_malta.py b/tests/acceptance/linux_ssh_mips_malta.py
> > index 134f10cac3..30c65568ce 100644
> > --- a/tests/acceptance/linux_ssh_mips_malta.py
> > +++ b/tests/acceptance/linux_ssh_mips_malta.py
> > @@ -9,13 +9,14 @@ import os
> > import re
> > import base64
> > import logging
> > -import paramiko
> > import time
> >
> > from avocado import skipIf
> > +from avocado import skipUnless
> > from avocado_qemu import Test
> > from avocado.utils import process
> > from avocado.utils import archive
> > +from avocado.utils import ssh
> >
> >
> > class LinuxSSH(Test):
> > @@ -26,35 +27,18 @@ class LinuxSSH(Test):
> > VM_IP = '127.0.0.1'
> >
> > IMAGE_INFO = {
> > - 'be': {
> > - 'image_url': 'https://people.debian.org/~aurel32/qemu/mips/'
> > - 'debian_wheezy_mips_standard.qcow2',
> > - 'image_hash': '8987a63270df67345b2135a6b7a4885a35e392d5',
> > - 'rsa_hostkey': b'AAAAB3NzaC1yc2EAAAADAQABAAABAQCca1VitiyLAdQOld'
> > - b'zT43IOEVJZ0wHD78GJi8wDAjMiYWUzNSSn0rXGQsINHuH5'
> > - b'IlF+kBZsHinb/FtKCAyS9a8uCHhQI4SuB4QhAb0+39MlUw'
> > - b'Mm0CLkctgM2eUUZ6MQMQvDlqnue6CCkxN62EZYbaxmby7j'
> > - b'CQa1125o1HRKBvdGm2zrJWxXAfA+f1v6jHLyE8Jnu83eQ+'
> > - b'BFY25G+Vzx1PVc3zQBwJ8r0NGTRqy2//oWQP0h+bMsgeFe'
> > - b'KH/J3RJM22vg6+I4JAdBFcxnK+l781h1FuRxOn4O/Xslbg'
> > - b'go6WtB4V4TOsw2E/KfxI5IZ/icxF+swVcnvF46Hf3uQc/0'
> > - b'BBqb',
> > - },
> > - 'le': {
> > - 'image_url': 'https://people.debian.org/~aurel32/qemu/mipsel/'
> > - 'debian_wheezy_mipsel_standard.qcow2',
> > - 'image_hash': '7866764d9de3ef536ffca24c9fb9f04ffdb45802',
> > - 'rsa_hostkey': b'AAAAB3NzaC1yc2EAAAADAQABAAABAQClXJlBT71HL5yKvv'
> > - b'gfC7jmxSWx5zSBCzET6CLZczwAafSIs7YKfNOy/dQTxhuk'
> > - b'yIGFUugZFoF3E9PzdhunuyvyTd56MPoNIqFbb5rGokwU5I'
> > - b'TOx3dBHZR0mClypL6MVrwe0bsiIb8GhF1zioNwcsaAZnAi'
> > - b'KfXStVDtXvn/kLLq+xLABYt48CC5KYWoFaCoICskLAY+qo'
> > - b'L+LWyAnQisj4jAH8VSaSKIImFpfkHWEXPhHcC4ZBlDKtnH'
> > - b'po9vhfCHgnfW3Pzrqmk8BI4HysqPFVmJWkJGlGUL+sGeg3'
> > - b'ZZolAYuDXGuBrw8ooPJq2v2dOH+z6dyD2q/ypmAbyPqj5C'
> > - b'rc8H',
> > - },
> > - }
> > + 'be': {'image_url': ('https://people.debian.org/~aurel32/qemu/mips/'
> > + 'debian_wheezy_mips_standard.qcow2'),
> > + 'image_hash': '8987a63270df67345b2135a6b7a4885a35e392d5'},
> > + 'le': {'image_url': ('https://people.debian.org/~aurel32/qemu/mipsel/'
> > + 'debian_wheezy_mipsel_standard.qcow2'),
> > + 'image_hash': '7866764d9de3ef536ffca24c9fb9f04ffdb45802'}
> > + }
> > +
> > +
> > + @skipUnless(ssh.SSH_CLIENT_BINARY, 'No SSH client available')
> > + def setUp(self):
> > + super(LinuxSSH, self).setUp()
> >
> > def wait_for_console_pattern(self, success_message,
> > failure_message='Oops'):
> > @@ -78,23 +62,14 @@ class LinuxSSH(Test):
> > self.log.debug("sshd listening on port:" + port)
> > return port
> >
> > - def ssh_connect(self, username, password, rsa_hostkey_b64=None):
> > + def ssh_connect(self, username, password):
> > self.ssh_logger = logging.getLogger('ssh')
> > - self.ssh_username = username
> > - self.ssh_ps1 = '# ' if username is 'root' else '$ '
> > - self.ssh_client = paramiko.SSHClient()
> > port = self.get_portfwd()
> > - if rsa_hostkey_b64:
> > - rsa_hostkey_bin = base64.b64decode(rsa_hostkey_b64)
> > - rsa_hostkey = paramiko.RSAKey(data = rsa_hostkey_bin)
> > - ipport = '[%s]:%s' % (self.VM_IP, port)
> > - self.ssh_logger.debug('ipport ' + ipport)
> > - self.ssh_client.get_host_keys().add(ipport, 'ssh-rsa', rsa_hostkey)
> > + self.ssh_session = ssh.Session(self.VM_IP, port=int(port),
> > + user=username, password=password)
> > for i in range(10):
> > try:
> > - self.ssh_client.connect(self.VM_IP, int(port),
> > - username, password, banner_timeout=90)
> > - self.ssh_logger.info("Entering interactive session.")
> > + self.ssh_session.connect()
> > return
> > except:
> > time.sleep(4)
> > @@ -102,15 +77,15 @@ class LinuxSSH(Test):
> > self.fail("sshd timeout")
> >
> > def ssh_disconnect_vm(self):
> > - self.ssh_client.close()
> > + self.ssh_session.quit()
> >
> > def ssh_command(self, command, is_root=True):
> > - self.ssh_logger.info(self.ssh_ps1 + command)
> > - stdin, stdout, stderr = self.ssh_client.exec_command(command)
> > - stdout_lines = [line.strip('\n') for line in stdout]
> > + self.ssh_logger.info(command)
> > + result = self.ssh_session.cmd(command)
> > + stdout_lines = [line.rstrip() for line in result.stdout_text.splitlines()]
> > for line in stdout_lines:
> > self.ssh_logger.info(line)
> > - stderr_lines = [line.strip('\n') for line in stderr]
> > + stderr_lines = [line.rstrip() for line in result.stderr_text.splitlines()]
> > for line in stderr_lines:
> > self.ssh_logger.warning(line)
> > return stdout_lines, stderr_lines
> > @@ -119,7 +94,6 @@ class LinuxSSH(Test):
> > image_url = self.IMAGE_INFO[endianess]['image_url']
> > image_hash = self.IMAGE_INFO[endianess]['image_hash']
> > image_path = self.fetch_asset(image_url, asset_hash=image_hash)
> > - rsa_hostkey_b64 = self.IMAGE_INFO[endianess]['rsa_hostkey']
> >
> > self.vm.set_machine('malta')
> > self.vm.set_console()
> > @@ -138,7 +112,7 @@ class LinuxSSH(Test):
> > self.wait_for_console_pattern(console_pattern)
> > self.log.info('sshd ready')
> >
> > - self.ssh_connect('root', 'root', rsa_hostkey_b64=rsa_hostkey_b64)
> > + self.ssh_connect('root', 'root')
> >
> > def shutdown_via_ssh(self):
> > self.ssh_command('poweroff')
> > diff --git a/tests/requirements.txt b/tests/requirements.txt
> > index bd1f7590ed..a2a587223a 100644
> > --- a/tests/requirements.txt
> > +++ b/tests/requirements.txt
> > @@ -1,5 +1,4 @@
> > # Add Python module requirements, one per line, to be installed
> > # in the tests/venv Python virtual environment. For more info,
> > # refer to: https://pip.pypa.io/en/stable/user_guide/#id1
> > -avocado-framework==68.0
> > -paramiko==2.4.2
> > +avocado-framework==72.0
> > --
> > 2.21.0
> >
> >
>
> --
> Eduardo
next prev parent reply other threads:[~2019-09-20 20:28 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-19 22:59 [PATCH] Acceptance tests: use avocado.utils.ssh for SSH interaction Cleber Rosa
2019-09-20 19:16 ` Eduardo Habkost
2019-09-20 20:27 ` Cleber Rosa [this message]
2019-09-20 20:33 ` Eduardo Habkost
2019-09-20 20:49 ` Cleber Rosa
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190920202732.GA7009@localhost.localdomain \
--to=crosa@redhat.com \
--cc=aleksandar.markovic@rt-rk.com \
--cc=amarkovic@wavecomp.com \
--cc=arikalo@wavecomp.com \
--cc=aurelien@aurel32.net \
--cc=david@gibson.dropbear.id.au \
--cc=ehabkost@redhat.com \
--cc=philmd@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=wainersm@redhat.com \
--cc=wrampazz@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).