Bug 1455010

Summary: Trim whitespaces in the user provided text in the cockpit UI
Product: [oVirt] cockpit-ovirt Reporter: SATHEESARAN <sasundar>
Component: GdeployAssignee: Gobinda Das <godas>
Status: CLOSED CURRENTRELEASE QA Contact: SATHEESARAN <sasundar>
Severity: high Docs Contact:
Priority: unspecified    
Version: ---CC: bugs, knarra, rbarry, sabose
Target Milestone: ovirt-4.1.5Flags: sasundar: ovirt-4.1?
sasundar: planning_ack?
rbarry: devel_ack+
sasundar: testing_ack+
Target Release: 0.10.7-0.0.23   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: cockpit-ovirt-0.10.7-0.0.23 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1483521 (view as bug list) Environment:
Last Closed: 2017-08-23 08:05:05 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Gluster RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1411323, 1480607, 1485863    

Description SATHEESARAN 2017-05-24 02:42:55 UTC
Description of problem:
-----------------------
Whitespaces in the value of the text fields should be trimmed as these values are passed straight away in to gdeploy config files.

Version-Release number of selected component (if applicable):
-------------------------------------------------------------
RHVH 4.1
cockpit-ovirt-dashboard-0.10.7-0.0.18

How reproducible:
-----------------
Always

Steps to Reproduce:
-------------------
1. While specifying the volume name and brick's mount point, leave a space after  specifying the mount point value

Actual results:
---------------
The space is passed on the gdeploy config file. [volume] section has bricks with space

Expected results:
-----------------
All the text that obtains the input from user should filter out the whitespaces

Comment 1 SATHEESARAN 2017-08-21 10:06:37 UTC
Verified with cockpit-ovirt-dashboard-0.10.7-0.0.23.el7ev.noarch
User input values for 'Bricks' and 'volume' text fields are  whitespaces trimmed

However the hostnames, package names, repo names are not whitespace trimmed.
I will open a separate bug for the fix