Bug 826463

Summary: Gluster - Backend: Exception on entering bad text in Volume Options
Product: [Retired] oVirt Reporter: Daniel Paikov <dpaikov>
Component: ovirt-engine-coreAssignee: Bala.FA <barumuga>
Status: CLOSED DUPLICATE QA Contact: Haim <hateya>
Severity: medium Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: acathrow, amureini, dpati, dyasny, hateya, iheim, mgoldboi, shireesh, yeylon, ykaul
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: gluster
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-06-11 10:11:59 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:
Attachments:
Description Flags
engine.log
none
vdsm.log none

Description Daniel Paikov 2012-05-30 09:14:59 UTC
Created attachment 587654 [details]
engine.log

Entering bad text in the Volume Options produces an exception. In my example I entered "xxx" in "nfs.disable", which only accepts values of "on" and "off".

Comment 1 Bala.FA 2012-05-31 01:59:21 UTC

Do you mean UI allows to enter other than  on/off?
Do you see exception similar to
https://bugzilla.redhat.com/show_bug.cgi?id=823565?
Can you attach respective vdsm log?

Comment 2 Daniel Paikov 2012-05-31 09:28:20 UTC
Yes, the UI allows entering any text in the textbox.

Attaching vdsm.log (from a new reproduction of the same flow).

Comment 3 Daniel Paikov 2012-05-31 09:28:49 UTC
Created attachment 588012 [details]
vdsm.log

Comment 4 Shireesh 2012-06-11 10:11:59 UTC
UI does not enforce validations on the values of options. If you enter invalid values for options, glusterfs will throw an error (seen in the vdsm.log attached by you). This error should be displayed on the UI as well.

There was a problem in vdsm (823565) when you raised this bug, because of which it used to crash and proper error didn't reach the UI. This is fixed now, and you should see the error returned by glusterfs on the UI as well.

Closing the bug, marking it as duplicate of 823565.

*** This bug has been marked as a duplicate of bug 823565 ***