Skip to content

CI: fix vagrant#10817

Merged
AkihiroSuda merged 4 commits intocontainerd:mainfrom
AkihiroSuda:fix-10816
Oct 12, 2024
Merged

CI: fix vagrant#10817
AkihiroSuda merged 4 commits intocontainerd:mainfrom
AkihiroSuda:fix-10816

Conversation

@AkihiroSuda
Copy link
Member

Revert c25183f 1bfdcce

Fix #10816

@dosubot dosubot bot added the area/github_actions Pull requests that update GitHub Actions code label Oct 11, 2024
@AkihiroSuda AkihiroSuda changed the title Revert "use older version of OVMF package" "use vagrant from jammy in noble" CI: fix vagrant Oct 11, 2024
Signed-By: /usr/share/keyrings/ubuntu-archive-keyring.gpg
EOF
sudo apt-get update
sudo apt-get install -y libvirt-daemon libvirt-daemon-system vagrant ovmf=2022.02-3ubuntu0.22.04.2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Without the ovmf package we were getting the error

/usr/share/OVMF/OVMF_CODE.fd' is not accessible

while booting the VM

@samuelkarp
Copy link
Member

Looks like the Vagrant jobs are all failing with qemu: could not load PC BIOS '/usr/share/OVMF/OVMF_CODE.fd' (Libvirt::Error)

@AkihiroSuda AkihiroSuda force-pushed the fix-10816 branch 2 times, most recently from 6f262ae to 14a242f Compare October 11, 2024 19:56
This reverts commit c25183f.

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
This reverts commit 1bfdcce.

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Copy link
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like they all booted properly!

@akhilerm
Copy link
Member

akhilerm commented Oct 12, 2024

Seeing the OOM error in rockylinux with systemd cgroup driver in the past few runs. https://github.com/containerd/containerd/actions/runs/11298785754/job/31438636822?pr=10817#step:9:216

May be related to opencontainers/runc#4427 🤔

@AkihiroSuda AkihiroSuda merged commit 61f91b9 into containerd:main Oct 12, 2024
@rata
Copy link
Contributor

rata commented Oct 12, 2024

@akhilerm do you know what is the mem limit for that container?

@akhilerm
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/github_actions Pull requests that update GitHub Actions code kind/test size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vagrant CI is failing (Package ovmf is not available)

6 participants