Bug 1720608 - Services dialog not getting created after import in service dialog accordion
Summary: Services dialog not getting created after import in service dialog accordion
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Automate
Version: 5.11.0
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: GA
: 5.11.0
Assignee: drew uhlmann
QA Contact: Niyaz Akhtar Ansari
Red Hat CloudForms Documentation
URL:
Whiteboard:
Depends On:
Blocks: 1696474
TreeView+ depends on / blocked
 
Reported: 2019-06-14 11:07 UTC by Niyaz Akhtar Ansari
Modified: 2019-12-13 15:01 UTC (History)
10 users (show)

Fixed In Version: 5.11.0.13
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-12-13 15:01:21 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Affected Service dialog (17.63 KB, text/plain)
2019-06-14 11:07 UTC, Niyaz Akhtar Ansari
no flags Details

Description Niyaz Akhtar Ansari 2019-06-14 11:07:07 UTC
Created attachment 1580642 [details]
Affected Service dialog

Description of problem:


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

How reproducible:
100%

Steps to Reproduce:
1.Import the attached Service dialog



Actual results:
No Service Dialog import  

Expected results:
It should import the dialog 

Additional info:

Comment 5 Tina Fitzgerald 2019-06-17 20:11:33 UTC
Hi Harpreet,

The screenshot looks like they're trying to import the service dialog from the datastore import/export page. Is that correct?

Thanks,
Tina

Comment 7 William Fitzgerald 2019-06-18 15:26:52 UTC
Dialog import appears to be broken.   I tried to import 2 dialogs, one from the BZ and another one.  
It appeared from the UI that it worked but the dialogs don't appear in the explorer.

Sprout Reproducer: 10.8.198.142

Comment 8 William Fitzgerald 2019-06-18 15:27:30 UTC
Version 5.11.0.8.20190611155126_01e077e

Comment 9 William Fitzgerald 2019-06-18 15:49:16 UTC
I was successful importing dialogs from 5.11.0.6 and 5.11.0.8 appliances.
Hope that helps

Comment 10 William Fitzgerald 2019-06-18 18:56:50 UTC
Dialog import not working in Version 5.11.0.5.20190521144849_d0a06e6

Sprout Reproducer: 10.8.197.178 


Working in Version 5.11.0.4.20190514210444_0c91ee1

Sprout Reproducer: 10.8.199.17

Comment 11 drew uhlmann 2019-06-19 19:28:31 UTC
I think it's this:
NoMethodError: undefined method `each' for false:FalseClass
from /Users/duhlmann/manageiq/app/models/dialog_import_validator.rb:29:in `check_dialogs_for_validity'

Comment 12 drew uhlmann 2019-06-19 19:28:54 UTC
[----] F, [2019-06-19T15:28:38.135294 #79576:3fef04eaf1b8] FATAL -- : Error caught: [NoMethodError] undefined method `each' for false:FalseClass
/Users/duhlmann/manageiq/app/models/dialog_import_validator.rb:29:in `check_dialogs_for_validity'
/Users/duhlmann/manageiq/app/models/dialog_import_validator.rb:15:in `determine_validity'
/Users/duhlmann/manageiq/lib/services/dialog_import_service.rb:68:in `store_for_import'
/Users/duhlmann/.rvm/gems/ruby-2.5.1/bundler/gems/manageiq-ui-classic-74de2e89c4e3/app/controllers/miq_ae_customization_controller.rb:44:in `upload_import_file'
/Users/duhlmann/.rvm/gems/ruby-2.5.1/gems/actionpack-5.0.7.2/lib/action_controller/metal/basic_implicit_render.rb:4:in `send_action'
/Users/duhlmann/.rvm/gems/ruby-2.5.1/gems/actionpack-5.0.7.2/lib/abstract_controller/base.rb:188:in `process_action'
/Users/duhlmann/.rvm/gems/ruby-2.5.1/gems/actionpack-5.0.7.2/lib/action_controller/metal/rendering.rb:30:in `process_action'
/Users/duhlmann/.rvm/gems/ruby-2.5.1/gems/actionpack-5.0.7.2/lib/abstract_controller/callbacks.rb:20:in `block in process_action'
/Users/duhlmann/.rvm/gems/ruby-2.5.1/gems/activesupport-5.0.7.2/lib/active_support/callbacks.rb:126:in `call'
/Users/duhlmann/.rvm/gems/ruby-2.5.1/gems/activesupport-5.0.7.2/lib/active_support/callbacks.rb:506:in `block (2 levels) in compile'
/Users/duhlmann/.rvm/gems/ruby-2.5.1/gems/activesupport-5.0.7.2/lib/active_support/callbacks.rb:455:in `call'
/Users/duhlmann/.rvm/gems/ruby-2.5.1/gems/activesupport-5.0.7.2/lib/active_support/callbacks.rb:101:in `__run_callbacks__'

Comment 13 drew uhlmann 2019-06-19 19:47:53 UTC
Okay fine that's wrong, ignore 11 and 12 please

Comment 14 drew uhlmann 2019-06-19 20:20:24 UTC
visible: false 
as a setting in old dialogs is the reason this broke, looking into the change that caused this to stop working, but the create for  ```def build_dialog_groups(dialog_tab, export_version = CURRENT_DIALOG_VERSION)``` in the main repo is failing because of 
ActiveRecord::RecordInvalid: Validation failed: DialogGroup: Dialog fields is invalid, Box Variables / DialogFieldCheckBox: Visible can't be blank

Comment 15 drew uhlmann 2019-06-25 18:35:02 UTC
https://github.com/ManageIQ/manageiq/pull/18914


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