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

draft: run multi-kernel go test (vmtests) on arm64 with emulation #2343

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

mtardy
Copy link
Member

@mtardy mtardy commented Apr 18, 2024

This is now failing because: using emulation on arm64 runners we bumped into an issue we had in the little VM helper images CI, we need to force libguestfs to use emulation (see cilium/little-vm-helper-images@e42f11b) on arm64 while rebuilding the new image with the tests inside of it.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Deduplicate some codes shared between vmtests and LVH:
- inject LVH runner.RunConf into vmtests RunConf and reuse existing
  LVH configuration fields.
- use LVH runner.BuildQemuArgs along with custom vmtests logic.
- removes the support for HVF, we would need to put it in LVH to get
  this again but I don't think this is used nowadays.

This is mainly so that we benefit from the arm64 support as LVH was
recently updated to support it.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Previously the fetch-data was defaulting to root-images:latest that is
more than a year old now (!) but most importantly, not multi-arch as
this new tag.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
@mtardy mtardy added area/ci Related to CI arch/arm64 arm64 issues release-note/ci This PR makes changes to the CI. labels Apr 18, 2024
Base automatically changed from pr/mtardy/vmtests-lvh to main May 31, 2024 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arch/arm64 arm64 issues area/ci Related to CI release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant