Bug 1129398

Summary: Missing parameter validation when adding new storage domain with advanced options
Product: [Retired] oVirt Reporter: Gal Amado <gamado>
Component: ovirt-engine-webadminAssignee: Tal Nisan <tnisan>
Status: CLOSED DUPLICATE QA Contact: Aharon Canan <acanan>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.5CC: amureini, bugs, ecohen, gamado, gklein, iheim, mgoldboi, rbalakri, yeylon
Target Milestone: ---Keywords: Triaged
Target Release: 3.5.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: storage
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-09-08 03:51:38 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:
Bug Depends On:    
Bug Blocks: 1193195    
Attachments:
Description Flags
engine log file
none
vdsm log file none

Description Gal Amado 2014-08-12 16:42:13 UTC
Created attachment 926159 [details]
engine log file

Description of problem:
When adding a new storage domain , for example of Data/NFS type , the advanced params are not validated .

Version-Release number of selected component (if applicable):
oVirt Engine Version: 3.5.0-0.0.master.20140804172041.git23b558e.el6

How reproducible:
Happans all the time 

Steps to Reproduce:
1.I've used up and running setup with 2 hosts and 2 iSCSI SD's , and 1 VM 
2.through GUI, Storage tab I've added "New Domain"  
3.Default Domain Function / Storage Type - Data/NFS
4.Fill in valid name and Export Path
5.Expand Advanced parameter
6.Check "Override Default Options"
7.Add an illegal value for example in the "Retransmissions (#)" field put "SOME_STRING"

Actual results:
8.The operation was passed though the illegal value.
9.running mount command on the host showes that the mount was created with retrans=6    

Expected results:
8.The GUI should validate illegal value and fail the operation.
9.The mount shouldn't be created.    


Additional info:

I've used netapp.qa.lab.tlv.redhat.com:/vol/vol_gamado_nfs01 for tha

Comment 1 Gal Amado 2014-08-12 16:46:11 UTC
Created attachment 926160 [details]
vdsm log file

Comment 2 Gal Amado 2014-08-12 16:48:00 UTC
In REST API , the transaction failed for "Illegal request format".

Comment 3 Allon Mureinik 2014-08-13 11:32:29 UTC
Isn't this a duplicate of bug 1116616 ?
If not - could you please highlight the differences?

Comment 4 Gal Amado 2014-09-08 03:51:38 UTC
Correct ,
This is a duplication of bug 1116616 .
Can I just close it ?

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