Skip to content

Add KVM based checks #160

Open
aerosouund wants to merge 4 commits into
edera-dev:mainfrom
aerosouund:checks
Open

Add KVM based checks #160
aerosouund wants to merge 4 commits into
edera-dev:mainfrom
aerosouund:checks

Conversation

@aerosouund

Copy link
Copy Markdown

Closes: #154

Explanation

Add a hypervisor flag to the preinstall command to specify that the kvm based group of checks should run (/dev/kvm exists, kvm_amd or kvm_intel exist, vhost_vsock exists)

Example successful run:

Running Group KVM checks [Optional] - Running edera in a KVM environment checks
    • Host contains the required modules for running edera on KVM: Passed
    • /dev/kvm check: Passed
✅ KVM checks: Passed

Example failed run:

Running Group KVM checks [Required] - Running edera in a KVM environment checks
    • Host contains the required modules for running edera on KVM: Failed: missing ["kvm_amd"]
    • /dev/kvm check: Failed: /dev/kvm doesn't exist on the host
❌ KVM checks: Failed

cc: @bleggett @antoineco

this logic may be used by other check groups that want to verify the
existence of certain modules. modify the KernelChecks has_modules check
to import and leverage this moved logic

Signed-off-by: aerosouund <aerosound161@gmail.com>
other check lanes may need this logic to check the cpu vendor or flags.

Signed-off-by: aerosouund <aerosound161@gmail.com>
@antoineco antoineco self-assigned this Jul 11, 2026
@antoineco antoineco self-requested a review July 11, 2026 12:13
@antoineco antoineco removed their assignment Jul 11, 2026
create a new lane of checks that assert the needed requirements presence
for running edera on a kvm based host.
which are:
- /dev/kvm exists
- the vhost_vsock module exists
- the one of vendor specific kvm modules exist on the host (kvm_amd,
  kvm_intel)

Signed-off-by: aerosouund <aerosound161@gmail.com>
if specified, it will signal the need to run the kvm based checks. which
are the Kvm, PVH and IOMMU lanes

Signed-off-by: aerosouund <aerosound161@gmail.com>
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.

Add proper KVM prerequisite checks

2 participants