Bug 1388431 - clicking on Unassigned Profiles Group from satellite provider
Summary: clicking on Unassigned Profiles Group from satellite provider
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.8.0
Assignee: Aparna Karve
QA Contact: Niyaz Akhtar Ansari
URL:
Whiteboard: ui:satellite:config_mgmt
Depends On:
Blocks: 1390731
TreeView+ depends on / blocked
 
Reported: 2016-10-25 10:42 UTC by Niyaz Akhtar Ansari
Modified: 2017-06-12 16:10 UTC (History)
7 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1390731 (view as bug list)
Environment:
Last Closed: 2017-06-12 16:10:50 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
screenshot (94.25 KB, image/png)
2016-10-25 10:42 UTC, Niyaz Akhtar Ansari
no flags Details

Description Niyaz Akhtar Ansari 2016-10-25 10:42:54 UTC
Created attachment 1213838 [details]
screenshot

Description of problem:


Version-Release number of selected component (if applicable):
Version 5.7.0.6-alpha3.20161019140041_ea8e259 

How reproducible:
100%

Steps to Reproduce:
1.Add satellite-6 Provider->Change the page view into Grid View 
2.Click on the Providers accordion,then click Red Hat Satellite Provider
3.then click on satellite-6 Configuration Manager
4.click on Unassigned Profiles Group icon and see production log 

Actual results:
when clicking on Unassigned Profiles Group from satellite provider page getting an error in production log.

Expected results:
no error 

Additional info:

Production Log:

