Skip to content

Makefile: Fail for unsupported hypervisor#5592

Merged
eriknordmark merged 1 commit intolf-edge:masterfrom
rene:fix-hv-make
Feb 7, 2026
Merged

Makefile: Fail for unsupported hypervisor#5592
eriknordmark merged 1 commit intolf-edge:masterfrom
rene:fix-hv-make

Conversation

@rene
Copy link
Copy Markdown
Contributor

@rene rene commented Feb 4, 2026

Description

Currently Makefile accepts any value for HV. This PR introduces a check to fail in case a unsupported hypervisor is passed through HV variable.

How to test and validate this PR

Try to build with non-valid HV values. It should fail:

make HV=bla rootfs
make HV=kubevirt rootfs

Of course it should build for valid HV values: acrn, kvm, k, mini and xen.

Changelog notes

No user-facing changes.

PR Backports

No backports are needed for this PR. This is an improvement.

Checklist

  • I've provided a proper description
  • I've added the proper documentation
  • I've tested my PR on amd64 device
  • I've tested my PR on arm64 device
  • I've written the test verification instructions
  • I've set the proper labels to this PR
  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

@rene rene requested a review from eriknordmark as a code owner February 4, 2026 09:52
Currently Makefile accepts any value for HV. This commit introduces a check
to fail in case a unsupported hypervisor is passed through HV variable.

Signed-off-by: Renê de Souza Pinto <rene@renesp.com.br>
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.49%. Comparing base (2281599) to head (7432e6b).
⚠️ Report is 267 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5592      +/-   ##
==========================================
+ Coverage   19.52%   29.49%   +9.96%     
==========================================
  Files          19       18       -1     
  Lines        3021     2417     -604     
==========================================
+ Hits          590      713     +123     
+ Misses       2310     1552     -758     
- Partials      121      152      +31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@eriknordmark eriknordmark merged commit ca36256 into lf-edge:master Feb 7, 2026
49 checks passed
@rene rene deleted the fix-hv-make branch March 3, 2026 09:20
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