-
Notifications
You must be signed in to change notification settings - Fork 34
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-1041 | Change created virt-v2v conversion output to -o kubeVirt #1020
Conversation
LGTM, is this part of #778 ? |
yeah I just rebased it and I'll add some additional commits in followup MRs |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1020 +/- ##
==========================================
- Coverage 16.44% 16.42% -0.02%
==========================================
Files 107 107
Lines 19645 19672 +27
==========================================
+ Hits 3230 3232 +2
- Misses 16127 16149 +22
- Partials 288 291 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Bella Khizgiyaev <[email protected]>
Signed-off-by: Martin Necas <[email protected]>
616ea2b
to
6faea4f
Compare
2a7448a
to
70671bf
Compare
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1020 +/- ##
=======================================
Coverage ? 16.28%
=======================================
Files ? 112
Lines ? 19772
Branches ? 0
=======================================
Hits ? 3220
Misses ? 16268
Partials ? 284
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Martin Necas <[email protected]>
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Issue:
We need to provide the firmware for the OVA and right now the firmware is not exposed from
virt-v2v -o local
but is available in the-o kubevirt
.Fix:
Change the output file of the virt-v2v to the
-o kubevirt
and expose that via the server to the controller.Notes:
In future, we could use also
virt-v2v-inspector
to expose this parameter https://issues.redhat.com/browse/RHEL-58047. But right now not available in the rhel 9.4 and as we have this parameter in the-o kubevirt
I don't think that we need to backport it.This is continuation of: #778