Bug 1328887 - Image Uploader: when not filling in mandatory fields: Size(GB), Alias - Uncaught exception Cannot read property 'H' of undefined
Summary: Image Uploader: when not filling in mandatory fields: Size(GB), Alias - Uncau...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: BLL.Storage
Version: 4.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ovirt-4.0.1
: 4.0.0
Assignee: Amit Aviram
QA Contact: Natalie Gavrielov
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-04-20 13:29 UTC by Natalie Gavrielov
Modified: 2016-08-04 13:31 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-08-04 13:31:46 UTC
oVirt Team: Storage
Embargoed:
rule-engine: ovirt-4.0.z+
rule-engine: planning_ack+
rule-engine: devel_ack+
acanan: testing_ack+


Attachments (Terms of Use)
logs: engine, proxy, vdsm, chrome (809.07 KB, application/x-gzip)
2016-04-20 13:29 UTC, Natalie Gavrielov
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 59439 0 master MERGED webadmin: fix UploadDiskModel validation and flush. 2016-06-19 12:32:05 UTC
oVirt gerrit 59455 0 ovirt-engine-4.0 MERGED webadmin: hide Interface selectbox when resuming upload 2016-06-22 06:44:57 UTC
oVirt gerrit 59456 0 ovirt-engine-4.0 MERGED webadmin: fix UploadDiskModel validation and flush. 2016-06-22 06:45:07 UTC

Description Natalie Gavrielov 2016-04-20 13:29:03 UTC
Created attachment 1149099 [details]
logs: engine, proxy, vdsm, chrome

Description of problem:

If a user misses filling out the mandatory fields under Disk Options section: Alias and Size(GB), then clicks "OK" it UI shows:

Uncaught exception occurred. Please try reloading the page. Details: Exception caught: (TypeError) __gwt$exception: <skipped>: Cannot read property 'H' of undefined
Please have your administrator check the UI logs

Version-Release number of selected component:

ovirt-engine-4.0.0-0.0.master.20160310051350.git3151c66.el7.centos.noarch
ovirt-image-common-0.1-1.noarch
vdsm-4.17.999-724.gitb8cb30a.el7.centos.noarch
ovirt-image-daemon-0.1-1.noarch

How reproducible:
100%

Steps to Reproduce:

1. Go to Disks tab.
2. Click on Upload.
3. Choose a file, do not fill in fields: Alias and Size(GB).
4. Click "OK".

Actual results:
UI shows:

Uncaught exception occurred. Please try reloading the page. Details: Exception caught: (TypeError) __gwt$exception: <skipped>: Cannot read property 'H' of undefined
Please have your administrator check the UI logs

From Chrome log:
Wed Apr 20 16:14:05 GMT+300 2016 
SEVERE: Uncaught exception: com.google.gwt.event.shared.UmbrellaException: Exception caught: (TypeError) 
 __gwt$exception: <skipped>: Cannot read property 'H' of undefined
	at Unknown.Ss(webadmin-0.js@24614)
	at Unknown.$s(webadmin-0.js@41)
	at Unknown.F4(webadmin-0.js@19)
	at Unknown.I4(webadmin-0.js@19)
	at Unknown.S3(webadmin-0.js@117)
	at Unknown.Bn(webadmin-0.js@30)
	at Unknown.Ln(webadmin-0.js@23330)
	at Unknown.G_(webadmin-0.js@149)
	at Unknown.Dn(webadmin-0.js@120)
	at Unknown.y3e(webadmin-0.js@699)
	at Unknown.fUe(webadmin-0.js@85)
	at Unknown.hWe(webadmin-0.js@73)
	at Unknown.ev(webadmin-0.js@29)
	at Unknown.iv(webadmin-0.js@57)
	at Unknown.eval(webadmin-0.js@54)
	at Unknown.aA(webadmin-0.js@20)
	at Unknown.c3e(webadmin-0.js@109)
	at Unknown.Qll(webadmin-0.js@2542)
	at Unknown.y3e(webadmin-0.js@317)
	at Unknown.fUe(webadmin-0.js@85)
	at Unknown.eUe(webadmin-0.js@60)
	at Unknown.fWe(webadmin-0.js@58)
	at Unknown.ev(webadmin-0.js@29)
	at Unknown.iv(webadmin-0.js@57)
	at Unknown.eval(webadmin-0.js@54)


Expected results:
1. No exceptions. 
2. Explanation for the users that these are mandatory fields. 

Additional info:

Comment 1 Tal Nisan 2016-04-21 06:47:54 UTC
There's an exception in the UI, Amit full info can be obtained by running webadmin in dev mode

Comment 2 Sandro Bonazzola 2016-05-02 10:04:36 UTC
Moving from 4.0 alpha to 4.0 beta since 4.0 alpha has been already released and bug is not ON_QA.

Comment 3 Yaniv Lavi 2016-05-23 13:19:11 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 4 Yaniv Lavi 2016-05-23 13:23:02 UTC
oVirt 4.0 beta has been released, moving to RC milestone.

Comment 5 Yaniv Lavi 2016-06-19 14:10:37 UTC
This require a backport.

Comment 6 Natalie Gavrielov 2016-07-20 12:48:43 UTC
Verified using:
rhevm-4.0.2-0.2.rc1.el7ev.noarch
ovirt-imageio-common-0.3.0-0.el7ev.noarch
ovirt-imageio-proxy-0.3.0-0.el7ev.noarch
vdsm-4.18.6-1.el7ev.x86_64
ovirt-imageio-daemon-0.3.0-0.el7ev.noarch


Note You need to log in before you can comment on or make changes to this bug.