[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 11/18] tests/acceptance/boot_linux: rename misleading cloudinit me
From: |
Philippe Mathieu-Daudé |
Subject: |
[PULL 11/18] tests/acceptance/boot_linux: rename misleading cloudinit method |
Date: |
Mon, 8 Feb 2021 14:37:04 +0100 |
From: Cleber Rosa <crosa@redhat.com>
There's no downloading happening on that method, so let's call it
"prepare" instead. While at it, and because of it, the current
"prepare_boot" and "prepare_cloudinit" are also renamed.
The reasoning here is that "prepare_" methods will just work on the
images, while "set_up_" will make them effective to the VM that will
be launched. Inspiration comes from the "virtiofs_submounts.py"
tests, which this expects to converge more into.
Signed-off-by: Cleber Rosa <crosa@redhat.com>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Beraldo Leal <bleal@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20210203172357.1422425-3-crosa@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
tests/acceptance/boot_linux.py | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tests/acceptance/boot_linux.py b/tests/acceptance/boot_linux.py
index 2ac3e575874..bcd923bb4a0 100644
--- a/tests/acceptance/boot_linux.py
+++ b/tests/acceptance/boot_linux.py
@@ -57,7 +57,7 @@ def download_boot(self):
self.cancel('Failed to download/prepare boot image')
return boot.path
- def download_cloudinit(self, ssh_pubkey=None):
+ def prepare_cloudinit(self, ssh_pubkey=None):
self.log.info('Preparing cloudinit image')
try:
cloudinit_iso = os.path.join(self.workdir, 'cloudinit.iso')
@@ -85,15 +85,15 @@ def setUp(self, ssh_pubkey=None):
super(BootLinux, self).setUp()
self.vm.add_args('-smp', '2')
self.vm.add_args('-m', '1024')
- self.prepare_boot()
- self.prepare_cloudinit(ssh_pubkey)
+ self.set_up_boot()
+ self.set_up_cloudinit(ssh_pubkey)
- def prepare_boot(self):
+ def set_up_boot(self):
path = self.download_boot()
self.vm.add_args('-drive', 'file=%s' % path)
- def prepare_cloudinit(self, ssh_pubkey=None):
- cloudinit_iso = self.download_cloudinit(ssh_pubkey)
+ def set_up_cloudinit(self, ssh_pubkey=None):
+ cloudinit_iso = self.prepare_cloudinit(ssh_pubkey)
self.vm.add_args('-drive', 'file=%s,format=raw' % cloudinit_iso)
def launch_and_wait(self):
--
2.26.2
- [PULL 00/18] Integration testing patches for 2021-02-08, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 02/18] tests/acceptance: Introduce tesseract_ocr() helper, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 01/18] tests/acceptance: Extract tesseract_available() helper in new namespace, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 03/18] tests/acceptance: Extract do_test_arm_orangepi_armbian_uboot() method, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 07/18] tests/acceptance: Add a test for the virtex-ml507 ppc machine, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 05/18] tests/acceptance: Move the pseries test to a separate file, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 06/18] tests/acceptance: Test the mpc8544ds machine, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 11/18] tests/acceptance/boot_linux: rename misleading cloudinit method,
Philippe Mathieu-Daudé <=
- [PULL 12/18] tests/acceptance/virtiofs_submounts: use workdir property, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 14/18] tests/acceptance/virtiofs_submounts: use a virtio-net device instead, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 15/18] tests/acceptance/virtiofs_submounts: standardize port as integer, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 04/18] tests/acceptance: Test U-Boot/Linux from Armbian 20.08 on Orange Pi PC, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 09/18] tests/acceptance: Re-enable the microblaze test, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 13/18] tests/acceptance/virtiofs_submounts: do not ask for ssh key password, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 16/18] tests/acceptance/virtiofs_submounts: required space between IP and port, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 10/18] tests/acceptance/boot_linux: fix typo on cloudinit error message, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 18/18] Acceptance Tests: remove unnecessary tag from documentation example, Philippe Mathieu-Daudé, 2021/02/08
- [PULL 08/18] tests/acceptance: Increase the timeout in the replay tests, Philippe Mathieu-Daudé, 2021/02/08