Bug 2012792

Summary:	[RFE] Add annotation to set the VDDK init image
Product:	Container Native Virtualization (CNV)	Reporter:	Fabien Dupont <fdupont>
Component:	Storage	Assignee:	Arik <ahadas>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Natalie Gavrielov <ngavrilo>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	4.9.0	CC:	ahadas, alitke, yadu, ycui
Target Milestone:	---	Keywords:	RFE
Target Release:	4.14.0	Flags:	ycui: needinfo? (fdupont)
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:	2023-08-13 14:59:10 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 Fabien Dupont 2021-10-11 10:20:17 UTC

Description of problem:

The configuration of the VDDK init image has been moved to the HCO CR. However, there are use cases where this value should be set per disk transfer, like using a specific version of VDDK depending on the VMware source provider.

In that situation, it would be great if an annotation could be set on the DataVolume CR and the controller built the Pod spec with the VDDK image set in the annotation, instead of the one configured in HCO.

Comment 1 Yan Du 2021-10-13 12:20:50 UTC

Flag the bug as RFE and set target release to 4.10 which is the soonest that could make it

Comment 2 Ying Cui 2021-12-09 02:57:09 UTC

As this is the RFE, do we need to create jira card to future follow? Thanks.

Comment 3 Adam Litke 2021-12-09 13:26:09 UTC

Ying, in my opinion, as long as the code merges with functional tests we should be good without the Jira issue.

Comment 5 Adam Litke 2023-08-09 18:01:11 UTC

Arik, is this still relevant?

Comment 6 Arik 2023-08-13 14:59:10 UTC

(In reply to Adam Litke from comment #5)
> Arik, is this still relevant?

Nope, it's done already (https://github.com/kubevirt/containerized-data-importer/pull/2102)