8000 test(e2e/pipelines): run tests on qemu runner by maxgio92 · Pull Request #1999 · chainguard-dev/melange · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

test(e2e/pipelines): run tests on qemu runner #1999

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

maxgio92
Copy link
Contributor
@maxgio92 maxgio92 commented May 21, 2025

This PR sets the QEMU runner as the default runner for e2e tests for pipelines.
It uses the Alpine kernel, the same way we do for the Wolfi presubmit CI workflow.

Melange Pull Request Template

Functional Changes

  • This change can build all of Wolfi without errors (describe results in notes)

Notes:

SCA Changes

  • Examining several representative APKs show no regression / the desired effect (details in notes)

Notes:

Linter

  • The new check is clean across Wolfi
  • The new check is opt-in or a warning

Notes:

@maxgio92 maxgio92 force-pushed the e2e-tests-pipelines-on-qemu branch from 6f1d7bf to 9fa6cb6 Compare May 21, 2025 16:02
Signed-off-by: Massimiliano Giovagnoli <massimiliano.giovagnoli@chainguard.dev>
@maxgio92 maxgio92 force-pushed the e2e-tests-pipelines-on-qemu branch from 9fa6cb6 to 4c8b749 Compare May 21, 2025 17:37
@maxgio92 maxgio92 changed the title wip(e2e-tests): run pipeline e2e tests on qemu runner wip(e2e-tests): run e2e tests on qemu runner May 21, 2025
maxgio92 added 2 commits May 21, 2025 20:58
Signed-off-by: Massimiliano Giovagnoli <massimiliano.giovagnoli@chainguard.dev>
Signed-off-by: Massimiliano Giovagnoli <massimiliano.giovagnoli@chainguard.dev>
@maxgio92 maxgio92 changed the title wip(e2e-tests): run e2e tests on qemu runner test(e2e): run e2e tests on qemu runner May 21, 2025
Signed-off-by: Massimiliano Giovagnoli <massimiliano.giovagnoli@chainguard.dev>
@maxgio92 maxgio92 changed the title test(e2e): run e2e tests on qemu runner test(e2e/pipelines): run e2e tests on qemu runner May 21, 2025
@maxgio92 maxgio92 changed the title test(e2e/pipelines): run e2e tests on qemu runner test(e2e/pipelines): run tests on qemu runner May 21, 2025
@maxgio92 maxgio92 marked this pull request as ready for review May 21, 2025 21:59
Comment on lines +79 to +80
KERNEL_PKG="$(curl -sL https://dl-cdn.alpinelinux.org/alpine/edge/main/x86_64/APKINDEX.tar.gz | tar -Oxz APKINDEX | awk -F':' '$1 == "P" {printf "%s-", $2} $1 == "V" {printf "%s.apk\n", $2}' | grep "linux-virt" | grep -v dev)"
curl -LSo linux-virt.apk "https://dl-cdn.alpinelinux.org/alpine/edge/main/x86_64/$KERNEL_PKG"
Copy link
Contributor

Choose a reason for hiding this comment

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

does it make sense to pin to a version here? and verify the checksum?

Copy link
Contributor Author
@maxgio92 maxgio92 May 22, 2025

Choose a reason for hiding this comment

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

Good point @joshrwolf. I think that would be easier to take the latest, to reduce maintenance since we're not really depending on specific kernel features for these tests. Also the one provided by linux-virt for virtualization is based on LTS https://wiki.alpinelinux.org/wiki/Kernels

@maxgio92 maxgio92 enabled auto-merge (squash) May 22, 2025 13:12
Signed-off-by: Massimiliano Giovagnoli <massimiliano.giovagnoli@chainguard.dev>
@maxgio92
Copy link
Contributor Author

I think we need force merge in this case, since test failures are not related to the changes, but CVEs found on fixture packages.
cc @chainguard-dev/tools-squad

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0