Bug 1109974
| Summary: | need a more graceful error when attempting to cache an origin that is already cached | |||
|---|---|---|---|---|
| Product: | Red Hat Enterprise Linux 7 | Reporter: | Corey Marthaler <cmarthal> | |
| Component: | lvm2 | Assignee: | Zdenek Kabelac <zkabelac> | |
| lvm2 sub component: | Cache Logical Volumes | QA Contact: | Cluster QE <mspqa-list> | |
| Status: | CLOSED ERRATA | Docs Contact: | ||
| Severity: | medium | |||
| Priority: | unspecified | CC: | agk, heinzm, jbrassow, msnitzer, prajnoha, zkabelac | |
| Version: | 7.0 | |||
| Target Milestone: | rc | |||
| Target Release: | --- | |||
| Hardware: | x86_64 | |||
| OS: | Linux | |||
| Whiteboard: | ||||
| Fixed In Version: | lvm2-2.02.112-1.el7 | Doc Type: | Bug Fix | |
| Doc Text: |
Lvm2 supports caching only certain LV types. Missed check allowed to pass unsupported volume type into internal code which then thrown internal error message. Check for supported types is now correctly checked at command level.
|
Story Points: | --- | |
| Clone Of: | ||||
| : | 1114078 (view as bug list) | Environment: | ||
| Last Closed: | 2015-03-05 13:09:01 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: | ||||
| Bug Depends On: | ||||
| Bug Blocks: | 1114078, 1119326 | |||
|
Description
Corey Marthaler
2014-06-16 19:09:43 UTC
this should actually be allowed to stack cache on cache... see bug 727072. It would be two layers of cache in that case. Although, we may not have to do this in 7.1. https://www.redhat.com/archives/lvm-devel/2014-November/msg00040.html As of now - disabled explicitly by lvm2 code. We do not support snapshotting of cached volumes until we resolve what this actually means - libdm currently requires snapshot to be a top-level LV so it's not going to work well in some orders. Verified the "Internal error" no longer shows up. However I'd argue the new warning message is actually more confusing than what the old internal error warning said, "The origin, $origin, cannot be of cache type" [root@host-115 ~]# lvs -a -o +devices LV Attr LSize Pool Origin Data% Meta% Cpy%Sync Devices already_cached Cwi-a-C--- 4.00g [pool_1] [already_cached_corig] 0.02 3.47 0.00 already_cached_corig(0) [already_cached_corig] owi-aoC--- 4.00g /dev/sdb1(0) [lvol0_pmspare] ewi------- 8.00m /dev/sda1(0) [pool_1] Cwi---C--- 2.00g 0.02 3.47 0.00 pool_1_cdata(0) [pool_1_cdata] Cwi-ao---- 2.00g /dev/sdd1(0) [pool_1_cmeta] ewi-ao---- 8.00m /dev/sdd1(512) pool_2 Cwi---C--- 2.00g pool_2_cdata(0) [pool_2_cdata] Cwi------- 2.00g /dev/sdd1(514) [pool_2_cmeta] ewi------- 8.00m /dev/sdd1(1026) [root@host-115 ~]# lvconvert --yes --type cache --cachepool cache_sanity/pool_2 cache_sanity/already_cached Cache is not supported with cache segment type of the original logical volume cache_sanity/already_cached. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://rhn.redhat.com/errata/RHBA-2015-0513.html |