Bug 683985

Summary: 2.2.7 getStoragePoolInfo fails due to missing POOL_DESCRIPTION in metadata
Product: Red Hat Enterprise Linux 5 Reporter: Chris Pelland <cpelland>
Component: vdsm22Assignee: Dan Kenigsberg <dkenigsb>
Status: CLOSED ERRATA QA Contact: Omri Hochman <ohochman>
Severity: urgent Docs Contact:
Priority: urgent    
Version: 5.6CC: abaron, bazulay, chetan, cpelland, danken, dpaikov, ewarszaw, iheim, oschreib, pm-eus, smizrahi, ykaul
Target Milestone: rcKeywords: ZStream
Target Release: ---   
Hardware: Unspecified   
OS: Linux   
Whiteboard:
Fixed In Version: vdsm22-4.5-63.24.el5_6 Doc Type: Bug Fix
Doc Text:
C: Editing the description of the master storage domain erased the POOL_DESCRIPTION field in the metadata C: The storage pool became unusable F: With this update, when the domain description is edited, running the “getStoragePoolInfo” command displays these changes in the POOL_DESCRIPTION field R: The storage pool remains operational Text: Editing the description of the master storage domain erased the POOL_DESCRIPTION field in the metadata, thus the storage pool became unusable. With this update, changes to POOL_DESCRIPTION display when the “getStoragePoolInfo” command is run, and the storage pool remains operational.
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-04-11 16:06:56 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On: 676582    
Bug Blocks:    

Description Chris Pelland 2011-03-10 20:44:07 UTC
This bug has been copied from bug #676582 and has been proposed
to be backported to 5.6 z-stream (EUS).

Comment 4 Daniel Paikov 2011-03-15 11:44:05 UTC
Checked on vdsm22-4.5-63.22.

Comment 5 Daniel Paikov 2011-03-15 11:45:02 UTC
Woops. Still reproduces on 4.5-63.22.

Comment 6 Dan Kenigsberg 2011-03-15 13:08:13 UTC
Actually, what you saw is our mishandling of missing POOL_DESCRIPTION (bug 684725). This bug is about our erasing of this field (the title is confusing...)

To avoid confusion, I'll ask to reverify the issue only on a build with both bugs resolved.

Comment 7 Omri Hochman 2011-03-21 16:09:52 UTC
Verification Description (Verified vdsm22-4.5-63.24.el5_6,Using RHEVM 2.2-GA):
*******************************************************************************

1) Create on host (with old version of vdsm_2.2.5 NFS data storage domain and attach it to NFS Data-Center. 

2) After host 'contending' and  become SPM--> on storage machine 
edit the NFS data storage-domain meta-data:
'vi /<storge_domain_uuid>/dom_md/metadata' 
         A) remove the 'POOL_DESCRIPTION' line.
         B) remove the '_SHA_CKSUM=c' line. 
3) maintenance the host.
4) upgrade vdsm22 rpm's on the host (to version 2.2.7).
5)activate the host and verified that host 'contending' and connectStoragePool Succeeded.

Note: 
*****
 verified the above test for switch between builds:
- 2.2.5 to 2.2.6 to 2.2.6.1 to 2.2.7.
- 2.2.5 directly to 2.2.7. 
- 2.2.6 directly to 2.2.7.

Comment 8 Dan Kenigsberg 2011-03-23 22:12:24 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
Editing the description of the master storage domain, used to render the storage pool useless. This no longer happens.

Comment 9 Cheryn Tan 2011-04-07 05:47:05 UTC
    Technical note updated. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    Diffed Contents:
@@ -1 +1,7 @@
-Editing the description of the master storage domain, used to render the storage pool useless. This no longer happens.+C: Editing the description of the master storage domain erased the POOL_DESCRIPTION field in the metadata
+C: The storage pool became unusable
+F: With this update, when the domain description is edited, running the “getStoragePoolInfo” command displays these changes in the POOL_DESCRIPTION field
+R: The storage pool remains operational
+
+Text:
+Editing the description of the master storage domain erased the POOL_DESCRIPTION field in the metadata, thus the storage pool became unusable. With this update, changes to POOL_DESCRIPTION display when the “getStoragePoolInfo” command is run, and the storage pool remains operational.

Comment 10 errata-xmlrpc 2011-04-11 16:06:56 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2011-0424.html