Bug 851835

Summary: [rhvem][admin-portal] Unable to rename storage domain name in case of migrate master flow (or reconstruct)
Product: Red Hat Enterprise Virtualization Manager Reporter: vvyazmin <vvyazmin>
Component: ovirt-engineAssignee: Maor <mlipchuk>
Status: CLOSED CURRENTRELEASE QA Contact: vvyazmin <vvyazmin>
Severity: urgent Docs Contact:
Priority: unspecified    
Version: 3.1.0CC: amureini, dyasny, hateya, iheim, lpeer, Rhev-m-bugs, sgrinber, yeylon, ykaul
Target Milestone: ---Keywords: Regression
Target Release: 3.1.0   
Hardware: x86_64   
OS: All   
Whiteboard: storage
Fixed In Version: SI17 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-04 20:03:26 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
## Logs vdsm, rhevm, screen-shots none

Description vvyazmin@redhat.com 2012-08-26 09:10:12 UTC
Created attachment 607050 [details]
## Logs vdsm, rhevm, screen-shots

Description of problem:
Unable to rename storage domain name

Version-Release number of selected component (if applicable):
Verified on RHEVM 3.1 - SI15

RHEVM: rhevm-3.1.0-13.el6ev.noarch
VDSM: vdsm-4.9.6-29.0.el6_3.x86_64
LIBVIRT: libvirt-0.9.10-21.el6.x86_64
QEMU & KVM: qemu-kvm-rhev-0.12.1.2-2.298.el6_3.x86_64
SANLOCK: sanlock-2.3-3.el6_3.x86_64


How reproducible:
100 %

Steps to Reproduce:
1. Edit storage domain
2. Rename storage domain name
  
Actual results:
Failed rename storage domain
Get an error: “Cannot edit Storage. Only Storage Domain name is updateable.

Expected results:
Succeed rename storage domain

Additional info:
Reproducible with iSCSI and NFS storage domain.

Rhevm  log:
2012-08-26 11:33:13,714 WARN  [org.ovirt.engine.core.bll.storage.UpdateStorageDomainCommand] (ajp-/127.0.0.1:8009-10) [368d3694] CanDoAction of action UpdateStorageDomain failed. Reasons:VAR__ACTION__UPDATE,VAR__TYPE__STORAGE__DOMAIN,ERROR_CANNOT_CHANGE_STORAGE_DOMAIN_FIELDS

Comment 3 Allon Mureinik 2012-08-29 13:49:26 UTC
The flow to reproduce the bug is incomplete - in some scenarios this edit can work, see below.


Steps to reproduce:
1. Create a DC with two active SDs.
2. set the master to maintenance - both SDs will go down
3. Activate the SD that was not the master - it will force a reconstructions of the master metadata on this domain.
4. Try to edit the name of the SD from (3) - the error will appear.

In UpdateStorageDomainCommand, line 39, there is a check that only the name is changed.
This fails since the UI creates a new storage_domains object without the lastTimeUsedAsMaster property (defaults to 0), and the database version has it.

Comment 4 Maor 2012-08-30 08:57:52 UTC
Merged upstream at commit #2abf376438ef81aeb4362a41ad4a512f7c29b32e

Comment 5 Allon Mureinik 2012-08-30 11:34:18 UTC
Change-Id for that commit, for easier tracking:
I77de69fb8bd28a3c2c1ec1a9be576cb7b4e9dd69

Comment 6 Allon Mureinik 2012-08-30 12:44:07 UTC
change merged.

Comment 7 Haim 2012-09-05 12:40:40 UTC
verified on SI17. managed to rename storage domain and extend lun.