diff mbox series

[2/2] automation: Remove installation of packages from test scripts

Message ID 20221117161643.3351-3-michal.orzel@amd.com (mailing list archive)
State New, archived
Headers show
Series CI improvements | expand

Commit Message

Michal Orzel Nov. 17, 2022, 4:16 p.m. UTC
Now, when these packages are already installed in the respective
containers, we can remove them from the test scripts.

Signed-off-by: Michal Orzel <michal.orzel@amd.com>
---
 automation/scripts/qemu-alpine-x86_64.sh        | 4 ----
 automation/scripts/qemu-smoke-dom0-arm32.sh     | 8 --------
 automation/scripts/qemu-smoke-dom0-arm64.sh     | 9 ---------
 automation/scripts/qemu-smoke-dom0less-arm64.sh | 9 ---------
 4 files changed, 30 deletions(-)

Comments

Stefano Stabellini Nov. 18, 2022, 11:48 p.m. UTC | #1
On Thu, 17 Nov 2022, Michal Orzel wrote:
> Now, when these packages are already installed in the respective
> containers, we can remove them from the test scripts.
> 
> Signed-off-by: Michal Orzel <michal.orzel@amd.com>

Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>


> ---
>  automation/scripts/qemu-alpine-x86_64.sh        | 4 ----
>  automation/scripts/qemu-smoke-dom0-arm32.sh     | 8 --------
>  automation/scripts/qemu-smoke-dom0-arm64.sh     | 9 ---------
>  automation/scripts/qemu-smoke-dom0less-arm64.sh | 9 ---------
>  4 files changed, 30 deletions(-)
> 
> diff --git a/automation/scripts/qemu-alpine-x86_64.sh b/automation/scripts/qemu-alpine-x86_64.sh
> index 2e9625109c56..8c8d0b0cd759 100755
> --- a/automation/scripts/qemu-alpine-x86_64.sh
> +++ b/automation/scripts/qemu-alpine-x86_64.sh
> @@ -2,10 +2,6 @@
>  
>  set -ex
>  
> -apt-get -qy update
> -apt-get -qy install --no-install-recommends cpio \
> -                                            busybox-static
> -
>  # DomU Busybox
>  cd binaries
>  mkdir -p initrd
> diff --git a/automation/scripts/qemu-smoke-dom0-arm32.sh b/automation/scripts/qemu-smoke-dom0-arm32.sh
> index ef19b988c2c4..98e4d481f65c 100755
> --- a/automation/scripts/qemu-smoke-dom0-arm32.sh
> +++ b/automation/scripts/qemu-smoke-dom0-arm32.sh
> @@ -2,14 +2,6 @@
>  
>  set -ex
>  
> -export DEBIAN_FRONTEND=noninteractive
> -apt-get -qy update
> -apt-get -qy install --no-install-recommends u-boot-qemu \
> -                                            u-boot-tools \
> -                                            device-tree-compiler \
> -                                            curl \
> -                                            cpio
> -
>  cd binaries
>  # Use the kernel from Debian
>  curl --fail --silent --show-error --location --output vmlinuz http://http.us.debian.org/debian/dists/bullseye/main/installer-armhf/current/images/netboot/vmlinuz
> diff --git a/automation/scripts/qemu-smoke-dom0-arm64.sh b/automation/scripts/qemu-smoke-dom0-arm64.sh
> index 9004c537e325..0bf5cfdc61f9 100755
> --- a/automation/scripts/qemu-smoke-dom0-arm64.sh
> +++ b/automation/scripts/qemu-smoke-dom0-arm64.sh
> @@ -2,15 +2,6 @@
>  
>  set -ex
>  
> -export DEBIAN_FRONTEND=noninteractive
> -apt-get -qy update
> -apt-get -qy install --no-install-recommends u-boot-qemu \
> -                                            u-boot-tools \
> -                                            device-tree-compiler \
> -                                            cpio \
> -                                            curl \
> -                                            busybox-static
> -
>  # DomU Busybox
>  cd binaries
>  mkdir -p initrd
> diff --git a/automation/scripts/qemu-smoke-dom0less-arm64.sh b/automation/scripts/qemu-smoke-dom0less-arm64.sh
> index 933209e2e01e..2b59346fdcfd 100755
> --- a/automation/scripts/qemu-smoke-dom0less-arm64.sh
> +++ b/automation/scripts/qemu-smoke-dom0less-arm64.sh
> @@ -38,15 +38,6 @@ fi
>  "
>  fi
>  
> -export DEBIAN_FRONTEND=noninteractive
> -apt-get -qy update
> -apt-get -qy install --no-install-recommends u-boot-qemu \
> -                                            u-boot-tools \
> -                                            device-tree-compiler \
> -                                            busybox-static \
> -                                            cpio \
> -                                            curl
> -
>  # XXX QEMU looks for "efi-virtio.rom" even if it is unneeded
>  curl -fsSLO https://github.com/qemu/qemu/raw/v5.2.0/pc-bios/efi-virtio.rom
>  ./binaries/qemu-system-aarch64 \
> -- 
> 2.25.1
>
diff mbox series