[----] I, [2016-10-25T06:23:06.331565 #3248:e3d6e0]  INFO -- : Started POST "/provider_foreman/x_show" for 127.0.0.1 at 2016-10-25 06:23:06 -0400
[----] I, [2016-10-25T06:23:06.333383 #3248:e3d6e0]  INFO -- : Processing by ProviderForemanController#x_show as JS
[----] F, [2016-10-25T06:23:06.343334 #3248:e3d6e0] FATAL -- : Error caught: [NoMethodError] undefined method `split' for nil:NilClass
/var/www/miq/vmdb/app/controllers/application_controller/tree_support.rb:279:in `parse_nodetype_and_id'
/var/www/miq/vmdb/app/controllers/provider_foreman_controller.rb:1063:in `unassigned_configuration_profile?'
/var/www/miq/vmdb/app/controllers/provider_foreman_controller.rb:323:in `x_show'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_controller/metal/basic_implicit_render.rb:4:in `send_action'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/abstract_controller/base.rb:188:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_controller/metal/rendering.rb:30:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/abstract_controller/callbacks.rb:20:in `block in process_action'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/callbacks.rb:126:in `call'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/callbacks.rb:506:in `block (2 levels) in compile'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/callbacks.rb:455:in `call'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/callbacks.rb:101:in `__run_callbacks__'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/callbacks.rb:750:in `_run_process_action_callbacks'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/callbacks.rb:90:in `run_callbacks'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/abstract_controller/callbacks.rb:19:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_controller/metal/rescue.rb:20:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_controller/metal/instrumentation.rb:32:in `block in process_action'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/notifications.rb:164:in `block in instrument'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/notifications/instrumenter.rb:21:in `instrument'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/notifications.rb:164:in `instrument'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_controller/metal/instrumentation.rb:30:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_controller/metal/params_wrapper.rb:248:in `process_action'
/opt/rh/cfme-gemset/gems/activerecord-5.0.0.1/lib/active_record/railties/controller_runtime.rb:18:in `process_action'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/abstract_controller/base.rb:126:in `process'
/opt/rh/cfme-gemset/gems/actionview-5.0.0.1/lib/action_view/rendering.rb:30:in `process'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_controller/metal.rb:190:in `dispatch'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_controller/metal.rb:262:in `dispatch'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/routing/route_set.rb:50:in `dispatch'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/routing/route_set.rb:32:in `serve'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/journey/router.rb:39:in `block in serve'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/journey/router.rb:26:in `each'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/journey/router.rb:26:in `serve'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/routing/route_set.rb:725:in `call'
/opt/rh/cfme-gemset/gems/secure_headers-3.0.3/lib/secure_headers/middleware.rb:10:in `call'
/opt/rh/cfme-gemset/gems/rack-2.0.1/lib/rack/etag.rb:25:in `call'
/opt/rh/cfme-gemset/gems/rack-2.0.1/lib/rack/conditional_get.rb:38:in `call'
/opt/rh/cfme-gemset/gems/rack-2.0.1/lib/rack/head.rb:12:in `call'
/opt/rh/cfme-gemset/gems/rack-2.0.1/lib/rack/session/abstract/id.rb:222:in `context'
/opt/rh/cfme-gemset/gems/rack-2.0.1/lib/rack/session/abstract/id.rb:216:in `call'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/middleware/cookies.rb:613:in `call'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/middleware/callbacks.rb:38:in `block in call'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/callbacks.rb:97:in `__run_callbacks__'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/callbacks.rb:750:in `_run_call_callbacks'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/callbacks.rb:90:in `run_callbacks'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/middleware/callbacks.rb:36:in `call'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/middleware/remote_ip.rb:79:in `call'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/middleware/debug_exceptions.rb:49:in `call'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/middleware/show_exceptions.rb:31:in `call'
/opt/rh/cfme-gemset/gems/railties-5.0.0.1/lib/rails/rack/logger.rb:36:in `call_app'
/opt/rh/cfme-gemset/gems/railties-5.0.0.1/lib/rails/rack/logger.rb:26:in `call'
/opt/rh/cfme-gemset/gems/request_store-1.3.1/lib/request_store/middleware.rb:9:in `call'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/middleware/request_id.rb:24:in `call'
/opt/rh/cfme-gemset/gems/rack-2.0.1/lib/rack/method_override.rb:22:in `call'
/opt/rh/cfme-gemset/gems/rack-2.0.1/lib/rack/runtime.rb:22:in `call'
/opt/rh/cfme-gemset/gems/activesupport-5.0.0.1/lib/active_support/cache/strategy/local_cache_middleware.rb:28:in `call'
/opt/rh/cfme-gemset/gems/actionpack-5.0.0.1/lib/action_dispatch/middleware/executor.rb:12:in `call'
/opt/rh/cfme-gemset/gems/rack-2.0.1/lib/rack/sendfile.rb:111:in `call'
/opt/rh/cfme-gemset/gems/railties-5.0.0.1/lib/rails/engine.rb:522:in `call'
/opt/rh/rh-ruby23/root/usr/share/gems/gems/puma-3.3.0/lib/puma/configuration.rb:224:in `call'
/opt/rh/rh-ruby23/root/usr/share/gems/gems/puma-3.3.0/lib/puma/server.rb:561:in `handle_request'
/opt/rh/rh-ruby23/root/usr/share/gems/gems/puma-3.3.0/lib/puma/server.rb:406:in `process_client'
/opt/rh/rh-ruby23/root/usr/share/gems/gems/puma-3.3.0/lib/puma/server.rb:271:in `block in run'
/opt/rh/rh-ruby23/root/usr/share/gems/gems/puma-3.3.0/lib/puma/thread_pool.rb:111:in `block in spawn_thread'
[----] I, [2016-10-25T06:23:06.344234 #3248:e3d6e0]  INFO -- :   Rendered layouts/_exception_contents.html.haml (0.1ms)

Comment 2 Harpreet Kataria 2016-10-26 03:09:17 UTC
Aparna,

Can you please investigate, looks like it is throwing an error when clicking on Unassigned Profiles Group folder in grid/tile view. I was able to recreate this on upstream. Works fine when clicking on Unassigned Profiles Group node in tree

Thanks,
~Harpreet

Comment 4 CFME Bot 2016-10-29 07:21:17 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/566d5498c626f85ffd2708cf54055be0f584d599

commit 566d5498c626f85ffd2708cf54055be0f584d599
Author:     Aparna Karve <akarve>
AuthorDate: Wed Oct 26 13:05:06 2016 -0700
Commit:     Aparna Karve <akarve>
CommitDate: Wed Oct 26 13:05:06 2016 -0700

    tree_select_unprovisioned_configured_system should use x_node
    
    to determine if a node is an unassigned configuration profile node
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1388431

 app/controllers/provider_foreman_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 5 CFME Bot 2016-11-01 16:36:04 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/0bea156b536f3a08a1ccbc71c1680f3849b421f3

commit 0bea156b536f3a08a1ccbc71c1680f3849b421f3
Author:     Martin Povolny <mpovolny>
AuthorDate: Sat Oct 29 09:16:59 2016 +0200
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Tue Nov 1 12:28:58 2016 -0400

    Merge pull request #12224 from AparnaKarve/bz1388431_fix_nav_to_unassigned_cs_from_list
    
    Use x_node to determine if the currently selected node is an Unassigned Profiles Group node
    (cherry picked from commit 41f5ae4dabfe4b874ce0add78c16afac7ff29bc6)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1388431

 app/controllers/provider_foreman_controller.rb       |  2 +-
 spec/controllers/provider_foreman_controller_spec.rb | 10 ++++++++++
 2 files changed, 11 insertions(+), 1 deletion(-)

Comment 8 Niyaz Akhtar Ansari 2017-03-09 08:15:22 UTC
Verified in version 5.8.0.4


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