Bug 1069706

Summary:	Explore improvement of handling of old snapshot code
Product:	[Fedora] Fedora	Reporter:	Zdenek Kabelac <zkabelac>
Component:	lvm2	Assignee:	Zdenek Kabelac <zkabelac>
Status:	POST ---	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	low	Docs Contact:
Priority:	low
Version:	rawhide	CC:	agk, bmarzins, bmr, dwysocha, heinzm, jonathan, lvm-team, msnitzer, prajnoha, zkabelac
Target Milestone:	---	Keywords:	FutureFeature
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		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 Zdenek Kabelac 2014-02-25 14:37:50 UTC

Description of problem:

While coding other pieces of lvm2 code it appears to me, we should improve handling of old-snapshot code inside lvm2 (mainly merge)

Code for snapshot seems to be spread into wrong layers and adds a lot of extra unneeded internal complexity.

Also we should try to eliminate multiphase execution of dm-tree - looks like preload of all children first before resuming them might resolve this problem.

This could be seen as low-prio task.



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

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Zdenek Kabelac 2023-02-12 16:56:14 UTC

As of the nature of old dm snapshot target - we can't reduce complexity - with recent patch we even needed to go the opposite way to ensure snapshots are scanned after origin LV.

Snapshot are required to be activated first - followed by the origin.

https://listman.redhat.com/archives/lvm-devel/2023-February/024556.html