-
Notifications
You must be signed in to change notification settings - Fork 35
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
MTV-1483 | Fix virtio-win driver version #1048
Conversation
Moving to draft as I'm not sure if I want to go this route |
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1048 +/- ##
==========================================
- Coverage 16.25% 16.20% -0.05%
==========================================
Files 112 112
Lines 19882 19882
==========================================
- Hits 3231 3222 -9
- Misses 16364 16375 +11
+ Partials 287 285 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Issue: The centos 9 AppStream repo has outdated windows virtio driver. This change updates the rpm to the fedora package. Signed-off-by: Martin Necas <[email protected]>
2cab2e3
to
b49c04c
Compare
Quality Gate passedIssues Measures |
Got build from koji for el9, moving from draft |
Will this be pushed to virt-v2v image in quay.io/kubev2v registry? Since , am using olm - I tried build now with the :latest - still its pointing to virtio-win-1.9.15.iso. Or do we still need to rebuild it only with bazel? @mnecas |
@jsakil14 strange, it should be updated, let me check. |
I opened #1112 you will need to update the controller image after it, hope it will help |
@mnecas was this pushed? |
Hi @mnecas just following up on this, the |
Issue:
The Centos 9 AppStream repo has an outdated Windows virtio driver which does not support Windows 2022 and has issues with 2019. This change updates the rpm to the fedora package.
This is not a proper fix but WA as the base image is ubi9 and we should not mix the packages.
Fixes: #982
Fixes: #1029
Fixes: https://issues.redhat.com/browse/MTV-1483