Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1168686

Summary: [engine-backend] updating a storage connection of a Gluster domain does not make any effect.
Product: Red Hat Enterprise Virtualization Manager Reporter: Elad <ebenahar>
Component: ovirt-engineAssignee: Amit Aviram <aaviram>
Status: CLOSED CURRENTRELEASE QA Contact: Elad <ebenahar>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 3.5.0CC: acanan, amureini, gklein, lpeer, lsurette, rbalakri, Rhev-m-bugs, tnisan, yeylon, ykaul
Target Milestone: ovirt-3.6.0-rc   
Target Release: 3.6.0   
Hardware: x86_64   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 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
engine and vdsm logs none

Description Elad 2014-11-27 15:28:12 UTC
Created attachment 962118 [details]
engine and vdsm logs

Description of problem:
Updating the connection details of a Gluster domain is not blocked to user. It is possible to edit the connection (tried using webadmin) and this should not be allowed since manage storage connections is not supported for Gluster.

Version-Release number of selected component (if applicable):
rhev 3.5 vt12
rhevm-3.5.0-0.22.el6ev.noarch

How reproducible:
Always

Steps to Reproduce:
1. Create a Gluster storage domain 
2. Edit the domain and change the connection details. For example, I chaged from:  10.35.160.6:/elad1 to  10.35.160.20:/elad1
3.

Actual results:
The operation is reported as success:

2014-11-27 17:23:47,641 INFO  [org.ovirt.engine.core.bll.storage.UpdateStorageDomainCommand] (ajp-/127.0.0.1:8702-5) [59b5a6f] Running command: UpdateStorageDomainCommand internal: false. Entities affected :  ID: 52dd06f6-944e-49dd-be38-8b9474d5728e Type: StorageAction group EDIT_STORAGE_DOMAIN_CONFIGURATION with role type ADMIN

But this is a false positive report since it hadn't done anything in the database. Checking the connection indicates that remained the same:

                  id                  |                              connection                              
--------------------------------------+----------------------------------------------------------------------
 1629fd83-a88e-4bc4-ae7c-655b330a1115 | elad-ovirt.qa.lab.tlv.redhat.com:/var/lib/exports/iso-20141126121445
 473880c8-13f3-4b9e-b6eb-fbe5ea45bb2b | 10.35.64.81:/export/elad/1
 70a2fba5-bc43-44a0-b911-cf78af4b4ba9 | 10.35.64.81:/export/elad/2
 54dc6a33-1f6f-4cd6-a140-bc5a66a67b92 | 10.35.146.193
 32efa148-338a-4b9d-8d70-e7f701b3302c | 10.35.146.161
 504be28c-d3f7-413a-9b1b-9e8111fd5ba4 | 10.35.146.129
 a1fab637-c3ed-45e7-8a0b-f4cf73e96ce8 | 10.35.146.225
 46ca8eeb-30cf-46de-b842-2bdcca8e1cfb | 10.35.160.6:/elad1
 5421d558-9a56-4488-86e7-d07a897c234b | 10.35.160.6:/elad
 b7d4ec51-b13a-4dd6-ab4f-9487ec9ae474 | 10.35.64.81:/export/rhevm-3-iso
 61d8261c-4b0a-41c3-9286-d67a6c7709db | lion.qa.lab.tlv.redhat.com:/export/elad/3



Expected results:
Updating a storage connection of Gluster should not be allowed 

Additional info:
engine and vdsm logs

Comment 1 Allon Mureinik 2014-11-28 19:38:34 UTC
The current state, where you can edit via the GUI but nothing happens in the DB is obviously wrong - but the proper result is not to block the GUI, but to be able to edit it properly.

Comment 2 Elad 2015-04-26 12:11:18 UTC
Update to storage connection of a Gluster domain is available and the change takes effect. The storage domain activation succeeds.

Verified using 3.6.0-1 
ovirt-engine-3.6.0-0.0.master.20150412172306.git55ba764.el6.noarch

Comment 4 Allon Mureinik 2016-03-10 10:43:57 UTC
RHEV 3.6.0 has been released, setting status to CLOSED CURRENTRELEASE

Comment 5 Allon Mureinik 2016-03-10 10:47:58 UTC
RHEV 3.6.0 has been released, setting status to CLOSED CURRENTRELEASE

Comment 6 Allon Mureinik 2016-03-10 12:04:29 UTC
RHEV 3.6.0 has been released, setting status to CLOSED CURRENTRELEASE