Bug 1366233

Summary: C&U - WEB UI crashes when moving from calendar to daily/hourly selection
Product: Red Hat CloudForms Management Engine Reporter: Einat Pacifici <epacific>
Component: UI - OPSAssignee: Yaacov Zamir <yzamir>
Status: CLOSED CURRENTRELEASE QA Contact: Einat Pacifici <epacific>
Severity: high Docs Contact:
Priority: high    
Version: 5.6.0CC: dron, fsimonce, hkataria, jhardy, mpovolny, nachandr, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.7.0Flags: epacific: automate_bug-
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: container:c&u
Fixed In Version: 5.7.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1370570 (view as bug list) Environment:
Last Closed: 2017-01-11 20:26:42 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:
Bug Depends On:    
Bug Blocks: 1370570    
Attachments:
Description Flags
evm.log attached.
none
screenshots none

Description Einat Pacifici 2016-08-11 10:41:26 UTC
Created attachment 1190009 [details]
evm.log attached.

Description of problem:
In C&U, when trying to select a date for metrics display, and moving away from calendar, web ui crashes: 
"undefined method `<' for nil:NilClass [ems_container/perf_chart_chooser] "

Version-Release number of selected component (if applicable):
5.6.1

How reproducible:
Always

Steps to Reproduce:
1.Setup CFME with Provider that has metrics in openshift
2.Go to Monitoring --> Utilization
3.Click on  Calendar (to select a day for metrics display)
4. Do not select any day
5. Click on the page - to close the calendar selection


Actual results:
Error is shown:  "undefined method `<' for nil:NilClass [container_replicator/perf_chart_chooser] "

Expected results:
User should be able to return to selection without UI crash

Additional info:
screenshots and evm.log attached

Comment 2 Einat Pacifici 2016-08-11 10:42:28 UTC
Created attachment 1190010 [details]
screenshots

Comment 3 Dave Johnson 2016-08-11 20:15:39 UTC
Nandini, are you seeing this on any other provider from cloud/infra or is it only related to containers?

Comment 4 Nandini Chandra 2016-08-11 21:21:25 UTC
Tested both infra(vsphere) and cloud(ec2) and I don't see the issue.

Comment 5 Yaacov Zamir 2016-08-14 11:48:37 UTC
Submitted upstream:
https://github.com/ManageIQ/manageiq/pull/10458

Comment 6 CFME Bot 2016-08-15 11:15:44 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/21ee8f35afce285cc6b7880b91dd4614ae9dacdf

commit 21ee8f35afce285cc6b7880b91dd4614ae9dacdf
Author:     Yaacov Zamir <yzamir>
AuthorDate: Sun Aug 14 14:38:07 2016 +0300
Commit:     Yaacov Zamir <yzamir>
CommitDate: Sun Aug 14 14:38:07 2016 +0300

    C&U - WEB UI crashes when moving from calendar to daily/hourly selection
    
    It is posible to send an empty string as a date from the UI, we currently only protect against nil date
    but not against an empty string. We add a check for empty string date.
    
    Bugzilla:
    https://bugzilla.redhat.com/show_bug.cgi?id=1366233

 app/controllers/application_controller/performance.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comment 7 Yaacov Zamir 2016-08-15 11:20:36 UTC
Merged upstream:
https://github.com/ManageIQ/manageiq/pull/10458