Bug 2012792 - [RFE] Add annotation to set the VDDK init image [NEEDINFO]
Summary: [RFE] Add annotation to set the VDDK init image
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Storage
Version: 4.9.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: 4.14.0
Assignee: Arik
QA Contact: Natalie Gavrielov
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-10-11 10:20 UTC by Fabien Dupont
Modified: 2023-08-13 14:59 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-08-13 14:59:10 UTC
Target Upstream Version:
Embargoed:
ycui: needinfo? (fdupont)

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker CNV-14431 0 None None None 2022-11-16 15:43:58 UTC

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)
Note You need to log in before you can comment on or make changes to this bug.