Bug 1751187 - Auto-detect source VM OS for VM Import
Summary: Auto-detect source VM OS for VM Import
Keywords:
Status: CLOSED DEFERRED
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: V2V
Version: 2.1.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: future
Assignee: Brett Thurber
QA Contact: Daniel Gur
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-09-11 11:13 UTC by Marek Libra
Modified: 2020-06-17 13:51 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-06-17 13:51:27 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Marek Libra 2019-09-11 11:13:06 UTC
Description of problem:

HCO should create create the "vmware-to-kubevirt-os configmap: in the kube-public namespace.

Its presence is optional. But if missing, the pre-selection of OS during VMware import (v2v) does not work.

More info and example:
- https://github.com/kubevirt/web-ui-components/pull/249
- https://github.com/kubevirt/web-ui-operator/blob/master/build/kubevirt-web-ui-ansible/roles/kubevirt_web_ui/files/vmware-to-kubevirt-os.yaml

Comment 1 Ryan Hallisey 2019-09-11 14:36:06 UTC
Was this not covered by Daniel Erez's patch?  https://github.com/kubevirt/hyperconverged-cluster-operator/pull/227

Comment 2 Ryan Hallisey 2019-09-16 15:03:54 UTC
Adding Adam

Comment 3 Marek Libra 2019-09-17 07:15:37 UTC
I don't see this configmap being added by the 227 patch mentioned above.

I believe the example mentioned in the Description above can be taken as it is and used to fix this BZ.
Nice to have: fill missing mappings.

Comment 4 Ryan Hallisey 2019-09-17 11:11:18 UTC
Does CDI have the capacity to take this?

Comment 6 Tomas Jelinek 2019-09-18 13:47:12 UTC
This is not so urgent, it is a nice to have feature which preselects you an OS. But it is anyway a best effort only. Marek have verified that this does not block the import.
@Brett, do you agree lowering the priority and potentially remove from the 2.1?

Comment 7 Brett Thurber 2019-09-18 14:30:53 UTC
(In reply to Tomas Jelinek from comment #6)
> This is not so urgent, it is a nice to have feature which preselects you an
> OS. But it is anyway a best effort only. Marek have verified that this does
> not block the import.
> @Brett, do you agree lowering the priority and potentially remove from the
> 2.1?

Yes.  I set priority and urgency to medium and target CNV 2.1.1.

Comment 21 Brett Thurber 2020-06-17 13:51:27 UTC
We may consider this for MTV.


Note You need to log in before you can comment on or make changes to this bug.