[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 04/10] tests/vm: use 'cp' instead of 'ln' for temporary vm ima
From: |
John Snow |
Subject: |
[PATCH v2 04/10] tests/vm: use 'cp' instead of 'ln' for temporary vm images |
Date: |
Thu, 16 Jun 2022 10:26:53 -0400 |
If the initial setup fails, you've permanently altered the state of the
downloaded image in an unknowable way. Use 'cp' like our other test
setup scripts do.
Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
---
tests/vm/centos | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/vm/centos b/tests/vm/centos
index 5c7bc1c1a9a..be4f6ff2f14 100755
--- a/tests/vm/centos
+++ b/tests/vm/centos
@@ -34,7 +34,7 @@ class CentosVM(basevm.BaseVM):
def build_image(self, img):
cimg =
self._download_with_cache("https://cloud.centos.org/centos/8/x86_64/images/CentOS-8-GenericCloud-8.3.2011-20201204.2.x86_64.qcow2")
img_tmp = img + ".tmp"
- subprocess.check_call(["ln", "-f", cimg, img_tmp])
+ subprocess.check_call(['cp', '-f', cimg, img_tmp])
self.exec_qemu_img("resize", img_tmp, "50G")
self.boot(img_tmp, extra_args = ["-cdrom", self.gen_cloud_init_iso()])
self.wait_ssh()
--
2.34.3
- [PATCH v2 01/10] tests/qemu-iotests: hotfix for 307, 223 output, (continued)
[PATCH v2 02/10] tests/qemu-iotests: skip 108 when FUSE is not loaded, John Snow, 2022/06/16
[PATCH v2 04/10] tests/vm: use 'cp' instead of 'ln' for temporary vm images,
John Snow <=
[PATCH v2 06/10] tests/vm: switch centos.aarch64 to CentOS 8 Stream, John Snow, 2022/06/16
[PATCH v2 07/10] tests/vm: update sha256sum for ubuntu.aarch64, John Snow, 2022/06/16
[PATCH v2 05/10] tests/vm: switch CentOS 8 to CentOS 8 Stream, John Snow, 2022/06/16
[PATCH v2 08/10] tests/vm: remove ubuntu.i386 VM test, John Snow, 2022/06/16
[PATCH v2 09/10] tests/vm: remove duplicate 'centos' VM test, John Snow, 2022/06/16
[PATCH v2 10/10] tests/vm: add 512MB extra memory per core, John Snow, 2022/06/16
Re: [PATCH v2 00/10] Improve reliability of VM tests, John Snow, 2022/06/17