Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Start using the free GH runners for e2e testing #421

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

ldoktor
Copy link
Contributor

@ldoktor ldoktor commented Aug 19, 2024

the free-runners come with pre-installed containerd which sometimes collides with the one we install. We also need to move docker images to /mnt due to limitted root (/) partition.

Note I tried testing this on my fork ldoktor#4 but for some reason I can only use ubuntu-latest while ubuntu-2204 (which is the same as ubuntu-latest) fails, which is why I left the az-* runners on for now, we can remove them once the GH runners stabilize.

the free-runners come with pre-installed containerd which sometimes
collides with the one we install. We also need to move docker images to
/mnt due to limitted root (/) partition.

Fixes: confidential-containers#407

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
@wainersm
Copy link
Member

Hi @ldoktor !

Good idea keeping the old jobs. We will be able to test the new ones only after the merge...

Copy link
Member

@wainersm wainersm left a comment

Choose a reason for hiding this comment

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

Thanks @ldoktor !

Copy link
Member

@beraldoleal beraldoleal left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

Copy link
Member

@fidencio fidencio left a comment

Choose a reason for hiding this comment

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

lgtm, thanks @ldoktor!

@wainersm wainersm merged commit 3cdc158 into confidential-containers:main Aug 20, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants