Bug 1220901

Summary:	Clicking on the 'Default filters' tab on Configure->My Settings causes comparison of Array with Array failed [configuration/change_tab] error
Product:	Red Hat CloudForms Management Engine	Reporter:	Nandini Chandra <nachandr>
Component:	UI - OPS	Assignee:	Harpreet Kataria <hkataria>
Status:	CLOSED ERRATA	QA Contact:	Nandini Chandra <nachandr>
Severity:	high	Docs Contact:
Priority:	high
Version:	5.4.0	CC:	cpelland, dajohnso, hkataria, jhardy, jprause, mfeifer, mpovolny, obarenbo
Target Milestone:	GA
Target Release:	5.4.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-06-16 13:02:16 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:

Description Nandini Chandra 2015-05-12 18:58:02 UTC

Description of problem:
------------------------
This exception occurs while clicking to the 'Default filters' tab on
Configure->My Settings.

Error Text:

comparison of Array with Array failed [configuration/change_tab]

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


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


Steps to Reproduce:
------------------
1.Navigate to Configure->My Settings
2.Click 'Default filters' tab
3.


Actual results:
---------------
Error occurs


Expected results:
-----------------


Additional info:
---------------

Comment 2 Nandini Chandra 2015-05-12 21:18:25 UTC

Snippet from production.log:

[----] I, [2015-05-12T14:46:52.844691 #2381:945ea8]  INFO -- : Started GET "/configuration/change_tab/?tab=3" for 127.0.0.1 at 2015-05-12 14:46:52 -0400
[----] I, [2015-05-12T14:46:52.881800 #2381:945ea8]  INFO -- : Processing by ConfigurationController#change_tab as HTML
[----] I, [2015-05-12T14:46:52.881949 #2381:945ea8]  INFO -- :   Parameters: {"tab"=>"3"}
[----] F, [2015-05-12T14:46:53.082956 #2381:945ea8] FATAL -- : Error caught: [ArgumentError] comparison of Array with Array failed
/var/www/miq/vmdb/app/controllers/configuration_controller.rb:773:in `sort_by'
/var/www/miq/vmdb/app/controllers/configuration_controller.rb:773:in `set_form_vars'
/var/www/miq/vmdb/app/controllers/configuration_controller.rb:83:in `edit'
/var/www/miq/vmdb/app/controllers/configuration_controller.rb:91:in `change_tab'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_controller/metal/implicit_render.rb:4:in `send_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/abstract_controller/base.rb:167:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_controller/metal/rendering.rb:10:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/abstract_controller/callbacks.rb:18:in `block in process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/activesupport/lib/active_support/callbacks.rb:568:in `_run__1550232514896681517__process_action__982762565923164105__callbacks'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/activesupport/lib/active_support/callbacks.rb:405:in `__run_callback'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/activesupport/lib/active_support/callbacks.rb:385:in `_run_process_action_callbacks'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/activesupport/lib/active_support/callbacks.rb:81:in `run_callbacks'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/abstract_controller/callbacks.rb:17:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_controller/metal/rescue.rb:29:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_controller/metal/instrumentation.rb:30:in `block in process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/activesupport/lib/active_support/notifications.rb:123:in `block in instrument'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/activesupport/lib/active_support/notifications/instrumenter.rb:20:in `instrument'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/activesupport/lib/active_support/notifications.rb:123:in `instrument'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_controller/metal/instrumentation.rb:29:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_controller/metal/params_wrapper.rb:207:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/activerecord/lib/active_record/railties/controller_runtime.rb:18:in `process_action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/abstract_controller/base.rb:121:in `process'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/abstract_controller/rendering.rb:45:in `process'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_controller/metal.rb:203:in `dispatch'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_controller/metal/rack_delegation.rb:14:in `dispatch'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_controller/metal.rb:246:in `block in action'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_dispatch/routing/route_set.rb:73:in `call'
/opt/rh/cfme-gemset/bundler/gems/rails-4842a8377644/actionpack/lib/action_dispatch/routing/route_set.rb:73:in `dispatch'

Comment 3 CFME Bot 2015-05-13 18:30:49 UTC

New commit detected on manageiq/master:
https://github.com/ManageIQ/manageiq/commit/e8d4581c5139b0d5587bd70e483d4dba51dbb313

commit e8d4581c5139b0d5587bd70e483d4dba51dbb313
Author:     Harpreet Kataria <hkataria>
AuthorDate: Tue May 12 18:05:30 2015 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Tue May 12 18:05:30 2015 -0400

    Added containerservice, containergroup keys to NAV_TAB_PATH
    
    - After new search default filters were added for Containers Service and Groups, needed to add those 2 keys to NAV_TAB_PATH constant that's used to build the default filters tree.
    - Added spec test to verify the issue.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1220901

 vmdb/app/controllers/configuration_controller.rb     | 20 +++++++++++---------
 .../controllers/configuration_controller_spec.rb     | 14 ++++++++++++++
 2 files changed, 25 insertions(+), 9 deletions(-)

Comment 4 John Prause 2015-05-14 18:37:57 UTC

This will need to be cherry-pick to 5.4.z since th efix was made after the 5.4.z branch was created yesterday.

Comment 5 CFME Bot 2015-05-14 21:35:38 UTC

New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=c0f6fd78fcad4b487f4509ff9faed6033060a4b9

commit c0f6fd78fcad4b487f4509ff9faed6033060a4b9
Author:     Harpreet Kataria <hkataria>
AuthorDate: Tue May 12 18:05:30 2015 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Thu May 14 15:30:13 2015 -0400

    Added containerservice, containergroup keys to NAV_TAB_PATH
    
    - After new search default filters were added for Containers Service and Groups, needed to add those 2 keys to NAV_TAB_PATH constant that's used to build the default filters tree.
    - Added spec test to verify the issue.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1220901
    (cherry picked from commit e8d4581)

 vmdb/app/controllers/configuration_controller.rb     | 20 +++++++++++---------
 .../controllers/configuration_controller_spec.rb     | 14 ++++++++++++++
 2 files changed, 25 insertions(+), 9 deletions(-)

Comment 6 CFME Bot 2015-05-14 21:36:02 UTC

New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=0575b6055439bcd67d98b48fa852c16538afd8b4

commit 0575b6055439bcd67d98b48fa852c16538afd8b4
Merge: 6da4559 c0f6fd7
Author:     Dan Clarizio <dclarizi>
AuthorDate: Thu May 14 17:34:57 2015 -0400
Commit:     Dan Clarizio <dclarizi>
CommitDate: Thu May 14 17:34:57 2015 -0400

    Merge branch 'bz_1220901' into '5.4.z'
    
    Added containerservice, containergroup keys to NAV_TAB_PATH
    
    - After new search default filters were added for Containers Service and Groups, needed to add those 2 keys to NAV_TAB_PATH constant that's used to build the default filters tree.
    - Added spec test to verify the issue.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1220901
    (cherry picked from commit e8d4581)
    
    @dclarizio please review. This a straight cherry-pick
    
    See merge request !58

 vmdb/app/controllers/configuration_controller.rb     | 20 +++++++++++---------
 .../controllers/configuration_controller_spec.rb     | 14 ++++++++++++++
 2 files changed, 25 insertions(+), 9 deletions(-)

Comment 8 Nandini Chandra 2015-05-27 01:07:27 UTC

Verified in 5.4.0.2

Comment 10 errata-xmlrpc 2015-06-16 13:02:16 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-1100.html