Bug 2003652

Summary:	Cloning DataVolumes fails with a 4M hardware block size
Product:	Container Native Virtualization (CNV)	Reporter:	Adam Litke <alitke>
Component:	Storage	Assignee:	Alexander Wels <awels>
Status:	CLOSED WORKSFORME	QA Contact:	Natalie Gavrielov <ngavrilo>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	2.5.5	CC:	cnv-qe-bugs, kgoldbla
Target Milestone:	---
Target Release:	4.10.0
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:	2021-12-17 18:52:51 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 Adam Litke 2021-09-13 11:51:25 UTC

Description of problem:  When using storage that is configured with a 4M block size, cloning fails with the following error:


data-processor.go:328] exit status 1, blockdev: ioctl error on BLKGETSIZE64: Inappropriate ioctl for device


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


How reproducible: Unsure


Steps to Reproduce:
1. Create source DV on storage with 4M block size
2. Create DV to clone source DV on storage with 4M block size


Actual results: Cloning fails with above error message


Expected results: Cloning should succeed


Additional info: We should consider detecting larger block sizes and adapting our cloning block size automatically.

Comment 1 Alexander Wels 2021-12-07 20:52:33 UTC

This GH issues [0] seems related. It appears calling blockdev on a non /dev path returns the above mentioned error. And we were in fact mounting our block devices on a non /dev path. However [1] fixed this for 2.5.x. So I am going to put this on QA

[0] https://github.com/kubernetes-sigs/azuredisk-csi-driver/issues/714
[1] https://github.com/kubevirt/containerized-data-importer/pull/2031

Comment 2 Alexander Wels 2021-12-07 20:54:10 UTC

Actually that PR was not merged, so it not fixed.

Comment 4 Adam Litke 2021-12-17 18:52:51 UTC

Seems like there is nothing left to do with this bug.  The related CDI PR (https://github.com/kubevirt/containerized-data-importer/pull/2031) was closed.