Bug 1666805 - Status field is required for DataVolumes while it should be optional.
Summary: Status field is required for DataVolumes while it should be optional.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Storage
Version: 1.4
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 1.4
Assignee: Alexander Wels
QA Contact: Kedar Bidarkar
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-01-16 15:40 UTC by Alexander Wels
Modified: 2019-03-05 16:50 UTC (History)
4 users (show)

Fixed In Version: v1.4.0-6
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-03-05 16:50:59 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
VM def with data volume that is missing a status field. (1.07 KB, text/plain)
2019-01-16 15:40 UTC, Alexander Wels
no flags Details
status field is no longer required for DataVolumes (3.57 KB, text/plain)
2019-01-29 09:13 UTC, Kedar Bidarkar
no flags Details

Description Alexander Wels 2019-01-16 15:40:05 UTC
Created attachment 1521076 [details]
VM def with data volume that is missing a status field.

Description of problem:
Creating a DataVolume requires on to add:
Status: {}
Into the spec, which should be optional if there is no status.

Version-Release number of selected component (if applicable):


How reproducible:
100%

Steps to Reproduce:
1. Create a VM using a DataVolumeTemplate yaml without the status field as part of the data volume. See attached example.
2. Try to create the data volume with a kubectl create -f <file>
3. Validation will reject the yaml.

Actual results:
Creation is rejected

Expected results:
Creation is successful.

Additional info:

Comment 2 Adam Litke 2019-01-17 13:35:11 UTC
Backport of fix to the cdi release-1.5 branch
https://github.com/kubevirt/containerized-data-importer/pull/610

Comment 3 Alexander Wels 2019-01-22 15:56:23 UTC
Vendor fix into kubevirt (where the validation was failing)

https://github.com/kubevirt/kubevirt/pull/1952

Comment 4 Kedar Bidarkar 2019-01-29 09:13:46 UTC
Created attachment 1524501 [details]
status field is no longer required for DataVolumes

Tested/VERIFIED this with the below CNV build.

cnv_build_name: "CNV-1.4-RHEL-7-20190128.n.0"


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