Bug 1610927 - Provision instances doesn't work when having ec2 provider with public images enabled
Summary: Provision instances doesn't work when having ec2 provider with public images ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.10.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.10.0
Assignee: Robin Knaur
QA Contact: Matouš Mojžíš
URL:
Whiteboard: ui:provision:ec2
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-08-01 16:16 UTC by Matouš Mojžíš
Modified: 2019-08-22 00:54 UTC (History)
8 users (show)

Fixed In Version: 5.10.0.28
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-02-12 16:49:49 UTC
Category: ---
Cloudforms Team: AWS
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
production.log (848.64 KB, text/plain)
2018-08-01 16:16 UTC, Matouš Mojžíš
no flags Details

Description Matouš Mojžíš 2018-08-01 16:16:58 UTC
Created attachment 1472151 [details]
production.log

Description of problem:


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

How reproducible:
Always

Steps to Reproduce:
1. Add an ec2 provider with public images enabled
2. Go to Compute -> Cloud -> Instances
3. Select provision an instance

Actual results:
Spinner for five minutes and then no redirection.
Also in production.log it looks like every second is loaded same page many times.

Expected results:


Additional info:

Comment 8 CFME Bot 2018-10-30 15:40:47 UTC
New commit detected on ManageIQ/manageiq/master:

https://github.com/ManageIQ/manageiq/commit/534a1bf3bca8f1f8bbafc83e7e28210b4c656587
commit 534a1bf3bca8f1f8bbafc83e7e28210b4c656587
Author:     PanSpagetka <rknaur>
AuthorDate: Tue Aug 21 08:32:33 2018 -0400
Commit:     PanSpagetka <rknaur>
CommitDate: Tue Aug 21 08:32:33 2018 -0400

    Add template methods needed for provision report #4509
    https://bugzilla.redhat.com/show_bug.cgi?id=1610927

 app/models/miq_template.rb | 79 +
 1 file changed, 79 insertions(+)

Comment 10 CFME Bot 2018-11-02 13:22:25 UTC
New commit detected on ManageIQ/manageiq/hammer:

https://github.com/ManageIQ/manageiq/commit/2b4a340fb31b52e62d7e24a088e3244e15116e6b
commit 2b4a340fb31b52e62d7e24a088e3244e15116e6b
Author:     Martin Povolny <mpovolny>
AuthorDate: Tue Oct 30 11:38:55 2018 -0400
Commit:     Martin Povolny <mpovolny>
CommitDate: Tue Oct 30 11:38:55 2018 -0400

    Merge pull request #17884 from PanSpagetka/add-template-provision-report-methods

    Add template methods needed for provision report

    (cherry picked from commit 67fd66a8b9bcb7856b8a684ee5a2452ba0f367d5)

    https://bugzilla.redhat.com/show_bug.cgi?id=1610927

 app/models/miq_template.rb | 79 +
 1 file changed, 79 insertions(+)

Comment 11 CFME Bot 2018-11-27 14:50:47 UTC
New commit detected on ManageIQ/manageiq/hammer:

https://github.com/ManageIQ/manageiq/commit/ee46c90eddf6be91b4b21bc9ef1cf3d4323e2970
commit ee46c90eddf6be91b4b21bc9ef1cf3d4323e2970
Author:     Martin Povolny <mpovolny>
AuthorDate: Tue Nov 27 09:10:33 2018 -0500
Commit:     Martin Povolny <mpovolny>
CommitDate: Tue Nov 27 09:10:33 2018 -0500

    Merge pull request #18217 from PanSpagetka/fix-miq-template-virtual-cols

    Dont return different types in virtual columns

    (cherry picked from commit ab470a4845a44e9dec6cd77613de31f284adfbb1)

    https://bugzilla.redhat.com/show_bug.cgi?id=1610927

 app/models/miq_template.rb | 26 +-
 1 file changed, 2 insertions(+), 24 deletions(-)

Comment 13 CFME Bot 2018-11-28 14:22:21 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/0491563f20396c5d30d21cc9d96a04fccf20a63b
commit 0491563f20396c5d30d21cc9d96a04fccf20a63b
Author:     PanSpagetka <rknaur>
AuthorDate: Thu Aug  9 02:49:31 2018 -0400
Commit:     PanSpagetka <rknaur>
CommitDate: Thu Aug  9 02:49:31 2018 -0400

    Let Select Provision Template screen use GTL
    https://bugzilla.redhat.com/show_bug.cgi?id=1610927

 app/assets/javascripts/controllers/report_data_controller.js | 16 +
 app/controllers/application_controller.rb | 6 +-
 app/controllers/application_controller/miq_request_methods.rb | 31 +-
 app/controllers/application_controller/report_data_additional_options.rb | 2 +
 app/helpers/gtl_helper.rb | 2 +
 app/javascript/packs/provisioning-actions-common.js | 16 +
 app/views/miq_request/_pre_prov.html.haml | 28 +-
 app/views/miq_request/pre_prov.html.haml | 1 +
 product/views/ProvisionTemplates.yaml | 105 +
 9 files changed, 180 insertions(+), 27 deletions(-)

