From: "Ross Burton" <ross@burtonini.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 4/7] oeqa/core/target: move server_ip to superclass
Date: Fri, 10 Sep 2021 11:20:47 +0100 [thread overview]
Message-ID: <20210910102050.1266413-4-ross.burton@arm.com> (raw)
In-Reply-To: <20210910102050.1266413-1-ross.burton@arm.com>
Move towards a consistent interface across OETarget subclasses by
moving server_ip to the superclass.
Signed-off-by: Ross Burton <ross.burton@arm.com>
---
meta/lib/oeqa/core/target/__init__.py | 3 ++-
meta/lib/oeqa/core/target/qemu.py | 4 +---
meta/lib/oeqa/core/target/ssh.py | 3 +--
3 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/meta/lib/oeqa/core/target/__init__.py b/meta/lib/oeqa/core/target/__init__.py
index 3da66038342..aff02ecc439 100644
--- a/meta/lib/oeqa/core/target/__init__.py
+++ b/meta/lib/oeqa/core/target/__init__.py
@@ -7,7 +7,8 @@
class OETarget:
- def __init__(self, logger, *args, **kwargs):
+ def __init__(self, logger, server_ip):
+ self.server_ip = server_ip
self.logger = logger
def start(self):
diff --git a/meta/lib/oeqa/core/target/qemu.py b/meta/lib/oeqa/core/target/qemu.py
index 79fd724f7d4..7c6e353edf2 100644
--- a/meta/lib/oeqa/core/target/qemu.py
+++ b/meta/lib/oeqa/core/target/qemu.py
@@ -26,10 +26,8 @@ class OEQemuTarget(OESSHTarget):
tmpdir='', dir_image='', boottime=60, serial_ports=2,
boot_patterns = defaultdict(str), ovmf=False, tmpfsdir=None, **kwargs):
- super(OEQemuTarget, self).__init__(logger, None, server_ip, timeout,
- user, port)
+ super().__init__(logger, None, server_ip, timeout, user, port)
- self.server_ip = server_ip
self.server_port = 0
self.machine = machine
self.rootfs = rootfs
diff --git a/meta/lib/oeqa/core/target/ssh.py b/meta/lib/oeqa/core/target/ssh.py
index 923a223b25b..4da1e4c2b2d 100644
--- a/meta/lib/oeqa/core/target/ssh.py
+++ b/meta/lib/oeqa/core/target/ssh.py
@@ -27,9 +27,8 @@ class OESSHTarget(OETarget):
fileHandler.setFormatter(formatter)
logger.addHandler(fileHandler)
- super(OESSHTarget, self).__init__(logger)
+ super().__init__(logger, server_ip)
self.ip = ip
- self.server_ip = server_ip
self.server_port = server_port
self.timeout = timeout
self.user = user
--
2.25.1
next prev parent reply other threads:[~2021-09-10 10:20 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-10 10:20 [PATCH 1/7] testimage: TEST_SERVER_IP isn't mandatory Ross Burton
2021-09-10 10:20 ` [PATCH 2/7] testimage: pass the base image name to the target class Ross Burton
2021-09-10 10:20 ` [PATCH 3/7] oeqa/core/target: remove abstract decorations Ross Burton
2021-09-10 10:20 ` Ross Burton [this message]
2021-09-10 10:20 ` [PATCH 5/7] oeqa/core/target: remove server port parameter Ross Burton
2021-09-10 10:20 ` [PATCH 6/7] oeqa/core/target: add target_ip to superclass Ross Burton
2021-09-10 10:20 ` [PATCH 7/7] oeqa/target/ssh: don't assume target_dumper is set Ross Burton
2021-09-10 12:21 ` [OE-core] [PATCH 1/7] testimage: TEST_SERVER_IP isn't mandatory Ross Burton
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=20210910102050.1266413-4-ross.burton@arm.com \
--to=ross@burtonini.com \
--cc=openembedded-core@lists.openembedded.org \
/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 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.