Bug 1751187

Summary:	Auto-detect source VM OS for VM Import
Product:	Container Native Virtualization (CNV)	Reporter:	Marek Libra <mlibra>
Component:	V2V	Assignee:	Brett Thurber <bthurber>
Status:	CLOSED DEFERRED	QA Contact:	Daniel Gur <dagur>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	2.1.0	CC:	alitke, bthurber, cnv-qe-bugs, dagur, derez, fdeutsch, ncredi, rhallise, tjelinek
Target Milestone:	---	Keywords:	Reopened
Target Release:	future
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-06-17 13:51:27 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

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.