Comment 14 CFME Bot 2018-11-28 15:21:33 UTC
New commit detected on ManageIQ/manageiq-ui-classic/hammer:

https://github.com/ManageIQ/manageiq-ui-classic/commit/1e43eb18b956e15c717407a363dfc029a8dab979
commit 1e43eb18b956e15c717407a363dfc029a8dab979
Author:     Martin Hradil <himdel>
AuthorDate: Wed Nov 28 09:18:50 2018 -0500
Commit:     Martin Hradil <himdel>
CommitDate: Wed Nov 28 09:18:50 2018 -0500

    Merge pull request #4509 from PanSpagetka/bz-1610927

    Let Select Provision Template screen use GTL

    (cherry picked from commit 3b71ea1e57c239245b43200a733e2fda4f357717)

    https://bugzilla.redhat.com/show_bug.cgi?id=1610927

 app/assets/javascripts/controllers/report_data_controller.js | 21 +
 app/controllers/application_controller.rb | 8 +-
 app/controllers/application_controller/miq_request_methods.rb | 44 +-
 app/controllers/application_controller/report_data_additional_options.rb | 4 +
 app/javascript/packs/provisioning-actions-common.js | 12 +
 app/views/miq_request/_pre_prov.html.haml | 27 +-
 app/views/miq_request/_pre_prov_images.html.haml | 31 -
 app/views/miq_request/_pre_prov_volumes_snapshots.html.haml | 32 -
 app/views/miq_request/pre_prov.html.haml | 1 -
 product/views/ProvisionCloudTemplates.yaml | 104 +
 product/views/ProvisionInfraTemplates.yaml | 95 +
 spec/controllers/application_controller/report_data_spec.rb | 12 +
 spec/controllers/vm_cloud_controller_spec.rb | 16 +
 spec/javascripts/support/jasmine.yml | 1 +
 spec/support/controller_helper.rb | 5 +
 15 files changed, 319 insertions(+), 94 deletions(-)

Comment 15 CFME Bot 2018-11-30 15:30:49 UTC
New commit detected on ManageIQ/manageiq/hammer:

https://github.com/ManageIQ/manageiq/commit/8dd25eb33d45ca3d748c8d3b13738fc78daa69c4
commit 8dd25eb33d45ca3d748c8d3b13738fc78daa69c4
Author:     Martin Povolny <mpovolny>
AuthorDate: Wed Nov 28 10:22:16 2018 -0500
Commit:     Martin Povolny <mpovolny>
CommitDate: Wed Nov 28 10:22:16 2018 -0500

    Merge pull request #18237 from PanSpagetka/template-display-bool-string

    Display deprecated as a true/false string

    (cherry picked from commit 12ecbe4b2945ceb5278a43d95336d70a004bab41)

    https://bugzilla.redhat.com/show_bug.cgi?id=1610927

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

Comment 16 Matouš Mojžíš 2018-12-11 15:13:58 UTC
I tried it in us-east-1 region which has over 120k images and when I select an image and then click on continue I get Error requesting data from server and in evm.log:
[----] E, [2018-12-11T09:58:27.492295 #13082:10d2f90] ERROR -- : MIQ(MiqServer#monitor) can't modify frozen Hash
[----] E, [2018-12-11T09:58:27.492440 #13082:10d2f90] ERROR -- : [RuntimeError]: can't modify frozen Hash  Method:[block (2 levels) in <class:LogProxy>]
[----] E, [2018-12-11T09:58:27.492501 #13082:10d2f90] ERROR -- : /opt/rh/cfme-gemset/gems/activerecord-5.0.7.1/lib/active_record/attribute_set.rb:53:in `write_from_user'
It looks same as in this BZ:https://bugzilla.redhat.com/show_bug.cgi?id=1588082
It loads the list of images with no issue so should it be part of this BZ or shall I created a new one for this issue?

Comment 17 Robin Knaur 2018-12-17 09:59:04 UTC
I think it should be new BZ, it loads the list even for big numbers of Images. I have tried to reproduce it, but after I clicked on continue every time I got into provision dialog. So, please create a new BZ and provide more detailed information when it happens and how to reproduce it. Does it happen every time? And even if public images are turned off?

Thanks

Comment 18 Matouš Mojžíš 2019-01-02 15:14:20 UTC
Verified in 5.10.0.28. It redirects me into provisioning dialog without long waiting.


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