Patch

diff --git a/automation/scripts/qemu-alpine-x86_64.sh b/automation/scripts/qemu-alpine-x86_64.sh
index 2e9625109c56..8c8d0b0cd759 100755
--- a/automation/scripts/qemu-alpine-x86_64.sh
+++ b/automation/scripts/qemu-alpine-x86_64.sh
@@ -2,10 +2,6 @@ 
 
 set -ex
 
-apt-get -qy update
-apt-get -qy install --no-install-recommends cpio \
-                                            busybox-static
-
 # DomU Busybox
 cd binaries
 mkdir -p initrd
diff --git a/automation/scripts/qemu-smoke-dom0-arm32.sh b/automation/scripts/qemu-smoke-dom0-arm32.sh
index ef19b988c2c4..98e4d481f65c 100755
--- a/automation/scripts/qemu-smoke-dom0-arm32.sh
+++ b/automation/scripts/qemu-smoke-dom0-arm32.sh
@@ -2,14 +2,6 @@ 
 
 set -ex
 
-export DEBIAN_FRONTEND=noninteractive
-apt-get -qy update
-apt-get -qy install --no-install-recommends u-boot-qemu \
-                                            u-boot-tools \
-                                            device-tree-compiler \
-                                            curl \
-                                            cpio
-
 cd binaries
 # Use the kernel from Debian
 curl --fail --silent --show-error --location --output vmlinuz http://http.us.debian.org/debian/dists/bullseye/main/installer-armhf/current/images/netboot/vmlinuz
diff --git a/automation/scripts/qemu-smoke-dom0-arm64.sh b/automation/scripts/qemu-smoke-dom0-arm64.sh
index 9004c537e325..0bf5cfdc61f9 100755
--- a/automation/scripts/qemu-smoke-dom0-arm64.sh
+++ b/automation/scripts/qemu-smoke-dom0-arm64.sh
@@ -2,15 +2,6 @@ 
 
 set -ex
 
-export DEBIAN_FRONTEND=noninteractive
-apt-get -qy update
-apt-get -qy install --no-install-recommends u-boot-qemu \
-                                            u-boot-tools \
-                                            device-tree-compiler \
-                                            cpio \
-                                            curl \
-                                            busybox-static
-
 # DomU Busybox
 cd binaries
 mkdir -p initrd
diff --git a/automation/scripts/qemu-smoke-dom0less-arm64.sh b/automation/scripts/qemu-smoke-dom0less-arm64.sh
index 933209e2e01e..2b59346fdcfd 100755
--- a/automation/scripts/qemu-smoke-dom0less-arm64.sh
+++ b/automation/scripts/qemu-smoke-dom0less-arm64.sh
@@ -38,15 +38,6 @@  fi
 "
 fi
 
-export DEBIAN_FRONTEND=noninteractive
-apt-get -qy update
-apt-get -qy install --no-install-recommends u-boot-qemu \
-                                            u-boot-tools \
-                                            device-tree-compiler \
-                                            busybox-static \
-                                            cpio \
-                                            curl
-
 # XXX QEMU looks for "efi-virtio.rom" even if it is unneeded
 curl -fsSLO https://github.com/qemu/qemu/raw/v5.2.0/pc-bios/efi-virtio.rom
 ./binaries/qemu-system-aarch64 \