Skip to content

Commit 53a22ca

Browse files
authored
Merge pull request #3970 from apostasie/minor-apt-cleanup
CI: trivial github action cleanup
2 parents aeac380 + d9a0e87 commit 53a22ca

File tree

2 files changed

+16
-14
lines changed

2 files changed

+16
-14
lines changed

.github/workflows/test-canary.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ jobs:
2929
- name: "Remove snap loopback devices (conflicts with our loopback devices in TestRunDevice)"
3030
run: |
3131
sudo systemctl disable --now snapd.service snapd.socket
32-
sudo apt-get purge -y snapd
32+
sudo apt-get purge -qq snapd
3333
sudo losetup -Dv
3434
sudo losetup -lv
3535
- name: "Register QEMU (tonistiigi/binfmt)"
@@ -75,7 +75,7 @@ jobs:
7575
go-version: ${{ env.GO_VERSION }}
7676
check-latest: true
7777
- run: go install ./cmd/nerdctl
78-
- run: go install -v gotest.tools/gotestsum@v1
78+
- run: make install-dev-tools
7979
# This here is solely to get the cni install script, which has not been modified in 3+ years.
8080
# There is little to no reason to update this to latest containerd
8181
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2

.github/workflows/test.yml

Lines changed: 14 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ jobs:
137137
- name: "Remove snap loopback devices (conflicts with our loopback devices in TestRunDevice)"
138138
run: |
139139
sudo systemctl disable --now snapd.service snapd.socket
140-
sudo apt-get purge -y snapd
140+
sudo apt-get purge -qq snapd
141141
sudo losetup -Dv
142142
sudo losetup -lv
143143
- name: "Register QEMU (tonistiigi/binfmt)"
@@ -195,7 +195,7 @@ jobs:
195195
- name: "Remove snap loopback devices (conflicts with our loopback devices in TestRunDevice)"
196196
run: |
197197
sudo systemctl disable --now snapd.service snapd.socket
198-
sudo apt-get purge -y snapd
198+
sudo apt-get purge -qq snapd
199199
sudo losetup -Dv
200200
sudo losetup -lv
201201
- name: "Register QEMU (tonistiigi/binfmt)"
@@ -337,8 +337,9 @@ jobs:
337337
docker run --privileged --rm tonistiigi/binfmt --install linux/arm/v7
338338
- name: "Prepare integration test environment"
339339
run: |
340-
sudo apt-get install -y expect
341-
go install -v gotest.tools/gotestsum@v1
340+
# FIXME: remove expect when we are done removing unbuffer from tests
341+
sudo apt-get install -qq expect
342+
make install-dev-tools
342343
- name: "Ensure that the integration test suite is compatible with Docker"
343344
run: WITH_SUDO=true ./hack/test-integration.sh -test.target=docker
344345
- name: "Ensure that the IPv6 integration test suite is compatible with Docker"
@@ -361,8 +362,9 @@ jobs:
361362
with:
362363
go-version: ${{ env.GO_VERSION }}
363364
check-latest: true
364-
- run: go install ./cmd/nerdctl
365-
- run: go install -v gotest.tools/gotestsum@v1
365+
- run: |
366+
go install ./cmd/nerdctl
367+
make install-dev-tools
366368
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
367369
with:
368370
repository: containerd/containerd
@@ -400,13 +402,13 @@ jobs:
400402
curl -fsSL https://apt.releases.hashicorp.com/gpg | sudo gpg --dearmor -o /usr/share/keyrings/hashicorp-archive-keyring.gpg
401403
echo "deb [signed-by=/usr/share/keyrings/hashicorp-archive-keyring.gpg] https://apt.releases.hashicorp.com $(lsb_release -cs) main" | sudo tee /etc/apt/sources.list.d/hashicorp.list
402404
sudo sed -i 's/^Types: deb$/Types: deb deb-src/' /etc/apt/sources.list.d/ubuntu.sources
403-
sudo apt-get update
404-
sudo apt-get install -y libvirt-daemon libvirt-daemon-system vagrant ovmf
405+
sudo apt-get update -qq
406+
sudo apt-get install -qq libvirt-daemon libvirt-daemon-system vagrant ovmf
405407
# https://github.com/vagrant-libvirt/vagrant-libvirt/issues/1725#issuecomment-1454058646
406408
sudo cp /usr/share/OVMF/OVMF_VARS_4M.fd /var/lib/libvirt/qemu/nvram/
407409
sudo systemctl enable --now libvirtd
408-
sudo apt-get build-dep -y ruby-libvirt
409-
sudo apt-get install -y --no-install-recommends libxslt-dev libxml2-dev libvirt-dev ruby-bundler ruby-dev zlib1g-dev
410+
sudo apt-get build-dep -qq ruby-libvirt
411+
sudo apt-get install -qq --no-install-recommends libxslt-dev libxml2-dev libvirt-dev ruby-bundler ruby-dev zlib1g-dev
410412
sudo vagrant plugin install vagrant-libvirt
411413
- name: Boot VM
412414
run: |
@@ -434,8 +436,8 @@ jobs:
434436
- name: "Install QEMU"
435437
run: |
436438
set -eux
437-
sudo apt-get update
438-
sudo apt-get -qq install -y --no-install-recommends ovmf qemu-system-x86 qemu-utils
439+
sudo apt-get update -qq
440+
sudo apt-get install -qq --no-install-recommends ovmf qemu-system-x86 qemu-utils
439441
sudo modprobe kvm
440442
# `sudo usermod -aG kvm $(whoami)` does not take an effect on GHA
441443
sudo chown $(whoami) /dev/kvm

0 commit comments

Comments
 (0)