Bug 2088220 - DV remains in cloneScheduled while the source dv is HPP and the target dv is OCS
Summary: DV remains in cloneScheduled while the source dv is HPP and the target dv is OCS
Keywords:
Status: CLOSED DUPLICATE of bug 2084122
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Storage
Version: 4.11.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: 4.11.0
Assignee: Bartosz Rybacki
QA Contact: Yan Du
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-05-19 03:49 UTC by Guohua Ouyang
Modified: 2022-06-28 15:00 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-06-28 15:00:26 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
dv (2.25 KB, text/plain)
2022-05-19 03:49 UTC, Guohua Ouyang
no flags Details
rhel9 (4.89 KB, text/plain)
2022-05-19 03:51 UTC, Guohua Ouyang
no flags Details
vm (6.57 KB, text/plain)
2022-05-19 03:51 UTC, Guohua Ouyang
no flags Details

Description Guohua Ouyang 2022-05-19 03:49:49 UTC
Created attachment 1881155 [details]
dv

Description of problem:
DV remains in cloneScheduled while the source dv is HPP and the target dv is OCS

Version-Release number of selected component (if applicable):
v4.11.0-369

How reproducible:


Steps to Reproduce:
1. on web console, create a VM and change the storageClass from HPP(default) to OCS
2. the dv remains in cloneScheduled.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Guohua Ouyang 2022-05-19 03:51:11 UTC
Created attachment 1881156 [details]
rhel9

Comment 2 Guohua Ouyang 2022-05-19 03:51:41 UTC
Created attachment 1881157 [details]
vm

Comment 3 Yan Du 2022-06-01 07:06:23 UTC
There is error showed in cdi deployment pod :

Error validating clone spec, ignoring","PVC":"default/rhel9-smoggy-bear","error":"target resources requests storage size is smaller than the source

workaround: set the disk size larger

Comment 4 Adam Litke 2022-06-01 12:00:23 UTC
Bartosz can you please take a look?

Comment 5 Yan Du 2022-06-22 12:12:18 UTC
Bartosz, do you have any updates for this bug?

Comment 6 Bartosz Rybacki 2022-06-27 11:18:36 UTC
Yes, it is now in review waiting to be merged and backported to correct version (UPSTREAM).

The problem is with the validation method we use in the webhook. It is not working correctly for block to filesystem clones. The fix removes the validation and improves the validation that is done later in the controller logic.

Comment 7 Bartosz Rybacki 2022-06-27 11:31:00 UTC
Well, after looking at the actual yaml dumps I can see that the source is 109G while the target is 30G. This will not work. This is actually not related to the other bug that is being fixed by the PR I mentioned. 

Who creates the template? Is it a manual template or something created by UI. If by UI then there is another question: how it was created? Was the source PVC resized in the meantime?


There is a new way of specifying DVTemplate. A template without size. I think it will be available in 4.12

Comment 8 Bartosz Rybacki 2022-06-27 11:33:29 UTC
@gouyang can you provide some more info on my questions?

Who creates the template? Is it a manual template or something created by UI. 

If it is created by UI then there is another question: how it was created? Was the source PVC resized in the meantime, between template creation and creating a VM?

Comment 9 Guohua Ouyang 2022-06-27 12:13:11 UTC
(In reply to Bartosz Rybacki from comment #8)
> @gouyang can you provide some more info on my questions?
> 
> Who creates the template? Is it a manual template or something created by
> UI. 

The templates are creating by common-templates component: https://github.com/kubevirt/common-templates

> If it is created by UI then there is another question: how it was created?
> Was the source PVC resized in the meantime, between template creation and
> creating a VM?

Comment 10 Bartosz Rybacki 2022-06-27 13:18:10 UTC
From what I can see (by looking at the few templates here: https://github.com/kubevirt/common-templates/tree/master/templates) the target size is hardcoded in the template. In that case it will only work if the source is smaller.

Comment 11 Adam Litke 2022-06-27 19:08:09 UTC
Bartosz,  In this case the templates are on HPP and have been expanded due to fs overhead but the VM wants to use block.  Shouldn't we accept a smaller size because fs overhead does not apply to block?  It seems that the rhel9 template is 30G in size so it really should work.

Comment 12 Bartosz Rybacki 2022-06-28 07:25:00 UTC
You are right Adam, in that case it is exactly the fix I did in kubevirt/containerized-data-importer/pull/2299.

I am backporting it right now.

Comment 13 Adam Litke 2022-06-28 15:00:26 UTC

*** This bug has been marked as a duplicate of bug 2084122 ***


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